On Tue, Oct 3, 2017 at 2:54 PM, Rob Clark <robdclark@xxxxxxxxx> wrote: > On Tue, Oct 3, 2017 at 1:49 PM, Nicolas Dechesne > <nicolas.dechesne@xxxxxxxxxx> wrote: >> On Tue, Oct 3, 2017 at 6:37 PM, Rob Clark <robdclark@xxxxxxxxx> wrote: >>> On Tue, Oct 3, 2017 at 8:25 AM, Srinivas Kandagatla >>> <srinivas.kandagatla@xxxxxxxxxx> wrote: >>>> >>>> >>>> On 03/10/17 10:11, Linus Walleij wrote: >>>>> >>>>> Naming the DragonBoard 410c "SBC" (I guess "single board computer") >>>>> is not very helpful for users looking for their device tree in >>>>> the kernel. Also the db820c is named properly. >>>>> >>>>> Rectify this, simply, do not change the compatible strings but name >>>>> the DTS files in a consistent manner. >>>>> >>>>> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> >>>> >>>> I did not like the sbc naming convention too, This was primarily done due to >>>> some legacy build tools + bootloaders which have used sbc compatible to >>>> identify boards from multiple dtb blobs. Now that we can append dtb to >>>> kernel with new LK, this should not be a issue anymore. >>> >>> maybe it would be an idea to update skales/dbttool first to look for >>> the new name, and then give people some time to upgrade, before >>> changing this in the kernel. Otherwise I expect you'll cause some >>> confusion.. >> >> iirc, skale is using the compatible name, not the filename. >> > > ahh, yes, I think you are right.. u-boot is using a different filename. > > So in this case, no issue to pick a saner name :-) > So, actually u-boot encodes the board's fdtname.. so as much as I'd like to rename it to something more sane, unless the kernel can manage to install a symlink for backwards compat, NAK I was trying to think of hacks for u-boot to check for multiple fdt names, but if we want to get to the point of being able to install multiple distro's on a single disk, we actually need to pass fdtname to grub (where distro specific grub.cfg is encoding the UUID of the distro's /boot partition, so it knows where to load the dtb from).. this gets nearly impossible when one name becomes multiple names. So hurray for the fdt name becoming accidental ABI! If we managed to get this in before the 4.14 LTS I might feel differently. But afterwards, totally not. :-( BR, -R -- To unsubscribe from this list: send the line "unsubscribe linux-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
