lkml.org 
[lkml]   [2023]   [Jan]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 04/11] dt-bindings: watchdog: convert meson-wdt.txt to dt-schema
On 10/01/2023 22:48, Martin Blumenstingl wrote:
> Hi Neil,
>
> On Mon, Jan 9, 2023 at 1:53 PM Neil Armstrong <neil.armstrong@linaro.org> wrote:
> [...]
>> + compatible:
>> + oneOf:
>> + - enum:
>> + - amlogic,meson6-wdt
>> + - amlogic,meson8-wdt
> Technically this is not an identical representation of what we had
> before which was:
> - "amlogic,meson8-wdt" and "amlogic,meson6-wdt" on Meson8 SoCs
>
> We're not using the "amlogic,meson8-wdt" compatible anywhere at the moment.
> In the meson_wdt driver it's defined with the same per-SoC data as
> "amlogic,meson6-wdt".
>
> Long story short: In my opinion there's no need to change what you
> have right now.
> If you have to re-spin this then maybe you can add a note to the patch
> description.
> Please add my:
> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

Sorry I was distracted, I'll add a node on the commit message.

Thanks,
Neil

>
>
> Best regards,
> Martin

\
 
 \ /
  Last update: 2023-03-26 23:52    [W:0.929 / U:0.192 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site