public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 29/32] nitrogen6x: config: configure usb_ether
Date: Tue, 07 Oct 2014 16:36:18 +0200	[thread overview]
Message-ID: <5433FA62.7050907@denx.de> (raw)
In-Reply-To: <5432C622.5010901@boundarydevices.com>

Hi Eric,

On 06/10/2014 18:41, Eric Nelson wrote:

>> I understand the use case, but it does not always work (I mean, in all
>> network configurations) and we regret generally having IP addresses hard
>> coded in the default configuration.
>>
> 
> Can you clarify which parts (mac/IP address/both) are a problem?
> 
> The 'usb_ether' is kind of an odd beast, in that it's a link-local
> protocol, which is why the the IP addresses aren't read from or written
> to a persistent environment.

This is not completely true. I mean, I understand that you want to have
such as situation, with addresses valid only in the link host / target.

However, if a customer / user has a PC belonging to the  10.0.0.0/24
network, there is a conflict. And this is not a rare case, because as I
have seen in companies 10.0.0.0/8 are used more often as 192.168.0.0/16.

You can have two interface (ethernet and USB) acting on the same address
range and packets originally sent to network are readdressed to the
target, letting the customer without network.

I understand that you want to provide is a special case - but as you can
see, you cannot cover all cases by setting a hard coded address.

> 
> Our goal was to only require configuration of one side of the link
> (the USB Host). It seems that without implementing a DHCP **server**,
> this is the most convenient.

If you really want, why don't you use a script as a 6x_ ? usbrecover can
load initially a script setting the network addresses, without
hardcoding to the u-boot image.

> 
> The mac addresses above are ours, and we can confirm that they are
> not in use on any other hardware,

I know that and it is exactly the same we had in the past with other
boards. Customers can buy more as one instance of the boards, having
then multiple boards with the same MAC address - and very bad case.


> so they're guaranteed to be unique
> unless you happen to hook up multiple of our boards to a Host at the
> same time.

Yes, exactly. Why do you want to restrict your sales chances ? :-D

Seriously, hard coding mac and network addresses was strictly rejected -
even if after your patch I have found a couple of boards doing that
(maybe it was not seen during review, see for example  v38b.h).

> 
> Since the configuration of network adapters on most hosts is based
> on mac addresses, hard-coding these prevents the need to re-configure
> each time a new board is connected to a host.

I understand that hard coding makes life easier, but I am not convinced
it is the correct way to do, specially with IP addresses.

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

  reply	other threads:[~2014-10-07 14:36 UTC|newest]

Thread overview: 71+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-02 19:16 [U-Boot] [PATCH 00/32] ARM: i.MX: nitrogen6x clean up Eric Nelson
2014-10-02 19:16 ` [U-Boot] [PATCH 01/32] nitrogen6x: implement board_cfb_skip() to disable text output Eric Nelson
2014-10-06 15:58   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 02/32] nitrogen6x: simplify board_mmc_getcd Eric Nelson
2014-10-06 15:58   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 03/32] nitrogen6x: configure SD2 pads for SDIO on USDHC2 Eric Nelson
2014-10-06 16:01   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 04/32] nitrogen6x: power-down miscellanous peripherals Eric Nelson
2014-10-06 16:01   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 05/32] nitrogen6x: configure SGTL5000, CSI camera clock outputs Eric Nelson
2014-10-06 16:01   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 06/32] nitrogen6x: staticize board file Eric Nelson
2014-10-06 16:02   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 07/32] nitrogen6x: Allow U-Boot to be silent on UART2 Eric Nelson
2014-10-06 15:42   ` Stefano Babic
2014-10-06 15:51     ` Eric Nelson
2014-10-02 19:16 ` [U-Boot] [PATCH 08/32] nitrogen6x: phy: add 100 us delay after phy reset Eric Nelson
2014-10-06 16:02   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 09/32] nitrogen6x: prevent warnings about board_ehci* callbacks Eric Nelson
2014-10-06 16:03   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 10/32] nitrogen6x: display: add support lvds jeida screen Eric Nelson
2014-10-06 16:04   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 11/32] nitrogen6x: display: add qvga panel Eric Nelson
2014-10-06 16:04   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 12/32] nitrogen6x: display: add support for LG-9.7 LVDS display Eric Nelson
2014-10-06 16:05   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 13/32] nitrogen6x: display: add LDB-WXGA-S for SPWG 1280x800 displays Eric Nelson
2014-10-06 16:05   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 14/32] nitrogen6x: display: add support for fusion 7 display Eric Nelson
2014-10-06 16:05   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 15/32] nitrogen6x: display: add svga display (800x600) Eric Nelson
2014-10-06 16:06   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 16/32] nitrogen6x: display: add Ampire 1024x600 panel Eric Nelson
2014-10-06 16:06   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 17/32] nitrogen6x: display: add wvga-lvds panel Eric Nelson
2014-10-06 16:06   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 18/32] nitrogen6x: display use I2C detect for HDMI Eric Nelson
2014-10-06 16:07   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 19/32] nitrogen6x: config: add USB Mass Storage (ums) support Eric Nelson
2014-10-06 16:07   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 20/32] nitrogen6x: config: allow boot to USB stick Eric Nelson
2014-10-06 16:07   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 21/32] nitrogen6x: config: use FS_GENERIC load command Eric Nelson
2014-10-06 16:07   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 22/32] nitrogen6x: config: add initrd_high Eric Nelson
2014-10-06 16:08   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 23/32] nitrogen6x: config: expose SATA, then MMC over USB Eric Nelson
2014-10-06 16:08   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 24/32] nitrogen6x: config: enable USB keyboard support Eric Nelson
2014-10-06 16:08   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 25/32] nitrogen6x: config: add CONFIG_CMD_MEMTEST Eric Nelson
2014-10-06 16:08   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 26/32] nitrogen6x: config: enable "i2c edid" Eric Nelson
2014-10-06 16:08   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 27/32] nitrogen6x: config: allow more bootargs parameters Eric Nelson
2014-10-06 16:09   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 28/32] nitrogen6x: config: disable logo Eric Nelson
2014-10-06 15:51   ` Stefano Babic
2014-10-06 16:09   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 29/32] nitrogen6x: config: configure usb_ether Eric Nelson
2014-10-06 15:54   ` Stefano Babic
2014-10-06 16:41     ` Eric Nelson
2014-10-07 14:36       ` Stefano Babic [this message]
2014-10-07 15:08         ` Eric Nelson
2014-10-02 19:16 ` [U-Boot] [PATCH 30/32] nitrogen6x: config: add gpio command Eric Nelson
2014-10-06 16:09   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 31/32] nitrogen6x: config: enable Android fastboot Eric Nelson
2014-10-06 16:09   ` Stefano Babic
2014-10-02 19:16 ` [U-Boot] [PATCH 32/32] nitrogen6x: config: enable EXT4 filesystem Eric Nelson
2014-10-06 16:10   ` Stefano Babic
2014-10-06 15:35 ` [U-Boot] [PATCH 00/32] ARM: i.MX: nitrogen6x clean up Stefano Babic

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=5433FA62.7050907@denx.de \
    --to=sbabic@denx.de \
    --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