* [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2014-11-07 8:31 ` Andrew Jackson
0 siblings, 0 replies; 12+ messages in thread
From: Andrew Jackson @ 2014-11-07 8:31 UTC (permalink / raw)
To: Russell King - ARM Linux, airlied, linux-kernel, dri-devel
Cc: linux-arm-kernel, moinejf, liviu Dudau
The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
but the two LSBs of the TDA19988's address are set by two configuration
pins on the chip. Irrespective of the chip, the associated CEC
peripheral's I2C address is based upon the main I2C address.
This patch avoids any special handling required to support
systems that contain multiple TDA19988 devices on the same
I2C bus.
Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
---
drivers/gpu/drm/i2c/tda998x_drv.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
index d476279..eeab17a 100644
--- a/drivers/gpu/drm/i2c/tda998x_drv.c
+++ b/drivers/gpu/drm/i2c/tda998x_drv.c
@@ -1255,6 +1255,7 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
struct device_node *np = client->dev.of_node;
u32 video;
int rev_lo, rev_hi, ret;
+ unsigned short cec_addr;
priv->vip_cntrl_0 = VIP_CNTRL_0_SWAP_A(2) | VIP_CNTRL_0_SWAP_B(3);
priv->vip_cntrl_1 = VIP_CNTRL_1_SWAP_C(0) | VIP_CNTRL_1_SWAP_D(1);
@@ -1262,7 +1263,9 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
priv->current_page = 0xff;
priv->hdmi = client;
- priv->cec = i2c_new_dummy(client->adapter, 0x34);
+ /* CEC I2C address bound to TDA998x I2C addr by configuration pins */
+ cec_addr = 0x34 + (client->addr & 0x03);
+ priv->cec = i2c_new_dummy(client->adapter, cec_addr);
if (!priv->cec)
return -ENODEV;
--
1.7.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2014-11-07 8:31 ` Andrew Jackson
0 siblings, 0 replies; 12+ messages in thread
From: Andrew Jackson @ 2014-11-07 8:31 UTC (permalink / raw)
To: Russell King - ARM Linux, airlied, linux-kernel, dri-devel
Cc: linux-arm-kernel, moinejf, liviu Dudau
The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
but the two LSBs of the TDA19988's address are set by two configuration
pins on the chip. Irrespective of the chip, the associated CEC
peripheral's I2C address is based upon the main I2C address.
This patch avoids any special handling required to support
systems that contain multiple TDA19988 devices on the same
I2C bus.
Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
---
drivers/gpu/drm/i2c/tda998x_drv.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
index d476279..eeab17a 100644
--- a/drivers/gpu/drm/i2c/tda998x_drv.c
+++ b/drivers/gpu/drm/i2c/tda998x_drv.c
@@ -1255,6 +1255,7 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
struct device_node *np = client->dev.of_node;
u32 video;
int rev_lo, rev_hi, ret;
+ unsigned short cec_addr;
priv->vip_cntrl_0 = VIP_CNTRL_0_SWAP_A(2) | VIP_CNTRL_0_SWAP_B(3);
priv->vip_cntrl_1 = VIP_CNTRL_1_SWAP_C(0) | VIP_CNTRL_1_SWAP_D(1);
@@ -1262,7 +1263,9 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
priv->current_page = 0xff;
priv->hdmi = client;
- priv->cec = i2c_new_dummy(client->adapter, 0x34);
+ /* CEC I2C address bound to TDA998x I2C addr by configuration pins */
+ cec_addr = 0x34 + (client->addr & 0x03);
+ priv->cec = i2c_new_dummy(client->adapter, cec_addr);
if (!priv->cec)
return -ENODEV;
--
1.7.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2014-11-07 8:31 ` Andrew Jackson
0 siblings, 0 replies; 12+ messages in thread
From: Andrew Jackson @ 2014-11-07 8:31 UTC (permalink / raw)
To: linux-arm-kernel
The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
but the two LSBs of the TDA19988's address are set by two configuration
pins on the chip. Irrespective of the chip, the associated CEC
peripheral's I2C address is based upon the main I2C address.
This patch avoids any special handling required to support
systems that contain multiple TDA19988 devices on the same
I2C bus.
Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
---
drivers/gpu/drm/i2c/tda998x_drv.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
index d476279..eeab17a 100644
--- a/drivers/gpu/drm/i2c/tda998x_drv.c
+++ b/drivers/gpu/drm/i2c/tda998x_drv.c
@@ -1255,6 +1255,7 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
struct device_node *np = client->dev.of_node;
u32 video;
int rev_lo, rev_hi, ret;
+ unsigned short cec_addr;
priv->vip_cntrl_0 = VIP_CNTRL_0_SWAP_A(2) | VIP_CNTRL_0_SWAP_B(3);
priv->vip_cntrl_1 = VIP_CNTRL_1_SWAP_C(0) | VIP_CNTRL_1_SWAP_D(1);
@@ -1262,7 +1263,9 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
priv->current_page = 0xff;
priv->hdmi = client;
- priv->cec = i2c_new_dummy(client->adapter, 0x34);
+ /* CEC I2C address bound to TDA998x I2C addr by configuration pins */
+ cec_addr = 0x34 + (client->addr & 0x03);
+ priv->cec = i2c_new_dummy(client->adapter, cec_addr);
if (!priv->cec)
return -ENODEV;
--
1.7.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
2014-11-07 8:31 ` Andrew Jackson
(?)
@ 2015-01-21 18:44 ` Liviu Dudau
-1 siblings, 0 replies; 12+ messages in thread
From: Liviu Dudau @ 2015-01-21 18:44 UTC (permalink / raw)
To: Andrew Jackson
Cc: Russell King - ARM Linux, airlied, linux-kernel, dri-devel,
linux-arm-kernel, moinejf
On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> but the two LSBs of the TDA19988's address are set by two configuration
> pins on the chip. Irrespective of the chip, the associated CEC
> peripheral's I2C address is based upon the main I2C address.
>
> This patch avoids any special handling required to support
> systems that contain multiple TDA19988 devices on the same
> I2C bus.
>
> Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
Hi Andrew,
What is the status of this patch? Is it going to be pulled by anyone?
Best regards,
Liviu
> ---
> drivers/gpu/drm/i2c/tda998x_drv.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
> index d476279..eeab17a 100644
> --- a/drivers/gpu/drm/i2c/tda998x_drv.c
> +++ b/drivers/gpu/drm/i2c/tda998x_drv.c
> @@ -1255,6 +1255,7 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
> struct device_node *np = client->dev.of_node;
> u32 video;
> int rev_lo, rev_hi, ret;
> + unsigned short cec_addr;
>
> priv->vip_cntrl_0 = VIP_CNTRL_0_SWAP_A(2) | VIP_CNTRL_0_SWAP_B(3);
> priv->vip_cntrl_1 = VIP_CNTRL_1_SWAP_C(0) | VIP_CNTRL_1_SWAP_D(1);
> @@ -1262,7 +1263,9 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
>
> priv->current_page = 0xff;
> priv->hdmi = client;
> - priv->cec = i2c_new_dummy(client->adapter, 0x34);
> + /* CEC I2C address bound to TDA998x I2C addr by configuration pins */
> + cec_addr = 0x34 + (client->addr & 0x03);
> + priv->cec = i2c_new_dummy(client->adapter, cec_addr);
> if (!priv->cec)
> return -ENODEV;
>
> --
> 1.7.1
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2015-01-21 18:44 ` Liviu Dudau
0 siblings, 0 replies; 12+ messages in thread
From: Liviu Dudau @ 2015-01-21 18:44 UTC (permalink / raw)
To: Andrew Jackson
Cc: Russell King - ARM Linux, airlied, linux-kernel, dri-devel,
linux-arm-kernel, moinejf
On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> but the two LSBs of the TDA19988's address are set by two configuration
> pins on the chip. Irrespective of the chip, the associated CEC
> peripheral's I2C address is based upon the main I2C address.
>
> This patch avoids any special handling required to support
> systems that contain multiple TDA19988 devices on the same
> I2C bus.
>
> Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
Hi Andrew,
What is the status of this patch? Is it going to be pulled by anyone?
Best regards,
Liviu
> ---
> drivers/gpu/drm/i2c/tda998x_drv.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
> index d476279..eeab17a 100644
> --- a/drivers/gpu/drm/i2c/tda998x_drv.c
> +++ b/drivers/gpu/drm/i2c/tda998x_drv.c
> @@ -1255,6 +1255,7 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
> struct device_node *np = client->dev.of_node;
> u32 video;
> int rev_lo, rev_hi, ret;
> + unsigned short cec_addr;
>
> priv->vip_cntrl_0 = VIP_CNTRL_0_SWAP_A(2) | VIP_CNTRL_0_SWAP_B(3);
> priv->vip_cntrl_1 = VIP_CNTRL_1_SWAP_C(0) | VIP_CNTRL_1_SWAP_D(1);
> @@ -1262,7 +1263,9 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
>
> priv->current_page = 0xff;
> priv->hdmi = client;
> - priv->cec = i2c_new_dummy(client->adapter, 0x34);
> + /* CEC I2C address bound to TDA998x I2C addr by configuration pins */
> + cec_addr = 0x34 + (client->addr & 0x03);
> + priv->cec = i2c_new_dummy(client->adapter, cec_addr);
> if (!priv->cec)
> return -ENODEV;
>
> --
> 1.7.1
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2015-01-21 18:44 ` Liviu Dudau
0 siblings, 0 replies; 12+ messages in thread
From: Liviu Dudau @ 2015-01-21 18:44 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> but the two LSBs of the TDA19988's address are set by two configuration
> pins on the chip. Irrespective of the chip, the associated CEC
> peripheral's I2C address is based upon the main I2C address.
>
> This patch avoids any special handling required to support
> systems that contain multiple TDA19988 devices on the same
> I2C bus.
>
> Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
Hi Andrew,
What is the status of this patch? Is it going to be pulled by anyone?
Best regards,
Liviu
> ---
> drivers/gpu/drm/i2c/tda998x_drv.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
> index d476279..eeab17a 100644
> --- a/drivers/gpu/drm/i2c/tda998x_drv.c
> +++ b/drivers/gpu/drm/i2c/tda998x_drv.c
> @@ -1255,6 +1255,7 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
> struct device_node *np = client->dev.of_node;
> u32 video;
> int rev_lo, rev_hi, ret;
> + unsigned short cec_addr;
>
> priv->vip_cntrl_0 = VIP_CNTRL_0_SWAP_A(2) | VIP_CNTRL_0_SWAP_B(3);
> priv->vip_cntrl_1 = VIP_CNTRL_1_SWAP_C(0) | VIP_CNTRL_1_SWAP_D(1);
> @@ -1262,7 +1263,9 @@ static int tda998x_create(struct i2c_client *client, struct tda998x_priv *priv)
>
> priv->current_page = 0xff;
> priv->hdmi = client;
> - priv->cec = i2c_new_dummy(client->adapter, 0x34);
> + /* CEC I2C address bound to TDA998x I2C addr by configuration pins */
> + cec_addr = 0x34 + (client->addr & 0x03);
> + priv->cec = i2c_new_dummy(client->adapter, cec_addr);
> if (!priv->cec)
> return -ENODEV;
>
> --
> 1.7.1
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
?\_(?)_/?
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
2015-01-21 18:44 ` Liviu Dudau
(?)
@ 2015-01-21 18:46 ` Russell King - ARM Linux
-1 siblings, 0 replies; 12+ messages in thread
From: Russell King - ARM Linux @ 2015-01-21 18:46 UTC (permalink / raw)
To: Liviu Dudau
Cc: Andrew Jackson, airlied, linux-kernel, dri-devel,
linux-arm-kernel, moinejf
On Wed, Jan 21, 2015 at 06:44:05PM +0000, Liviu Dudau wrote:
> On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> > The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> > but the two LSBs of the TDA19988's address are set by two configuration
> > pins on the chip. Irrespective of the chip, the associated CEC
> > peripheral's I2C address is based upon the main I2C address.
> >
> > This patch avoids any special handling required to support
> > systems that contain multiple TDA19988 devices on the same
> > I2C bus.
> >
> > Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
>
> Hi Andrew,
>
> What is the status of this patch? Is it going to be pulled by anyone?
It's been sent to David.
--
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2015-01-21 18:46 ` Russell King - ARM Linux
0 siblings, 0 replies; 12+ messages in thread
From: Russell King - ARM Linux @ 2015-01-21 18:46 UTC (permalink / raw)
To: Liviu Dudau; +Cc: Andrew Jackson, linux-kernel, dri-devel, linux-arm-kernel
On Wed, Jan 21, 2015 at 06:44:05PM +0000, Liviu Dudau wrote:
> On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> > The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> > but the two LSBs of the TDA19988's address are set by two configuration
> > pins on the chip. Irrespective of the chip, the associated CEC
> > peripheral's I2C address is based upon the main I2C address.
> >
> > This patch avoids any special handling required to support
> > systems that contain multiple TDA19988 devices on the same
> > I2C bus.
> >
> > Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
>
> Hi Andrew,
>
> What is the status of this patch? Is it going to be pulled by anyone?
It's been sent to David.
--
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2015-01-21 18:46 ` Russell King - ARM Linux
0 siblings, 0 replies; 12+ messages in thread
From: Russell King - ARM Linux @ 2015-01-21 18:46 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jan 21, 2015 at 06:44:05PM +0000, Liviu Dudau wrote:
> On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> > The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> > but the two LSBs of the TDA19988's address are set by two configuration
> > pins on the chip. Irrespective of the chip, the associated CEC
> > peripheral's I2C address is based upon the main I2C address.
> >
> > This patch avoids any special handling required to support
> > systems that contain multiple TDA19988 devices on the same
> > I2C bus.
> >
> > Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
>
> Hi Andrew,
>
> What is the status of this patch? Is it going to be pulled by anyone?
It's been sent to David.
--
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
2015-01-21 18:46 ` Russell King - ARM Linux
(?)
@ 2015-01-21 19:15 ` Liviu Dudau
-1 siblings, 0 replies; 12+ messages in thread
From: Liviu Dudau @ 2015-01-21 19:15 UTC (permalink / raw)
To: Russell King - ARM Linux
Cc: Andrew Jackson, airlied, linux-kernel, dri-devel,
linux-arm-kernel, moinejf
On Wed, Jan 21, 2015 at 06:46:58PM +0000, Russell King - ARM Linux wrote:
> On Wed, Jan 21, 2015 at 06:44:05PM +0000, Liviu Dudau wrote:
> > On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> > > The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> > > but the two LSBs of the TDA19988's address are set by two configuration
> > > pins on the chip. Irrespective of the chip, the associated CEC
> > > peripheral's I2C address is based upon the main I2C address.
> > >
> > > This patch avoids any special handling required to support
> > > systems that contain multiple TDA19988 devices on the same
> > > I2C bus.
> > >
> > > Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> > > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
> >
> > Hi Andrew,
> >
> > What is the status of this patch? Is it going to be pulled by anyone?
>
> It's been sent to David.
Thanks, Russell!
Liviu
>
> --
> FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
> according to speedtest.net.
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2015-01-21 19:15 ` Liviu Dudau
0 siblings, 0 replies; 12+ messages in thread
From: Liviu Dudau @ 2015-01-21 19:15 UTC (permalink / raw)
To: Russell King - ARM Linux
Cc: Andrew Jackson, linux-kernel, dri-devel, linux-arm-kernel
On Wed, Jan 21, 2015 at 06:46:58PM +0000, Russell King - ARM Linux wrote:
> On Wed, Jan 21, 2015 at 06:44:05PM +0000, Liviu Dudau wrote:
> > On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> > > The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> > > but the two LSBs of the TDA19988's address are set by two configuration
> > > pins on the chip. Irrespective of the chip, the associated CEC
> > > peripheral's I2C address is based upon the main I2C address.
> > >
> > > This patch avoids any special handling required to support
> > > systems that contain multiple TDA19988 devices on the same
> > > I2C bus.
> > >
> > > Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> > > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
> >
> > Hi Andrew,
> >
> > What is the status of this patch? Is it going to be pulled by anyone?
>
> It's been sent to David.
Thanks, Russell!
Liviu
>
> --
> FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
> according to speedtest.net.
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address.
@ 2015-01-21 19:15 ` Liviu Dudau
0 siblings, 0 replies; 12+ messages in thread
From: Liviu Dudau @ 2015-01-21 19:15 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jan 21, 2015 at 06:46:58PM +0000, Russell King - ARM Linux wrote:
> On Wed, Jan 21, 2015 at 06:44:05PM +0000, Liviu Dudau wrote:
> > On Fri, Nov 07, 2014 at 08:31:25AM +0000, Andrew Jackson wrote:
> > > The I2C address for the TDA9989 and TDA19989 is fixed at 0x34
> > > but the two LSBs of the TDA19988's address are set by two configuration
> > > pins on the chip. Irrespective of the chip, the associated CEC
> > > peripheral's I2C address is based upon the main I2C address.
> > >
> > > This patch avoids any special handling required to support
> > > systems that contain multiple TDA19988 devices on the same
> > > I2C bus.
> > >
> > > Signed-off-by: Andrew Jackson <Andrew.Jackson@arm.com>
> > > Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
> >
> > Hi Andrew,
> >
> > What is the status of this patch? Is it going to be pulled by anyone?
>
> It's been sent to David.
Thanks, Russell!
Liviu
>
> --
> FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
> according to speedtest.net.
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
?\_(?)_/?
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2015-01-21 19:15 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-07 8:31 [PATCH v2] drm/i2c: tda998x: Set the CEC I2C address based on the slave I2C address Andrew Jackson
2014-11-07 8:31 ` Andrew Jackson
2014-11-07 8:31 ` Andrew Jackson
2015-01-21 18:44 ` Liviu Dudau
2015-01-21 18:44 ` Liviu Dudau
2015-01-21 18:44 ` Liviu Dudau
2015-01-21 18:46 ` Russell King - ARM Linux
2015-01-21 18:46 ` Russell King - ARM Linux
2015-01-21 18:46 ` Russell King - ARM Linux
2015-01-21 19:15 ` Liviu Dudau
2015-01-21 19:15 ` Liviu Dudau
2015-01-21 19:15 ` Liviu Dudau
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.