From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH v7 6/9] nvmem: qfprom: Add Qualcomm QFPROM support. Date: Tue, 14 Jul 2015 14:18:31 -0700 Message-ID: <20150714211831.GN30412@codeaurora.org> References: <1436521427-10568-1-git-send-email-srinivas.kandagatla@linaro.org> <1436521529-10931-1-git-send-email-srinivas.kandagatla@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1436521529-10931-1-git-send-email-srinivas.kandagatla-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Srinivas Kandagatla Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Greg Kroah-Hartman , Rob Herring , Kumar Gala , Mark Brown , s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, arnd-r2nGTMty4D4@public.gmane.org, pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org, mporter-OWPKS81ov/FWk0Htik3J/w@public.gmane.org, stefan.wahren-eS4NqCHxEME@public.gmane.org, wxt-TNX95d0MmH7DzftRWevZcw@public.gmane.org List-Id: linux-arm-msm@vger.kernel.org On 07/10, Srinivas Kandagatla wrote: > This patch adds QFPROM support driver which is used by other drivers > like thermal sensor and cpufreq. > > On MSM parts there are some efuses (called qfprom) these fuses store > things like calibration data, speed bins.. etc. Drivers like cpufreq, > thermal sensors would read out this data for configuring the driver. > > Signed-off-by: Srinivas Kandagatla > --- I didn't see any reply on v6, but at least delay.h was removed. Same comments from v6 below. > diff --git a/drivers/nvmem/qfprom.c b/drivers/nvmem/qfprom.c > new file mode 100644 > index 0000000..0e22a03 > --- /dev/null > +++ b/drivers/nvmem/qfprom.c > @@ -0,0 +1,87 @@ > +/* > + * Copyright (C) 2015 Srinivas Kandagatla > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 and > + * only version 2 as published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#include > +#include Is this include used? > +#include > +#include > +#include > +#include > +#include > + > +static struct regmap_config qfprom_regmap_config = { Can this be const? > + .reg_bits = 32, > + .val_bits = 8, > + .reg_stride = 1, > +}; > + > +static struct nvmem_config econfig = { Can this be const? > + .name = "qfprom", > + .owner = THIS_MODULE, > +}; > + -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project