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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox