> -----Original Message----- > From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- > ow...@vger.kernel.org] On Behalf Of Srinivas Kandagatla > Sent: Tuesday, May 10, 2016 6:32 PM > To: Peng Fan <van.free...@gmail.com> > Cc: shawn...@kernel.org; maxime.rip...@free-electrons.com; linux-arm- > ker...@lists.infradead.org; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH V2 3/3] nvmem: imx-ocotp: handling clock > > > > On 09/05/16 13:41, Peng Fan wrote: > > Hi Srinivas, > > > > On Mon, May 09, 2016 at 01:28:14PM +0100, Srinivas Kandagatla wrote: > >> Thanks for the Ping, I totally missed this thread. > >> > >> On 20/04/16 18:26, Peng Fan wrote: > >>> Before access ocotp nvmem area, the clock should be enabled. > >>> Or, `hexdump nvmem` will hang the system. So, use such flow: > >>> " > >>> 1. clock_enable_prepare > >>> 2. read nvmem ocotp area > >>> 3. clock_disable_unprepare > >>> " > >>> > >>> Signed-off-by: Peng Fan <van.free...@gmail.com> > >>> Cc: Srinivas Kandagatla <srinivas.kandaga...@linaro.org> > >>> Cc: Maxime Ripard <maxime.rip...@free-electrons.com> > >>> Cc: Shawn Guo <shawn...@kernel.org> > >>> --- > >>> > >>> V2: > >>> Follow Fabio's comments, check return value of clk_prepare_enable > >>> > >>> drivers/nvmem/imx-ocotp.c | 16 +++++++++++++++- > >>> 1 file changed, 15 insertions(+), 1 deletion(-) > >> > >> Changes to the driver looks good to me, Only one comment, As the > >> clock property is now made mandatory, you should update the > >> dt-bindings with this info too. > > > > In Documentation/devicetree/bindings/nvmem/imx-ocotp.txt, > > the clock property is already marked Required properties: > > Ok, I can take this patch, dt patch should go via the dt maintainers.
Thanks, Srinivas. Shawn, Will you review and pick up the other two patches, since changes to i.MX dts files? https://lkml.org/lkml/2016/4/20/150 arm: dts: imx6qdl: add clocks property for ocotp node https://lkml.org/lkml/2016/4/20/153 arm: dts: imx6sl: add clocks property for ocotp node Thanks, Peng. > > thanks, > --srini
