lkml.org 
[lkml]   [2020]   [May]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v6 3/4] usb: ehci: Add new EHCI driver for Broadcom STB SoC's
On Wed, May 06, 2020 at 04:23:01PM -0400, Alan Cooper wrote:
> > > + irq = platform_get_irq(pdev, 0);
> > > + if (irq < 0)
> >
> > "<=" right?
>
> The example in the comment above platform_get_irq in platform.c says
> "if (irq < 0)"

There is work to fix that up on the mailing lists, 0 is not a valid irq :)

> > > +static int __init ehci_brcm_init(void)
> > > +{
> > > + if (usb_disabled())
> > > + return -ENODEV;
> >
> > You check this here, so why are you also checking it in the probe
> > function?
>
> Good point. I'll remove the one in probe. This was another thing that
> came from the ehci and ohci platform drivers. About a third of the
> drivers in this directory do this, probably because they started with
> the platform drivers. Do you think it's worth me submitting a patch to
> clean this up at some point?

Yes please.

>
> >
> > thanks,
> >
> > greg k-h
>
> I'll wait for your response before sending a V7.

Never wait for someone who you have no idea how much email they get :)

thanks,

greg k-h

\
 
 \ /
  Last update: 2020-05-07 08:43    [W:0.130 / U:0.280 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site