RE: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h

2016-10-07 Thread Tan, Jui Nee


> -Original Message-
> From: Lee Jones [mailto:lee.jo...@linaro.org]
> Sent: Friday, September 30, 2016 8:33 AM
> To: Tan, Jui Nee <jui.nee@intel.com>
> Cc: mika.westerb...@linux.intel.com; heikki.kroge...@linux.intel.com;
> andriy.shevche...@linux.intel.com; t...@linutronix.de; mi...@redhat.com;
> h...@zytor.com; x...@kernel.org; pty...@xes-inc.com;
> linus.wall...@linaro.org; linux-g...@vger.kernel.org; linux-
> ker...@vger.kernel.org; Yong, Jonathan <jonathan.y...@intel.com>; Yu, Ong
> Hock <ong.hock...@intel.com>; Luck, Tony <tony.l...@intel.com>; Wan
> Mohamad, Wan Ahmad Zainie <wan.ahmad.zainie.wan.moha...@intel.com>
> Subject: Re: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h
> 
> On Wed, 28 Sep 2016, Tan Jui Nee wrote:
> 
> > Move the enum's definition into a standalone header file which can be
> > used wherever its definition is needed.
> >
> > Signed-off-by: Tan Jui Nee <jui.nee@intel.com>
> > ---
> >  drivers/mfd/lpc_ich-core.c  | 71
> > -
> >  include/linux/mfd/lpc_ich.h | 71
> > +
> >  2 files changed, 71 insertions(+), 71 deletions(-)
> 
> Did you use -M when creating this patch?
> 
This patch is to move part of the file and not the entire file to lpc_ich.h. It 
will not show similarity % info even I have used -M option when creating the 
patch.
> > diff --git a/drivers/mfd/lpc_ich-core.c b/drivers/mfd/lpc_ich-core.c
> > index bd3aa45..05ed985 100644
> > --- a/drivers/mfd/lpc_ich-core.c
> > +++ b/drivers/mfd/lpc_ich-core.c
> > @@ -147,77 +147,6 @@ static struct mfd_cell lpc_ich_gpio_cell = {
> > .ignore_resource_conflicts = true,
> >  };
> >
> > -/* chipset related info */
> > -enum lpc_chipsets {
> > -   LPC_ICH = 0,/* ICH */
> > -   LPC_ICH0,   /* ICH0 */
> > -   LPC_ICH2,   /* ICH2 */
> > -   LPC_ICH2M,  /* ICH2-M */
> > -   LPC_ICH3,   /* ICH3-S */
> > -   LPC_ICH3M,  /* ICH3-M */
> > -   LPC_ICH4,   /* ICH4 */
> > -   LPC_ICH4M,  /* ICH4-M */
> > -   LPC_CICH,   /* C-ICH */
> > -   LPC_ICH5,   /* ICH5 & ICH5R */
> > -   LPC_6300ESB,/* 6300ESB */
> > -   LPC_ICH6,   /* ICH6 & ICH6R */
> > -   LPC_ICH6M,  /* ICH6-M */
> > -   LPC_ICH6W,  /* ICH6W & ICH6RW */
> > -   LPC_631XESB,/* 631xESB/632xESB */
> > -   LPC_ICH7,   /* ICH7 & ICH7R */
> > -   LPC_ICH7DH, /* ICH7DH */
> > -   LPC_ICH7M,  /* ICH7-M & ICH7-U */
> > -   LPC_ICH7MDH,/* ICH7-M DH */
> > -   LPC_NM10,   /* NM10 */
> > -   LPC_ICH8,   /* ICH8 & ICH8R */
> > -   LPC_ICH8DH, /* ICH8DH */
> > -   LPC_ICH8DO, /* ICH8DO */
> > -   LPC_ICH8M,  /* ICH8M */
> > -   LPC_ICH8ME, /* ICH8M-E */
> > -   LPC_ICH9,   /* ICH9 */
> > -   LPC_ICH9R,  /* ICH9R */
> > -   LPC_ICH9DH, /* ICH9DH */
> > -   LPC_ICH9DO, /* ICH9DO */
> > -   LPC_ICH9M,  /* ICH9M */
> > -   LPC_ICH9ME, /* ICH9M-E */
> > -   LPC_ICH10,  /* ICH10 */
> > -   LPC_ICH10R, /* ICH10R */
> > -   LPC_ICH10D, /* ICH10D */
> > -   LPC_ICH10DO,/* ICH10DO */
> > -   LPC_PCH,/* PCH Desktop Full Featured */
> > -   LPC_PCHM,   /* PCH Mobile Full Featured */
> > -   LPC_P55,/* P55 */
> > -   LPC_PM55,   /* PM55 */
> > -   LPC_H55,/* H55 */
> > -   LPC_QM57,   /* QM57 */
> > -   LPC_H57,/* H57 */
> > -   LPC_HM55,   /* HM55 */
> > -   LPC_Q57,/* Q57 */
> > -   LPC_HM57,   /* HM57 */
> > -   LPC_PCHMSFF,/* PCH Mobile SFF Full Featured */
> > -   LPC_QS57,   /* QS57 */
> > -   LPC_3400,   /* 3400 */
> > -   LPC_3420,   /* 3420 */
> > -   LPC_3450,   /* 3450 */
> > -   LPC_EP80579,/* EP80579 */
> > -   LPC_CPT,/* Cougar Point */
> > -   LPC_CPTD,   /* Cougar Point Desktop */
> > -   LPC_CPTM,   /* Cougar Point Mobile */
> > -   LPC_PBG,/* Patsburg */
> > -   LPC_DH89XXCC,   /* DH89xxCC */
> > -   LPC_PPT,/* Panther Point */
> > -   LPC_LPT,/* Lynx Point */
> > -   LPC_LPT_LP, /* Lynx Point-LP */
> > -   LPC_WBG,/* Wellsburg */
> > -   LPC_AVN,/* Avoton SoC */
> > -   LPC_BAYTRAIL,   /* Bay Trail SoC */
> > -   LPC_COLETO, /* Coleto Creek */
> > -   LPC_WPT_LP, /* Wildcat Point-LP */
> > -   LPC_BRASWELL,   /* Braswell SoC */
> > -   LPC_LEWISBURG,  /* L

RE: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h

2016-10-07 Thread Tan, Jui Nee


> -Original Message-
> From: Lee Jones [mailto:lee.jo...@linaro.org]
> Sent: Friday, September 30, 2016 8:33 AM
> To: Tan, Jui Nee 
> Cc: mika.westerb...@linux.intel.com; heikki.kroge...@linux.intel.com;
> andriy.shevche...@linux.intel.com; t...@linutronix.de; mi...@redhat.com;
> h...@zytor.com; x...@kernel.org; pty...@xes-inc.com;
> linus.wall...@linaro.org; linux-g...@vger.kernel.org; linux-
> ker...@vger.kernel.org; Yong, Jonathan ; Yu, Ong
> Hock ; Luck, Tony ; Wan
> Mohamad, Wan Ahmad Zainie 
> Subject: Re: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h
> 
> On Wed, 28 Sep 2016, Tan Jui Nee wrote:
> 
> > Move the enum's definition into a standalone header file which can be
> > used wherever its definition is needed.
> >
> > Signed-off-by: Tan Jui Nee 
> > ---
> >  drivers/mfd/lpc_ich-core.c  | 71
> > -
> >  include/linux/mfd/lpc_ich.h | 71
> > +
> >  2 files changed, 71 insertions(+), 71 deletions(-)
> 
> Did you use -M when creating this patch?
> 
This patch is to move part of the file and not the entire file to lpc_ich.h. It 
will not show similarity % info even I have used -M option when creating the 
patch.
> > diff --git a/drivers/mfd/lpc_ich-core.c b/drivers/mfd/lpc_ich-core.c
> > index bd3aa45..05ed985 100644
> > --- a/drivers/mfd/lpc_ich-core.c
> > +++ b/drivers/mfd/lpc_ich-core.c
> > @@ -147,77 +147,6 @@ static struct mfd_cell lpc_ich_gpio_cell = {
> > .ignore_resource_conflicts = true,
> >  };
> >
> > -/* chipset related info */
> > -enum lpc_chipsets {
> > -   LPC_ICH = 0,/* ICH */
> > -   LPC_ICH0,   /* ICH0 */
> > -   LPC_ICH2,   /* ICH2 */
> > -   LPC_ICH2M,  /* ICH2-M */
> > -   LPC_ICH3,   /* ICH3-S */
> > -   LPC_ICH3M,  /* ICH3-M */
> > -   LPC_ICH4,   /* ICH4 */
> > -   LPC_ICH4M,  /* ICH4-M */
> > -   LPC_CICH,   /* C-ICH */
> > -   LPC_ICH5,   /* ICH5 & ICH5R */
> > -   LPC_6300ESB,/* 6300ESB */
> > -   LPC_ICH6,   /* ICH6 & ICH6R */
> > -   LPC_ICH6M,  /* ICH6-M */
> > -   LPC_ICH6W,  /* ICH6W & ICH6RW */
> > -   LPC_631XESB,/* 631xESB/632xESB */
> > -   LPC_ICH7,   /* ICH7 & ICH7R */
> > -   LPC_ICH7DH, /* ICH7DH */
> > -   LPC_ICH7M,  /* ICH7-M & ICH7-U */
> > -   LPC_ICH7MDH,/* ICH7-M DH */
> > -   LPC_NM10,   /* NM10 */
> > -   LPC_ICH8,   /* ICH8 & ICH8R */
> > -   LPC_ICH8DH, /* ICH8DH */
> > -   LPC_ICH8DO, /* ICH8DO */
> > -   LPC_ICH8M,  /* ICH8M */
> > -   LPC_ICH8ME, /* ICH8M-E */
> > -   LPC_ICH9,   /* ICH9 */
> > -   LPC_ICH9R,  /* ICH9R */
> > -   LPC_ICH9DH, /* ICH9DH */
> > -   LPC_ICH9DO, /* ICH9DO */
> > -   LPC_ICH9M,  /* ICH9M */
> > -   LPC_ICH9ME, /* ICH9M-E */
> > -   LPC_ICH10,  /* ICH10 */
> > -   LPC_ICH10R, /* ICH10R */
> > -   LPC_ICH10D, /* ICH10D */
> > -   LPC_ICH10DO,/* ICH10DO */
> > -   LPC_PCH,/* PCH Desktop Full Featured */
> > -   LPC_PCHM,   /* PCH Mobile Full Featured */
> > -   LPC_P55,/* P55 */
> > -   LPC_PM55,   /* PM55 */
> > -   LPC_H55,/* H55 */
> > -   LPC_QM57,   /* QM57 */
> > -   LPC_H57,/* H57 */
> > -   LPC_HM55,   /* HM55 */
> > -   LPC_Q57,/* Q57 */
> > -   LPC_HM57,   /* HM57 */
> > -   LPC_PCHMSFF,/* PCH Mobile SFF Full Featured */
> > -   LPC_QS57,   /* QS57 */
> > -   LPC_3400,   /* 3400 */
> > -   LPC_3420,   /* 3420 */
> > -   LPC_3450,   /* 3450 */
> > -   LPC_EP80579,/* EP80579 */
> > -   LPC_CPT,/* Cougar Point */
> > -   LPC_CPTD,   /* Cougar Point Desktop */
> > -   LPC_CPTM,   /* Cougar Point Mobile */
> > -   LPC_PBG,/* Patsburg */
> > -   LPC_DH89XXCC,   /* DH89xxCC */
> > -   LPC_PPT,/* Panther Point */
> > -   LPC_LPT,/* Lynx Point */
> > -   LPC_LPT_LP, /* Lynx Point-LP */
> > -   LPC_WBG,/* Wellsburg */
> > -   LPC_AVN,/* Avoton SoC */
> > -   LPC_BAYTRAIL,   /* Bay Trail SoC */
> > -   LPC_COLETO, /* Coleto Creek */
> > -   LPC_WPT_LP, /* Wildcat Point-LP */
> > -   LPC_BRASWELL,   /* Braswell SoC */
> > -   LPC_LEWISBURG,  /* Lewisburg */
> > -   LPC_9S, /* 9 Series */
> > -};
> > -
> >  static struct lpc_ich_info lpc_chipset_info[] = {
> > [LPC_ICH] = {
> > .name = 

Re: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h

2016-09-29 Thread Lee Jones
On Wed, 28 Sep 2016, Tan Jui Nee wrote:

> Move the enum's definition into a standalone header file which can be used
> wherever its definition is needed.
> 
> Signed-off-by: Tan Jui Nee 
> ---
>  drivers/mfd/lpc_ich-core.c  | 71 
> -
>  include/linux/mfd/lpc_ich.h | 71 
> +
>  2 files changed, 71 insertions(+), 71 deletions(-)

Did you use -M when creating this patch?

> diff --git a/drivers/mfd/lpc_ich-core.c b/drivers/mfd/lpc_ich-core.c
> index bd3aa45..05ed985 100644
> --- a/drivers/mfd/lpc_ich-core.c
> +++ b/drivers/mfd/lpc_ich-core.c
> @@ -147,77 +147,6 @@ static struct mfd_cell lpc_ich_gpio_cell = {
>   .ignore_resource_conflicts = true,
>  };
>  
> -/* chipset related info */
> -enum lpc_chipsets {
> - LPC_ICH = 0,/* ICH */
> - LPC_ICH0,   /* ICH0 */
> - LPC_ICH2,   /* ICH2 */
> - LPC_ICH2M,  /* ICH2-M */
> - LPC_ICH3,   /* ICH3-S */
> - LPC_ICH3M,  /* ICH3-M */
> - LPC_ICH4,   /* ICH4 */
> - LPC_ICH4M,  /* ICH4-M */
> - LPC_CICH,   /* C-ICH */
> - LPC_ICH5,   /* ICH5 & ICH5R */
> - LPC_6300ESB,/* 6300ESB */
> - LPC_ICH6,   /* ICH6 & ICH6R */
> - LPC_ICH6M,  /* ICH6-M */
> - LPC_ICH6W,  /* ICH6W & ICH6RW */
> - LPC_631XESB,/* 631xESB/632xESB */
> - LPC_ICH7,   /* ICH7 & ICH7R */
> - LPC_ICH7DH, /* ICH7DH */
> - LPC_ICH7M,  /* ICH7-M & ICH7-U */
> - LPC_ICH7MDH,/* ICH7-M DH */
> - LPC_NM10,   /* NM10 */
> - LPC_ICH8,   /* ICH8 & ICH8R */
> - LPC_ICH8DH, /* ICH8DH */
> - LPC_ICH8DO, /* ICH8DO */
> - LPC_ICH8M,  /* ICH8M */
> - LPC_ICH8ME, /* ICH8M-E */
> - LPC_ICH9,   /* ICH9 */
> - LPC_ICH9R,  /* ICH9R */
> - LPC_ICH9DH, /* ICH9DH */
> - LPC_ICH9DO, /* ICH9DO */
> - LPC_ICH9M,  /* ICH9M */
> - LPC_ICH9ME, /* ICH9M-E */
> - LPC_ICH10,  /* ICH10 */
> - LPC_ICH10R, /* ICH10R */
> - LPC_ICH10D, /* ICH10D */
> - LPC_ICH10DO,/* ICH10DO */
> - LPC_PCH,/* PCH Desktop Full Featured */
> - LPC_PCHM,   /* PCH Mobile Full Featured */
> - LPC_P55,/* P55 */
> - LPC_PM55,   /* PM55 */
> - LPC_H55,/* H55 */
> - LPC_QM57,   /* QM57 */
> - LPC_H57,/* H57 */
> - LPC_HM55,   /* HM55 */
> - LPC_Q57,/* Q57 */
> - LPC_HM57,   /* HM57 */
> - LPC_PCHMSFF,/* PCH Mobile SFF Full Featured */
> - LPC_QS57,   /* QS57 */
> - LPC_3400,   /* 3400 */
> - LPC_3420,   /* 3420 */
> - LPC_3450,   /* 3450 */
> - LPC_EP80579,/* EP80579 */
> - LPC_CPT,/* Cougar Point */
> - LPC_CPTD,   /* Cougar Point Desktop */
> - LPC_CPTM,   /* Cougar Point Mobile */
> - LPC_PBG,/* Patsburg */
> - LPC_DH89XXCC,   /* DH89xxCC */
> - LPC_PPT,/* Panther Point */
> - LPC_LPT,/* Lynx Point */
> - LPC_LPT_LP, /* Lynx Point-LP */
> - LPC_WBG,/* Wellsburg */
> - LPC_AVN,/* Avoton SoC */
> - LPC_BAYTRAIL,   /* Bay Trail SoC */
> - LPC_COLETO, /* Coleto Creek */
> - LPC_WPT_LP, /* Wildcat Point-LP */
> - LPC_BRASWELL,   /* Braswell SoC */
> - LPC_LEWISBURG,  /* Lewisburg */
> - LPC_9S, /* 9 Series */
> -};
> -
>  static struct lpc_ich_info lpc_chipset_info[] = {
>   [LPC_ICH] = {
>   .name = "ICH",
> diff --git a/include/linux/mfd/lpc_ich.h b/include/linux/mfd/lpc_ich.h
> index 2b300b4..42307ee 100644
> --- a/include/linux/mfd/lpc_ich.h
> +++ b/include/linux/mfd/lpc_ich.h
> @@ -43,4 +43,75 @@ struct lpc_ich_info {
>   u8 use_gpio;
>  };
>  
> +/* chipset related info */
> +enum lpc_chipsets {
> + LPC_ICH = 0,/* ICH */
> + LPC_ICH0,   /* ICH0 */
> + LPC_ICH2,   /* ICH2 */
> + LPC_ICH2M,  /* ICH2-M */
> + LPC_ICH3,   /* ICH3-S */
> + LPC_ICH3M,  /* ICH3-M */
> + LPC_ICH4,   /* ICH4 */
> + LPC_ICH4M,  /* ICH4-M */
> + LPC_CICH,   /* C-ICH */
> + LPC_ICH5,   /* ICH5 & ICH5R */
> + LPC_6300ESB,/* 6300ESB */
> + LPC_ICH6,   /* ICH6 & ICH6R */
> + LPC_ICH6M,  /* ICH6-M */
> + LPC_ICH6W,  /* ICH6W & ICH6RW */
> + LPC_631XESB,/* 631xESB/632xESB */
> + LPC_ICH7,   /* ICH7 & ICH7R */
> + LPC_ICH7DH, /* ICH7DH */
> + LPC_ICH7M,  /* ICH7-M & ICH7-U */
> + LPC_ICH7MDH,/* ICH7-M DH */
> + LPC_NM10,   /* NM10 */
> + LPC_ICH8,   /* ICH8 & ICH8R */
> + LPC_ICH8DH, /* ICH8DH */
> + LPC_ICH8DO, /* ICH8DO */
> + LPC_ICH8M,  /* ICH8M */
> + LPC_ICH8ME, /* ICH8M-E */
> + LPC_ICH9,   /* ICH9 */
> + LPC_ICH9R,  /* ICH9R */
> + LPC_ICH9DH, /* ICH9DH */
> + LPC_ICH9DO, /* 

Re: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h

2016-09-29 Thread Lee Jones
On Wed, 28 Sep 2016, Tan Jui Nee wrote:

> Move the enum's definition into a standalone header file which can be used
> wherever its definition is needed.
> 
> Signed-off-by: Tan Jui Nee 
> ---
>  drivers/mfd/lpc_ich-core.c  | 71 
> -
>  include/linux/mfd/lpc_ich.h | 71 
> +
>  2 files changed, 71 insertions(+), 71 deletions(-)

Did you use -M when creating this patch?

> diff --git a/drivers/mfd/lpc_ich-core.c b/drivers/mfd/lpc_ich-core.c
> index bd3aa45..05ed985 100644
> --- a/drivers/mfd/lpc_ich-core.c
> +++ b/drivers/mfd/lpc_ich-core.c
> @@ -147,77 +147,6 @@ static struct mfd_cell lpc_ich_gpio_cell = {
>   .ignore_resource_conflicts = true,
>  };
>  
> -/* chipset related info */
> -enum lpc_chipsets {
> - LPC_ICH = 0,/* ICH */
> - LPC_ICH0,   /* ICH0 */
> - LPC_ICH2,   /* ICH2 */
> - LPC_ICH2M,  /* ICH2-M */
> - LPC_ICH3,   /* ICH3-S */
> - LPC_ICH3M,  /* ICH3-M */
> - LPC_ICH4,   /* ICH4 */
> - LPC_ICH4M,  /* ICH4-M */
> - LPC_CICH,   /* C-ICH */
> - LPC_ICH5,   /* ICH5 & ICH5R */
> - LPC_6300ESB,/* 6300ESB */
> - LPC_ICH6,   /* ICH6 & ICH6R */
> - LPC_ICH6M,  /* ICH6-M */
> - LPC_ICH6W,  /* ICH6W & ICH6RW */
> - LPC_631XESB,/* 631xESB/632xESB */
> - LPC_ICH7,   /* ICH7 & ICH7R */
> - LPC_ICH7DH, /* ICH7DH */
> - LPC_ICH7M,  /* ICH7-M & ICH7-U */
> - LPC_ICH7MDH,/* ICH7-M DH */
> - LPC_NM10,   /* NM10 */
> - LPC_ICH8,   /* ICH8 & ICH8R */
> - LPC_ICH8DH, /* ICH8DH */
> - LPC_ICH8DO, /* ICH8DO */
> - LPC_ICH8M,  /* ICH8M */
> - LPC_ICH8ME, /* ICH8M-E */
> - LPC_ICH9,   /* ICH9 */
> - LPC_ICH9R,  /* ICH9R */
> - LPC_ICH9DH, /* ICH9DH */
> - LPC_ICH9DO, /* ICH9DO */
> - LPC_ICH9M,  /* ICH9M */
> - LPC_ICH9ME, /* ICH9M-E */
> - LPC_ICH10,  /* ICH10 */
> - LPC_ICH10R, /* ICH10R */
> - LPC_ICH10D, /* ICH10D */
> - LPC_ICH10DO,/* ICH10DO */
> - LPC_PCH,/* PCH Desktop Full Featured */
> - LPC_PCHM,   /* PCH Mobile Full Featured */
> - LPC_P55,/* P55 */
> - LPC_PM55,   /* PM55 */
> - LPC_H55,/* H55 */
> - LPC_QM57,   /* QM57 */
> - LPC_H57,/* H57 */
> - LPC_HM55,   /* HM55 */
> - LPC_Q57,/* Q57 */
> - LPC_HM57,   /* HM57 */
> - LPC_PCHMSFF,/* PCH Mobile SFF Full Featured */
> - LPC_QS57,   /* QS57 */
> - LPC_3400,   /* 3400 */
> - LPC_3420,   /* 3420 */
> - LPC_3450,   /* 3450 */
> - LPC_EP80579,/* EP80579 */
> - LPC_CPT,/* Cougar Point */
> - LPC_CPTD,   /* Cougar Point Desktop */
> - LPC_CPTM,   /* Cougar Point Mobile */
> - LPC_PBG,/* Patsburg */
> - LPC_DH89XXCC,   /* DH89xxCC */
> - LPC_PPT,/* Panther Point */
> - LPC_LPT,/* Lynx Point */
> - LPC_LPT_LP, /* Lynx Point-LP */
> - LPC_WBG,/* Wellsburg */
> - LPC_AVN,/* Avoton SoC */
> - LPC_BAYTRAIL,   /* Bay Trail SoC */
> - LPC_COLETO, /* Coleto Creek */
> - LPC_WPT_LP, /* Wildcat Point-LP */
> - LPC_BRASWELL,   /* Braswell SoC */
> - LPC_LEWISBURG,  /* Lewisburg */
> - LPC_9S, /* 9 Series */
> -};
> -
>  static struct lpc_ich_info lpc_chipset_info[] = {
>   [LPC_ICH] = {
>   .name = "ICH",
> diff --git a/include/linux/mfd/lpc_ich.h b/include/linux/mfd/lpc_ich.h
> index 2b300b4..42307ee 100644
> --- a/include/linux/mfd/lpc_ich.h
> +++ b/include/linux/mfd/lpc_ich.h
> @@ -43,4 +43,75 @@ struct lpc_ich_info {
>   u8 use_gpio;
>  };
>  
> +/* chipset related info */
> +enum lpc_chipsets {
> + LPC_ICH = 0,/* ICH */
> + LPC_ICH0,   /* ICH0 */
> + LPC_ICH2,   /* ICH2 */
> + LPC_ICH2M,  /* ICH2-M */
> + LPC_ICH3,   /* ICH3-S */
> + LPC_ICH3M,  /* ICH3-M */
> + LPC_ICH4,   /* ICH4 */
> + LPC_ICH4M,  /* ICH4-M */
> + LPC_CICH,   /* C-ICH */
> + LPC_ICH5,   /* ICH5 & ICH5R */
> + LPC_6300ESB,/* 6300ESB */
> + LPC_ICH6,   /* ICH6 & ICH6R */
> + LPC_ICH6M,  /* ICH6-M */
> + LPC_ICH6W,  /* ICH6W & ICH6RW */
> + LPC_631XESB,/* 631xESB/632xESB */
> + LPC_ICH7,   /* ICH7 & ICH7R */
> + LPC_ICH7DH, /* ICH7DH */
> + LPC_ICH7M,  /* ICH7-M & ICH7-U */
> + LPC_ICH7MDH,/* ICH7-M DH */
> + LPC_NM10,   /* NM10 */
> + LPC_ICH8,   /* ICH8 & ICH8R */
> + LPC_ICH8DH, /* ICH8DH */
> + LPC_ICH8DO, /* ICH8DO */
> + LPC_ICH8M,  /* ICH8M */
> + LPC_ICH8ME, /* ICH8M-E */
> + LPC_ICH9,   /* ICH9 */
> + LPC_ICH9R,  /* ICH9R */
> + LPC_ICH9DH, /* ICH9DH */
> + LPC_ICH9DO, /* ICH9DO */
> + 

[PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h

2016-09-28 Thread Tan Jui Nee
Move the enum's definition into a standalone header file which can be used
wherever its definition is needed.

Signed-off-by: Tan Jui Nee 
---
 drivers/mfd/lpc_ich-core.c  | 71 -
 include/linux/mfd/lpc_ich.h | 71 +
 2 files changed, 71 insertions(+), 71 deletions(-)

diff --git a/drivers/mfd/lpc_ich-core.c b/drivers/mfd/lpc_ich-core.c
index bd3aa45..05ed985 100644
--- a/drivers/mfd/lpc_ich-core.c
+++ b/drivers/mfd/lpc_ich-core.c
@@ -147,77 +147,6 @@ static struct mfd_cell lpc_ich_gpio_cell = {
.ignore_resource_conflicts = true,
 };
 
-/* chipset related info */
-enum lpc_chipsets {
-   LPC_ICH = 0,/* ICH */
-   LPC_ICH0,   /* ICH0 */
-   LPC_ICH2,   /* ICH2 */
-   LPC_ICH2M,  /* ICH2-M */
-   LPC_ICH3,   /* ICH3-S */
-   LPC_ICH3M,  /* ICH3-M */
-   LPC_ICH4,   /* ICH4 */
-   LPC_ICH4M,  /* ICH4-M */
-   LPC_CICH,   /* C-ICH */
-   LPC_ICH5,   /* ICH5 & ICH5R */
-   LPC_6300ESB,/* 6300ESB */
-   LPC_ICH6,   /* ICH6 & ICH6R */
-   LPC_ICH6M,  /* ICH6-M */
-   LPC_ICH6W,  /* ICH6W & ICH6RW */
-   LPC_631XESB,/* 631xESB/632xESB */
-   LPC_ICH7,   /* ICH7 & ICH7R */
-   LPC_ICH7DH, /* ICH7DH */
-   LPC_ICH7M,  /* ICH7-M & ICH7-U */
-   LPC_ICH7MDH,/* ICH7-M DH */
-   LPC_NM10,   /* NM10 */
-   LPC_ICH8,   /* ICH8 & ICH8R */
-   LPC_ICH8DH, /* ICH8DH */
-   LPC_ICH8DO, /* ICH8DO */
-   LPC_ICH8M,  /* ICH8M */
-   LPC_ICH8ME, /* ICH8M-E */
-   LPC_ICH9,   /* ICH9 */
-   LPC_ICH9R,  /* ICH9R */
-   LPC_ICH9DH, /* ICH9DH */
-   LPC_ICH9DO, /* ICH9DO */
-   LPC_ICH9M,  /* ICH9M */
-   LPC_ICH9ME, /* ICH9M-E */
-   LPC_ICH10,  /* ICH10 */
-   LPC_ICH10R, /* ICH10R */
-   LPC_ICH10D, /* ICH10D */
-   LPC_ICH10DO,/* ICH10DO */
-   LPC_PCH,/* PCH Desktop Full Featured */
-   LPC_PCHM,   /* PCH Mobile Full Featured */
-   LPC_P55,/* P55 */
-   LPC_PM55,   /* PM55 */
-   LPC_H55,/* H55 */
-   LPC_QM57,   /* QM57 */
-   LPC_H57,/* H57 */
-   LPC_HM55,   /* HM55 */
-   LPC_Q57,/* Q57 */
-   LPC_HM57,   /* HM57 */
-   LPC_PCHMSFF,/* PCH Mobile SFF Full Featured */
-   LPC_QS57,   /* QS57 */
-   LPC_3400,   /* 3400 */
-   LPC_3420,   /* 3420 */
-   LPC_3450,   /* 3450 */
-   LPC_EP80579,/* EP80579 */
-   LPC_CPT,/* Cougar Point */
-   LPC_CPTD,   /* Cougar Point Desktop */
-   LPC_CPTM,   /* Cougar Point Mobile */
-   LPC_PBG,/* Patsburg */
-   LPC_DH89XXCC,   /* DH89xxCC */
-   LPC_PPT,/* Panther Point */
-   LPC_LPT,/* Lynx Point */
-   LPC_LPT_LP, /* Lynx Point-LP */
-   LPC_WBG,/* Wellsburg */
-   LPC_AVN,/* Avoton SoC */
-   LPC_BAYTRAIL,   /* Bay Trail SoC */
-   LPC_COLETO, /* Coleto Creek */
-   LPC_WPT_LP, /* Wildcat Point-LP */
-   LPC_BRASWELL,   /* Braswell SoC */
-   LPC_LEWISBURG,  /* Lewisburg */
-   LPC_9S, /* 9 Series */
-};
-
 static struct lpc_ich_info lpc_chipset_info[] = {
[LPC_ICH] = {
.name = "ICH",
diff --git a/include/linux/mfd/lpc_ich.h b/include/linux/mfd/lpc_ich.h
index 2b300b4..42307ee 100644
--- a/include/linux/mfd/lpc_ich.h
+++ b/include/linux/mfd/lpc_ich.h
@@ -43,4 +43,75 @@ struct lpc_ich_info {
u8 use_gpio;
 };
 
+/* chipset related info */
+enum lpc_chipsets {
+   LPC_ICH = 0,/* ICH */
+   LPC_ICH0,   /* ICH0 */
+   LPC_ICH2,   /* ICH2 */
+   LPC_ICH2M,  /* ICH2-M */
+   LPC_ICH3,   /* ICH3-S */
+   LPC_ICH3M,  /* ICH3-M */
+   LPC_ICH4,   /* ICH4 */
+   LPC_ICH4M,  /* ICH4-M */
+   LPC_CICH,   /* C-ICH */
+   LPC_ICH5,   /* ICH5 & ICH5R */
+   LPC_6300ESB,/* 6300ESB */
+   LPC_ICH6,   /* ICH6 & ICH6R */
+   LPC_ICH6M,  /* ICH6-M */
+   LPC_ICH6W,  /* ICH6W & ICH6RW */
+   LPC_631XESB,/* 631xESB/632xESB */
+   LPC_ICH7,   /* ICH7 & ICH7R */
+   LPC_ICH7DH, /* ICH7DH */
+   LPC_ICH7M,  /* ICH7-M & ICH7-U */
+   LPC_ICH7MDH,/* ICH7-M DH */
+   LPC_NM10,   /* NM10 */
+   LPC_ICH8,   /* ICH8 & ICH8R */
+   LPC_ICH8DH, /* ICH8DH */
+   LPC_ICH8DO, /* ICH8DO */
+   LPC_ICH8M,  /* ICH8M */
+   LPC_ICH8ME, /* ICH8M-E */
+   LPC_ICH9,   /* ICH9 */
+   LPC_ICH9R,  /* ICH9R */
+   LPC_ICH9DH, /* ICH9DH */
+   LPC_ICH9DO, /* ICH9DO */
+   LPC_ICH9M,  /* ICH9M */
+   LPC_ICH9ME, /* ICH9M-E */
+   LPC_ICH10,  /* ICH10 */
+   LPC_ICH10R, /* 

[PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h

2016-09-28 Thread Tan Jui Nee
Move the enum's definition into a standalone header file which can be used
wherever its definition is needed.

Signed-off-by: Tan Jui Nee 
---
 drivers/mfd/lpc_ich-core.c  | 71 -
 include/linux/mfd/lpc_ich.h | 71 +
 2 files changed, 71 insertions(+), 71 deletions(-)

diff --git a/drivers/mfd/lpc_ich-core.c b/drivers/mfd/lpc_ich-core.c
index bd3aa45..05ed985 100644
--- a/drivers/mfd/lpc_ich-core.c
+++ b/drivers/mfd/lpc_ich-core.c
@@ -147,77 +147,6 @@ static struct mfd_cell lpc_ich_gpio_cell = {
.ignore_resource_conflicts = true,
 };
 
-/* chipset related info */
-enum lpc_chipsets {
-   LPC_ICH = 0,/* ICH */
-   LPC_ICH0,   /* ICH0 */
-   LPC_ICH2,   /* ICH2 */
-   LPC_ICH2M,  /* ICH2-M */
-   LPC_ICH3,   /* ICH3-S */
-   LPC_ICH3M,  /* ICH3-M */
-   LPC_ICH4,   /* ICH4 */
-   LPC_ICH4M,  /* ICH4-M */
-   LPC_CICH,   /* C-ICH */
-   LPC_ICH5,   /* ICH5 & ICH5R */
-   LPC_6300ESB,/* 6300ESB */
-   LPC_ICH6,   /* ICH6 & ICH6R */
-   LPC_ICH6M,  /* ICH6-M */
-   LPC_ICH6W,  /* ICH6W & ICH6RW */
-   LPC_631XESB,/* 631xESB/632xESB */
-   LPC_ICH7,   /* ICH7 & ICH7R */
-   LPC_ICH7DH, /* ICH7DH */
-   LPC_ICH7M,  /* ICH7-M & ICH7-U */
-   LPC_ICH7MDH,/* ICH7-M DH */
-   LPC_NM10,   /* NM10 */
-   LPC_ICH8,   /* ICH8 & ICH8R */
-   LPC_ICH8DH, /* ICH8DH */
-   LPC_ICH8DO, /* ICH8DO */
-   LPC_ICH8M,  /* ICH8M */
-   LPC_ICH8ME, /* ICH8M-E */
-   LPC_ICH9,   /* ICH9 */
-   LPC_ICH9R,  /* ICH9R */
-   LPC_ICH9DH, /* ICH9DH */
-   LPC_ICH9DO, /* ICH9DO */
-   LPC_ICH9M,  /* ICH9M */
-   LPC_ICH9ME, /* ICH9M-E */
-   LPC_ICH10,  /* ICH10 */
-   LPC_ICH10R, /* ICH10R */
-   LPC_ICH10D, /* ICH10D */
-   LPC_ICH10DO,/* ICH10DO */
-   LPC_PCH,/* PCH Desktop Full Featured */
-   LPC_PCHM,   /* PCH Mobile Full Featured */
-   LPC_P55,/* P55 */
-   LPC_PM55,   /* PM55 */
-   LPC_H55,/* H55 */
-   LPC_QM57,   /* QM57 */
-   LPC_H57,/* H57 */
-   LPC_HM55,   /* HM55 */
-   LPC_Q57,/* Q57 */
-   LPC_HM57,   /* HM57 */
-   LPC_PCHMSFF,/* PCH Mobile SFF Full Featured */
-   LPC_QS57,   /* QS57 */
-   LPC_3400,   /* 3400 */
-   LPC_3420,   /* 3420 */
-   LPC_3450,   /* 3450 */
-   LPC_EP80579,/* EP80579 */
-   LPC_CPT,/* Cougar Point */
-   LPC_CPTD,   /* Cougar Point Desktop */
-   LPC_CPTM,   /* Cougar Point Mobile */
-   LPC_PBG,/* Patsburg */
-   LPC_DH89XXCC,   /* DH89xxCC */
-   LPC_PPT,/* Panther Point */
-   LPC_LPT,/* Lynx Point */
-   LPC_LPT_LP, /* Lynx Point-LP */
-   LPC_WBG,/* Wellsburg */
-   LPC_AVN,/* Avoton SoC */
-   LPC_BAYTRAIL,   /* Bay Trail SoC */
-   LPC_COLETO, /* Coleto Creek */
-   LPC_WPT_LP, /* Wildcat Point-LP */
-   LPC_BRASWELL,   /* Braswell SoC */
-   LPC_LEWISBURG,  /* Lewisburg */
-   LPC_9S, /* 9 Series */
-};
-
 static struct lpc_ich_info lpc_chipset_info[] = {
[LPC_ICH] = {
.name = "ICH",
diff --git a/include/linux/mfd/lpc_ich.h b/include/linux/mfd/lpc_ich.h
index 2b300b4..42307ee 100644
--- a/include/linux/mfd/lpc_ich.h
+++ b/include/linux/mfd/lpc_ich.h
@@ -43,4 +43,75 @@ struct lpc_ich_info {
u8 use_gpio;
 };
 
+/* chipset related info */
+enum lpc_chipsets {
+   LPC_ICH = 0,/* ICH */
+   LPC_ICH0,   /* ICH0 */
+   LPC_ICH2,   /* ICH2 */
+   LPC_ICH2M,  /* ICH2-M */
+   LPC_ICH3,   /* ICH3-S */
+   LPC_ICH3M,  /* ICH3-M */
+   LPC_ICH4,   /* ICH4 */
+   LPC_ICH4M,  /* ICH4-M */
+   LPC_CICH,   /* C-ICH */
+   LPC_ICH5,   /* ICH5 & ICH5R */
+   LPC_6300ESB,/* 6300ESB */
+   LPC_ICH6,   /* ICH6 & ICH6R */
+   LPC_ICH6M,  /* ICH6-M */
+   LPC_ICH6W,  /* ICH6W & ICH6RW */
+   LPC_631XESB,/* 631xESB/632xESB */
+   LPC_ICH7,   /* ICH7 & ICH7R */
+   LPC_ICH7DH, /* ICH7DH */
+   LPC_ICH7M,  /* ICH7-M & ICH7-U */
+   LPC_ICH7MDH,/* ICH7-M DH */
+   LPC_NM10,   /* NM10 */
+   LPC_ICH8,   /* ICH8 & ICH8R */
+   LPC_ICH8DH, /* ICH8DH */
+   LPC_ICH8DO, /* ICH8DO */
+   LPC_ICH8M,  /* ICH8M */
+   LPC_ICH8ME, /* ICH8M-E */
+   LPC_ICH9,   /* ICH9 */
+   LPC_ICH9R,  /* ICH9R */
+   LPC_ICH9DH, /* ICH9DH */
+   LPC_ICH9DO, /* ICH9DO */
+   LPC_ICH9M,  /* ICH9M */
+   LPC_ICH9ME, /* ICH9M-E */
+   LPC_ICH10,  /* ICH10 */
+   LPC_ICH10R, /* ICH10R */
+