From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srinivas Kandagatla Subject: Re: [PATCH 2/3] nvmem: imx-ocotp: Add i.MX6 OCOTP driver Date: Thu, 06 Aug 2015 20:36:47 +0100 Message-ID: <55C3B74F.5010501@linaro.org> References: <1438693342-605-1-git-send-email-p.zabel@pengutronix.de> <1438693342-605-2-git-send-email-p.zabel@pengutronix.de> <55C38941.9090709@linaro.org> <1438878490.3223.10.camel@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1438878490.3223.10.camel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Philipp Zabel Cc: Stefan Wahren , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, Maxime Ripard , Shawn Guo , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Philipp, Could you add Greg KH in the CC, for next merge window, we can request Greg to take this driver via his tree. Once the comments are fixed you can add my Acked-by: Srinivas Kandagatla On 06/08/15 17:28, Philipp Zabel wrote: > Hi Srinivas, > > Am Donnerstag, den 06.08.2015, 17:20 +0100 schrieb Srinivas Kandagatla: >> Few Nits.. >> >> On 04/08/15 14:02, Philipp Zabel wrote: >>> +#include >> May be you can drop this? > > Yes, that's a left-over and can be removed. > >> BTW, who is taking care of the gated peripheral clock controlled for >> this driver? > > Nobody. I had initially copied the code that actually senses the fuses, > but since this happens during power-on reset anyway (all values are > stored in shadow registers), I didn't see the point and dropped it for > now. > The shadow registers are clocked directly by the ungated ipg_root clock. So, are you planning to drop the clocks property from the bindings too? --srini > > [...] >>> +MODULE_LICENSE("GPL"); >> >> GPL v2 ? > > Will do. > > thanks > Philipp > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html