diff mbox series

i2c: i801: Consolidate chipset names in documentation and Kconfig

Message ID 20180614110058.24116-1-jarkko.nikula@linux.intel.com
State Superseded
Headers show
Series i2c: i801: Consolidate chipset names in documentation and Kconfig | expand

Commit Message

Jarkko Nikula June 14, 2018, 11 a.m. UTC
Make list of supported chipsets a little bit shorter by consolidating
names that belong to the same family.

Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
---
Jean et all: does this look sensible to you? If yes then how about
consolidating those older 82801xx? Although they seems to belong to
different families/revision so I didn't touch them.
---
 Documentation/i2c/busses/i2c-i801 | 10 +++-------
 drivers/i2c/busses/Kconfig        | 10 +++-------
 2 files changed, 6 insertions(+), 14 deletions(-)

Comments

Jean Delvare June 21, 2018, 5:48 p.m. UTC | #1
On Thu, 14 Jun 2018 14:00:58 +0300, Jarkko Nikula wrote:
> Make list of supported chipsets a little bit shorter by consolidating
> names that belong to the same family.
> 
> Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> ---
> Jean et all: does this look sensible to you? If yes then how about

I think it is a good idea.

> consolidating those older 82801xx? Although they seems to belong to
> different families/revision so I didn't touch them.

I would leave them separate because they are actually different
chipsets with different features.

> ---
>  Documentation/i2c/busses/i2c-i801 | 10 +++-------
>  drivers/i2c/busses/Kconfig        | 10 +++-------
>  2 files changed, 6 insertions(+), 14 deletions(-)
> (...)

Reviewed-by: Jean Delvare <jdelvare@suse.de>
Jarkko Nikula June 28, 2018, 11:16 a.m. UTC | #2
On 06/14/2018 02:00 PM, Jarkko Nikula wrote:
> Make list of supported chipsets a little bit shorter by consolidating
> names that belong to the same family.
> 
> Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> ---
> Jean et all: does this look sensible to you? If yes then how about
> consolidating those older 82801xx? Although they seems to belong to
> different families/revision so I didn't touch them.
> ---
>   Documentation/i2c/busses/i2c-i801 | 10 +++-------
>   drivers/i2c/busses/Kconfig        | 10 +++-------
>   2 files changed, 6 insertions(+), 14 deletions(-)
> 
> diff --git a/Documentation/i2c/busses/i2c-i801 b/Documentation/i2c/busses/i2c-i801
> index 65514c251318..4807a11536d1 100644
> --- a/Documentation/i2c/busses/i2c-i801
> +++ b/Documentation/i2c/busses/i2c-i801
...
> -  * Intel Cannon Lake-H (PCH)
> -  * Intel Cannon Lake-LP (PCH)
> +  * Intel Cannon Lake

I dropped here "(PCH)" by accident. I'll fix and resend with Jean's 
reviewed-by.
diff mbox series

Patch

diff --git a/Documentation/i2c/busses/i2c-i801 b/Documentation/i2c/busses/i2c-i801
index 65514c251318..4807a11536d1 100644
--- a/Documentation/i2c/busses/i2c-i801
+++ b/Documentation/i2c/busses/i2c-i801
@@ -21,23 +21,19 @@  Supported adapters:
   * Intel DH89xxCC (PCH)
   * Intel Panther Point (PCH)
   * Intel Lynx Point (PCH)
-  * Intel Lynx Point-LP (PCH)
   * Intel Avoton (SOC)
   * Intel Wellsburg (PCH)
   * Intel Coleto Creek (PCH)
   * Intel Wildcat Point (PCH)
-  * Intel Wildcat Point-LP (PCH)
   * Intel BayTrail (SOC)
   * Intel Braswell (SOC)
-  * Intel Sunrise Point-H (PCH)
-  * Intel Sunrise Point-LP (PCH)
-  * Intel Kaby Lake-H (PCH)
+  * Intel Sunrise Point (PCH)
+  * Intel Kaby Lake (PCH)
   * Intel DNV (SOC)
   * Intel Broxton (SOC)
   * Intel Lewisburg (PCH)
   * Intel Gemini Lake (SOC)
-  * Intel Cannon Lake-H (PCH)
-  * Intel Cannon Lake-LP (PCH)
+  * Intel Cannon Lake
   * Intel Cedar Fork (PCH)
    Datasheets: Publicly available at the Intel website
 
diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 99edffae27f6..297b100c6c90 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -116,23 +116,19 @@  config I2C_I801
 	    DH89xxCC (PCH)
 	    Panther Point (PCH)
 	    Lynx Point (PCH)
-	    Lynx Point-LP (PCH)
 	    Avoton (SOC)
 	    Wellsburg (PCH)
 	    Coleto Creek (PCH)
 	    Wildcat Point (PCH)
-	    Wildcat Point-LP (PCH)
 	    BayTrail (SOC)
 	    Braswell (SOC)
-	    Sunrise Point-H (PCH)
-	    Sunrise Point-LP (PCH)
-	    Kaby Lake-H (PCH)
+	    Sunrise Point (PCH)
+	    Kaby Lake (PCH)
 	    DNV (SOC)
 	    Broxton (SOC)
 	    Lewisburg (PCH)
 	    Gemini Lake (SOC)
-	    Cannon Lake-H (PCH)
-	    Cannon Lake-LP (PCH)
+	    Cannon Lake (PCH)
 	    Cedar Fork (PCH)
 
 	  This driver can also be built as a module.  If so, the module