2021-01-31 14:26:48

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH 0/2] Add MediaTek MT8192 clock provider device nodes



On 22/12/2020 14:40, Weiyi Lu wrote:
> This series is based on v5.10-rc1, MT8192 dts v6[1] and
> MT8192 clock v6 series[2].
>
> [1] https://patchwork.kernel.org/project/linux-mediatek/list/?series=373899
> [2] https://patchwork.kernel.org/project/linux-mediatek/list/?series=405295
>

[1] is already mainline. You could add this patch as a new one to [2]. But
please try to improve the series, before sending just a new version with this
patch added.

Regards,
Matthias

> Weiyi Lu (2):
> arm64: dts: mediatek: Add mt8192 clock controllers
> arm64: dts: mediatek: Correct UART0 bus clock of MT8192
>
> arch/arm64/boot/dts/mediatek/mt8192.dtsi | 165 ++++++++++++++++++++++-
> 1 file changed, 164 insertions(+), 1 deletion(-)
>


2021-02-01 09:33:38

by Weiyi Lu

[permalink] [raw]
Subject: Re: [PATCH 0/2] Add MediaTek MT8192 clock provider device nodes

On Sun, 2021-01-31 at 14:27 +0100, Matthias Brugger wrote:
>
> On 22/12/2020 14:40, Weiyi Lu wrote:
> > This series is based on v5.10-rc1, MT8192 dts v6[1] and
> > MT8192 clock v6 series[2].
> >
> > [1] https://patchwork.kernel.org/project/linux-mediatek/list/?series=373899
> > [2] https://patchwork.kernel.org/project/linux-mediatek/list/?series=405295
> >
>
> [1] is already mainline. You could add this patch as a new one to [2]. But
> please try to improve the series, before sending just a new version with this
> patch added.
>
> Regards,
> Matthias
>
Hi Matthias,

Actually I'm a little confused now. Stephen suggested me to send clock
dts separately because dts may not go through his tree.
So I separated it from the MT8192 clock series since clock v6.
What do you suggest me to do next time?

> > Weiyi Lu (2):
> > arm64: dts: mediatek: Add mt8192 clock controllers
> > arm64: dts: mediatek: Correct UART0 bus clock of MT8192
> >
> > arch/arm64/boot/dts/mediatek/mt8192.dtsi | 165 ++++++++++++++++++++++-
> > 1 file changed, 164 insertions(+), 1 deletion(-)
> >

2021-02-01 10:16:49

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH 0/2] Add MediaTek MT8192 clock provider device nodes

Hi Weiyi,

On 01/02/2021 10:31, Weiyi Lu wrote:
> On Sun, 2021-01-31 at 14:27 +0100, Matthias Brugger wrote:
>>
>> On 22/12/2020 14:40, Weiyi Lu wrote:
>>> This series is based on v5.10-rc1, MT8192 dts v6[1] and
>>> MT8192 clock v6 series[2].
>>>
>>> [1] https://patchwork.kernel.org/project/linux-mediatek/list/?series=373899
>>> [2] https://patchwork.kernel.org/project/linux-mediatek/list/?series=405295
>>>
>>
>> [1] is already mainline. You could add this patch as a new one to [2]. But
>> please try to improve the series, before sending just a new version with this
>> patch added.
>>
>> Regards,
>> Matthias
>>
> Hi Matthias,
>
> Actually I'm a little confused now. Stephen suggested me to send clock
> dts separately because dts may not go through his tree.
> So I separated it from the MT8192 clock series since clock v6.
> What do you suggest me to do next time?
>

Yes, now that you mention that, I remember...
OK, then I'd propose to resend the DTS patches once the clock patches are accepted.

Regards,
Matthias

>>> Weiyi Lu (2):
>>> arm64: dts: mediatek: Add mt8192 clock controllers
>>> arm64: dts: mediatek: Correct UART0 bus clock of MT8192
>>>
>>> arch/arm64/boot/dts/mediatek/mt8192.dtsi | 165 ++++++++++++++++++++++-
>>> 1 file changed, 164 insertions(+), 1 deletion(-)
>>>
>