On Fri, Jun 09, 2017 at 04:59:00PM +0200, Stefan Wahren wrote: Hi Stefan, > Hi Oleksij, > > Am 09.06.2017 um 14:57 schrieb Oleksij Rempel: > > Documentation bindings for the Low Power General Purpose Register > > available on i.MX6 SoCs in the Secure Non-Volatile Storage. > > > > Signed-off-by: Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx> > > --- > > .../devicetree/bindings/nvmem/snvs-lpgpr.txt | 19 +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > > > > diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > > new file mode 100644 > > index 000000000000..21910fb3159f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > > @@ -0,0 +1,19 @@ > > +Device tree bindings for Low Power General Purpose Register found in i.MX6Q/D > > +Secure Non-Volatile Storage. > > + > > +This DT node should be represented as a sub-node of a "syscon", > > +"simple-mfd" node. > > + > > +Required properties: > > +- compatible: should be: > > + "fsl,imx6q-snvs-lpgpr" for Freescale i.MX6Q/D/DL/S > > + > > +Example: > > +snvs: snvs@020cc000 { > > + compatible = "fsl,sec-v4.0-mon", "syscon", "simple-mfd"; > > + reg = <0x020cc000 0x4000>; > > + > > + snvs_lpgpr: snvs-lpgpr { > > + compatible = "fsl,imx6q-snvs-lpgpr"; > > according to the reference manual at least the clock "lp_ipg_clk_s" is > required for register R/W access. > So it should be added to the binding and enabled by the driver. Hm... Non of current SNVS drives use, set or defines *_ipg_clk_s. I can't find in the docs how can I control this clocks. lp_ipg_clk_s and hp_ipg_clk_s depend on ipg_clk_root, which seems to be not gated. So, it is always on. Or do I miss something. -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html