2024-06-12 16:43:19

by Bryan Brattlof

[permalink] [raw]
Subject: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

The JTAG_USER_ID_USERCODE efuse address, which is located inside the
WKUP_CTRL_MMR0 range holds information to identify the speed grades of
various components on TI's K3 SoCs. Add a compatible to allow the
cpufreq driver to obtain the data to limit the maximum frequency for the
CPUs under Linux control.

Signed-off-by: Bryan Brattlof <[email protected]>
---
Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
index 7ed12a938baa3..ab1fcbe2148f7 100644
--- a/Documentation/devicetree/bindings/mfd/syscon.yaml
+++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
@@ -88,6 +88,7 @@ properties:
- rockchip,rv1126-qos
- starfive,jh7100-sysmain
- ti,am62-usb-phy-ctrl
+ - ti,am62-opp-efuse-table
- ti,am62p-cpsw-mac-efuse
- ti,am654-dss-oldi-io-ctrl
- ti,am654-serdes-ctrl

--
2.45.2



2024-06-12 17:18:14

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
> The JTAG_USER_ID_USERCODE efuse address, which is located inside the
> WKUP_CTRL_MMR0 range holds information to identify the speed grades of
> various components on TI's K3 SoCs. Add a compatible to allow the
> cpufreq driver to obtain the data to limit the maximum frequency for the
> CPUs under Linux control.
>
> Signed-off-by: Bryan Brattlof <[email protected]>

$subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

Okay, if this isn't for merging then I won't Ack it.

Thanks,
Conor.

> ---
> Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index 7ed12a938baa3..ab1fcbe2148f7 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -88,6 +88,7 @@ properties:
> - rockchip,rv1126-qos
> - starfive,jh7100-sysmain
> - ti,am62-usb-phy-ctrl
> + - ti,am62-opp-efuse-table
> - ti,am62p-cpsw-mac-efuse
> - ti,am654-dss-oldi-io-ctrl
> - ti,am654-serdes-ctrl
>
> --
> 2.45.2
>


Attachments:
(No filename) (1.30 kB)
signature.asc (235.00 B)
Download all attachments

2024-06-12 17:55:19

by Bryan Brattlof

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On June 12, 2024 thus sayeth Conor Dooley:
> On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
> > The JTAG_USER_ID_USERCODE efuse address, which is located inside the
> > WKUP_CTRL_MMR0 range holds information to identify the speed grades of
> > various components on TI's K3 SoCs. Add a compatible to allow the
> > cpufreq driver to obtain the data to limit the maximum frequency for the
> > CPUs under Linux control.
> >
> > Signed-off-by: Bryan Brattlof <[email protected]>
>
> $subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible
>
> Okay, if this isn't for merging then I won't Ack it.

Ha! Nice. If I don't hear anything from anyone else I'll send a v2 in a
few hours.

~Bryan

2024-06-13 12:13:15

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On Wed, 12 Jun 2024, Bryan Brattlof wrote:

> On June 12, 2024 thus sayeth Conor Dooley:
> > On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
> > > The JTAG_USER_ID_USERCODE efuse address, which is located inside the
> > > WKUP_CTRL_MMR0 range holds information to identify the speed grades of
> > > various components on TI's K3 SoCs. Add a compatible to allow the
> > > cpufreq driver to obtain the data to limit the maximum frequency for the
> > > CPUs under Linux control.
> > >
> > > Signed-off-by: Bryan Brattlof <[email protected]>
> >
> > $subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible
> >
> > Okay, if this isn't for merging then I won't Ack it.
>
> Ha! Nice. If I don't hear anything from anyone else I'll send a v2 in a
> few hours.

What's the point of all the DONOTMERGE nonsense?

--
Lee Jones [李琼斯]

2024-06-13 12:23:54

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On Thu, Jun 13, 2024 at 01:09:23PM +0100, Lee Jones wrote:
> On Wed, 12 Jun 2024, Bryan Brattlof wrote:
>
> > On June 12, 2024 thus sayeth Conor Dooley:
> > > On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
> > > > The JTAG_USER_ID_USERCODE efuse address, which is located inside the
> > > > WKUP_CTRL_MMR0 range holds information to identify the speed grades of
> > > > various components on TI's K3 SoCs. Add a compatible to allow the
> > > > cpufreq driver to obtain the data to limit the maximum frequency for the
> > > > CPUs under Linux control.
> > > >
> > > > Signed-off-by: Bryan Brattlof <[email protected]>
> > >
> > > $subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible
> > >
> > > Okay, if this isn't for merging then I won't Ack it.
> >
> > Ha! Nice. If I don't hear anything from anyone else I'll send a v2 in a
> > few hours.
>
> What's the point of all the DONOTMERGE nonsense?

AFAICT, TI live in fear of subsystem maintainers merging the dts patches,
so they do this.


Attachments:
(No filename) (1.03 kB)
signature.asc (235.00 B)
Download all attachments

2024-06-13 13:34:51

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On 13/06/2024 14:20, Conor Dooley wrote:
> On Thu, Jun 13, 2024 at 01:09:23PM +0100, Lee Jones wrote:
>> On Wed, 12 Jun 2024, Bryan Brattlof wrote:
>>
>>> On June 12, 2024 thus sayeth Conor Dooley:
>>>> On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
>>>>> The JTAG_USER_ID_USERCODE efuse address, which is located inside the
>>>>> WKUP_CTRL_MMR0 range holds information to identify the speed grades of
>>>>> various components on TI's K3 SoCs. Add a compatible to allow the
>>>>> cpufreq driver to obtain the data to limit the maximum frequency for the
>>>>> CPUs under Linux control.
>>>>>
>>>>> Signed-off-by: Bryan Brattlof <[email protected]>
>>>>
>>>> $subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible
>>>>
>>>> Okay, if this isn't for merging then I won't Ack it.
>>>
>>> Ha! Nice. If I don't hear anything from anyone else I'll send a v2 in a
>>> few hours.
>>
>> What's the point of all the DONOTMERGE nonsense?
>
> AFAICT, TI live in fear of subsystem maintainers merging the dts patches,
> so they do this.

And want some strict timeframe of merging bindings (via subsystem) and
DTS (via SoC tree), which causes all weird submissions like this above
or sending bindings without users.

So far I can live with it but if more such peculiarities come up, then
sorry, fix your process/tools instead of putting burden on maintainers
and community.

Best regards,
Krzysztof


2024-06-13 17:30:34

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On Thu, 13 Jun 2024, Krzysztof Kozlowski wrote:

> On 13/06/2024 14:20, Conor Dooley wrote:
> > On Thu, Jun 13, 2024 at 01:09:23PM +0100, Lee Jones wrote:
> >> On Wed, 12 Jun 2024, Bryan Brattlof wrote:
> >>
> >>> On June 12, 2024 thus sayeth Conor Dooley:
> >>>> On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
> >>>>> The JTAG_USER_ID_USERCODE efuse address, which is located inside the
> >>>>> WKUP_CTRL_MMR0 range holds information to identify the speed grades of
> >>>>> various components on TI's K3 SoCs. Add a compatible to allow the
> >>>>> cpufreq driver to obtain the data to limit the maximum frequency for the
> >>>>> CPUs under Linux control.
> >>>>>
> >>>>> Signed-off-by: Bryan Brattlof <[email protected]>
> >>>>
> >>>> $subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible
> >>>>
> >>>> Okay, if this isn't for merging then I won't Ack it.
> >>>
> >>> Ha! Nice. If I don't hear anything from anyone else I'll send a v2 in a
> >>> few hours.
> >>
> >> What's the point of all the DONOTMERGE nonsense?
> >
> > AFAICT, TI live in fear of subsystem maintainers merging the dts patches,
> > so they do this.
>
> And want some strict timeframe of merging bindings (via subsystem) and
> DTS (via SoC tree), which causes all weird submissions like this above
> or sending bindings without users.
>
> So far I can live with it but if more such peculiarities come up, then
> sorry, fix your process/tools instead of putting burden on maintainers
> and community.

FWIW, if I see DONOTMERGE in the subject line, I tend to skip over them.

--
Lee Jones [李琼斯]

2024-06-15 16:56:00

by Bryan Brattlof

[permalink] [raw]
Subject: Re: [PATCH 3/5] DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible

On June 13, 2024 thus sayeth Krzysztof Kozlowski:
> On 13/06/2024 14:20, Conor Dooley wrote:
> > On Thu, Jun 13, 2024 at 01:09:23PM +0100, Lee Jones wrote:
> >> On Wed, 12 Jun 2024, Bryan Brattlof wrote:
> >>
> >>> On June 12, 2024 thus sayeth Conor Dooley:
> >>>> On Wed, Jun 12, 2024 at 11:41:52AM -0500, Bryan Brattlof wrote:
> >>>>> The JTAG_USER_ID_USERCODE efuse address, which is located inside the
> >>>>> WKUP_CTRL_MMR0 range holds information to identify the speed grades of
> >>>>> various components on TI's K3 SoCs. Add a compatible to allow the
> >>>>> cpufreq driver to obtain the data to limit the maximum frequency for the
> >>>>> CPUs under Linux control.
> >>>>>
> >>>>> Signed-off-by: Bryan Brattlof <[email protected]>
> >>>>
> >>>> $subject: DONOTMERGE: dt-bindings: mfd: syscon: add TI's opp table compatible
> >>>>
> >>>> Okay, if this isn't for merging then I won't Ack it.
> >>>
> >>> Ha! Nice. If I don't hear anything from anyone else I'll send a v2 in a
> >>> few hours.
> >>
> >> What's the point of all the DONOTMERGE nonsense?
> >
> > AFAICT, TI live in fear of subsystem maintainers merging the dts patches,
> > so they do this.
>
> And want some strict timeframe of merging bindings (via subsystem) and
> DTS (via SoC tree), which causes all weird submissions like this above
> or sending bindings without users.
>
> So far I can live with it but if more such peculiarities come up, then
> sorry, fix your process/tools instead of putting burden on maintainers
> and community.
>

Yeah my worry was all the DTB additions filter in would require a lot of
coordination between the maintainers of all the different trees.

Having a few DONOTMERGE patches gave the driver maintainer a full look
at the outcome of the series without having to worry about DTB conflicts
when another tree picked something up, however I guess if everyone
participates in -next it shouldn't be that big of a problem.

~Bryan