Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-04-01 Thread Prafulla Wadaskar
> -Original Message- > From: Michael Walle [mailto:mich...@walle.cc] > Sent: 30 March 2012 19:27 > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Wolfgang Denk > Subject: Re: [PATCH v2] lsxl: add support for lschlv2 and lsxhl > > > Hi Prafulla, > > [ I'm CCing Wolfgang on this, be

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-30 Thread Michael Walle
Hi Prafulla, [ I'm CCing Wolfgang on this, because i don't know who's responsible for the build/make infrastructure. ] > AFAIK, you need to pass only "u-boot.kwb" to the make (no $(okj) needed) Sorry you're wrong here, that won't work for out-of-tree builds, see below. [mw@thanatos b-u-boot]$

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-30 Thread Prafulla Wadaskar
> -Original Message- > From: Michael Walle [mailto:mich...@walle.cc] > Sent: 29 March 2012 20:09 > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de > Subject: Re: [PATCH v2] lsxl: add support for lschlv2 and lsxhl > > Am Donnerstag 29 März 2012, 16:24:23 schrieb Prafulla Wadaskar: > > >

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-29 Thread Michael Walle
Am Donnerstag 29 März 2012, 16:24:23 schrieb Prafulla Wadaskar: > > -Original Message- > > From: Michael Walle [mailto:mich...@walle.cc] > > Sent: 29 March 2012 18:08 > > To: Prafulla Wadaskar > > Cc: u-boot@lists.denx.de > > Subject: Re: [PATCH v2] lsxl: add support for lschlv2 and lsxhl >

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-29 Thread Prafulla Wadaskar
> -Original Message- > From: Michael Walle [mailto:mich...@walle.cc] > Sent: 29 March 2012 18:08 > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de > Subject: Re: [PATCH v2] lsxl: add support for lschlv2 and lsxhl > > Am Dienstag 27 März 2012, 09:27:49 schrieb Prafulla Wadaskar: > > > #

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-29 Thread Michael Walle
Am Dienstag 27 März 2012, 09:27:49 schrieb Prafulla Wadaskar: > > # diff --git a/board/buffalo/lsxl/config.mk > > b/board/buffalo/lsxl/config.mk new file mode 100644 > > index 000..062554b > > --- /dev/null > > +++ b/board/buffalo/lsxl/config.mk > > @@ -0,0 +1 @@ > > +ALL-y += $(obj)u-boot.kwb

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-29 Thread Michael Walle
Hi Wolfgang, > I still fail to understand why you would need fixed network settings. > Evenin emergency cases, the only parameter you really need is a MAC > address, all others can be fetched for example from a DHCP server. bootp/dhcp should work for getting the ip address. My initial thought was

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Wolfgang Denk
Dear Michael, In message <201203272255.59259.mich...@walle.cc> you wrote: > > Wolfgang, what do you think about relaxing this policy a bit and allowing > this > class of devices (no service port/serial port available, no storage for the > MAC address besides the environment) to use hardcoded v

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Michael Walle
Hi, Am Dienstag 27 März 2012, 13:05:05 schrieb Prafulla Wadaskar: > > I removed the hardcoded values from the environment and put it into a > > special rescue mode, so it won't show up until the user is explicitly > > choosing that mode. I can understand, that hardcoded values are bad > > but in >

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Michael Walle
Am Dienstag 27 März 2012, 09:27:49 schrieb Prafulla Wadaskar: > Is the chip 6281/6282 or 6192? It' a 88F6281-A1. -- Michael ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Prafulla Wadaskar
> -Original Message- > From: Michael Walle [mailto:mich...@walle.cc] > Sent: 27 March 2012 16:22 > To: Prafulla Wadaskar > Cc: Michael Walle; u-boot@lists.denx.de > Subject: RE: [PATCH v2] lsxl: add support for lschlv2 and lsxhl > > Hi Prafulla, > > thanks for the review. > > >> +++ b/

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Michael Walle
Hi Prafulla, thanks for the review. >> +++ b/board/buffalo/lsxl/kwbimage-lsxhl.cfg > BTW: What is difference between them? if it is very small you can manage > it through c file. mh? arent these the values which end up in the head of an SPI image, the SoC bootstrap rom will execute? how can thes

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Prafulla Wadaskar
> -Original Message- > From: Michael Walle [mailto:mich...@walle.cc] > Sent: 26 March 2012 15:46 > To: u-boot@lists.denx.de > Cc: Prafulla Wadaskar; Wolfgang Denk > Subject: Re: [PATCH v2] lsxl: add support for lschlv2 and lsxhl > > On Fri, March 2, 2012 17:02, Michael Walle wrote: > > T

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-27 Thread Prafulla Wadaskar
> Please use "Kirkwood: add xxx board support" as a patch heading to identify the patch for kirkwood > This patchset adds support for both the Linkstation Live (LS-CHLv2) and > Linkstation Pro (LS-XHL) by Buffalo. > > Signed-off-by: Michael Walle walle.cc> > Cc: Prafulla Wadaskar marvell.com>

Re: [U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-26 Thread Michael Walle
On Fri, March 2, 2012 17:02, Michael Walle wrote: > This patchset adds support for both the Linkstation Live (LS-CHLv2) and > Linkstation Pro (LS-XHL) by Buffalo. > > Signed-off-by: Michael Walle > Cc: Prafulla Wadaskar > Cc: Wolfgang Denk > --- > > v2: > - add to buffalo vendor directory inste

[U-Boot] [PATCH v2] lsxl: add support for lschlv2 and lsxhl

2012-03-02 Thread Michael Walle
This patchset adds support for both the Linkstation Live (LS-CHLv2) and Linkstation Pro (LS-XHL) by Buffalo. Signed-off-by: Michael Walle Cc: Prafulla Wadaskar Cc: Wolfgang Denk --- v2: - add to buffalo vendor directory instead of Marvell - add both boards to MAINTAINERS - don't define valu