[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0158A29DB680F54A88142ED28D55B1D0082F77DE@PGSMSX107.gar.corp.intel.com>
Date: Fri, 7 Oct 2016 07:39:26 +0000
From: "Tan, Jui Nee" <jui.nee.tan@...el.com>
To: Lee Jones <lee.jones@...aro.org>
CC: "mika.westerberg@...ux.intel.com" <mika.westerberg@...ux.intel.com>,
"heikki.krogerus@...ux.intel.com" <heikki.krogerus@...ux.intel.com>,
"andriy.shevchenko@...ux.intel.com"
<andriy.shevchenko@...ux.intel.com>,
"tglx@...utronix.de" <tglx@...utronix.de>,
"mingo@...hat.com" <mingo@...hat.com>,
"hpa@...or.com" <hpa@...or.com>, "x86@...nel.org" <x86@...nel.org>,
"ptyser@...-inc.com" <ptyser@...-inc.com>,
"linus.walleij@...aro.org" <linus.walleij@...aro.org>,
"linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"Yong, Jonathan" <jonathan.yong@...el.com>,
"Yu, Ong Hock" <ong.hock.yu@...el.com>,
"Luck, Tony" <tony.luck@...el.com>,
"Wan Mohamad, Wan Ahmad Zainie"
<wan.ahmad.zainie.wan.mohamad@...el.com>
Subject: RE: [PATCH v7 4/6] mfd: move enum lpc_chipsets into lpc_ich.h
> -----Original Message-----
> From: Lee Jones [mailto:lee.jones@...aro.org]
> Sent: Friday, September 30, 2016 8:33 AM
> To: Tan, Jui Nee <jui.nee.tan@...el.com>
> Cc: mika.westerberg@...ux.intel.com; heikki.krogerus@...ux.intel.com;
> andriy.shevchenko@...ux.intel.com; tglx@...utronix.de; mingo@...hat.com;
> hpa@...or.com; x86@...nel.org; ptyser@...-inc.com;
> linus.walleij@...aro.org; linux-gpio@...r.kernel.org; linux-
> kernel@...r.kernel.org; Yong, Jonathan <jonathan.yong@...el.com>; Yu, Ong
> Hock <ong.hock.yu@...el.com>; Luck, Tony <tony.luck@...el.com>; Wan
> Mohamad, Wan Ahmad Zainie <wan.ahmad.zainie.wan.mohamad@...el.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.tan@...el.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, /* 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 */
> > + 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 */
> > +};
> > +
> > #endif
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source
> software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
Powered by blists - more mailing lists