lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ