devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188
@ 2022-08-18 12:46 Johnson Wang
  2022-08-18 12:51 ` Krzysztof Kozlowski
  2022-09-20  5:20 ` Johnson Wang
  0 siblings, 2 replies; 6+ messages in thread
From: Johnson Wang @ 2022-08-18 12:46 UTC (permalink / raw)
  To: daniel.lezcano, matthias.bgg, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-kernel, devicetree, linux-arm-kernel, linux-mediatek,
	Project_Global_Chrome_Upstream_Group, Johnson Wang

Add dt-binding documentation of timer for MediaTek MT8188 SoC
platform.

Signed-off-by: Johnson Wang <johnson.wang@mediatek.com>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
index f1c848af91d3..8bbb6e94508b 100644
--- a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
+++ b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
@@ -25,6 +25,7 @@ Required properties:
 	For those SoCs that use SYST
 	* "mediatek,mt8183-timer" for MT8183 compatible timers (SYST)
 	* "mediatek,mt8186-timer" for MT8186 compatible timers (SYST)
+	* "mediatek,mt8188-timer" for MT8188 compatible timers (SYST)
 	* "mediatek,mt8192-timer" for MT8192 compatible timers (SYST)
 	* "mediatek,mt8195-timer" for MT8195 compatible timers (SYST)
 	* "mediatek,mt7629-timer" for MT7629 compatible timers (SYST)
-- 
2.18.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188
  2022-08-18 12:46 [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188 Johnson Wang
@ 2022-08-18 12:51 ` Krzysztof Kozlowski
  2022-08-18 13:40   ` Krzysztof Kozlowski
  2022-09-20  5:20 ` Johnson Wang
  1 sibling, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-18 12:51 UTC (permalink / raw)
  To: Johnson Wang, daniel.lezcano, matthias.bgg, robh+dt,
	krzysztof.kozlowski+dt
  Cc: linux-kernel, devicetree, linux-arm-kernel, linux-mediatek,
	Project_Global_Chrome_Upstream_Group

On 18/08/2022 15:46, Johnson Wang wrote:
> Add dt-binding documentation of timer for MediaTek MT8188 SoC
> platform.
> 
> Signed-off-by: Johnson Wang <johnson.wang@mediatek.com>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> index f1c848af91d3..8bbb6e94508b 100644
> --- a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> +++ b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> @@ -25,6 +25,7 @@ Required properties:
>  	For those SoCs that use SYST
>  	* "mediatek,mt8183-timer" for MT8183 compatible timers (SYST)
>  	* "mediatek,mt8186-timer" for MT8186 compatible timers (SYST)
> +	* "mediatek,mt8188-timer" for MT8188 compatible timers (SYST)

Where is the driver change implementing this?

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188
  2022-08-18 12:51 ` Krzysztof Kozlowski
@ 2022-08-18 13:40   ` Krzysztof Kozlowski
  2022-08-19  1:46     ` Johnson Wang
  0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-18 13:40 UTC (permalink / raw)
  To: Johnson Wang, daniel.lezcano, matthias.bgg, robh+dt,
	krzysztof.kozlowski+dt
  Cc: linux-kernel, devicetree, linux-arm-kernel, linux-mediatek,
	Project_Global_Chrome_Upstream_Group

On 18/08/2022 15:51, Krzysztof Kozlowski wrote:
> On 18/08/2022 15:46, Johnson Wang wrote:
>> Add dt-binding documentation of timer for MediaTek MT8188 SoC
>> platform.
>>
>> Signed-off-by: Johnson Wang <johnson.wang@mediatek.com>
>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>>  Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
>> index f1c848af91d3..8bbb6e94508b 100644
>> --- a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
>> +++ b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
>> @@ -25,6 +25,7 @@ Required properties:
>>  	For those SoCs that use SYST
>>  	* "mediatek,mt8183-timer" for MT8183 compatible timers (SYST)
>>  	* "mediatek,mt8186-timer" for MT8186 compatible timers (SYST)
>> +	* "mediatek,mt8188-timer" for MT8188 compatible timers (SYST)
> 
> Where is the driver change implementing this?

Or is there a generic fallback and you just added a specific one (not
visible on the diff)?

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188
  2022-08-18 13:40   ` Krzysztof Kozlowski
@ 2022-08-19  1:46     ` Johnson Wang
  0 siblings, 0 replies; 6+ messages in thread
From: Johnson Wang @ 2022-08-19  1:46 UTC (permalink / raw)
  To: Krzysztof Kozlowski, daniel.lezcano, matthias.bgg, robh+dt,
	krzysztof.kozlowski+dt
  Cc: linux-kernel, devicetree, linux-arm-kernel, linux-mediatek,
	Project_Global_Chrome_Upstream_Group

On Thu, 2022-08-18 at 16:40 +0300, Krzysztof Kozlowski wrote:
> On 18/08/2022 15:51, Krzysztof Kozlowski wrote:
> > On 18/08/2022 15:46, Johnson Wang wrote:
> > > Add dt-binding documentation of timer for MediaTek MT8188 SoC
> > > platform.
> > > 
> > > Signed-off-by: Johnson Wang <johnson.wang@mediatek.com>
> > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > > ---
> > >  Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt |
> > > 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git
> > > a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> > > b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> > > index f1c848af91d3..8bbb6e94508b 100644
> > > --- a/Documentation/devicetree/bindings/timer/mediatek,mtk-
> > > timer.txt
> > > +++ b/Documentation/devicetree/bindings/timer/mediatek,mtk-
> > > timer.txt
> > > @@ -25,6 +25,7 @@ Required properties:
> > >  	For those SoCs that use SYST
> > >  	* "mediatek,mt8183-timer" for MT8183 compatible timers (SYST)
> > >  	* "mediatek,mt8186-timer" for MT8186 compatible timers (SYST)
> > > +	* "mediatek,mt8188-timer" for MT8188 compatible timers (SYST)
> > 
> > Where is the driver change implementing this?
> 
> Or is there a generic fallback and you just added a specific one (not
> visible on the diff)?
> 
> Best regards,
> Krzysztof

Hi Krzysztof,

Yes, MT8188 uses "mediatek,mt6765-timer" as fallback compatible.

BRs,
Johnson Wang


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188
  2022-08-18 12:46 [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188 Johnson Wang
  2022-08-18 12:51 ` Krzysztof Kozlowski
@ 2022-09-20  5:20 ` Johnson Wang
  2022-09-20  8:31   ` Daniel Lezcano
  1 sibling, 1 reply; 6+ messages in thread
From: Johnson Wang @ 2022-09-20  5:20 UTC (permalink / raw)
  To: daniel.lezcano, matthias.bgg, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-kernel, devicetree, linux-arm-kernel, linux-mediatek,
	Project_Global_Chrome_Upstream_Group

On Thu, 2022-08-18 at 20:46 +0800, Johnson Wang wrote:
> Add dt-binding documentation of timer for MediaTek MT8188 SoC
> platform.
> 
> Signed-off-by: Johnson Wang <johnson.wang@mediatek.com>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/timer/mediatek,mtk-
> timer.txt b/Documentation/devicetree/bindings/timer/mediatek,mtk-
> timer.txt
> index f1c848af91d3..8bbb6e94508b 100644
> --- a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> +++ b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> @@ -25,6 +25,7 @@ Required properties:
>  	For those SoCs that use SYST
>  	* "mediatek,mt8183-timer" for MT8183 compatible timers (SYST)
>  	* "mediatek,mt8186-timer" for MT8186 compatible timers (SYST)
> +	* "mediatek,mt8188-timer" for MT8188 compatible timers (SYST)
>  	* "mediatek,mt8192-timer" for MT8192 compatible timers (SYST)
>  	* "mediatek,mt8195-timer" for MT8195 compatible timers (SYST)
>  	* "mediatek,mt7629-timer" for MT7629 compatible timers (SYST)

Hi Daniel,

Just a gentle ping on this.
Could you please review this patch and give us some suggestion?

Thanks.


BRs,
Johnson Wang


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188
  2022-09-20  5:20 ` Johnson Wang
@ 2022-09-20  8:31   ` Daniel Lezcano
  0 siblings, 0 replies; 6+ messages in thread
From: Daniel Lezcano @ 2022-09-20  8:31 UTC (permalink / raw)
  To: Johnson Wang, matthias.bgg, robh+dt, krzysztof.kozlowski+dt
  Cc: linux-kernel, devicetree, linux-arm-kernel, linux-mediatek,
	Project_Global_Chrome_Upstream_Group

On 20/09/2022 07:20, Johnson Wang wrote:
> On Thu, 2022-08-18 at 20:46 +0800, Johnson Wang wrote:
>> Add dt-binding documentation of timer for MediaTek MT8188 SoC
>> platform.
>>
>> Signed-off-by: Johnson Wang <johnson.wang@mediatek.com>
>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>>   Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/timer/mediatek,mtk-
>> timer.txt b/Documentation/devicetree/bindings/timer/mediatek,mtk-
>> timer.txt
>> index f1c848af91d3..8bbb6e94508b 100644
>> --- a/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
>> +++ b/Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
>> @@ -25,6 +25,7 @@ Required properties:
>>   	For those SoCs that use SYST
>>   	* "mediatek,mt8183-timer" for MT8183 compatible timers (SYST)
>>   	* "mediatek,mt8186-timer" for MT8186 compatible timers (SYST)
>> +	* "mediatek,mt8188-timer" for MT8188 compatible timers (SYST)
>>   	* "mediatek,mt8192-timer" for MT8192 compatible timers (SYST)
>>   	* "mediatek,mt8195-timer" for MT8195 compatible timers (SYST)
>>   	* "mediatek,mt7629-timer" for MT7629 compatible timers (SYST)
> 
> Hi Daniel,
> 
> Just a gentle ping on this.
> Could you please review this patch and give us some suggestion?


I've applied the patch

Thanks

   -- D.


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-09-20  8:32 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-08-18 12:46 [PATCH RESEND] dt-bindings: timer: Add compatible for MediaTek MT8188 Johnson Wang
2022-08-18 12:51 ` Krzysztof Kozlowski
2022-08-18 13:40   ` Krzysztof Kozlowski
2022-08-19  1:46     ` Johnson Wang
2022-09-20  5:20 ` Johnson Wang
2022-09-20  8:31   ` Daniel Lezcano

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).