Re: [PATCH V2] d-i hd-media support for armhf

2014-11-13 Thread Ian Campbell
On Fri, 2014-10-24 at 21:39 +0100, Ian Campbell wrote: > On Sat, 2014-10-11 at 08:47 +0100, Ian Campbell wrote: > > On Wed, 2014-10-01 at 00:47 +0200, Karsten Merker wrote: > > > +.PHONY: hd-media_dtbs > > > +hd-media_dtbs: $(TEMP_DTBS) > > > + mkdir -p $(SOME_DEST)/$(EXTRANAME)dtbs > > > +

Re: [PATCH V2] d-i hd-media support for armhf

2014-11-03 Thread Stefan Monnier
> Unless console=ttyS0,115200,115200 happens to be safe, but I don't know > that I would suggest relying on that. What about "console=tty0,115200"? I think adding ",115200" is a bad idea trying to paper over a mis-configured u-boot env. Better say no and let people fix their config. Ste

Re: [PATCH V2] d-i hd-media support for armhf

2014-11-03 Thread Ian Campbell
On Sun, 2014-11-02 at 15:33 +0100, Karsten Merker wrote: > On Sat, Nov 01, 2014 at 12:06:55AM -0700, Vagrant Cascadian wrote: > > [d-i hd-media support for armhf, boot script] > > Overall, it appears to be working quite well. I've thought about > > creating a similar bootscript for the netboot ima

Re: [PATCH V2] d-i hd-media support for armhf

2014-11-02 Thread Karsten Merker
On Sat, Nov 01, 2014 at 12:06:55AM -0700, Vagrant Cascadian wrote: [d-i hd-media support for armhf, boot script] > Overall, it appears to be working quite well. I've thought about > creating a similar bootscript for the netboot images. > > > +if test -n "${console}"; then > > + setenv bootargs "

Re: [PATCH V2] d-i hd-media support for armhf

2014-11-01 Thread Vagrant Cascadian
On 2014-10-01, Karsten Merker wrote: > On Wed, Oct 01, 2014 at 07:01:44AM +0200, Cyril Brulebois wrote: >> Karsten Merker (2014-10-01): > attached is V3 of the patch. I have run a full armhf installer > build with it without problems and I have done two test installs > on a Cubietruck with the re

Re: [PATCH V2] d-i hd-media support for armhf

2014-10-24 Thread Ian Campbell
On Sat, 2014-10-11 at 08:47 +0100, Ian Campbell wrote: > On Wed, 2014-10-01 at 00:47 +0200, Karsten Merker wrote: > > +.PHONY: hd-media_dtbs > > +hd-media_dtbs: $(TEMP_DTBS) > > + mkdir -p $(SOME_DEST)/$(EXTRANAME)dtbs > > + set -ex ; for dtb in $(TEMP_DTBS)/*.dtb ; do \ > > +

Re: [PATCH V2] d-i hd-media support for armhf

2014-10-11 Thread Ian Campbell
On Wed, 2014-10-01 at 00:47 +0200, Karsten Merker wrote: > +.PHONY: hd-media_dtbs > +hd-media_dtbs: $(TEMP_DTBS) > + mkdir -p $(SOME_DEST)/$(EXTRANAME)dtbs > + set -ex ; for dtb in $(TEMP_DTBS)/*.dtb ; do \ > + tgt=$(SOME_DEST)/$(EXTRANAME)dtbs/$$(basename $$dtb); \ > +

Re: [PATCH V2] d-i hd-media support for armhf

2014-09-30 Thread Cyril Brulebois
Karsten Merker (2014-10-01): > Vagrant is right, the script fragment added in V2 for conditionally > setting the console parameter is the wrong way round. I am very > much in a hurry now to catch a train and don't want to risk > breaking anything by rushing an untested last-minute change into > t

Re: [PATCH V2] d-i hd-media support for armhf

2014-09-30 Thread Cyril Brulebois
Cyril Brulebois (2014-10-01): > Karsten Merker (2014-10-01): > > On Mon, Sep 22, 2014 at 12:17:23AM +0200, Karsten Merker wrote: > > > > > I have started working on implementing hd-media support for the > > > armhf platform in debian-installer. > > [snip] > > > I have run some tests with the res

Re: [PATCH V2] d-i hd-media support for armhf

2014-09-30 Thread Vagrant Cascadian
On 2014-09-30 15:47:13 -0700, Karsten Merker wrote: > * Add the "console=" parameter to bootargs only if ${console} > is set. ... > diff --git a/build/boot/arm/bootscr.mainline_common > b/build/boot/arm/bootscr.mainline_common ... > +if test -z "${console}"; then > + setenv bootargs "${boot

Re: [PATCH V2] d-i hd-media support for armhf

2014-09-30 Thread Cyril Brulebois
Karsten Merker (2014-10-01): > On Mon, Sep 22, 2014 at 12:17:23AM +0200, Karsten Merker wrote: > > > I have started working on implementing hd-media support for the > > armhf platform in debian-installer. > [snip] > > I have run some tests with the resulting tarball contents and a > > Jesse CD1 i