On Oct 18, 2011, at 10:19 PM, Mike Frysinger wrote:

> On Tuesday 18 October 2011 19:41:22 Kyle Moffett wrote:
>> --- a/README
>> +++ b/README
>> 
>> - 85xx CPU Options:
>> +            CONFIG_MPC85XX_GENERIC_GPIO
>> +
>> +            Provide a generic gpio_request() interface to the onboard
>> +            processor GPIOs in the MPC85XX-series chips.
> 
> do you really need a CONFIG knob for this ?  defining this adds no overhead 
> that i can see.

I agree, wasn't paying attention that this is just in a .h, so lets remove the 
CONFIG_ option bits (both in README and in .h)

>> --- /dev/null
>> +++ b/arch/powerpc/include/asm/mpc85xx_gpio.h
> 
> missing #ifdef protection against multiple inclusion

please fix.

- k
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to