Message ID | 20200306130401.C07838030795@mail.baikalelectronics.ru |
---|---|
State | Not Applicable, archived |
Headers | show |
Series | [1/4] dt-bindings: syscon: Add syscon endian properties support | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | warning | "total: 0 errors, 2 warnings, 55 lines checked" |
robh/dt-meta-schema | success |
Hi, On Fri, Mar 06, 2020 at 04:03:39PM +0300, Sergey.Semin@baikalelectronics.ru wrote: > From: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > Modern device tree bindings are supposed to be created as YAML-files > in accordance with dt-schema. This commit replaces SYSCON reboot-mode > legacy bare text bindings with YAML file. As before the bindings file > states that the corresponding dts node is supposed to be compatible > "syscon-reboot-mode" device and necessarily have an offset property > to determine which register from the regmap is supposed to keep the > mode on reboot. > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > Signed-off-by: Alexey Malahov <Alexey.Malahov@baikalelectronics.ru> > Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de> > Cc: Paul Burton <paulburton@kernel.org> > Cc: Ralf Baechle <ralf@linux-mips.org> > --- I'm missing patch 1 and would like an Acked-by from Rob Herring, so for now: Acked-by: Sebastian Reichel <sebastian.reichel@collabora.com> -- Sebastian > .../power/reset/syscon-reboot-mode.txt | 35 ------------ > .../power/reset/syscon-reboot-mode.yaml | 55 +++++++++++++++++++ > 2 files changed, 55 insertions(+), 35 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > create mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > > diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > deleted file mode 100644 > index f7ce1d8af04a..000000000000 > --- a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > +++ /dev/null > @@ -1,35 +0,0 @@ > -SYSCON reboot mode driver > - > -This driver gets reboot mode magic value form reboot-mode driver > -and stores it in a SYSCON mapped register. Then the bootloader > -can read it and take different action according to the magic > -value stored. > - > -This DT node should be represented as a sub-node of a "syscon", "simple-mfd" > -node. > - > -Required properties: > -- compatible: should be "syscon-reboot-mode" > -- offset: offset in the register map for the storage register (in bytes) > - > -Optional property: > -- mask: bits mask of the bits in the register to store the reboot mode magic value, > - default set to 0xffffffff if missing. > - > -The rest of the properties should follow the generic reboot-mode description > -found in reboot-mode.txt > - > -Example: > - pmu: pmu@20004000 { > - compatible = "rockchip,rk3066-pmu", "syscon", "simple-mfd"; > - reg = <0x20004000 0x100>; > - > - reboot-mode { > - compatible = "syscon-reboot-mode"; > - offset = <0x40>; > - mode-normal = <BOOT_NORMAL>; > - mode-recovery = <BOOT_RECOVERY>; > - mode-bootloader = <BOOT_FASTBOOT>; > - mode-loader = <BOOT_BL_DOWNLOAD>; > - }; > - }; > diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > new file mode 100644 > index 000000000000..e09bb07b1abb > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > @@ -0,0 +1,55 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/power/reset/syscon-reboot-mode.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Generic SYSCON reboot mode driver > + > +maintainers: > + - Sebastian Reichel <sre@kernel.org> > + > +description: | > + This driver gets reboot mode magic value from reboot-mode driver > + and stores it in a SYSCON mapped register. Then the bootloader > + can read it and take different action according to the magic > + value stored. The SYSCON mapped register is retrieved from the > + parental dt-node plus the offset. So the SYSCON reboot-mode node > + should be represented as a sub-node of a "syscon", "simple-mfd" node. > + > +properties: > + compatible: > + const: syscon-reboot-mode > + > + mask: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Update only the register bits defined by the mask (32 bit). > + > + offset: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Offset in the register map for the mode register (in bytes). > + > +patternProperties: > + "^mode-.+": > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Vendor-specific mode value written to the mode register. > + > +additionalProperties: false > + > +required: > + - compatible > + - offset > + > +examples: > + - | > + #include <dt-bindings/soc/rockchip,boot-mode.h> > + > + reboot-mode { > + compatible = "syscon-reboot-mode"; > + offset = <0x40>; > + mode-normal = <BOOT_NORMAL>; > + mode-recovery = <BOOT_RECOVERY>; > + mode-bootloader = <BOOT_FASTBOOT>; > + mode-loader = <BOOT_BL_DOWNLOAD>; > + }; > +... > -- > 2.25.1 >
On Fri, Mar 06, 2020 at 08:56:38PM +0100, Sebastian Reichel wrote: > Hi, > > On Fri, Mar 06, 2020 at 04:03:39PM +0300, Sergey.Semin@baikalelectronics.ru wrote: > > From: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > > > Modern device tree bindings are supposed to be created as YAML-files > > in accordance with dt-schema. This commit replaces SYSCON reboot-mode > > legacy bare text bindings with YAML file. As before the bindings file > > states that the corresponding dts node is supposed to be compatible > > "syscon-reboot-mode" device and necessarily have an offset property > > to determine which register from the regmap is supposed to keep the > > mode on reboot. > > > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > Signed-off-by: Alexey Malahov <Alexey.Malahov@baikalelectronics.ru> > > Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de> > > Cc: Paul Burton <paulburton@kernel.org> > > Cc: Ralf Baechle <ralf@linux-mips.org> > > --- > > I'm missing patch 1 and would like an Acked-by from Rob Herring, so > for now: > > Acked-by: Sebastian Reichel <sebastian.reichel@collabora.com> > Great. Thanks. I'll resend the patchset very soon. You aren't in the first patch Cc because it doesn't concern power/reset subsystem, but mfd/syscon. That's why my submission script didn't add you to the list. Sorry about that. I'll send a v2 copy to you. Regards, -Sergey > -- Sebastian > > > .../power/reset/syscon-reboot-mode.txt | 35 ------------ > > .../power/reset/syscon-reboot-mode.yaml | 55 +++++++++++++++++++ > > 2 files changed, 55 insertions(+), 35 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > > create mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > > > > diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > > deleted file mode 100644 > > index f7ce1d8af04a..000000000000 > > --- a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > > +++ /dev/null > > @@ -1,35 +0,0 @@ > > -SYSCON reboot mode driver > > - > > -This driver gets reboot mode magic value form reboot-mode driver > > -and stores it in a SYSCON mapped register. Then the bootloader > > -can read it and take different action according to the magic > > -value stored. > > - > > -This DT node should be represented as a sub-node of a "syscon", "simple-mfd" > > -node. > > - > > -Required properties: > > -- compatible: should be "syscon-reboot-mode" > > -- offset: offset in the register map for the storage register (in bytes) > > - > > -Optional property: > > -- mask: bits mask of the bits in the register to store the reboot mode magic value, > > - default set to 0xffffffff if missing. > > - > > -The rest of the properties should follow the generic reboot-mode description > > -found in reboot-mode.txt > > - > > -Example: > > - pmu: pmu@20004000 { > > - compatible = "rockchip,rk3066-pmu", "syscon", "simple-mfd"; > > - reg = <0x20004000 0x100>; > > - > > - reboot-mode { > > - compatible = "syscon-reboot-mode"; > > - offset = <0x40>; > > - mode-normal = <BOOT_NORMAL>; > > - mode-recovery = <BOOT_RECOVERY>; > > - mode-bootloader = <BOOT_FASTBOOT>; > > - mode-loader = <BOOT_BL_DOWNLOAD>; > > - }; > > - }; > > diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > > new file mode 100644 > > index 000000000000..e09bb07b1abb > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > > @@ -0,0 +1,55 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/power/reset/syscon-reboot-mode.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Generic SYSCON reboot mode driver > > + > > +maintainers: > > + - Sebastian Reichel <sre@kernel.org> > > + > > +description: | > > + This driver gets reboot mode magic value from reboot-mode driver > > + and stores it in a SYSCON mapped register. Then the bootloader > > + can read it and take different action according to the magic > > + value stored. The SYSCON mapped register is retrieved from the > > + parental dt-node plus the offset. So the SYSCON reboot-mode node > > + should be represented as a sub-node of a "syscon", "simple-mfd" node. > > + > > +properties: > > + compatible: > > + const: syscon-reboot-mode > > + > > + mask: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: Update only the register bits defined by the mask (32 bit). > > + > > + offset: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: Offset in the register map for the mode register (in bytes). > > + > > +patternProperties: > > + "^mode-.+": > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: Vendor-specific mode value written to the mode register. > > + > > +additionalProperties: false > > + > > +required: > > + - compatible > > + - offset > > + > > +examples: > > + - | > > + #include <dt-bindings/soc/rockchip,boot-mode.h> > > + > > + reboot-mode { > > + compatible = "syscon-reboot-mode"; > > + offset = <0x40>; > > + mode-normal = <BOOT_NORMAL>; > > + mode-recovery = <BOOT_RECOVERY>; > > + mode-bootloader = <BOOT_FASTBOOT>; > > + mode-loader = <BOOT_BL_DOWNLOAD>; > > + }; > > +... > > -- > > 2.25.1 > >
On Fri, 6 Mar 2020 16:03:39 +0300, <Sergey.Semin@baikalelectronics.ru> wrote: > From: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > Modern device tree bindings are supposed to be created as YAML-files > in accordance with dt-schema. This commit replaces SYSCON reboot-mode > legacy bare text bindings with YAML file. As before the bindings file > states that the corresponding dts node is supposed to be compatible > "syscon-reboot-mode" device and necessarily have an offset property > to determine which register from the regmap is supposed to keep the > mode on reboot. > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > Signed-off-by: Alexey Malahov <Alexey.Malahov@baikalelectronics.ru> > Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de> > Cc: Paul Burton <paulburton@kernel.org> > Cc: Ralf Baechle <ralf@linux-mips.org> > --- > .../power/reset/syscon-reboot-mode.txt | 35 ------------ > .../power/reset/syscon-reboot-mode.yaml | 55 +++++++++++++++++++ > 2 files changed, 55 insertions(+), 35 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt > create mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt deleted file mode 100644 index f7ce1d8af04a..000000000000 --- a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt +++ /dev/null @@ -1,35 +0,0 @@ -SYSCON reboot mode driver - -This driver gets reboot mode magic value form reboot-mode driver -and stores it in a SYSCON mapped register. Then the bootloader -can read it and take different action according to the magic -value stored. - -This DT node should be represented as a sub-node of a "syscon", "simple-mfd" -node. - -Required properties: -- compatible: should be "syscon-reboot-mode" -- offset: offset in the register map for the storage register (in bytes) - -Optional property: -- mask: bits mask of the bits in the register to store the reboot mode magic value, - default set to 0xffffffff if missing. - -The rest of the properties should follow the generic reboot-mode description -found in reboot-mode.txt - -Example: - pmu: pmu@20004000 { - compatible = "rockchip,rk3066-pmu", "syscon", "simple-mfd"; - reg = <0x20004000 0x100>; - - reboot-mode { - compatible = "syscon-reboot-mode"; - offset = <0x40>; - mode-normal = <BOOT_NORMAL>; - mode-recovery = <BOOT_RECOVERY>; - mode-bootloader = <BOOT_FASTBOOT>; - mode-loader = <BOOT_BL_DOWNLOAD>; - }; - }; diff --git a/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml new file mode 100644 index 000000000000..e09bb07b1abb --- /dev/null +++ b/Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.yaml @@ -0,0 +1,55 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/power/reset/syscon-reboot-mode.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Generic SYSCON reboot mode driver + +maintainers: + - Sebastian Reichel <sre@kernel.org> + +description: | + This driver gets reboot mode magic value from reboot-mode driver + and stores it in a SYSCON mapped register. Then the bootloader + can read it and take different action according to the magic + value stored. The SYSCON mapped register is retrieved from the + parental dt-node plus the offset. So the SYSCON reboot-mode node + should be represented as a sub-node of a "syscon", "simple-mfd" node. + +properties: + compatible: + const: syscon-reboot-mode + + mask: + $ref: /schemas/types.yaml#/definitions/uint32 + description: Update only the register bits defined by the mask (32 bit). + + offset: + $ref: /schemas/types.yaml#/definitions/uint32 + description: Offset in the register map for the mode register (in bytes). + +patternProperties: + "^mode-.+": + $ref: /schemas/types.yaml#/definitions/uint32 + description: Vendor-specific mode value written to the mode register. + +additionalProperties: false + +required: + - compatible + - offset + +examples: + - | + #include <dt-bindings/soc/rockchip,boot-mode.h> + + reboot-mode { + compatible = "syscon-reboot-mode"; + offset = <0x40>; + mode-normal = <BOOT_NORMAL>; + mode-recovery = <BOOT_RECOVERY>; + mode-bootloader = <BOOT_FASTBOOT>; + mode-loader = <BOOT_BL_DOWNLOAD>; + }; +...