<div dir="ltr">control: severity -1 normal<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 14, 2016 at 7:42 AM, karl156 <span dir="ltr"><<a href="mailto:karl156@abwesend.de" target="_blank">karl156@abwesend.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Package: raspi3-firmware<br>
Version: 1.20161123-2<br>
Severity: important<br>
Tags: sid<br>
<br>
Thanks for packaging the raspi firmware.<br>
<br>
As this package basically only contains the blobs which are working on<br>
all Raspberry Pis (1-3), wouldn't it be more appropriate to name this<br>
package "raspi-firmware"? (And make it available on armhf too.)<br></blockquote><div><br></div><div>The reason why this package is targeted at the Raspberry Pi 3 and above is that only the Raspberry Pi 3 in ARM64 mode is supported by kernels which are included in Debian. All other Raspberry Pi models require custom kernels from the Raspberry Pi foundation (<a href="https://github.com/raspberrypi/linux/">https://github.com/raspberrypi/linux/</a>).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
Also config.txt & cmdline.txt should be treated as conffiles so that<br>
changes the user has made (e.g. special hdmi parameters or codec<br>
licenses) won't get overridden on every kernel update.<br></blockquote><div><br></div><div>I was thinking we could implement something similar to how Raspbian et al. manage their configs, i.e. using a config-available/config-enabled like system where users don’t need to modify conffiles directly at all.</div><div><br></div><div>What do you think about that suggestion?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I would suggest to place them somewhere in /etc instead of<br>
auto-generating them in the kernel postinst script and just copy them to<br>
the firmware folder in the kernel postinst script.<br>
If you copy the kernel, initramfs + dtb to fixed positions (like the<br>
"/vmlinuz" & "/initrd.img" symlinks) in the kernel postinst script then<br>
config.txt & cmdline.txt don't need to be touched on every update.<br>
Another idea would be to only write them if they don't exist yet.<br>
<br>
I've put these two issues together because without a customizable<br>
config.txt the package indeed only works on a raspi3 because of the<br>
"arm_control=0x200" & "device_tree=bcm2837-rpi-3-b.<wbr>dtb" lines.<br>
<br>
Severity set to important, cause all user changes to the config files<br>
are lost on every update, rendering this package useless for many users.<br></blockquote><div><br></div><div>I’m downgrading this to normal. You are correct in that the experience is not ideal for some users, but at the same time, there are other gaping holes to fix in Raspberry Pi support in Debian, see <a href="https://wiki.debian.org/RaspberryPi3">https://wiki.debian.org/RaspberryPi3</a> (help very welcome!). These gaps make me believe the raspi3-firmware package doesn’t have any serious users yet, so I’d prefer to take some time to think about the best way to manage these configuration files. </div></div><div><br></div>-- <br><div class="gmail_signature">Best regards,<br>Michael</div>
</div></div>