Re: [PATCH 2/2] arm64: dts: qcom: Rename SBC to DragonBoard 410c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 :-)

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



[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux