Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <30d3f753-187f-4a3f-c14a-f3b003f23773@gmail.com>
Date:   Mon, 5 Sep 2022 09:19:06 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.2.1
Subject: Re: [PATCH] ARM: dts: turris-omnia: Add mcu node
Content-Language: en-US
To:     =?UTF-8?Q?Pali_Roh=c3=a1r?= <pali@kernel.org>,
        Andrew Lunn <andrew@lunn.ch>
Cc:     Gregory Clement <gregory.clement@bootlin.com>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Marek Behun <marek.behun@nic.cz>,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20220819131152.6513-1-pali@kernel.org>
 <20220831142809.lcmnv3l4rnulo522@pali> <YxE2JqJutZ0ilghH@lunn.ch>
 <20220903152735.2ga3iwrivh6zhibf@pali>
From:   Florian Fainelli <f.fainelli@gmail.com>
In-Reply-To: <20220903152735.2ga3iwrivh6zhibf@pali>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 9/3/2022 8:27 AM, Pali Rohár wrote:
> On Friday 02 September 2022 00:45:58 Andrew Lunn wrote:
>> On Wed, Aug 31, 2022 at 04:28:09PM +0200, Pali Rohár wrote:
>>> PING?
>>>
>>> On Friday 19 August 2022 15:11:52 Pali Rohár wrote:
>>>> At i2c address 0x2a is MCU command interface which provides access to GPIOs
>>>> connected to Turris Omnia MCU. So define mcu node in Turris Omnia DTS file.
>>>>
>>>> Signed-off-by: Pali Rohár <pali@kernel.org>
>>>>
>>>> ---
>>>> Same change was already sent to U-Boot project together with driver. As
>>>> Turris Omnia DTS file is shared between Linux kernel U-Boot, I'm sending
>>>> this change also in Linux. There is a plan to write also Linux driver for
>>>> Turris Omnia MCU, like there is already in U-Boot.
>>>>
>>>> https://source.denx.de/u-boot/u-boot/-/commit/832738974806e6264a3d0ac2aaa92d0f662fd128
>>>> https://source.denx.de/u-boot/u-boot/-/blob/master/drivers/gpio/turris_omnia_mcu.c
>>>> ---
>>>>   arch/arm/boot/dts/armada-385-turris-omnia.dts | 8 +++++++-
>>>>   1 file changed, 7 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts
>>>> index f4878df39753..f655e9229d68 100644
>>>> --- a/arch/arm/boot/dts/armada-385-turris-omnia.dts
>>>> +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts
>>>> @@ -184,7 +184,13 @@
>>>>   			#size-cells = <0>;
>>>>   			reg = <0>;
>>>>   
>>>> -			/* STM32F0 command interface at address 0x2a */
>>>> +			/* MCU command i2c API */
>>>> +			mcu: mcu@2a {
>>>> +				compatible = "cznic,turris-omnia-mcu";
>>>> +				reg = <0x2a>;
>>>> +				gpio-controller;
>>>> +				#gpio-cells = <3>;
>>>> +			};
>>
>> Please document the binding, preferably in yaml.
> 
> I'm not going to send any new yaml dt binding document as I see that
> dt bindings is clearly deprecated project. Either patches for dt
> bindings are waiting without any answer for months (maybe year?) or
> patches are ignored/not accepted by beyond reasons or there are request
> for changes which cannot work on the real hardware or that new yaml
> cannot be parsed/validated due to ton of bugs in other schemas.

These are some pretty hard statements that are not nearly true, maybe 
they are based upon your past experience, but Rob and Krysztof provide 
feedback within a few days at most on DT bindings and DTS files.

> 
> Sorry, this is just a waste of time and energy to write new those yamls
> as it does not bring any value.

So you say.
-- 
Florian