Re: [PATCH 3/3] usb: gadget: s3c-hsudc: Replace 0 with NULL for pointers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 03, 2012 at 05:00:44PM +0530, Sachin Kamat wrote:
> Hi
> 
> On 3 September 2012 16:37, ABRAHAM, KISHON VIJAY <kishon@xxxxxx> wrote:
> > Hi,
> >
> > On Mon, Sep 3, 2012 at 3:48 PM, Sachin Kamat <sachin.kamat@xxxxxxxxxx> wrote:
> >> Silences the following type of sparse warnings:
> >> warning: Using plain integer as NULL pointer
> >>
> >> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>
> >> ---
> >>  drivers/usb/gadget/s3c-hsudc.c |    8 ++++----
> >>  1 files changed, 4 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/drivers/usb/gadget/s3c-hsudc.c b/drivers/usb/gadget/s3c-hsudc.c
> >> index 35cdc6a..d8e785d 100644
> >> --- a/drivers/usb/gadget/s3c-hsudc.c
> >> +++ b/drivers/usb/gadget/s3c-hsudc.c
> >> @@ -835,7 +835,7 @@ static struct usb_request *s3c_hsudc_alloc_request(struct usb_ep *_ep,
> >>
> >>         hsreq = kzalloc(sizeof(*hsreq), gfp_flags);
> >>         if (!hsreq)
> >> -               return 0;
> >> +               return NULL;
> >
> > shouldn't this be -ENOMEM?
> 
> That should have been the obvious return value. However, I thought
> probably it was made so with some reasoning and did not change it as I
> am not familiar with this driver.

it's just the way the API was designed. If we fail to allocate a
usb_request, we must return NULL so function/gadget drivers can check if
it was successful or not.

Of course it could be changed into ERR_PTR(-ENOMEM) or something, but
that's an API change... ;-)

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux