Re: [PATCH 1/2] thermal: mtk: Cleanup unused defines

From: Zhang Rui
Date: Thu Dec 21 2017 - 08:40:23 EST


On Thu, 2017-12-21 at 11:01 +0100, Matthias Brugger wrote:
>
> On 12/01/2017 11:43 AM, Matthias Brugger wrote:
> >
> > The mtk_thermal has some defiens which are never used within the
> > driver.
> > This patch delets them.
> >
> > Signed-off-by: Matthias Brugger <mbrugger@xxxxxxxx>
> > ---
> Rui, Eduardo, do you have any comments on this patch?
>
No, I don't have any.
I suppose Eduardo will review them soon as I need the thermal soc
material for next merge window.

thanks,
rui
> Regards,
> Matthias
>
> >
> > Âdrivers/thermal/mtk_thermal.c | 9 +--------
> > Â1 file changed, 1 insertion(+), 8 deletions(-)
> >
> > diff --git a/drivers/thermal/mtk_thermal.c
> > b/drivers/thermal/mtk_thermal.c
> > index 1e61c09153c9..c75661a3801a 100644
> > --- a/drivers/thermal/mtk_thermal.c
> > +++ b/drivers/thermal/mtk_thermal.c
> > @@ -32,15 +32,10 @@
> > Â#include <linux/types.h>
> > Â
> > Â/* AUXADC Registers */
> > -#define AUXADC_CON0_V 0x000
> > -#define AUXADC_CON1_V 0x004
> > Â#define AUXADC_CON1_SET_V 0x008
> > Â#define AUXADC_CON1_CLR_V 0x00c
> > Â#define AUXADC_CON2_V 0x010
> > Â#define AUXADC_DATA(channel) (0x14 + (channel) * 4)
> > -#define AUXADC_MISC_V 0x094
> > -
> > -#define AUXADC_CON1_CHANNEL(x) BIT(x)
> > Â
> > Â#define APMIXED_SYS_TS_CON1 0x604
> > Â
> > @@ -158,8 +153,6 @@
> > Â/* The number of sensing points per bank */
> > Â#define MT2712_NUM_SENSORS_PER_ZONE 4
> > Â
> > -#define THERMAL_NAMEÂÂÂÂ"mtk-thermal"
> > -
> > Âstruct mtk_thermal;
> > Â
> > Âstruct thermal_bank_cfg {
> > @@ -765,7 +758,7 @@ static struct platform_driver
> > mtk_thermal_driver = {
> > Â .probe = mtk_thermal_probe,
> > Â .remove = mtk_thermal_remove,
> > Â .driver = {
> > - .name = THERMAL_NAME,
> > + .name = "mtk-thermal",
> > Â .of_match_table = mtk_thermal_of_match,
> > Â },
> > Â};
> >