linux-kernel - RE: [PATCH V2 3/3] ARM: dts: imx6ull-dhcor: Add Marantec maveo box

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: <a7fcfe695623491da96639079eb14c8f@dh-electronics.com>
Date:   Thu, 6 Apr 2023 17:54:10 +0000
From:   Christoph Niedermaier <cniedermaier@...electronics.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>
CC:     Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Shawn Guo <shawnguo@...nel.org>, Li Yang <leoyang.li@....com>,
        Marek Vasut <marex@...x.de>, Fabio Estevam <festevam@...x.de>,
        NXP Linux Team <linux-imx@....com>,
        kernel <kernel@...electronics.com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH V2 3/3] ARM: dts: imx6ull-dhcor: Add Marantec maveo box

From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@...aro.org]
Sent: Thursday, April 6, 2023 6:40 PM
> On 06/04/2023 17:49, Christoph Niedermaier wrote:
>> Add support for Marantec maveo box. The system is used to get a
>> smart conntection to a door drive. It has USB, WiFi, Bluetooth,
>> Zigbee and NFC interfaces. The core of this system is a soldered
>> i.MX6ULL DHCOR SoM from DH electronics.
>>
>> Signed-off-by: Christoph Niedermaier <cniedermaier@...electronics.com>
>> ---
>> Cc: Rob Herring <robh+dt@...nel.org>
>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
>> Cc: Shawn Guo <shawnguo@...nel.org>
>> Cc: Li Yang <leoyang.li@....com>
>> Cc: Marek Vasut <marex@...x.de>
>> Cc: Fabio Estevam <festevam@...x.de>
>> Cc: NXP Linux Team <linux-imx@....com>
>> Cc: kernel@...electronics.com
>> Cc: devicetree@...r.kernel.org
>> Cc: linux-kernel@...r.kernel.org
>> To: linux-arm-kernel@...ts.infradead.org
>> ---
>> V2: - Correct the vendor prefix
>>     - Change pinconfig of User and Reset button
>> ---
>>  arch/arm/boot/dts/Makefile                    |   1 +
>>  arch/arm/boot/dts/imx6ull-dhcor-maveo-box.dts | 361 ++++++++++++++++++++++++++
>>  2 files changed, 362 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/imx6ull-dhcor-maveo-box.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index aa8fb4b7cdcc..58d3334164e2 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -751,6 +751,7 @@ dtb-$(CONFIG_SOC_IMX6UL) += \
>>       imx6ull-dhcom-drc02.dtb \
>>       imx6ull-dhcom-pdk2.dtb \
>>       imx6ull-dhcom-picoitx.dtb \
>> +     imx6ull-dhcor-maveo-box.dtb \
>>       imx6ull-jozacp.dtb \
>>       imx6ull-kontron-bl.dtb \
>>       imx6ull-myir-mys-6ulx-eval.dtb \
>> diff --git a/arch/arm/boot/dts/imx6ull-dhcor-maveo-box.dts b/arch/arm/boot/dts/imx6ull-dhcor-maveo-box.dts
>> new file mode 100644
>> index 000000000000..83bacac19933
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/imx6ull-dhcor-maveo-box.dts
>> @@ -0,0 +1,361 @@
>> +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT
>> +/*
>> + * Copyright (C) 2023 DH electronics GmbH
>> + * Copyright (C) 2023 Marantec electronics GmbH
>> + *
>> + * DHCOM iMX6ULL variant:
>> + * DHCR-iMX6ULL-C080-R051-SPI-WBT-I-01LG
>> + * DHCOR PCB number: 578-200 or newer
>> + * maveo box PCB number: 525-200 or newer
>> + */
>> +
>> +/dts-v1/;
>> +
>> +#include "imx6ull-dhcor-som.dtsi"
>> +
>> +/ {
>> +     model = "DH electronics i.MX6ULL DHCOR on maveo box";
>> +     compatible = "marantec,imx6ull-dhcor-maveo-box", "dh,imx6ull-dhcor-som",
>> +                  "fsl,imx6ull";
>> +
>> +     aliases {
>> +             /delete-property/ mmc0; /* Avoid double definitions */
> 
> I don't understand it. What is "double definition" of aliases?

Otherwise I end up like this:
mmc0 = &usdhc1;
mmc1 = &usdhc2;
mmc2 = &usdhc2;

Is "Ensure unique allocation" a better comment here?

> 
>> +             /delete-property/ mmc1;
>> +             mmc2 = &usdhc2; /* eMMC should be mmc2 */
> 
> Why? How is this labeled on the board (physically or on schematics)? If
> you answer here "for booting", then the answer is NAK. Don't add
> software policies to Devicetree.

The name in the schematics is "SD2".


Regards
Christoph