Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
On Thu, 2013-09-05 at 21:30 -0500, Tang Yuantian-B29983 wrote: -Original Message- From: Wood Scott-B07421 Sent: 2013年9月6日 星期五 2:41 To: Tang Yuantian-B29983 Cc: Yang,Wei; Jia Hongtao-B38951; Wood Scott-B07421; linuxppc- d...@lists.ozlabs.org Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On Tue, 2013-09-03 at 22:30 -0500, Tang Yuantian-B29983 wrote: Hi, These eeproms are never used by kernel. So no need to add them. The device tree describes the hardware, not what Linux does with it. Missing some nodes doesn't mean it is not describing the hardware. There are almost fifty I2C devices on T4 connected to PCA9547. Do you think we need to list them all? Ideally, yes. I realize it's not uncommon for some things to be missing, but that's not a reason to tell people to leave things out just because Linux doesn't use them. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
On Tue, 2013-09-03 at 22:30 -0500, Tang Yuantian-B29983 wrote: Hi, These eeproms are never used by kernel. So no need to add them. The device tree describes the hardware, not what Linux does with it. -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
-Original Message- From: Wood Scott-B07421 Sent: 2013年9月6日 星期五 2:41 To: Tang Yuantian-B29983 Cc: Yang,Wei; Jia Hongtao-B38951; Wood Scott-B07421; linuxppc- d...@lists.ozlabs.org Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On Tue, 2013-09-03 at 22:30 -0500, Tang Yuantian-B29983 wrote: Hi, These eeproms are never used by kernel. So no need to add them. The device tree describes the hardware, not what Linux does with it. Missing some nodes doesn't mean it is not describing the hardware. There are almost fifty I2C devices on T4 connected to PCA9547. Do you think we need to list them all? Regards, Yuantian -Scott ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
On 09/03/2013 03:51 PM, Jia Hongtao wrote: In both B4 and T4240QDS platform PCA9547 I2C bus multiplexer is used. Hi Hongtao, If you want to support I2C bus multiplexer, for T4 and B4QDS platform, since some eeprom devices is connected to PCA9574 I2C bus multiplexer, so these devices should be connected to pca9547 node. Just like the following, what do you think of it? + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + #address-cells = 1; + #size-cells = 0; + channel@0 { + #address-cells = 1; + #size-cells = 0; + reg = 0; + eeprom@51 { + compatible = at24,24c256; + reg = 0x51; + }; + eeprom@52 { + compatible = at24,24c256; + reg = 0x52; + }; + eeprom@53 { + compatible = at24,24c256; + reg = 0x53; + }; + eeprom@54 { + compatible = at24,24c256; + reg = 0x54; + }; + eeprom@55 { + compatible = at24,24c256; + reg = 0x55; + }; + eeprom@56 { + compatible = at24,24c256; + reg = 0x56; + }; + rtc@68 { + compatible = dallas,ds3232; + reg = 0x68; + interrupts = 0x1 0x1 0 0; + }; + }; Wei Signed-off-by: Jia Hongtao hongtao@freescale.com --- arch/powerpc/boot/dts/b4qds.dtsi | 4 arch/powerpc/boot/dts/t4240qds.dts | 4 2 files changed, 8 insertions(+) diff --git a/arch/powerpc/boot/dts/b4qds.dtsi b/arch/powerpc/boot/dts/b4qds.dtsi index e6d2f8f..2aa3399 100644 --- a/arch/powerpc/boot/dts/b4qds.dtsi +++ b/arch/powerpc/boot/dts/b4qds.dtsi @@ -120,6 +120,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@50 { compatible = at24,24c64; reg = 0x50; diff --git a/arch/powerpc/boot/dts/t4240qds.dts b/arch/powerpc/boot/dts/t4240qds.dts index 0555976..084db57 100644 --- a/arch/powerpc/boot/dts/t4240qds.dts +++ b/arch/powerpc/boot/dts/t4240qds.dts @@ -118,6 +118,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@51 { compatible = at24,24c256; reg = 0x51; ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
Hi, These eeproms are never used by kernel. So no need to add them. Thanks, Yuantian -Original Message- From: Linuxppc-dev [mailto:linuxppc-dev- bounces+b29983=freescale@lists.ozlabs.org] On Behalf Of Yang,Wei Sent: 2013年9月4日 星期三 9:27 To: Jia Hongtao-B38951 Cc: Wood Scott-B07421; linuxppc-dev@lists.ozlabs.org; Jia Hongtao-B38951 Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On 09/03/2013 03:51 PM, Jia Hongtao wrote: In both B4 and T4240QDS platform PCA9547 I2C bus multiplexer is used. Hi Hongtao, If you want to support I2C bus multiplexer, for T4 and B4QDS platform, since some eeprom devices is connected to PCA9574 I2C bus multiplexer, so these devices should be connected to pca9547 node. Just like the following, what do you think of it? + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + #address-cells = 1; + #size-cells = 0; + channel@0 { + #address-cells = 1; + #size-cells = 0; + reg = 0; + eeprom@51 { + compatible = at24,24c256; + reg = 0x51; + }; + eeprom@52 { + compatible = at24,24c256; + reg = 0x52; + }; + eeprom@53 { + compatible = at24,24c256; + reg = 0x53; + }; + eeprom@54 { + compatible = at24,24c256; + reg = 0x54; + }; + eeprom@55 { + compatible = at24,24c256; + reg = 0x55; + }; + eeprom@56 { + compatible = at24,24c256; + reg = 0x56; + }; + rtc@68 { + compatible = dallas,ds3232; + reg = 0x68; + interrupts = 0x1 0x1 0 0; + }; + }; Wei Signed-off-by: Jia Hongtao hongtao@freescale.com --- arch/powerpc/boot/dts/b4qds.dtsi | 4 arch/powerpc/boot/dts/t4240qds.dts | 4 2 files changed, 8 insertions(+) diff --git a/arch/powerpc/boot/dts/b4qds.dtsi b/arch/powerpc/boot/dts/b4qds.dtsi index e6d2f8f..2aa3399 100644 --- a/arch/powerpc/boot/dts/b4qds.dtsi +++ b/arch/powerpc/boot/dts/b4qds.dtsi @@ -120,6 +120,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@50 { compatible = at24,24c64; reg = 0x50; diff --git a/arch/powerpc/boot/dts/t4240qds.dts b/arch/powerpc/boot/dts/t4240qds.dts index 0555976..084db57 100644 --- a/arch/powerpc/boot/dts/t4240qds.dts +++ b/arch/powerpc/boot/dts/t4240qds.dts @@ -118,6 +118,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@51 { compatible = at24,24c256; reg = 0x51; ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
Hi Wei, I totally agree that the i2c nodes topology should end up like you said. But I think adding sub-nodes should step by step. Actually the hardware i2c topology are huge like on T4. So I'd like to adding nodes when we needed. If you think the sub-nodes are needed please send another patch based on mine. I think this is the more reasonable way. Thanks. -Hongtao -Original Message- From: Yang,Wei [mailto:wei.y...@windriver.com] Sent: Wednesday, September 04, 2013 9:27 AM To: Jia Hongtao-B38951 Cc: linuxppc-dev@lists.ozlabs.org; Wood Scott-B07421; Jia Hongtao-B38951 Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On 09/03/2013 03:51 PM, Jia Hongtao wrote: In both B4 and T4240QDS platform PCA9547 I2C bus multiplexer is used. Hi Hongtao, If you want to support I2C bus multiplexer, for T4 and B4QDS platform, since some eeprom devices is connected to PCA9574 I2C bus multiplexer, so these devices should be connected to pca9547 node. Just like the following, what do you think of it? + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + #address-cells = 1; + #size-cells = 0; + channel@0 { + #address-cells = 1; + #size-cells = 0; + reg = 0; + eeprom@51 { + compatible = at24,24c256; + reg = 0x51; + }; + eeprom@52 { + compatible = at24,24c256; + reg = 0x52; + }; + eeprom@53 { + compatible = at24,24c256; + reg = 0x53; + }; + eeprom@54 { + compatible = at24,24c256; + reg = 0x54; + }; + eeprom@55 { + compatible = at24,24c256; + reg = 0x55; + }; + eeprom@56 { + compatible = at24,24c256; + reg = 0x56; + }; + rtc@68 { + compatible = dallas,ds3232; + reg = 0x68; + interrupts = 0x1 0x1 0 0; + }; + }; Wei Signed-off-by: Jia Hongtao hongtao@freescale.com --- arch/powerpc/boot/dts/b4qds.dtsi | 4 arch/powerpc/boot/dts/t4240qds.dts | 4 2 files changed, 8 insertions(+) diff --git a/arch/powerpc/boot/dts/b4qds.dtsi b/arch/powerpc/boot/dts/b4qds.dtsi index e6d2f8f..2aa3399 100644 --- a/arch/powerpc/boot/dts/b4qds.dtsi +++ b/arch/powerpc/boot/dts/b4qds.dtsi @@ -120,6 +120,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@50 { compatible = at24,24c64; reg = 0x50; diff --git a/arch/powerpc/boot/dts/t4240qds.dts b/arch/powerpc/boot/dts/t4240qds.dts index 0555976..084db57 100644 --- a/arch/powerpc/boot/dts/t4240qds.dts +++ b/arch/powerpc/boot/dts/t4240qds.dts @@ -118,6 +118,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@51 { compatible = at24,24c256; reg = 0x51; ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo
RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
Hi, I noticed that there are already some nodes in i2c bus. You should at least move the existing node into PCA9547. Thanks, Yuantian -Original Message- From: Linuxppc-dev [mailto:linuxppc-dev- bounces+b29983=freescale@lists.ozlabs.org] On Behalf Of Jia Hongtao- B38951 Sent: 2013年9月4日 星期三 11:38 To: Yang,Wei Cc: Wood Scott-B07421; linuxppc-dev@lists.ozlabs.org Subject: RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS Hi Wei, I totally agree that the i2c nodes topology should end up like you said. But I think adding sub-nodes should step by step. Actually the hardware i2c topology are huge like on T4. So I'd like to adding nodes when we needed. If you think the sub-nodes are needed please send another patch based on mine. I think this is the more reasonable way. Thanks. -Hongtao -Original Message- From: Yang,Wei [mailto:wei.y...@windriver.com] Sent: Wednesday, September 04, 2013 9:27 AM To: Jia Hongtao-B38951 Cc: linuxppc-dev@lists.ozlabs.org; Wood Scott-B07421; Jia Hongtao-B38951 Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On 09/03/2013 03:51 PM, Jia Hongtao wrote: In both B4 and T4240QDS platform PCA9547 I2C bus multiplexer is used. Hi Hongtao, If you want to support I2C bus multiplexer, for T4 and B4QDS platform, since some eeprom devices is connected to PCA9574 I2C bus multiplexer, so these devices should be connected to pca9547 node. Just like the following, what do you think of it? + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + #address-cells = 1; + #size-cells = 0; + channel@0 { + #address-cells = 1; + #size-cells = 0; + reg = 0; + eeprom@51 { + compatible = at24,24c256; + reg = 0x51; + }; + eeprom@52 { + compatible = at24,24c256; + reg = 0x52; + }; + eeprom@53 { + compatible = at24,24c256; + reg = 0x53; + }; + eeprom@54 { + compatible = at24,24c256; + reg = 0x54; + }; + eeprom@55 { + compatible = at24,24c256; + reg = 0x55; + }; + eeprom@56 { + compatible = at24,24c256; + reg = 0x56; + }; + rtc@68 { + compatible = dallas,ds3232; + reg = 0x68; + interrupts = 0x1 0x1 0 0; + }; + }; Wei Signed-off-by: Jia Hongtao hongtao@freescale.com --- arch/powerpc/boot/dts/b4qds.dtsi | 4 arch/powerpc/boot/dts/t4240qds.dts | 4 2 files changed, 8 insertions(+) diff --git a/arch/powerpc/boot/dts/b4qds.dtsi b/arch/powerpc/boot/dts/b4qds.dtsi index e6d2f8f..2aa3399 100644 --- a/arch/powerpc/boot/dts/b4qds.dtsi +++ b/arch/powerpc/boot/dts/b4qds.dtsi @@ -120,6 +120,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@50 { compatible = at24,24c64; reg = 0x50; diff --git a/arch/powerpc/boot/dts/t4240qds.dts b/arch/powerpc/boot/dts/t4240qds.dts index 0555976..084db57 100644
RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
Hi Yuantian, Yes, you are right. Hi Wei, I misunderstood your idea. I agree it and I will submit V2 patch to update it soon. Thanks. -Hongtao -Original Message- From: Tang Yuantian-B29983 Sent: Wednesday, September 04, 2013 12:04 PM To: Jia Hongtao-B38951; Yang,Wei Cc: Wood Scott-B07421; linuxppc-dev@lists.ozlabs.org Subject: RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS Hi, I noticed that there are already some nodes in i2c bus. You should at least move the existing node into PCA9547. Thanks, Yuantian -Original Message- From: Linuxppc-dev [mailto:linuxppc-dev- bounces+b29983=freescale@lists.ozlabs.org] On Behalf Of Jia bounces+Hongtao- B38951 Sent: 2013年9月4日 星期三 11:38 To: Yang,Wei Cc: Wood Scott-B07421; linuxppc-dev@lists.ozlabs.org Subject: RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS Hi Wei, I totally agree that the i2c nodes topology should end up like you said. But I think adding sub-nodes should step by step. Actually the hardware i2c topology are huge like on T4. So I'd like to adding nodes when we needed. If you think the sub-nodes are needed please send another patch based on mine. I think this is the more reasonable way. Thanks. -Hongtao -Original Message- From: Yang,Wei [mailto:wei.y...@windriver.com] Sent: Wednesday, September 04, 2013 9:27 AM To: Jia Hongtao-B38951 Cc: linuxppc-dev@lists.ozlabs.org; Wood Scott-B07421; Jia Hongtao-B38951 Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On 09/03/2013 03:51 PM, Jia Hongtao wrote: In both B4 and T4240QDS platform PCA9547 I2C bus multiplexer is used. Hi Hongtao, If you want to support I2C bus multiplexer, for T4 and B4QDS platform, since some eeprom devices is connected to PCA9574 I2C bus multiplexer, so these devices should be connected to pca9547 node. Just like the following, what do you think of it? + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + #address-cells = 1; + #size-cells = 0; + channel@0 { + #address-cells = 1; + #size-cells = 0; + reg = 0; + eeprom@51 { + compatible = at24,24c256; + reg = 0x51; + }; + eeprom@52 { + compatible = at24,24c256; + reg = 0x52; + }; + eeprom@53 { + compatible = at24,24c256; + reg = 0x53; + }; + eeprom@54 { + compatible = at24,24c256; + reg = 0x54; + }; + eeprom@55 { + compatible = at24,24c256; + reg = 0x55; + }; + eeprom@56 { + compatible = at24,24c256; + reg = 0x56; + }; + rtc@68 { + compatible = dallas,ds3232; + reg = 0x68; + interrupts = 0x1 0x1 0 0; + }; + }; Wei Signed-off-by: Jia Hongtao hongtao@freescale.com --- arch/powerpc/boot/dts/b4qds.dtsi | 4 arch/powerpc/boot/dts/t4240qds.dts | 4 2 files changed, 8 insertions(+) diff --git a/arch/powerpc/boot/dts/b4qds.dtsi b/arch/powerpc/boot/dts/b4qds.dtsi index e6d2f8f..2aa3399 100644 --- a/arch/powerpc/boot/dts/b4qds.dtsi +++ b/arch/powerpc/boot
Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS
On 09/04/2013 12:03 PM, Tang Yuantian-B29983 wrote: Hi, I noticed that there are already some nodes in i2c bus. Sorry for my late response, yeah, just as you noticed, some eeprom nodes already are inside i2c bus.:-) Wei You should at least move the existing node into PCA9547. Thanks, Yuantian -Original Message- From: Linuxppc-dev [mailto:linuxppc-dev- bounces+b29983=freescale@lists.ozlabs.org] On Behalf Of Jia Hongtao- B38951 Sent: 2013年9月4日 星期三 11:38 To: Yang,Wei Cc: Wood Scott-B07421; linuxppc-dev@lists.ozlabs.org Subject: RE: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS Hi Wei, I totally agree that the i2c nodes topology should end up like you said. But I think adding sub-nodes should step by step. Actually the hardware i2c topology are huge like on T4. So I'd like to adding nodes when we needed. If you think the sub-nodes are needed please send another patch based on mine. I think this is the more reasonable way. Thanks. -Hongtao -Original Message- From: Yang,Wei [mailto:wei.y...@windriver.com] Sent: Wednesday, September 04, 2013 9:27 AM To: Jia Hongtao-B38951 Cc: linuxppc-dev@lists.ozlabs.org; Wood Scott-B07421; Jia Hongtao-B38951 Subject: Re: [PATCH] powerpc: Add I2C bus multiplexer node for B4 and T4240QDS On 09/03/2013 03:51 PM, Jia Hongtao wrote: In both B4 and T4240QDS platform PCA9547 I2C bus multiplexer is used. Hi Hongtao, If you want to support I2C bus multiplexer, for T4 and B4QDS platform, since some eeprom devices is connected to PCA9574 I2C bus multiplexer, so these devices should be connected to pca9547 node. Just like the following, what do you think of it? + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + #address-cells = 1; + #size-cells = 0; + channel@0 { + #address-cells = 1; + #size-cells = 0; + reg = 0; + eeprom@51 { + compatible = at24,24c256; + reg = 0x51; + }; + eeprom@52 { + compatible = at24,24c256; + reg = 0x52; + }; + eeprom@53 { + compatible = at24,24c256; + reg = 0x53; + }; + eeprom@54 { + compatible = at24,24c256; + reg = 0x54; + }; + eeprom@55 { + compatible = at24,24c256; + reg = 0x55; + }; + eeprom@56 { + compatible = at24,24c256; + reg = 0x56; + }; + rtc@68 { + compatible = dallas,ds3232; + reg = 0x68; + interrupts = 0x1 0x1 0 0; + }; + }; Wei Signed-off-by: Jia Hongtao hongtao@freescale.com --- arch/powerpc/boot/dts/b4qds.dtsi | 4 arch/powerpc/boot/dts/t4240qds.dts | 4 2 files changed, 8 insertions(+) diff --git a/arch/powerpc/boot/dts/b4qds.dtsi b/arch/powerpc/boot/dts/b4qds.dtsi index e6d2f8f..2aa3399 100644 --- a/arch/powerpc/boot/dts/b4qds.dtsi +++ b/arch/powerpc/boot/dts/b4qds.dtsi @@ -120,6 +120,10 @@ }; i2c@118000 { + pca9547@77 { + compatible = philips,pca9547; + reg = 0x77; + }; eeprom@50 { compatible = at24,24c64; reg = 0x50; diff --git a/arch/powerpc/boot/dts/t4240qds.dts b/arch/powerpc/boot/dts/t4240qds.dts index 0555976..084db57