[PATCH/RFC] powerpc: Add of_platform support for OHCI/Bigendian HC
Benjamin Herrenschmidt
benh at kernel.crashing.org
Thu Nov 2 10:27:07 EST 2006
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.
More information about the Linuxppc-embedded
mailing list