linux-kernel - Re: [PATCH v1] arm64: dts: mt7986: add overlay for SATA power socket on BPI-R3

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <76060BF0-B432-4BB8-A5C8-39EFF3D628EA@public-files.de>
Date:   Mon, 11 Sep 2023 22:10:46 +0200
From:   Frank Wunderlich <frank-w@...lic-files.de>
To:     Rob Herring <robh+dt@...nel.org>,
        Frank Wunderlich <linux@...web.de>
CC:     linux-mediatek@...ts.infradead.org,
        Matthias Brugger <matthias.bgg@...il.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@...labora.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        devicetree@...r.kernel.org
Subject: Re: [PATCH v1] arm64: dts: mt7986: add overlay for SATA power socket on BPI-R3

Am 11. September 2023 21:51:42 MESZ schrieb Rob Herring <robh+dt@...nel.org>:
>On Fri, Sep 1, 2023 at 2:27 AM Frank Wunderlich <linux@...web.de> wrote:
>>
>> From: Frank Wunderlich <frank-w@...lic-files.de>
>>
>> Bananapi R3 has a Power socket entended for using external SATA drives.
>> This Socket is off by default but can be switched with gpio 8.
>>
>> Add an overlay to activate it.
>>
>> Signed-off-by: Frank Wunderlich <frank-w@...lic-files.de>
>> ---
>>  arch/arm64/boot/dts/mediatek/Makefile         |  1 +
>>  .../mt7986a-bananapi-bpi-r3-sata.dtso         | 39 +++++++++++++++++++
>>  2 files changed, 40 insertions(+)
>>  create mode 100644 arch/arm64/boot/dts/mediatek/mt7986a-bananapi-bpi-r3-sata.dtso
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile
>> index c99c3372a4b5..822d3e36d3df 100644
>> --- a/arch/arm64/boot/dts/mediatek/Makefile
>> +++ b/arch/arm64/boot/dts/mediatek/Makefile
>> @@ -13,6 +13,7 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-emmc.dtbo
>>  dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-nand.dtbo
>>  dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-nor.dtbo
>>  dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-sd.dtbo
>> +dtb-$(CONFIG_ARCH_MEDIATEK) += mt7986a-bananapi-bpi-r3-sata.dtbo
>
>The requirement for overlays is they have a target base dt in tree and
>that you apply the overlay to it. All these existing overlays have the
>same problem which I pointed out when you submitted them. Please fix
>the existing ones before adding more.
>
>
>Rob

Hi Rob,

i do not understand the problem as there is a target base dt...mt7986a-bananapi-bpi-r3.dtb.

Do you mean that overlays should be merged with basedt at compiletime?

We pack the base-dt and all overlays into one fit image and let uboot dynamically select the combination to apply. E.g. check if emmc is available, if yes apply emmc overlay else the sd one,same for nand/nor.

Else i do not know *how* to fix as i don't see it as broken.

regards Frank