* Ryan Tandy <tarp...@gmail.com> [2010-05-07 16:17]: > The attached patch adds a target to debian-installer to build a kernel > image and ramdisk for the Linkstation Pro/Live platform, similar to the > Kurobox Pro which is already supported.
I've copied Tim Small and John with whom I've previously had conversations about the LS Pro. Since I don't know a lot about the LS and don't have such a device, I hope they can help with the code review. 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 In any case, I'm fine with leaving it there as long as it will work both with Debian and the original LS firmware. Also, from Tim's message above, it seems that you've removed panic=5 from the command line. Any reason why? -- Martin Michlmayr http://www.cyrius.com/ -- 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/20100519200158.gf1...@jirafa.cyrius.com
