[U-Boot-Users] [PATCH 3/7] sbc8560: define eth0 and eth1 instead of eth1 and eth2

Andy Fleming afleming at gmail.com
Tue Jul 15 02:51:26 CEST 2008


On Fri, Jul 11, 2008 at 5:03 PM, Paul Gortmaker
<paul.gortmaker at windriver.com> wrote:
> Wolfgang Denk wrote:
>>
>>
>>> + * MAC addresses directly on it.
>>> + */
>>> +#if 0
>>> +#define CONFIG_ETHADDR              00:01:af:07:9b:8a
>>> +#define CONFIG_ETH1ADDR             00:01:af:07:9b:8b
>>>  #define CONFIG_SERVERIP             192.168.0.131
>>>  #define CONFIG_IPADDR               192.168.0.105
>>> -#define CONFIG_GATEWAYIP    0.0.0.0
>>> +#define CONFIG_GATEWAYIP    192.168.0.1
>>>  #define CONFIG_NETMASK              255.255.255.0
>>> +#endif
>>> +
>>>
>>
>> I think this should be removed. There is really no good reason to add
>> network configuration at all to  the  board  config  file  (with  the
>> exception of the 2 or 3 boards which actually boot from real ROM).
>>
>
> No problem.  I'll remove and resend.

As a compromise between removal and keeping the useful information for
developers who are doing early bringup on similar boards, I have
applied this patch, and replaced the above with:

/* You can compile in a MAC address and your custom net settings by using
 * the following syntax.  Your board should be marked with the assigned
 * MAC addresses directly on it.
 *
 * #define CONFIG_ETHADDR		de:ad:be:ef:00:00
 * #define CONFIG_ETH1ADDR		fa:ke:ad:dr:es:s!
 * #define CONFIG_SERVERIP		<server ip>
 * #define CONFIG_IPADDR		<board ip>
 * #define CONFIG_GATEWAYIP		<gateway ip>
 * #define CONFIG_NETMASK		<your netmask>
 */

Wolfgang, if you still object, I can totally remove it before you pull
(git rebase --interactive is a wonderful thing).

I also corrected some whitespace issues where comments trailed off the
end of the line for no good reason.

Thanks,
Andy




More information about the U-Boot mailing list