On Fri Dec 20, 2024 at 10:36 AM CET, Konrad Dybcio wrote: > On 20.12.2024 9:55 AM, Luca Weiss wrote: > > The ID 434 is for SM6350 while 459 is for SM7225. Fairphone 4 is only > > SM7225, so drop the unused 434 entry. > > > > Fixes: 4cbea668767d ("arm64: dts: qcom: sm7225: Add device tree for Fairphone 4") > > Signed-off-by: Luca Weiss <luca.weiss@xxxxxxxxxxxxx> > > --- > > Completely dropping these values also appears to work when using > > boot.img header version 2, but doing this change now would mean any > > tooling in the wild for packaging up the boot.img using header v0 would > > stop producing working without adjusting that mkbootimg invocation. > > --- > > It would be nice if you could start shifting that tooling to bootimgv2 > slowly, and when it trickles down to appropriate places, switch it up > upstream as well. Of course if that works for you. In postmarketOS for sure it's simple to switch, I know Mobian also has something for FP4, but apart from that I'm not sure if there's more build scripts floating around somewhere. > > Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx> > > Konrad