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


Groups > linux.kernel > #1679073 > unrolled thread

[PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver

Started bysathyanarayanan.kuppuswamy@linux.intel.com
First post2017-06-30 21:30 +0200
Last post2017-06-30 23:10 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver sathyanarayanan.kuppuswamy@linux.intel.com - 2017-06-30 21:30 +0200
    Re: [PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-06-30 23:00 +0200
      Re: [PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO  driver sathyanarayanan kuppuswamy          <sathyanarayanan.kuppuswamy@linux.intel.com> - 2017-06-30 23:10 +0200

#1679073 — [PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver

Fromsathyanarayanan.kuppuswamy@linux.intel.com
Date2017-06-30 21:30 +0200
Subject[PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver
Message-ID<tY9M6-13z-19@gated-at.bofh.it>
From: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>

Added maintainer info for Whiskey Cove PMIC GPIO driver.

Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
---
 MAINTAINERS | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 09b5ab6..f76c349 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -13814,6 +13814,12 @@ F:	drivers/watchdog/
 F:	include/linux/watchdog.h
 F:	include/uapi/linux/watchdog.h
 
+WHISKEYCOVE PMIC GPIO DRIVER
+M:	Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
+L:	linux-gpio@vger.kernel.org
+S:	Maintained
+F:	drivers/gpio/gpio-wcove.c
+
 WIIMOTE HID DRIVER
 M:	David Herrmann <dh.herrmann@googlemail.com>
 L:	linux-input@vger.kernel.org
-- 
2.7.4

[toc] | [next] | [standalone]


#1679128

FromAndy Shevchenko <andy.shevchenko@gmail.com>
Date2017-06-30 23:00 +0200
Message-ID<tYbbc-1O8-13@gated-at.bofh.it>
In reply to#1679073
On Fri, Jun 30, 2017 at 10:20 PM,
<sathyanarayanan.kuppuswamy@linux.intel.com> wrote:
> From: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>

> +F:     drivers/gpio/gpio-wcove.c

If we are going to rename (which I personally consider as a good
idea), this should follow it, and not otherwise.

-- 
With Best Regards,
Andy Shevchenko

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


#1679134 — Re: [PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver

Fromsathyanarayanan kuppuswamy <sathyanarayanan.kuppuswamy@linux.intel.com>
Date2017-06-30 23:10 +0200
SubjectRe: [PATCH 1/1] MAINTAINERS: Add entry for Whiskey Cove PMIC GPIO driver
Message-ID<tYbkT-27u-45@gated-at.bofh.it>
In reply to#1679128
Hi Andy,


On 06/30/2017 01:51 PM, Andy Shevchenko wrote:
> If we are going to rename (which I personally consider as a good
> idea), this should follow it, and not otherwise.
I am having second thoughts about the rename. After going through the 
spec documents, I found that there are two main differences between CHT 
and BXT PMIC GPIO controllers.

1. BXT variant has 3 GPIO banks where as CHT variant has only 2 GPIO banks.
2. GPIO I/O control register and IRQ register mapping addresses are 
different in both BXT and CHT variants.

Given above differences, I think we have a choice to use the current 
driver and pass the above data difference to driver via .driver_data or 
platform_data. This is a bit messy method because we are not getting the 
register address or info about number of gpio banks from BIOS.

Other solution is to rename the existing driver to gpio-bxt-wcove.c and 
create a new one for CHT (if required).

Let me know your comments.

-- 
Sathyanarayanan Kuppuswamy
Linux kernel developer

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web