* [PATCH v1 0/2] AUXADC: support efuse calibration on MT818
@ 2019-07-18 8:01 Zhiyong Tao
2019-07-18 8:01 ` [PATCH v1 1/2] arm64: dts: mt8183: auxadc: add efuse information in device tree Zhiyong Tao
2019-07-18 8:01 ` [PATCH v1 2/2] auxadc: mediatek: support efuse calibration in auxadc driver Zhiyong Tao
0 siblings, 2 replies; 4+ messages in thread
From: Zhiyong Tao @ 2019-07-18 8:01 UTC (permalink / raw)
To: robh+dt, jic23, knaack.h, lars, pmeerw
Cc: devicetree, sean.wang, srv_heupstream, linux-iio, erin.lo,
hui.liu, linux-kernel, sj.huang, linux-mediatek, eddie.huang,
matthias.bgg, yingjoe.chen, jg_poxu, s.hauer, linux-arm-kernel
This series includes two patches:
1.Add mt8183 auxadc efuse information in device tree.
1.Add mt8183 efuse calibration function in auxadc driver.
Zhiyong Tao (2):
arm64: dts: mt8183: auxadc: add efuse information in device tree
auxadc: mediatek: support efuse calibration in auxadc driver
arch/arm64/boot/dts/mediatek/mt8183.dtsi | 5 +++
drivers/iio/adc/mt6577_auxadc.c | 71 ++++++++++++++++++++++++++++++++
2 files changed, 76 insertions(+)
--
2.12.5
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v1 1/2] arm64: dts: mt8183: auxadc: add efuse information in device tree
2019-07-18 8:01 [PATCH v1 0/2] AUXADC: support efuse calibration on MT818 Zhiyong Tao
@ 2019-07-18 8:01 ` Zhiyong Tao
2019-07-18 8:01 ` [PATCH v1 2/2] auxadc: mediatek: support efuse calibration in auxadc driver Zhiyong Tao
1 sibling, 0 replies; 4+ messages in thread
From: Zhiyong Tao @ 2019-07-18 8:01 UTC (permalink / raw)
To: robh+dt, jic23, knaack.h, lars, pmeerw
Cc: devicetree, sean.wang, srv_heupstream, linux-iio, Zhiyong Tao,
erin.lo, hui.liu, linux-kernel, sj.huang, linux-mediatek,
eddie.huang, matthias.bgg, yingjoe.chen, jg_poxu, s.hauer,
linux-arm-kernel
The commit add add efuse information in device tree
Signed-off-by: Zhiyong Tao <zhiyong.tao@mediatek.com>
Signed-off-by: jg_poxu <jg_poxu@mediatek.com>
---
arch/arm64/boot/dts/mediatek/mt8183.dtsi | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
index c2749c4631bc..59b4f3d933b5 100644
--- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
@@ -261,6 +261,8 @@
clocks = <&infracfg CLK_INFRA_AUXADC>;
clock-names = "main";
#io-channel-cells = <1>;
+ nvmem-cells = <&auxadc_calibration>;
+ nvmem-cell-names = "calibration-data";
status = "disabled";
};
@@ -382,6 +384,9 @@
compatible = "mediatek,mt8183-efuse",
"mediatek,efuse";
reg = <0 0x11f10000 0 0x1000>;
+ auxadc_calibration: auxadccali@1b4 {
+ reg = <0x1b4 0x4>;
+ };
};
mfgcfg: syscon@13000000 {
--
2.12.5
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v1 2/2] auxadc: mediatek: support efuse calibration in auxadc driver
2019-07-18 8:01 [PATCH v1 0/2] AUXADC: support efuse calibration on MT818 Zhiyong Tao
2019-07-18 8:01 ` [PATCH v1 1/2] arm64: dts: mt8183: auxadc: add efuse information in device tree Zhiyong Tao
@ 2019-07-18 8:01 ` Zhiyong Tao
2019-07-20 14:13 ` Nicolas Boichat
1 sibling, 1 reply; 4+ messages in thread
From: Zhiyong Tao @ 2019-07-18 8:01 UTC (permalink / raw)
To: robh+dt, jic23, knaack.h, lars, pmeerw
Cc: devicetree, sean.wang, srv_heupstream, linux-iio, Zhiyong Tao,
erin.lo, hui.liu, linux-kernel, sj.huang, linux-mediatek,
eddie.huang, matthias.bgg, yingjoe.chen, jg_poxu, s.hauer,
linux-arm-kernel
This patch support efuse calibration in auxadc driver
Signed-off-by: Zhiyong Tao <zhiyong.tao@mediatek.com>
Signed-off-by: jg_poxu <jg_poxu@mediatek.com>
---
drivers/iio/adc/mt6577_auxadc.c | 71 +++++++++++++++++++++++++++++++++++++++++
1 file changed, 71 insertions(+)
diff --git a/drivers/iio/adc/mt6577_auxadc.c b/drivers/iio/adc/mt6577_auxadc.c
index 95d76abb64ec..e30d9736b1a5 100644
--- a/drivers/iio/adc/mt6577_auxadc.c
+++ b/drivers/iio/adc/mt6577_auxadc.c
@@ -17,12 +17,14 @@
#include <linux/err.h>
#include <linux/kernel.h>
#include <linux/module.h>
+#include <linux/nvmem-consumer.h>
#include <linux/of.h>
#include <linux/of_device.h>
#include <linux/platform_device.h>
#include <linux/iopoll.h>
#include <linux/io.h>
#include <linux/iio/iio.h>
+#include <linux/slab.h>
/* Register definitions */
#define MT6577_AUXADC_CON0 0x00
@@ -42,6 +44,13 @@
#define MT6577_AUXADC_POWER_READY_MS 1
#define MT6577_AUXADC_SAMPLE_READY_US 25
+#define ADC_GE_A_SHIFT 10
+#define ADC_GE_A_MASK (0x3ff << ADC_GE_A_SHIFT)
+#define ADC_OE_A_SHIFT 0
+#define ADC_OE_A_MASK (0x3ff << ADC_OE_A_SHIFT)
+#define ADC_CALI_EN_A_SHIFT 20
+#define ADC_CALI_EN_A_MASK (0x1 << ADC_CALI_EN_A_SHIFT)
+
struct mt6577_auxadc_device {
void __iomem *reg_base;
struct clk *adc_clk;
@@ -74,6 +83,64 @@ static const struct iio_chan_spec mt6577_auxadc_iio_channels[] = {
MT6577_AUXADC_CHANNEL(15),
};
+s32 cali_oe;
+s32 cali_ge;
+struct adc_cali_info {
+ u32 cali_ge_a;
+ u32 cali_oe_a;
+ u32 gain;
+};
+static struct adc_cali_info adc_cali;
+
+static int mt6577_auxadc_update_cali(struct device *dev)
+{
+ struct nvmem_cell *cell;
+ u32 *buf;
+ size_t len;
+ int ret = 0;
+
+ cali_oe = 0;
+ cali_ge = 0;
+
+ cell = nvmem_cell_get(dev, "calibration-data");
+ if (IS_ERR(cell)) {
+ if (PTR_ERR(cell) == -EPROBE_DEFER)
+ return PTR_ERR(cell);
+ return 0;
+ }
+
+ buf = (u32 *)nvmem_cell_read(cell, &len);
+
+ nvmem_cell_put(cell);
+
+ if (IS_ERR(buf))
+ return PTR_ERR(buf);
+
+ if (len < sizeof(u32)) {
+ dev_warn(dev, "invalid calibration data\n");
+ ret = -EINVAL;
+ goto out;
+ }
+
+ if (((buf[0] & ADC_CALI_EN_A_MASK) >> ADC_CALI_EN_A_SHIFT) != 0) {
+ adc_cali.cali_oe_a =
+ (buf[0] & ADC_OE_A_MASK) >> ADC_OE_A_SHIFT;
+ adc_cali.cali_ge_a =
+ ((buf[0] & ADC_GE_A_MASK) >> ADC_GE_A_SHIFT);
+
+ cali_ge = adc_cali.cali_ge_a - 512;
+ cali_oe = adc_cali.cali_oe_a - 512;
+ adc_cali.gain = 1 + cali_ge;
+ } else {
+ dev_info(dev, "Device not calibrated, using default calibration values\n");
+ }
+
+out:
+ kfree(buf);
+
+ return ret;
+}
+
static inline void mt6577_auxadc_mod_reg(void __iomem *reg,
u32 or_mask, u32 and_mask)
{
@@ -274,6 +341,10 @@ static int mt6577_auxadc_probe(struct platform_device *pdev)
goto err_power_off;
}
+ ret = mt6577_auxadc_update_cali(&pdev->dev);
+ if (ret)
+ return ret;
+
return 0;
err_power_off:
--
2.12.5
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v1 2/2] auxadc: mediatek: support efuse calibration in auxadc driver
2019-07-18 8:01 ` [PATCH v1 2/2] auxadc: mediatek: support efuse calibration in auxadc driver Zhiyong Tao
@ 2019-07-20 14:13 ` Nicolas Boichat
0 siblings, 0 replies; 4+ messages in thread
From: Nicolas Boichat @ 2019-07-20 14:13 UTC (permalink / raw)
To: Zhiyong Tao
Cc: devicetree, Lars-Peter Clausen, srv_heupstream, linux-iio,
Erin Lo, hui.liu, Eddie Huang, lkml, sj.huang, Rob Herring,
moderated list:ARM/Mediatek SoC support, linux-arm Mailing List,
pmeerw, knaack.h, Matthias Brugger, Yingjoe Chen, jg_poxu,
Sean Wang, Sascha Hauer, jic23
On Thu, Jul 18, 2019 at 4:01 PM Zhiyong Tao <zhiyong.tao@mediatek.com> wrote:
>
> This patch support efuse calibration in auxadc driver
>
> Signed-off-by: Zhiyong Tao <zhiyong.tao@mediatek.com>
> Signed-off-by: jg_poxu <jg_poxu@mediatek.com>
> ---
> drivers/iio/adc/mt6577_auxadc.c | 71 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 71 insertions(+)
>
> diff --git a/drivers/iio/adc/mt6577_auxadc.c b/drivers/iio/adc/mt6577_auxadc.c
> index 95d76abb64ec..e30d9736b1a5 100644
> --- a/drivers/iio/adc/mt6577_auxadc.c
> +++ b/drivers/iio/adc/mt6577_auxadc.c
> @@ -17,12 +17,14 @@
> #include <linux/err.h>
> #include <linux/kernel.h>
> #include <linux/module.h>
> +#include <linux/nvmem-consumer.h>
> #include <linux/of.h>
> #include <linux/of_device.h>
> #include <linux/platform_device.h>
> #include <linux/iopoll.h>
> #include <linux/io.h>
> #include <linux/iio/iio.h>
> +#include <linux/slab.h>
>
> /* Register definitions */
> #define MT6577_AUXADC_CON0 0x00
> @@ -42,6 +44,13 @@
> #define MT6577_AUXADC_POWER_READY_MS 1
> #define MT6577_AUXADC_SAMPLE_READY_US 25
>
> +#define ADC_GE_A_SHIFT 10
> +#define ADC_GE_A_MASK (0x3ff << ADC_GE_A_SHIFT)
> +#define ADC_OE_A_SHIFT 0
> +#define ADC_OE_A_MASK (0x3ff << ADC_OE_A_SHIFT)
> +#define ADC_CALI_EN_A_SHIFT 20
> +#define ADC_CALI_EN_A_MASK (0x1 << ADC_CALI_EN_A_SHIFT)
> +
> struct mt6577_auxadc_device {
> void __iomem *reg_base;
> struct clk *adc_clk;
> @@ -74,6 +83,64 @@ static const struct iio_chan_spec mt6577_auxadc_iio_channels[] = {
> MT6577_AUXADC_CHANNEL(15),
> };
>
> +s32 cali_oe;
> +s32 cali_ge;
Why do you need these 2 (global) variables? They are set in
mt6577_auxadc_update_cali but never read anywhere.
> +struct adc_cali_info {
> + u32 cali_ge_a;
> + u32 cali_oe_a;
> + u32 gain;
> +};
> +static struct adc_cali_info adc_cali;
This should be inside struct mt6577_auxadc_device structure. And... I
still do not understand what this does, since nothing reads back
adc_cali either...
On gerrit (https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1624410),
you mentioned that thermal module will use these, can you please
upload these changes as part of this series, so that we can review the
whole thing in one piece (and understand better how to fix this
interface).
> +
> +static int mt6577_auxadc_update_cali(struct device *dev)
> +{
> + struct nvmem_cell *cell;
> + u32 *buf;
> + size_t len;
> + int ret = 0;
> +
> + cali_oe = 0;
> + cali_ge = 0;
> +
> + cell = nvmem_cell_get(dev, "calibration-data");
> + if (IS_ERR(cell)) {
> + if (PTR_ERR(cell) == -EPROBE_DEFER)
> + return PTR_ERR(cell);
> + return 0;
> + }
> +
> + buf = (u32 *)nvmem_cell_read(cell, &len);
Cast is not necessary.
> +
> + nvmem_cell_put(cell);
> +
> + if (IS_ERR(buf))
> + return PTR_ERR(buf);
> +
> + if (len < sizeof(u32)) {
> + dev_warn(dev, "invalid calibration data\n");
> + ret = -EINVAL;
> + goto out;
> + }
> +
> + if (((buf[0] & ADC_CALI_EN_A_MASK) >> ADC_CALI_EN_A_SHIFT) != 0) {
> + adc_cali.cali_oe_a =
> + (buf[0] & ADC_OE_A_MASK) >> ADC_OE_A_SHIFT;
> + adc_cali.cali_ge_a =
> + ((buf[0] & ADC_GE_A_MASK) >> ADC_GE_A_SHIFT);
> +
> + cali_ge = adc_cali.cali_ge_a - 512;
> + cali_oe = adc_cali.cali_oe_a - 512;
> + adc_cali.gain = 1 + cali_ge;
> + } else {
> + dev_info(dev, "Device not calibrated, using default calibration values\n");
> + }
> +
> +out:
> + kfree(buf);
> +
> + return ret;
> +}
> +
> static inline void mt6577_auxadc_mod_reg(void __iomem *reg,
> u32 or_mask, u32 and_mask)
> {
> @@ -274,6 +341,10 @@ static int mt6577_auxadc_probe(struct platform_device *pdev)
> goto err_power_off;
> }
>
> + ret = mt6577_auxadc_update_cali(&pdev->dev);
> + if (ret)
> + return ret;
> +
> return 0;
Or just
return mt6577_auxadc_update_cali(&pdev->dev);
>
> err_power_off:
> --
> 2.12.5
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-07-20 14:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-18 8:01 [PATCH v1 0/2] AUXADC: support efuse calibration on MT818 Zhiyong Tao
2019-07-18 8:01 ` [PATCH v1 1/2] arm64: dts: mt8183: auxadc: add efuse information in device tree Zhiyong Tao
2019-07-18 8:01 ` [PATCH v1 2/2] auxadc: mediatek: support efuse calibration in auxadc driver Zhiyong Tao
2019-07-20 14:13 ` Nicolas Boichat
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).