On Wed, May 19, 2010 at 1:01 PM, Martin Michlmayr <t...@cyrius.com> wrote: > To me, this patch looks fine with one exception. I wonder if this is > the right thing to do: > > | $SETENV bootargs_root "root=/dev/sda2 rw initrd=0x00800040" > > I know we have to modify the initrd= parameter because the default one > will not work. However, afaict, Tim originally suggested to remove > the whole initrd= parameter; see > http://lists.debian.org/debian-arm/2008/08/msg00036.html
I thought I'd read that entire thread, but as you can see the form I used in the patch is from an older message that Tim quotes. My Linkstations work fine (on stock firmware as well) with the initrd= parameter removed entirely, so let's go with that. Reading that message again reminded me that I did not implement a test for the bootloader version. A note in the documentation recommending the latest firmware revision might suffice, but I'll also have a look at how the u-boot version might be detected from userspace. > Also, from Tim's message above, it seems that you've removed panic=5 > from the command line. Any reason why? A typo on my part; thanks for catching it. So indeed that line should read: $SETENV bootargs_root "root=/dev/sda2 rw panic=5" precisely as in Tim's email. Thanks, Ryan -- To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/aanlktinsdxvx1ih-j6vsg3rffs5teibdmzbk8inwe...@mail.gmail.com
