public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

      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