Subject: Re: [PATCH/RFC] powerpc: Add of_platform support for OHCI/Bigendian HC

From: Benjamin Herrenschmidt <[email-masked]>
Date: Thu, 02 Nov 2006 10:27:07 +1100

On Thu, 2006-11-02 at 09:23 +1100, Benjamin Herrenschmidt wrote:

And in english here too....

> On Wed, 2006-11-01 at 21:31 +0100, Nicolas DET wrote:
> 
> > +/**+	ohci = hcd_to_ohci(hcd);
> > +	ohci->flags |= OHCI_BIG_ENDIAN;
> > +	ohci_hcd_init(ohci);
> 
> Bon, c'est pas mal. Le seul truc ici, c'est que je prefererais que
> big-endian ne soit pas hard-code comme ca mais depende du device-tree.
> Ce code doit pouvoir etre re-utilise pour des implementations
> little-endian.

Ok, so it looks good. The only thing here is that I would prefer if the
big-endian setting wasn't hard-coded, but set based on the device-tree,
so that this code can be re-used for little-endian implementations.

> > +	retval = usb_add_hcd(hcd, irq, IRQF_DISABLED);
> > +	if (retval == 0)
> > +		return retval;
> 
> Pourquoi IRQF_DISABLED ?

Why IRQF_DISABLED ?

> > +module_init(ohci_hcd_ppc_of_init);
> > +module_exit(ohci_hcd_ppc_of_cleanup);
> > --- a/drivers/usb/host/Kconfig	2006-11-01 09:18:56.000000000 +0100
> > +++ b/drivers/usb/host/Kconfig	2006-11-01 21:16:06.000000000 +0100
> > @@ -106,6 +106,14 @@ config USB_OHCI_HCD_PPC_SOC
> >  	  Enables support for the USB controller on the MPC52xx or
> >  	  STB03xxx processor chip.  If unsure, say Y.
> >  
> > +config USB_OHCI_HCD_PPC_OF
> > +	bool "OHCI support for PPC USB controller for OpenFirmware platform"
> > +	depends on USB_OHCI_HCD && PPC_OF
> > +	default y
> > +	select USB_OHCI_BIG_ENDIAN
> > +	---help---
> > +	  Enables support for the USB controller PowerPC OpenFirmware platform
> > +
> 
> Je prefererais que USB_OHCI_BIG_ENDIAN soit une option a choisir.

I'd prefer if USB_OHCI_BIG_ENDIAN was a user selectable option

Ben.