[PATCH 0/6] Use -ENOSYS consistently

Tom Rini trini at konsulko.com
Sun Mar 21 17:38:13 CET 2021


On Sun, Mar 21, 2021 at 04:18:18PM +1300, Simon Glass wrote:

> A few places use -ENOTSUPP when they should use -ENOSYS. In two cases both
> are used. This little series tidies this up.
> 
> 
> Simon Glass (6):
>   usb: Return -ENOSYS when system call is not available
>   spi: Return -ENOSYS when system call is not available
>   tlv_eeprom: Return -ENOSYS when system call is not available
>   clk: Return -ENOSYS when system call is not available
>   simple-pm-bus: Use -ENOSYS for checking missing system call
>   pinctrl: Return -ENOSYS when system call is not available
> 
>  drivers/clk/clk-composite.c         |  8 ++++----
>  drivers/core/simple-pm-bus.c        |  4 ++--
>  drivers/pinctrl/pinctrl-uclass.c    | 10 ++++++----
>  drivers/usb/gadget/udc/udc-uclass.c |  2 +-
>  include/spi-mem.h                   |  2 +-
>  include/tlv_eeprom.h                |  6 +++---
>  6 files changed, 17 insertions(+), 15 deletions(-)

I think we need to start this series with a doc update that makes it
clear why this is correct and when it should be used.   There's two
instances of "ENOSYS" in doc/driver-model/ today and neither explains
why this series is correct (which I'm sure it is).  Thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210321/f063b944/attachment.sig>


More information about the U-Boot mailing list