[U-Boot] [PATCH v2] add dm9000 eeprom read/write command

Eric Jarrige eric.jarrige at armadeus.org
Tue Aug 30 23:17:12 CEST 2011


Hi Stefano,

On 30 août 2011, at 11:47, Stefano Babic wrote:

> On 08/28/2011 11:47 PM, Eric Jarrige wrote:
>> Signed-off-by: Eric Jarrige <eric.jarrige at armadeus.org>
>> Signed-off-by: Stefano Babic <sbabic at denx.de>
>> Cc: Wolfgang Denk <wd at denx.de>
>> Cc: Detlev Zundel <dzu at denx.de>
>> 
>> Changes for v2:
>> 	- remove DM9000 driver dependant compilation flag
>> ---
> 
> Hi Eric,
> 
> why should this code put in a separate file and not inserted in the
> driver itself ? I think this remains an open point from our previous
> discussion.
> 

Sorry for the confusion, I did not understood that your remark was
not related to the compilation flags.
Now, I've checked how to have this U-Boot commands in the driver
itself. I think it's doable if  I can have a compilation
CONFIG_CMD_XXX to enable the command line feature.
Without this option, there is a direct impact one memory footprint for
every board featuring the DM9000 controller and I failed to find any
model of driver with an optional command line interface in a driver.

Why do you recommend to put the command interface in drivers
as it is seems to be something quite unusual in U-Boot ? 

Best regards,
Eric
 







More information about the U-Boot mailing list