* [RESEND PATCH] arm64: dts: mediatek: Fix "status" values
@ 2023-08-04 22:58 Rob Herring
2023-08-12 9:14 ` patchwork-bot+linux-soc
2023-09-12 12:54 ` AngeloGioacchino Del Regno
0 siblings, 2 replies; 5+ messages in thread
From: Rob Herring @ 2023-08-04 22:58 UTC (permalink / raw)
To: soc, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
AngeloGioacchino Del Regno
Cc: Alexandre Mergnat, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek
The defined value for "status" is "disabled", not "disable".
Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Rob Herring <robh@kernel.org>
---
Arnd, Please take this directly I guess. I think Mediatek maintainership
needs some help. Maybe AngeloGioacchino should be co-maintainer as
that's the only response I seem to be getting.
I think for future .dts patches I will not bother splitting them up by
sub-arch because it's a pain to chase down the maintainers to apply
stuff in a timely manner. /rant
arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
index e4605d23fdc8..86cedb0bf1a9 100644
--- a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
+++ b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
@@ -585,11 +585,11 @@ &pwrap {
};
&sata {
- status = "disable";
+ status = "disabled";
};
&sata_phy {
- status = "disable";
+ status = "disabled";
};
&spi0 {
--
2.40.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH] arm64: dts: mediatek: Fix "status" values
2023-08-04 22:58 [RESEND PATCH] arm64: dts: mediatek: Fix "status" values Rob Herring
@ 2023-08-12 9:14 ` patchwork-bot+linux-soc
2023-09-12 12:54 ` AngeloGioacchino Del Regno
1 sibling, 0 replies; 5+ messages in thread
From: patchwork-bot+linux-soc @ 2023-08-12 9:14 UTC (permalink / raw)
To: Rob Herring; +Cc: soc
Hello:
This patch was applied to soc/soc.git (for-next)
by Arnd Bergmann <arnd@arndb.de>:
On Fri, 4 Aug 2023 16:58:13 -0600 you wrote:
> The defined value for "status" is "disabled", not "disable".
>
> Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Arnd, Please take this directly I guess. I think Mediatek maintainership
> needs some help. Maybe AngeloGioacchino should be co-maintainer as
> that's the only response I seem to be getting.
>
> [...]
Here is the summary with links:
- [RESEND] arm64: dts: mediatek: Fix "status" values
https://git.kernel.org/soc/soc/c/c718f36c9bc3
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH] arm64: dts: mediatek: Fix "status" values
2023-08-04 22:58 [RESEND PATCH] arm64: dts: mediatek: Fix "status" values Rob Herring
2023-08-12 9:14 ` patchwork-bot+linux-soc
@ 2023-09-12 12:54 ` AngeloGioacchino Del Regno
2023-09-12 13:10 ` Alexandre Mergnat
1 sibling, 1 reply; 5+ messages in thread
From: AngeloGioacchino Del Regno @ 2023-09-12 12:54 UTC (permalink / raw)
To: Rob Herring, soc, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger
Cc: Alexandre Mergnat, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek, Nícolas F. R. A. Prado
Il 05/08/23 00:58, Rob Herring ha scritto:
> The defined value for "status" is "disabled", not "disable".
>
> Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Arnd, Please take this directly I guess. I think Mediatek maintainership
> needs some help. Maybe AngeloGioacchino should be co-maintainer as
> that's the only response I seem to be getting.
Sorry but I've seen this message just now as I've been on holiday in August.
Thing is, the MediaTek scene is starting to see more and more code, bringing
an obvious increase in the amount of reviews to be done and in the required
efforts to maintain the MTK bits - and we will possibly (hopefully) see even
more of that.
If Matthias needs/wants a co-maintainer for MediaTek I'm here and I will be
proud to become one.
Cheers,
Angelo
>
> I think for future .dts patches I will not bother splitting them up by
> sub-arch because it's a pain to chase down the maintainers to apply
> stuff in a timely manner. /rant
>
> arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> index e4605d23fdc8..86cedb0bf1a9 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> @@ -585,11 +585,11 @@ &pwrap {
> };
>
> &sata {
> - status = "disable";
> + status = "disabled";
> };
>
> &sata_phy {
> - status = "disable";
> + status = "disabled";
> };
>
> &spi0 {
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH] arm64: dts: mediatek: Fix "status" values
2023-09-12 12:54 ` AngeloGioacchino Del Regno
@ 2023-09-12 13:10 ` Alexandre Mergnat
2023-09-13 8:49 ` Chen-Yu Tsai
0 siblings, 1 reply; 5+ messages in thread
From: Alexandre Mergnat @ 2023-09-12 13:10 UTC (permalink / raw)
To: AngeloGioacchino Del Regno, Rob Herring, soc,
Krzysztof Kozlowski, Conor Dooley, Matthias Brugger
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Nícolas F. R. A. Prado
On 12/09/2023 14:54, AngeloGioacchino Del Regno wrote:
> Il 05/08/23 00:58, Rob Herring ha scritto:
>> The defined value for "status" is "disabled", not "disable".
>>
>> Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
>> Reviewed-by: AngeloGioacchino Del Regno
>> <angelogioacchino.delregno@collabora.com>
>> Signed-off-by: Rob Herring <robh@kernel.org>
>> ---
>> Arnd, Please take this directly I guess. I think Mediatek maintainership
>> needs some help. Maybe AngeloGioacchino should be co-maintainer as
>> that's the only response I seem to be getting.
>
> Sorry but I've seen this message just now as I've been on holiday in
> August.
>
> Thing is, the MediaTek scene is starting to see more and more code,
> bringing
> an obvious increase in the amount of reviews to be done and in the required
> efforts to maintain the MTK bits - and we will possibly (hopefully) see
> even
> more of that.
>
> If Matthias needs/wants a co-maintainer for MediaTek I'm here and I will be
> proud to become one.
>
> Cheers,
> Angelo
>
I started reviewing MediaTek patches for some months now, so I'm still
new. From my PoV, I'm agree with Angelo when he say the amount of patch
is increasing. I support Angelo if he wants to become a co-maintainer
and continue to do my best to review patches. ;)
>>
>> I think for future .dts patches I will not bother splitting them up by
>> sub-arch because it's a pain to chase down the maintainers to apply
>> stuff in a timely manner. /rant
>>
>> arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> index e4605d23fdc8..86cedb0bf1a9 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
>> @@ -585,11 +585,11 @@ &pwrap {
>> };
>> &sata {
>> - status = "disable";
>> + status = "disabled";
>> };
>> &sata_phy {
>> - status = "disable";
>> + status = "disabled";
>> };
>> &spi0 {
>
>
--
Regards,
Alexandre
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH] arm64: dts: mediatek: Fix "status" values
2023-09-12 13:10 ` Alexandre Mergnat
@ 2023-09-13 8:49 ` Chen-Yu Tsai
0 siblings, 0 replies; 5+ messages in thread
From: Chen-Yu Tsai @ 2023-09-13 8:49 UTC (permalink / raw)
To: Alexandre Mergnat
Cc: AngeloGioacchino Del Regno, Rob Herring, soc,
Krzysztof Kozlowski, Conor Dooley, Matthias Brugger, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek,
Nícolas F. R. A. Prado
On Tue, Sep 12, 2023 at 9:10 PM Alexandre Mergnat <amergnat@baylibre.com> wrote:
>
>
>
> On 12/09/2023 14:54, AngeloGioacchino Del Regno wrote:
> > Il 05/08/23 00:58, Rob Herring ha scritto:
> >> The defined value for "status" is "disabled", not "disable".
> >>
> >> Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
> >> Reviewed-by: AngeloGioacchino Del Regno
> >> <angelogioacchino.delregno@collabora.com>
> >> Signed-off-by: Rob Herring <robh@kernel.org>
> >> ---
> >> Arnd, Please take this directly I guess. I think Mediatek maintainership
> >> needs some help. Maybe AngeloGioacchino should be co-maintainer as
> >> that's the only response I seem to be getting.
> >
> > Sorry but I've seen this message just now as I've been on holiday in
> > August.
> >
> > Thing is, the MediaTek scene is starting to see more and more code,
> > bringing
> > an obvious increase in the amount of reviews to be done and in the required
> > efforts to maintain the MTK bits - and we will possibly (hopefully) see
> > even
> > more of that.
> >
> > If Matthias needs/wants a co-maintainer for MediaTek I'm here and I will be
> > proud to become one.
> >
> > Cheers,
> > Angelo
> >
>
> I started reviewing MediaTek patches for some months now, so I'm still
> new. From my PoV, I'm agree with Angelo when he say the amount of patch
> is increasing. I support Angelo if he wants to become a co-maintainer
> and continue to do my best to review patches. ;)
I might not have enough time to do a lot of reviews, but if help is needed
for administrative tasks like queueing patches and sending pull requests,
that's something I can do.
ChenYu
> >>
> >> I think for future .dts patches I will not bother splitting them up by
> >> sub-arch because it's a pain to chase down the maintainers to apply
> >> stuff in a timely manner. /rant
> >>
> >> arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts | 4 ++--
> >> 1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> >> b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> >> index e4605d23fdc8..86cedb0bf1a9 100644
> >> --- a/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> >> +++ b/arch/arm64/boot/dts/mediatek/mt7622-bananapi-bpi-r64.dts
> >> @@ -585,11 +585,11 @@ &pwrap {
> >> };
> >> &sata {
> >> - status = "disable";
> >> + status = "disabled";
> >> };
> >> &sata_phy {
> >> - status = "disable";
> >> + status = "disabled";
> >> };
> >> &spi0 {
> >
> >
>
> --
> Regards,
> Alexandre
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-09-13 8:49 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-04 22:58 [RESEND PATCH] arm64: dts: mediatek: Fix "status" values Rob Herring
2023-08-12 9:14 ` patchwork-bot+linux-soc
2023-09-12 12:54 ` AngeloGioacchino Del Regno
2023-09-12 13:10 ` Alexandre Mergnat
2023-09-13 8:49 ` Chen-Yu Tsai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).