Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1566904 > unrolled thread

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

Started byAndy Shevchenko <andriy.shevchenko@linux.intel.com>
First post2017-01-25 21:20 +0100
Last post2017-01-26 11:40 +0100
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-01-25 21:20 +0100
    [PATCH v1 2/2] mfd: lpc_ich: Remove useless comments in core part Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-01-25 21:20 +0100
    Re: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-01-26 03:10 +0100
    Re: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo  Lake PCH Mika Westerberg <mika.westerberg@linux.intel.com> - 2017-01-26 10:30 +0100
      Re: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-01-26 11:40 +0100

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

FromAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Date2017-01-25 21:20 +0100
Subject[PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH
Message-ID<t3CcV-6rc-13@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


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

FromAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Date2017-01-25 21:20 +0100
Subject[PATCH v1 2/2] mfd: lpc_ich: Remove useless comments in core part
Message-ID<t3CcW-6rc-17@gated-at.bofh.it>
In reply to#1566904
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

[toc] | [prev] | [next] | [standalone]


#1567080

FromAndy Shevchenko <andy.shevchenko@gmail.com>
Date2017-01-26 03:10 +0100
Message-ID<t3HFD-1sp-1@gated-at.bofh.it>
In reply to#1566904
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

[toc] | [prev] | [next] | [standalone]


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

FromMika Westerberg <mika.westerberg@linux.intel.com>
Date2017-01-26 10:30 +0100
SubjectRe: [PATCH v1 1/2] mfd: lpc_ich: Add Device IDs for Intel Apollo Lake PCH
Message-ID<t3Oxs-5zz-15@gated-at.bofh.it>
In reply to#1566904
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.

[toc] | [prev] | [next] | [standalone]


#1567313

FromAndy Shevchenko <andy.shevchenko@gmail.com>
Date2017-01-26 11:40 +0100
Message-ID<t3PDd-6bY-29@gated-at.bofh.it>
In reply to#1567217
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web