diff mbox

[3/4] gpio: pca953x: Add pca9506 as DT compatible

Message ID 1406618686-22385-4-git-send-email-mpa@pengutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Markus Pargmann July 29, 2014, 7:24 a.m. UTC
This driver can also be used for the nxp,pca9506 device.

Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
---
 Documentation/devicetree/bindings/gpio/gpio-pca953x.txt | 1 +
 drivers/gpio/gpio-pca953x.c                             | 1 +
 2 files changed, 2 insertions(+)

Comments

Markus Pargmann July 30, 2014, 12:52 p.m. UTC | #1
On Tue, Jul 29, 2014 at 09:24:45AM +0200, Markus Pargmann wrote:
> This driver can also be used for the nxp,pca9506 device.
> 
> Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
> ---
>  Documentation/devicetree/bindings/gpio/gpio-pca953x.txt | 1 +
>  drivers/gpio/gpio-pca953x.c                             | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt b/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
> index b9a42f294dd0..eb65157d47f6 100644
> --- a/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
> +++ b/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
> @@ -3,6 +3,7 @@
>  Required properties:
>   - compatible: Has to contain one of the following:
>  	nxp,pca9505
> +	nxp,pca9506
>  	nxp,pca9534
>  	nxp,pca9535
>  	nxp,pca9536
> diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c
> index fda3eae835d0..df5eb6e6be1e 100644
> --- a/drivers/gpio/gpio-pca953x.c
> +++ b/drivers/gpio/gpio-pca953x.c
> @@ -729,6 +729,7 @@ static int pca953x_remove(struct i2c_client *client)
>  
>  static const struct of_device_id pca953x_dt_ids[] = {
>  	{ .compatible = "nxp,pca9505", },
> +	{ .compatible = "nxp,pca9506", },

The item for the i2c_device_id table is missing in this patch. I will
send a v2.

Regards,

Markus
Markus Pargmann July 30, 2014, 1:09 p.m. UTC | #2
On Wed, Jul 30, 2014 at 02:52:16PM +0200, Markus Pargmann wrote:
> On Tue, Jul 29, 2014 at 09:24:45AM +0200, Markus Pargmann wrote:
> > This driver can also be used for the nxp,pca9506 device.
> > 
> > Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
> > ---
> >  Documentation/devicetree/bindings/gpio/gpio-pca953x.txt | 1 +
> >  drivers/gpio/gpio-pca953x.c                             | 1 +
> >  2 files changed, 2 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt b/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
> > index b9a42f294dd0..eb65157d47f6 100644
> > --- a/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
> > +++ b/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
> > @@ -3,6 +3,7 @@
> >  Required properties:
> >   - compatible: Has to contain one of the following:
> >  	nxp,pca9505
> > +	nxp,pca9506
> >  	nxp,pca9534
> >  	nxp,pca9535
> >  	nxp,pca9536
> > diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c
> > index fda3eae835d0..df5eb6e6be1e 100644
> > --- a/drivers/gpio/gpio-pca953x.c
> > +++ b/drivers/gpio/gpio-pca953x.c
> > @@ -729,6 +729,7 @@ static int pca953x_remove(struct i2c_client *client)
> >  
> >  static const struct of_device_id pca953x_dt_ids[] = {
> >  	{ .compatible = "nxp,pca9505", },
> > +	{ .compatible = "nxp,pca9506", },
> 
> The item for the i2c_device_id table is missing in this patch. I will
> send a v2.

This patch can actually be dropped. There is no difference between
pca9505 and pca9506 so it doesn't make sense to add pca9506.

Regards,

Markus
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt b/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
index b9a42f294dd0..eb65157d47f6 100644
--- a/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
+++ b/Documentation/devicetree/bindings/gpio/gpio-pca953x.txt
@@ -3,6 +3,7 @@ 
 Required properties:
  - compatible: Has to contain one of the following:
 	nxp,pca9505
+	nxp,pca9506
 	nxp,pca9534
 	nxp,pca9535
 	nxp,pca9536
diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c
index fda3eae835d0..df5eb6e6be1e 100644
--- a/drivers/gpio/gpio-pca953x.c
+++ b/drivers/gpio/gpio-pca953x.c
@@ -729,6 +729,7 @@  static int pca953x_remove(struct i2c_client *client)
 
 static const struct of_device_id pca953x_dt_ids[] = {
 	{ .compatible = "nxp,pca9505", },
+	{ .compatible = "nxp,pca9506", },
 	{ .compatible = "nxp,pca9534", },
 	{ .compatible = "nxp,pca9535", },
 	{ .compatible = "nxp,pca9536", },