From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [RESEND PATCH v4 1/9] mfd: axp20x: correct name of temperature data ADC registers Date: Mon, 20 Mar 2017 10:43:29 +0000 Message-ID: <20170320104329.ymusypmxljnitmwr@dell> References: <20170320081653.8884-1-quentin.schulz@free-electrons.com> <20170320081653.8884-2-quentin.schulz@free-electrons.com> Reply-To: lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Content-Disposition: inline In-Reply-To: <20170320081653.8884-2-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Quentin Schulz Cc: sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, wens-jdAy2FN1RRM@public.gmane.org, linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org, maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org, jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, knaack.h-Mmb7MZpHnFY@public.gmane.org, lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org, pmeerw-jW+XmwGofnusTnJN9+BGXg@public.gmane.org, icenowy-ymACFijhrKM@public.gmane.org, liam-RYWXG+zxWwBdeoIcmNTgJF6hYfS7NtTn@public.gmane.org, thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org, linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: devicetree@vger.kernel.org On Mon, 20 Mar 2017, Quentin Schulz wrote: > The registers 0x56 and 0x57 of AXP22X PMIC store the value of the > internal temperature of the PMIC. >=20 > This patch modifies the name of these registers from AXP22X_PMIC_ADC_H/L > to AXP22X_PMIC_TEMP_H/L so their purpose is clearer. >=20 > Signed-off-by: Quentin Schulz > Acked-by: Chen-Yu Tsai > Acked-by: Maxime Ripard > Acked-for-MFD-by: Lee Jones > --- >=20 > added in v2 >=20 > drivers/mfd/axp20x.c | 2 +- > include/linux/mfd/axp20x.h | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) Applied, thanks. > diff --git a/drivers/mfd/axp20x.c b/drivers/mfd/axp20x.c > index 0512900..e02edf1 100644 > --- a/drivers/mfd/axp20x.c > +++ b/drivers/mfd/axp20x.c > @@ -101,7 +101,7 @@ static const struct regmap_range axp22x_volatile_rang= es[] =3D { > regmap_reg_range(AXP20X_PWR_INPUT_STATUS, AXP20X_PWR_OP_MODE), > regmap_reg_range(AXP20X_IRQ1_EN, AXP20X_IRQ5_STATE), > regmap_reg_range(AXP22X_GPIO_STATE, AXP22X_GPIO_STATE), > - regmap_reg_range(AXP22X_PMIC_ADC_H, AXP20X_IPSOUT_V_HIGH_L), > + regmap_reg_range(AXP22X_PMIC_TEMP_H, AXP20X_IPSOUT_V_HIGH_L), > regmap_reg_range(AXP20X_FG_RES, AXP20X_FG_RES), > }; > =20 > diff --git a/include/linux/mfd/axp20x.h b/include/linux/mfd/axp20x.h > index 0d9a1ff..dc8798c 100644 > --- a/include/linux/mfd/axp20x.h > +++ b/include/linux/mfd/axp20x.h > @@ -228,8 +228,8 @@ enum axp20x_variants { > #define AXP20X_OCV_MAX 0xf > =20 > /* AXP22X specific registers */ > -#define AXP22X_PMIC_ADC_H 0x56 > -#define AXP22X_PMIC_ADC_L 0x57 > +#define AXP22X_PMIC_TEMP_H 0x56 > +#define AXP22X_PMIC_TEMP_L 0x57 > #define AXP22X_TS_ADC_H 0x58 > #define AXP22X_TS_ADC_L 0x59 > #define AXP22X_BATLOW_THRES1 0xe6 --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog --=20 You received this message because you are subscribed to the Google Groups "= linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an e= mail to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout.