diff mbox series

dt-bindings: Add compatibles for undocumented trivial syscons

Message ID 20220228221537.1700071-1-robh@kernel.org (mailing list archive)
State New, archived
Headers show
Series dt-bindings: Add compatibles for undocumented trivial syscons | expand

Commit Message

Rob Herring Feb. 28, 2022, 10:15 p.m. UTC
'intel,lgm-syscon', 'marvell,armada-3700-usb2-host-misc',
'mediatek,mt8135-pctl-a-syscfg', and 'mediatek,mt8135-pctl-b-syscfg' are
all used in DT binding examples and/or dts files, but are not
documented. Add them to syscon.yaml as they are all trivial cases.

Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/mfd/syscon.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Matthias Brugger March 1, 2022, 7:14 a.m. UTC | #1
Hi Rob,

On 28/02/2022 23:15, Rob Herring wrote:
> 'intel,lgm-syscon', 'marvell,armada-3700-usb2-host-misc',
> 'mediatek,mt8135-pctl-a-syscfg', and 'mediatek,mt8135-pctl-b-syscfg' are
> all used in DT binding examples and/or dts files, but are not
> documented. Add them to syscon.yaml as they are all trivial cases.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>   Documentation/devicetree/bindings/mfd/syscon.yaml | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index eeac1cbc5a17..29d4a97f7108 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -44,6 +44,10 @@ properties:
>                 - hisilicon,hi6220-sramctrl
>                 - hisilicon,pcie-sas-subctrl
>                 - hisilicon,peri-subctrl
> +              - intel,lgm-syscon
> +              - marvell,armada-3700-usb2-host-misc
> +              - mediatek,mt8135-pctl-a-syscfg
> +              - mediatek,mt8135-pctl-b-syscfg

Thanks for filling the gap. Would you mind to add as well:
"mediatek,mt2701-pctl-a-syscfg"
"mediatek,mt2712-pctl-a-syscfg"
"mediatek,mt7623-pctl-a-syscfg"
"mediatek,mt8173-pctl-a-syscfg

With that:
Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

>                 - microchip,sparx5-cpu-syscon
>                 - mstar,msc313-pmsleep
>                 - rockchip,px30-qos
Lee Jones March 1, 2022, 10 a.m. UTC | #2
On Mon, 28 Feb 2022, Rob Herring wrote:

> 'intel,lgm-syscon', 'marvell,armada-3700-usb2-host-misc',
> 'mediatek,mt8135-pctl-a-syscfg', and 'mediatek,mt8135-pctl-b-syscfg' are
> all used in DT binding examples and/or dts files, but are not
> documented. Add them to syscon.yaml as they are all trivial cases.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 4 ++++
>  1 file changed, 4 insertions(+)

Applied, thanks.
Lee Jones March 1, 2022, 10 a.m. UTC | #3
On Tue, 01 Mar 2022, Matthias Brugger wrote:

> Hi Rob,
> 
> On 28/02/2022 23:15, Rob Herring wrote:
> > 'intel,lgm-syscon', 'marvell,armada-3700-usb2-host-misc',
> > 'mediatek,mt8135-pctl-a-syscfg', and 'mediatek,mt8135-pctl-b-syscfg' are
> > all used in DT binding examples and/or dts files, but are not
> > documented. Add them to syscon.yaml as they are all trivial cases.
> > 
> > Signed-off-by: Rob Herring <robh@kernel.org>
> > ---
> >   Documentation/devicetree/bindings/mfd/syscon.yaml | 4 ++++
> >   1 file changed, 4 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> > index eeac1cbc5a17..29d4a97f7108 100644
> > --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> > +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> > @@ -44,6 +44,10 @@ properties:
> >                 - hisilicon,hi6220-sramctrl
> >                 - hisilicon,pcie-sas-subctrl
> >                 - hisilicon,peri-subctrl
> > +              - intel,lgm-syscon
> > +              - marvell,armada-3700-usb2-host-misc
> > +              - mediatek,mt8135-pctl-a-syscfg
> > +              - mediatek,mt8135-pctl-b-syscfg
> 
> Thanks for filling the gap. Would you mind to add as well:
> "mediatek,mt2701-pctl-a-syscfg"
> "mediatek,mt2712-pctl-a-syscfg"
> "mediatek,mt7623-pctl-a-syscfg"
> "mediatek,mt8173-pctl-a-syscfg
> 
> With that:
> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

Please follow-up with a separate patch.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
index eeac1cbc5a17..29d4a97f7108 100644
--- a/Documentation/devicetree/bindings/mfd/syscon.yaml
+++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
@@ -44,6 +44,10 @@  properties:
               - hisilicon,hi6220-sramctrl
               - hisilicon,pcie-sas-subctrl
               - hisilicon,peri-subctrl
+              - intel,lgm-syscon
+              - marvell,armada-3700-usb2-host-misc
+              - mediatek,mt8135-pctl-a-syscfg
+              - mediatek,mt8135-pctl-b-syscfg
               - microchip,sparx5-cpu-syscon
               - mstar,msc313-pmsleep
               - rockchip,px30-qos