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

Nicolas DET nd at bplan-gmbh.de
Fri Nov 3 07:49:48 EST 2006


Benjamin Herrenschmidt wrote:
> 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 ?
> 

It's like this in ofsoc-ppc. As I am not sure what it does exactly I 
left it like this. This module performs well here so far.


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: archpowerpc_ohciof.patch
Url: http://ozlabs.org/pipermail/linuxppc-embedded/attachments/20061102/83adcee4/attachment.txt 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nd.vcf
Type: text/x-vcard
Size: 249 bytes
Desc: not available
Url : http://ozlabs.org/pipermail/linuxppc-embedded/attachments/20061102/83adcee4/attachment.vcf 


More information about the Linuxppc-embedded mailing list