Re: [PATCH] ARM: dts: Revert disabling of smc91x for n900

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 06 January 2015 21:17:59 Tony Lindgren wrote:
> * Tony Lindgren <tony@xxxxxxxxxxx> [150106 12:01]:
> > * Aaro Koskinen <aaro.koskinen@xxxxxx> [150106 11:47]:
> > > Hi,
> > > 
> > > On Tue, Jan 06, 2015 at 08:59:03AM -0800, Tony Lindgren 
wrote:
> > > > * Pavel Machek <pavel@xxxxxx> [150106 00:03]:
> > > > > On Mon 2015-01-05 15:02:29, Tony Lindgren wrote:
> > > > > > Revert "ARM: dts: Disable smc91x on n900 until
> > > > > > bootloader dependency is removed". We've now fixed
> > > > > > the issues that caused problems with uninitialized
> > > > > > hardware depending on the bootloader version.
> > > > > > Mostly things got fixed with the following commits:
> > > > > > 
> > > > > > 9a894953a97b ("ARM: dts: Fix bootloader version
> > > > > > dependencies by muxing n900 smc91x pins")
> > > > > > 7d2911c43815 ("net: smc91x: Fix gpios for device
> > > > > > tree based booting")
> > > > > > 
> > > > > > Note that this only affects the early development
> > > > > > boards with Ethernet that we still have in a few
> > > > > > automated boot test systems.
> > > > > > 
> > > > > > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
> > > > > 
> > > > > Normally, the early development boards should have
> > > > > separate dts file (then include common parts), no?
> > > > 
> > > > In this case it won't matter. The GPMC hardware is
> > > > there, the probe just fails if no smsc91x is found.
> > > > 
> > > > > Could you at least add a note to the dts file what is
> > > > > it? Because I always thought it is a bug.
> > > > 
> > > > Sure, updated patch below. Can somebody please test boot
> > > > it on a production n900 too to make sure it no longer
> > > > causes issues?
> > > 
> > > Seems to work fine with normal n900.
> > > 
> > > Tested-by: Aaro Koskinen <aaro.koskinen@xxxxxx>
> > 
> > OK good to hear, thanks for testing.
> > 
> > > I also tested with a development board, eth0 seemed to
> > > appear, but couldn't configure the MAC address with
> > > busybox ifconfig. How should it be done, I guess the
> > > interface does not have any MAC by default?
> > 
> > You need to write the eeprom with ethtool from Linux,
> > something like this:
> > 
> > Run u-boot/tools/gen_eth_addr to generate a random local
> > mac, then swap the bytes for it for big endian. Enter them
> > into a file with hexedit in big endian order. Then just do:
> > 
> > # cat mac | ethtool -E eth0 offset 0x40 length 6
> > 
> > Then ethtool -e eth0 should show you the configuration.
> 
> Oh and I have some u-boot patches that I'll post that allow
> booting n900 with bootz and to use smsc91x tftp booting. I'll
> try to post those shortly..
> 
> Regards,
> 
> Tony

N900 uboot support is broken, see my email on uboot ML with 
bisected commits:

http://lists.denx.de/pipermail/u-boot/2015-January/200154.html

-- 
Pali Rohár
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux