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

Timestamp:

2018-01-13T03:10:29Z (8 years ago)

Author:

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

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

a53ed3a

Parents:

36f0738

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.

Although this is a massive commit, it is a simple text replacement, and thus
is very easy to verify. Simply do the following:

`
git checkout <this commit's hash>
git reset HEAD
<sup>git add .
tools/srepl '\berrno_t\b' int
git add .
tools/srepl '\bsys_errno_t\b' sysarg_t
git reset
git diff
`</sup>

While this doesn't ensure that the replacements are correct, it does ensure
that the commit doesn't do anything except those replacements. Since errno_t
is typedef'd to int in the usual case (and sys_errno_t to sysarg_t), even if
incorrect, this commit cannot change behavior.

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