Re: [PATCH] HID: usbkbd: return proper error code
From: Sudip Mukherjee
Date: Thu Sep 01 2016 - 18:10:21 EST
On Thursday 01 September 2016 04:51 PM, Jiri Kosina wrote:
On Wed, 31 Aug 2016, Sudip Mukherjee wrote:
Use proper error code instead of using -1 on failure to allocate
memory. We may use the error code later in the caller.
But we don't. usb_kbd_probe() returns -ENOMEM in case usb_kbd_alloc_mem()
fails anyway, so I fail to see the point of the change really.
Well, yes, we don't as of now.
When I was reading the code for something related to my day job I was a
bit confused with -1 instead of a proper error code. I am sure there
will be many others like me.
Its fine if you think the change is not needed.
regards
sudip