From: "Jiří Prchal" <jiri.prchal@aksignal.cz>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Subject: [PATCH v.2] at91: add support for CDU9G25 board
Date: Wed, 11 Sep 2013 09:39:44 +0200 [thread overview]
Message-ID: <52301E40.9010704@aksignal.cz> (raw)
In-Reply-To: <20130910225202.ED144380E12@gemini.denx.de>
Hi Wolfgang,
Dne 11.9.2013 00:52, Wolfgang Denk napsal(a):
> Dear Ji?? Prchal,
>
> In message <522ED13F.90303@aksignal.cz> you wrote:
>>
>>> This comment does not belong into the commit message - please move it
>>> to the comment section, i. e. below the "---" line.
>> I'm not sure if I even should number versions of this patch?
>
> But this is mandatory, see
> http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions
>
>>>> +#define CONFIG_BOOTARGS "console=ttyS0,115200 ubi.mtd=root"\
>>>> + "root=ubi0:root rootfstype=ubifs rw"\
>>>> + "g_ether.dev_addr=02:04:25:aa:55:5e"\
>>>> + "g_ether.host_addr=02:04:25:aa:55:5f"
>> This is here for linux connection with PC via USB ethernet gadget. I'd like static IP for this connection but if there
>> is random MAC addr I must set up IP again and again. Is another solution for this?
I will remove this.
>>
>>>> +#define CONFIG_SERVERIP 10.0.1.1
>> Please help me with this:
>> I'd like to have board prepared for download images from TFTP server on address 10.0.1.1.
>
> We do not allow static networkj configuations in the U-Boot binary
> image. what works for you, may be bad for others. It's especially
> bad when all boards using the same image come p using the same MAC (or
> IP) addresses.
I thought that this is default configuration of serverip. Does not
affect network communication. And user can change it any time.
>
>> Is there other way how to do it? Even with MAC addr, I solved it with random MAC, is that OK? I keep in mind that would
>
> No. The image must not contain any static network addresses.
>
>> be nice read unique serial number from F-RAM (is on boart too) on SPI and use it as MAC addr. Will be this way better?
>
> There are many ways how to initialize the MAC address on the system
> during production or by reading any form of product information from
> EEPROM or F-RAM or whatever you have on your board. This is OK, as
> long as boards get assigned correct (i. e. different) addresses.
I'll try use F-RAM for MAC, but it will take some time.
Can I post patch now with random MAC and sometime in the future post
another patch with F-RAM?
Thanks,
Jiri
prev parent reply other threads:[~2013-09-11 7:39 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-09 14:22 [U-Boot] [PATCH] Subject: [PATCH v.2] at91: add support for CDU9G25 board Jiri Prchal
2013-09-09 15:34 ` Wolfgang Denk
2013-09-10 7:58 ` Jiří Prchal
2013-09-10 9:26 ` Andreas Bießmann
2013-09-10 22:52 ` Wolfgang Denk
2013-09-11 7:39 ` Jiří Prchal [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52301E40.9010704@aksignal.cz \
--to=jiri.prchal@aksignal.cz \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.