[PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH

[PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH

[Andy Shevchenko]

Adding Intel codename Apollo Lake platform device IDs for PCH.

Signed-off-by: Tan Jui Nee <jui.nee.tan@intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/mfd/lpc_ich.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/mfd/lpc_ich.c b/drivers/mfd/lpc_ich.c
index 1ef7575547e6..262ddd9330b7 100644
--- a/drivers/mfd/lpc_ich.c
+++ b/drivers/mfd/lpc_ich.c
@@ -216,6 +216,7 @@ enum lpc_chipsets {
 	LPC_BRASWELL,	/* Braswell SoC */
 	LPC_LEWISBURG,	/* Lewisburg */
 	LPC_9S,		/* 9 Series */
+	LPC_APL,	/* Apollo Lake SoC */
 };
 
 static struct lpc_ich_info lpc_chipset_info[] = {
@@ -533,6 +534,10 @@ static struct lpc_ich_info lpc_chipset_info[] = {
 		.iTCO_version = 2,
 		.gpio_version = ICH_V5_GPIO,
 	},
+	[LPC_APL]  = {
+		.name = "Apollo Lake SoC",
+		.iTCO_version = 5,
+	},
 };
 
 /*
@@ -681,6 +686,7 @@ static const struct pci_device_id lpc_ich_ids[] = {
 	{ PCI_VDEVICE(INTEL, 0x3b14), LPC_3420},
 	{ PCI_VDEVICE(INTEL, 0x3b16), LPC_3450},
 	{ PCI_VDEVICE(INTEL, 0x5031), LPC_EP80579},
+	{ PCI_VDEVICE(INTEL, 0x5ae8), LPC_APL},
 	{ PCI_VDEVICE(INTEL, 0x8c40), LPC_LPT},
 	{ PCI_VDEVICE(INTEL, 0x8c41), LPC_LPT},
 	{ PCI_VDEVICE(INTEL, 0x8c42), LPC_LPT},
-- 
2.11.0

[PATCH v1 2/2] mfd: lpc_ich: Remove useless comments in core part

[Andy Shevchenko]

First of all, remove stalled references to datasheets. If someone knows
the document numbers, it would be added later.

Second, remove FSF snail address since it's subject to change. Actual
information can be found on FSF site on the internet.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/mfd/lpc_ich.c | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/drivers/mfd/lpc_ich.c b/drivers/mfd/lpc_ich.c
index 262ddd9330b7..0350e14ff84b 100644
--- a/drivers/mfd/lpc_ich.c
+++ b/drivers/mfd/lpc_ich.c
@@ -20,10 +20,6 @@
  *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  *  GNU General Public License for more details.
  *
- *  You should have received a copy of the GNU General Public License
- *  along with this program; see the file COPYING.  If not, write to
- *  the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
- *
  *  This driver supports the following I/O Controller hubs:
  *	(See the intel documentation on http://developer.intel.com.)
  *	document number 290655-003, 290677-014: 82801AA (ICH), 82801AB (ICHO)
@@ -45,17 +41,6 @@
  *	document number 322169-001, 322170-003: 5 Series, 3400 Series (PCH)
  *	document number 320066-003, 320257-008: EP80597 (IICH)
  *	document number 324645-001, 324646-001: Cougar Point (CPT)
- *	document number TBD : Patsburg (PBG)
- *	document number TBD : DH89xxCC
- *	document number TBD : Panther Point
- *	document number TBD : Lynx Point
- *	document number TBD : Lynx Point-LP
- *	document number TBD : Wellsburg
- *	document number TBD : Avoton SoC
- *	document number TBD : Coleto Creek
- *	document number TBD : Wildcat Point-LP
- *	document number TBD : 9 Series
- *	document number TBD : Lewisburg
  */
 
 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
-- 
2.11.0

Re: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH

[Andy Shevchenko]

On Wed, Jan 25, 2017 at 10:18 PM, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
> Adding Intel codename Apollo Lake platform device IDs for PCH.
>

Ah, now this is in conflict with Mika's SPI-NOR patchset.
It's strange I have no such conflict locally. Seems linux-next missed
that series / tree somehow.

> Signed-off-by: Tan Jui Nee <jui.nee.tan@intel.com>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  drivers/mfd/lpc_ich.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/drivers/mfd/lpc_ich.c b/drivers/mfd/lpc_ich.c
> index 1ef7575547e6..262ddd9330b7 100644
> --- a/drivers/mfd/lpc_ich.c
> +++ b/drivers/mfd/lpc_ich.c
> @@ -216,6 +216,7 @@ enum lpc_chipsets {
>         LPC_BRASWELL,   /* Braswell SoC */
>         LPC_LEWISBURG,  /* Lewisburg */
>         LPC_9S,         /* 9 Series */
> +       LPC_APL,        /* Apollo Lake SoC */
>  };
>
>  static struct lpc_ich_info lpc_chipset_info[] = {
> @@ -533,6 +534,10 @@ static struct lpc_ich_info lpc_chipset_info[] = {
>                 .iTCO_version = 2,
>                 .gpio_version = ICH_V5_GPIO,
>         },
> +       [LPC_APL]  = {
> +               .name = "Apollo Lake SoC",
> +               .iTCO_version = 5,
> +       },
>  };
>
>  /*
> @@ -681,6 +686,7 @@ static const struct pci_device_id lpc_ich_ids[] = {
>         { PCI_VDEVICE(INTEL, 0x3b14), LPC_3420},
>         { PCI_VDEVICE(INTEL, 0x3b16), LPC_3450},
>         { PCI_VDEVICE(INTEL, 0x5031), LPC_EP80579},
> +       { PCI_VDEVICE(INTEL, 0x5ae8), LPC_APL},
>         { PCI_VDEVICE(INTEL, 0x8c40), LPC_LPT},
>         { PCI_VDEVICE(INTEL, 0x8c41), LPC_LPT},
>         { PCI_VDEVICE(INTEL, 0x8c42), LPC_LPT},
> --
> 2.11.0
>



-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH

[Mika Westerberg]

On Wed, Jan 25, 2017 at 10:18:28PM +0200, Andy Shevchenko wrote:
> Adding Intel codename Apollo Lake platform device IDs for PCH.

Didn't this originate from Tan Jui Nee and not from you? Please make
sure you have the correct From: here and the other patch as well.

You can use 'git commit --amend --author' for fixing it.

Re: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH

[Andy Shevchenko]

On Thu, Jan 26, 2017 at 11:24 AM, Mika Westerberg
<mika.westerberg@linux.intel.com> wrote:
> On Wed, Jan 25, 2017 at 10:18:28PM +0200, Andy Shevchenko wrote:
>> Adding Intel codename Apollo Lake platform device IDs for PCH.
>
> Didn't this originate from Tan Jui Nee and not from you? Please make
> sure you have the correct From: here and the other patch as well.

Oh, right, thanks. I changed a lot in the whole series, though this
one must keep original authorship. I will fix this as well.

-- 
With Best Regards,
Andy Shevchenko