[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5758c2af-c2c5-dfbe-c7d8-036bbdaf71c7@kernel.org>
Date: Wed, 9 Nov 2022 10:04:09 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Matthias Brugger <matthias.bgg@...il.com>,
Allen-KH Cheng <allen-kh.cheng@...iatek.com>,
Wim Van Sebroeck <wim@...ux-watchdog.org>,
Guenter Roeck <linux@...ck-us.net>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Rob Herring <robh+dt@...nel.org>
Cc: nfraprado@...labora.com, angelogioacchino.delregno@...labora.com,
Project_Global_Chrome_Upstream_Group@...iatek.com,
devicetree@...r.kernel.org, linux-watchdog@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH v5 5/7] dt-bindings: watchdog: mediatek: Convert mtk-wdt
to json-schema
On 08/11/2022 11:50, Matthias Brugger wrote:
>
>
> On 08/11/2022 04:32, Allen-KH Cheng wrote:
>> From: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
>>
>> Convert the MediaTek watchdog bindings to schema.
>>
>> The original binding only had 4 without a fallback but there is a reset
>> controller on the "mediatek,mt7986-wdt", "mediatek,mt8186-wdt",
>> "mediatek,mt8188-wdt" and "mediatek,mt8195-wdt" Since there is no reset
>> controller for the mt6589, we remove "mediatek,mt6589-wdt" as a
>> fallback.
>>
>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
>> Co-developed-by: Allen-KH Cheng <allen-kh.cheng@...iatek.com>
>> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@...iatek.com>
>> Reviewed-by: Rob Herring <robh@...nel.org>
>> Reviewed-by: Guenter Roeck <linux@...ck-us.net>
>> Reviewed-by: NĂcolas F. R. A. Prado <nfraprado@...labora.com>
>
> As I'm put as the maintainer:
> Acked-by: Matthias Brugger <matthias.bgg@...il.com>
>
> Shall I take that through my tree or shall it go through the watchdog tree?
>
In general, bindings should go via subsystem trees (so watchdog), just
like drivers. However this got Guenter's review tag, so usually it means
also an ack... Dunno... :)
Best regards,
Krzysztof
Powered by blists - more mailing lists