Changeset 46577995 in mainline for uspace/lib/c/include/device

Timestamp:

2018-01-04T20:50:52Z (8 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Children:

e211ea04

Parents:

facacc71

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:47:53)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:50:52)

Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

After this commit, HelenOS is free of code that mixes error codes with non-error
values on the assumption that error codes are negative.

Location:

uspace/lib/c/include/device

Files:

• clock_dev.h (modified) (1 diff)
• hw_res.h (modified) (1 diff)
• hw_res_parsed.h (modified) (1 diff)
• led_dev.h (modified) (1 diff)
• pio_window.h (modified) (1 diff)

uspace/lib/c/include/device/clock_dev.h

@@ -44 +44 @@
 } clock_dev_method_t;
 
-extern int clock_dev_time_get(async_sess_t *, struct tm *);
-extern int clock_dev_time_set(async_sess_t *, struct tm *);
+extern errno_t clock_dev_time_get(async_sess_t *, struct tm *);
+extern errno_t clock_dev_time_set(async_sess_t *, struct tm *);
 
 #endif

uspace/lib/c/include/device/hw_res.h

@@ -116 +116 @@
 }
 
-extern int hw_res_get_resource_list(async_sess_t *, hw_resource_list_t *);
-extern int hw_res_enable_interrupt(async_sess_t *, int);
-extern int hw_res_disable_interrupt(async_sess_t *, int);
-extern int hw_res_clear_interrupt(async_sess_t *, int);
+extern errno_t hw_res_get_resource_list(async_sess_t *, hw_resource_list_t *);
+extern errno_t hw_res_enable_interrupt(async_sess_t *, int);
+extern errno_t hw_res_disable_interrupt(async_sess_t *, int);
+extern errno_t hw_res_clear_interrupt(async_sess_t *, int);
 
-extern int hw_res_dma_channel_setup(async_sess_t *, unsigned int, uint32_t,
+extern errno_t hw_res_dma_channel_setup(async_sess_t *, unsigned int, uint32_t,
     uint32_t, uint8_t);
-extern int hw_res_dma_channel_remain(async_sess_t *, unsigned, size_t *);
+extern errno_t hw_res_dma_channel_remain(async_sess_t *, unsigned, size_t *);
 
 #endif

uspace/lib/c/include/device/hw_res_parsed.h

@@ -154 +154 @@
 }
 
-extern int hw_res_list_parse(const pio_window_t *, const hw_resource_list_t *,
+extern errno_t hw_res_list_parse(const pio_window_t *, const hw_resource_list_t *,
     hw_res_list_parsed_t *, int);
-extern int hw_res_get_list_parsed(async_sess_t *, hw_res_list_parsed_t *, int);
+extern errno_t hw_res_get_list_parsed(async_sess_t *, hw_res_list_parsed_t *, int);
 
 #endif

uspace/lib/c/include/device/led_dev.h

@@ -43 +43 @@
 } led_dev_method_t;
 
-extern int led_dev_color_set(async_sess_t *, pixel_t);
+extern errno_t led_dev_color_set(async_sess_t *, pixel_t);
 
 #endif

uspace/lib/c/include/device/pio_window.h

@@ -51 +51 @@
 } pio_window_t;
 
-extern int pio_window_get(async_sess_t *, pio_window_t *);
+extern errno_t pio_window_get(async_sess_t *, pio_window_t *);
 
 #endif