* [PATCH v2 0/3] ARM: dts: BCM5301X: GIC_PPI bug, TWD WD and i2c Support
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: linux-arm-kernel
Changes in v2:
* Rafal discovered an issue with the GIC_PPI flags being incorrect.
Since there was a dependency on that DT entry in this series, I added
a patch to correct the issue and reworked the TWD patch with the
changes.
These seemed to have been lost sometime last year. See
https://lkml.org/lkml/2016/5/11/953
I am dropping the syscon reboot from the series, as I believe that was
the sticking point, and resending the TWD and i2c patches.
Jon Mason (3):
ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
ARM: dts: bcm5301x: Add TWD WD Support to DT
ARM: dts: bcm5301x: Add I2C support to the DT
arch/arm/boot/dts/bcm5301x.dtsi | 26 ++++++++++++++++++++++----
1 file changed, 22 insertions(+), 4 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 0/3] ARM: dts: BCM5301X: GIC_PPI bug, TWD WD and i2c Support
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Mark Rutland, devicetree, Florian Fainelli, linux-kernel,
Rob Herring, bcm-kernel-feedback-list, linux-arm-kernel
Changes in v2:
* Rafal discovered an issue with the GIC_PPI flags being incorrect.
Since there was a dependency on that DT entry in this series, I added
a patch to correct the issue and reworked the TWD patch with the
changes.
These seemed to have been lost sometime last year. See
https://lkml.org/lkml/2016/5/11/953
I am dropping the syscon reboot from the series, as I believe that was
the sticking point, and resending the TWD and i2c patches.
Jon Mason (3):
ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
ARM: dts: bcm5301x: Add TWD WD Support to DT
ARM: dts: bcm5301x: Add I2C support to the DT
arch/arm/boot/dts/bcm5301x.dtsi | 26 ++++++++++++++++++++++----
1 file changed, 22 insertions(+), 4 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 1/3] ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
2017-03-03 0:21 ` Jon Mason
(?)
@ 2017-03-03 0:21 ` Jon Mason
-1 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Rob Herring, Mark Rutland, Florian Fainelli, devicetree,
linux-arm-kernel, linux-kernel, bcm-kernel-feedback-list
GIC_PPI flags were misconfigured for the timers, resulting in errors
like:
[ 0.000000] GIC: PPI11 is secure or misconfigured
Changing them to being edge triggered corrects the issue
Suggested-by: Rafał Miłecki <rafal@milecki.pl>
Signed-off-by: Jon Mason <jon.mason@broadcom.com>
Fixes: d27509f1 ("ARM: BCM5301X: add dts files for BCM4708 SoC")
---
arch/arm/boot/dts/bcm5301x.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 4fbb089..00de62d 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -66,14 +66,14 @@
timer@20200 {
compatible = "arm,cortex-a9-global-timer";
reg = <0x20200 0x100>;
- interrupts = <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_PPI 11 IRQ_TYPE_EDGE_RISING>;
clocks = <&periph_clk>;
};
local-timer@20600 {
compatible = "arm,cortex-a9-twd-timer";
reg = <0x20600 0x100>;
- interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_PPI 13 IRQ_TYPE_EDGE_RISING>;
clocks = <&periph_clk>;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v2 1/3] ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: linux-arm-kernel
GIC_PPI flags were misconfigured for the timers, resulting in errors
like:
[ 0.000000] GIC: PPI11 is secure or misconfigured
Changing them to being edge triggered corrects the issue
Suggested-by: Rafa? Mi?ecki <rafal@milecki.pl>
Signed-off-by: Jon Mason <jon.mason@broadcom.com>
Fixes: d27509f1 ("ARM: BCM5301X: add dts files for BCM4708 SoC")
---
arch/arm/boot/dts/bcm5301x.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 4fbb089..00de62d 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -66,14 +66,14 @@
timer at 20200 {
compatible = "arm,cortex-a9-global-timer";
reg = <0x20200 0x100>;
- interrupts = <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_PPI 11 IRQ_TYPE_EDGE_RISING>;
clocks = <&periph_clk>;
};
local-timer at 20600 {
compatible = "arm,cortex-a9-twd-timer";
reg = <0x20600 0x100>;
- interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_PPI 13 IRQ_TYPE_EDGE_RISING>;
clocks = <&periph_clk>;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v2 1/3] ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Mark Rutland, devicetree, Florian Fainelli, linux-kernel,
Rob Herring, bcm-kernel-feedback-list, linux-arm-kernel
GIC_PPI flags were misconfigured for the timers, resulting in errors
like:
[ 0.000000] GIC: PPI11 is secure or misconfigured
Changing them to being edge triggered corrects the issue
Suggested-by: Rafał Miłecki <rafal@milecki.pl>
Signed-off-by: Jon Mason <jon.mason@broadcom.com>
Fixes: d27509f1 ("ARM: BCM5301X: add dts files for BCM4708 SoC")
---
arch/arm/boot/dts/bcm5301x.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 4fbb089..00de62d 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -66,14 +66,14 @@
timer@20200 {
compatible = "arm,cortex-a9-global-timer";
reg = <0x20200 0x100>;
- interrupts = <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_PPI 11 IRQ_TYPE_EDGE_RISING>;
clocks = <&periph_clk>;
};
local-timer@20600 {
compatible = "arm,cortex-a9-twd-timer";
reg = <0x20600 0x100>;
- interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_PPI 13 IRQ_TYPE_EDGE_RISING>;
clocks = <&periph_clk>;
};
--
2.7.4
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH v2 1/3] ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
2017-03-03 0:21 ` Jon Mason
@ 2017-03-06 1:11 ` Florian Fainelli
-1 siblings, 0 replies; 20+ messages in thread
From: Florian Fainelli @ 2017-03-06 1:11 UTC (permalink / raw)
To: Jon Mason, Hauke Mehrtens, zajec5
Cc: Mark Rutland, devicetree, Florian Fainelli, linux-kernel,
Rob Herring, bcm-kernel-feedback-list, linux-arm-kernel
On 03/02/2017 04:21 PM, Jon Mason wrote:
> GIC_PPI flags were misconfigured for the timers, resulting in errors
> like:
> [ 0.000000] GIC: PPI11 is secure or misconfigured
>
> Changing them to being edge triggered corrects the issue
>
> Suggested-by: Rafał Miłecki <rafal@milecki.pl>
> Signed-off-by: Jon Mason <jon.mason@broadcom.com>
> Fixes: d27509f1 ("ARM: BCM5301X: add dts files for BCM4708 SoC")
Applied, thanks!
--
Florian
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 20+ messages in thread* [PATCH v2 1/3] ARM: dts: BCM5301X: Correct GIC_PPI interrupt flags
@ 2017-03-06 1:11 ` Florian Fainelli
0 siblings, 0 replies; 20+ messages in thread
From: Florian Fainelli @ 2017-03-06 1:11 UTC (permalink / raw)
To: linux-arm-kernel
On 03/02/2017 04:21 PM, Jon Mason wrote:
> GIC_PPI flags were misconfigured for the timers, resulting in errors
> like:
> [ 0.000000] GIC: PPI11 is secure or misconfigured
>
> Changing them to being edge triggered corrects the issue
>
> Suggested-by: Rafa? Mi?ecki <rafal@milecki.pl>
> Signed-off-by: Jon Mason <jon.mason@broadcom.com>
> Fixes: d27509f1 ("ARM: BCM5301X: add dts files for BCM4708 SoC")
Applied, thanks!
--
Florian
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 2/3] ARM: dts: bcm5301x: Add TWD WD Support to DT
2017-03-03 0:21 ` Jon Mason
(?)
@ 2017-03-03 0:21 ` Jon Mason
-1 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Rob Herring, Mark Rutland, Florian Fainelli, devicetree,
linux-arm-kernel, linux-kernel, bcm-kernel-feedback-list,
Jon Mason
From: Jon Mason <jonmason@broadcom.com>
Add support for the ARM TWD Watchdog to the bcm5301x device tree. The
ARM TWD timer allocated the register space for the WDT, so this patch
necessitated shrinking that. Also, the GIC masks were added for these.
Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
arch/arm/boot/dts/bcm5301x.dtsi | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 00de62d..4e3fbce 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -70,10 +70,19 @@
clocks = <&periph_clk>;
};
- local-timer@20600 {
+ timer@20600 {
compatible = "arm,cortex-a9-twd-timer";
- reg = <0x20600 0x100>;
- interrupts = <GIC_PPI 13 IRQ_TYPE_EDGE_RISING>;
+ reg = <0x20600 0x20>;
+ interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(2) |
+ IRQ_TYPE_EDGE_RISING)>;
+ clocks = <&periph_clk>;
+ };
+
+ watchdog@20620 {
+ compatible = "arm,cortex-a9-twd-wdt";
+ reg = <0x20620 0x20>;
+ interrupts = <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(2) |
+ IRQ_TYPE_EDGE_RISING)>;
clocks = <&periph_clk>;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v2 2/3] ARM: dts: bcm5301x: Add TWD WD Support to DT
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: linux-arm-kernel
From: Jon Mason <jonmason@broadcom.com>
Add support for the ARM TWD Watchdog to the bcm5301x device tree. The
ARM TWD timer allocated the register space for the WDT, so this patch
necessitated shrinking that. Also, the GIC masks were added for these.
Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
arch/arm/boot/dts/bcm5301x.dtsi | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 00de62d..4e3fbce 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -70,10 +70,19 @@
clocks = <&periph_clk>;
};
- local-timer at 20600 {
+ timer at 20600 {
compatible = "arm,cortex-a9-twd-timer";
- reg = <0x20600 0x100>;
- interrupts = <GIC_PPI 13 IRQ_TYPE_EDGE_RISING>;
+ reg = <0x20600 0x20>;
+ interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(2) |
+ IRQ_TYPE_EDGE_RISING)>;
+ clocks = <&periph_clk>;
+ };
+
+ watchdog at 20620 {
+ compatible = "arm,cortex-a9-twd-wdt";
+ reg = <0x20620 0x20>;
+ interrupts = <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(2) |
+ IRQ_TYPE_EDGE_RISING)>;
clocks = <&periph_clk>;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v2 2/3] ARM: dts: bcm5301x: Add TWD WD Support to DT
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Mark Rutland, devicetree, Florian Fainelli, Jon Mason,
linux-kernel, Rob Herring, bcm-kernel-feedback-list,
linux-arm-kernel
From: Jon Mason <jonmason@broadcom.com>
Add support for the ARM TWD Watchdog to the bcm5301x device tree. The
ARM TWD timer allocated the register space for the WDT, so this patch
necessitated shrinking that. Also, the GIC masks were added for these.
Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
arch/arm/boot/dts/bcm5301x.dtsi | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 00de62d..4e3fbce 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -70,10 +70,19 @@
clocks = <&periph_clk>;
};
- local-timer@20600 {
+ timer@20600 {
compatible = "arm,cortex-a9-twd-timer";
- reg = <0x20600 0x100>;
- interrupts = <GIC_PPI 13 IRQ_TYPE_EDGE_RISING>;
+ reg = <0x20600 0x20>;
+ interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(2) |
+ IRQ_TYPE_EDGE_RISING)>;
+ clocks = <&periph_clk>;
+ };
+
+ watchdog@20620 {
+ compatible = "arm,cortex-a9-twd-wdt";
+ reg = <0x20620 0x20>;
+ interrupts = <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(2) |
+ IRQ_TYPE_EDGE_RISING)>;
clocks = <&periph_clk>;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
2017-03-03 0:21 ` Jon Mason
(?)
@ 2017-03-03 0:21 ` Jon Mason
-1 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Rob Herring, Mark Rutland, Florian Fainelli, devicetree,
linux-arm-kernel, linux-kernel, bcm-kernel-feedback-list,
Jon Mason
From: Jon Mason <jonmason@broadcom.com>
Add I2C support to the bcm5301x Device Tree. Since no driver changes
are needed to enable this hardware, only the device tree changes are
required to make this functional.
Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 4e3fbce..2a343f2 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -338,6 +338,15 @@
};
};
+ i2c0: i2c@18009000 {
+ compatible = "brcm,iproc-i2c";
+ reg = <0x18009000 0x50>;
+ interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ clock-frequency = <100000>;
+ };
+
lcpll0: lcpll0@1800c100 {
#clock-cells = <1>;
compatible = "brcm,nsp-lcpll0";
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: linux-arm-kernel
From: Jon Mason <jonmason@broadcom.com>
Add I2C support to the bcm5301x Device Tree. Since no driver changes
are needed to enable this hardware, only the device tree changes are
required to make this functional.
Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 4e3fbce..2a343f2 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -338,6 +338,15 @@
};
};
+ i2c0: i2c at 18009000 {
+ compatible = "brcm,iproc-i2c";
+ reg = <0x18009000 0x50>;
+ interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ clock-frequency = <100000>;
+ };
+
lcpll0: lcpll0 at 1800c100 {
#clock-cells = <1>;
compatible = "brcm,nsp-lcpll0";
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
@ 2017-03-03 0:21 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-03 0:21 UTC (permalink / raw)
To: Hauke Mehrtens, zajec5
Cc: Mark Rutland, devicetree, Florian Fainelli, Jon Mason,
linux-kernel, Rob Herring, bcm-kernel-feedback-list,
linux-arm-kernel
From: Jon Mason <jonmason@broadcom.com>
Add I2C support to the bcm5301x Device Tree. Since no driver changes
are needed to enable this hardware, only the device tree changes are
required to make this functional.
Signed-off-by: Jon Mason <jonmason@broadcom.com>
---
arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 4e3fbce..2a343f2 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -338,6 +338,15 @@
};
};
+ i2c0: i2c@18009000 {
+ compatible = "brcm,iproc-i2c";
+ reg = <0x18009000 0x50>;
+ interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ clock-frequency = <100000>;
+ };
+
lcpll0: lcpll0@1800c100 {
#clock-cells = <1>;
compatible = "brcm,nsp-lcpll0";
--
2.7.4
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
2017-03-03 0:21 ` Jon Mason
(?)
@ 2017-03-06 1:13 ` Florian Fainelli
-1 siblings, 0 replies; 20+ messages in thread
From: Florian Fainelli @ 2017-03-06 1:13 UTC (permalink / raw)
To: Jon Mason, Hauke Mehrtens, zajec5
Cc: Rob Herring, Mark Rutland, Florian Fainelli, devicetree,
linux-arm-kernel, linux-kernel, bcm-kernel-feedback-list,
Jon Mason
On 03/02/2017 04:21 PM, Jon Mason wrote:
> From: Jon Mason <jonmason@broadcom.com>
>
> Add I2C support to the bcm5301x Device Tree. Since no driver changes
> are needed to enable this hardware, only the device tree changes are
> required to make this functional.
>
> Signed-off-by: Jon Mason <jonmason@broadcom.com>
> ---
> arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
> index 4e3fbce..2a343f2 100644
> --- a/arch/arm/boot/dts/bcm5301x.dtsi
> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
> @@ -338,6 +338,15 @@
> };
> };
>
> + i2c0: i2c@18009000 {
> + compatible = "brcm,iproc-i2c";
> + reg = <0x18009000 0x50>;
> + interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clock-frequency = <100000>;
Should not this have a:
status = "disabled"
property by default such that boards can override this by phandle when
there are actual I2C devices available?
> + };
> +
> lcpll0: lcpll0@1800c100 {
> #clock-cells = <1>;
> compatible = "brcm,nsp-lcpll0";
>
--
Florian
^ permalink raw reply [flat|nested] 20+ messages in thread* [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
@ 2017-03-06 1:13 ` Florian Fainelli
0 siblings, 0 replies; 20+ messages in thread
From: Florian Fainelli @ 2017-03-06 1:13 UTC (permalink / raw)
To: linux-arm-kernel
On 03/02/2017 04:21 PM, Jon Mason wrote:
> From: Jon Mason <jonmason@broadcom.com>
>
> Add I2C support to the bcm5301x Device Tree. Since no driver changes
> are needed to enable this hardware, only the device tree changes are
> required to make this functional.
>
> Signed-off-by: Jon Mason <jonmason@broadcom.com>
> ---
> arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
> index 4e3fbce..2a343f2 100644
> --- a/arch/arm/boot/dts/bcm5301x.dtsi
> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
> @@ -338,6 +338,15 @@
> };
> };
>
> + i2c0: i2c at 18009000 {
> + compatible = "brcm,iproc-i2c";
> + reg = <0x18009000 0x50>;
> + interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clock-frequency = <100000>;
Should not this have a:
status = "disabled"
property by default such that boards can override this by phandle when
there are actual I2C devices available?
> + };
> +
> lcpll0: lcpll0 at 1800c100 {
> #clock-cells = <1>;
> compatible = "brcm,nsp-lcpll0";
>
--
Florian
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
@ 2017-03-06 1:13 ` Florian Fainelli
0 siblings, 0 replies; 20+ messages in thread
From: Florian Fainelli @ 2017-03-06 1:13 UTC (permalink / raw)
To: Jon Mason, Hauke Mehrtens, zajec5
Cc: Mark Rutland, devicetree, Florian Fainelli, Jon Mason,
linux-kernel, Rob Herring, bcm-kernel-feedback-list,
linux-arm-kernel
On 03/02/2017 04:21 PM, Jon Mason wrote:
> From: Jon Mason <jonmason@broadcom.com>
>
> Add I2C support to the bcm5301x Device Tree. Since no driver changes
> are needed to enable this hardware, only the device tree changes are
> required to make this functional.
>
> Signed-off-by: Jon Mason <jonmason@broadcom.com>
> ---
> arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
> index 4e3fbce..2a343f2 100644
> --- a/arch/arm/boot/dts/bcm5301x.dtsi
> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
> @@ -338,6 +338,15 @@
> };
> };
>
> + i2c0: i2c@18009000 {
> + compatible = "brcm,iproc-i2c";
> + reg = <0x18009000 0x50>;
> + interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clock-frequency = <100000>;
Should not this have a:
status = "disabled"
property by default such that boards can override this by phandle when
there are actual I2C devices available?
> + };
> +
> lcpll0: lcpll0@1800c100 {
> #clock-cells = <1>;
> compatible = "brcm,nsp-lcpll0";
>
--
Florian
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
2017-03-06 1:13 ` Florian Fainelli
(?)
@ 2017-03-06 15:12 ` Jon Mason
-1 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-06 15:12 UTC (permalink / raw)
To: Florian Fainelli
Cc: Hauke Mehrtens, Rafał Miłecki, Rob Herring,
Mark Rutland,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
linux-arm-kernel, open list, BCM Kernel Feedback, Jon Mason
On Sun, Mar 5, 2017 at 8:13 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>
>
> On 03/02/2017 04:21 PM, Jon Mason wrote:
>> From: Jon Mason <jonmason@broadcom.com>
>>
>> Add I2C support to the bcm5301x Device Tree. Since no driver changes
>> are needed to enable this hardware, only the device tree changes are
>> required to make this functional.
>>
>> Signed-off-by: Jon Mason <jonmason@broadcom.com>
>> ---
>> arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
>> 1 file changed, 9 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
>> index 4e3fbce..2a343f2 100644
>> --- a/arch/arm/boot/dts/bcm5301x.dtsi
>> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
>> @@ -338,6 +338,15 @@
>> };
>> };
>>
>> + i2c0: i2c@18009000 {
>> + compatible = "brcm,iproc-i2c";
>> + reg = <0x18009000 0x50>;
>> + interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + clock-frequency = <100000>;
>
> Should not this have a:
> status = "disabled"
>
> property by default such that boards can override this by phandle when
> there are actual I2C devices available?
Your comment appears to match the way it is being done in most device trees.
NSP implementation is identical to this. So, if I should change it
here, I should make there too.
Thanks,
Jon
>> + };
>> +
>> lcpll0: lcpll0@1800c100 {
>> #clock-cells = <1>;
>> compatible = "brcm,nsp-lcpll0";
>>
>
> --
> Florian
^ permalink raw reply [flat|nested] 20+ messages in thread* [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
@ 2017-03-06 15:12 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-06 15:12 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Mar 5, 2017 at 8:13 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>
>
> On 03/02/2017 04:21 PM, Jon Mason wrote:
>> From: Jon Mason <jonmason@broadcom.com>
>>
>> Add I2C support to the bcm5301x Device Tree. Since no driver changes
>> are needed to enable this hardware, only the device tree changes are
>> required to make this functional.
>>
>> Signed-off-by: Jon Mason <jonmason@broadcom.com>
>> ---
>> arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
>> 1 file changed, 9 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
>> index 4e3fbce..2a343f2 100644
>> --- a/arch/arm/boot/dts/bcm5301x.dtsi
>> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
>> @@ -338,6 +338,15 @@
>> };
>> };
>>
>> + i2c0: i2c at 18009000 {
>> + compatible = "brcm,iproc-i2c";
>> + reg = <0x18009000 0x50>;
>> + interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + clock-frequency = <100000>;
>
> Should not this have a:
> status = "disabled"
>
> property by default such that boards can override this by phandle when
> there are actual I2C devices available?
Your comment appears to match the way it is being done in most device trees.
NSP implementation is identical to this. So, if I should change it
here, I should make there too.
Thanks,
Jon
>> + };
>> +
>> lcpll0: lcpll0 at 1800c100 {
>> #clock-cells = <1>;
>> compatible = "brcm,nsp-lcpll0";
>>
>
> --
> Florian
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH v2 3/3] ARM: dts: bcm5301x: Add I2C support to the DT
@ 2017-03-06 15:12 ` Jon Mason
0 siblings, 0 replies; 20+ messages in thread
From: Jon Mason @ 2017-03-06 15:12 UTC (permalink / raw)
To: Florian Fainelli
Cc: Mark Rutland,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Jon Mason, Hauke Mehrtens, Rafał Miłecki, open list,
Rob Herring, BCM Kernel Feedback, linux-arm-kernel
On Sun, Mar 5, 2017 at 8:13 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>
>
> On 03/02/2017 04:21 PM, Jon Mason wrote:
>> From: Jon Mason <jonmason@broadcom.com>
>>
>> Add I2C support to the bcm5301x Device Tree. Since no driver changes
>> are needed to enable this hardware, only the device tree changes are
>> required to make this functional.
>>
>> Signed-off-by: Jon Mason <jonmason@broadcom.com>
>> ---
>> arch/arm/boot/dts/bcm5301x.dtsi | 9 +++++++++
>> 1 file changed, 9 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
>> index 4e3fbce..2a343f2 100644
>> --- a/arch/arm/boot/dts/bcm5301x.dtsi
>> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
>> @@ -338,6 +338,15 @@
>> };
>> };
>>
>> + i2c0: i2c@18009000 {
>> + compatible = "brcm,iproc-i2c";
>> + reg = <0x18009000 0x50>;
>> + interrupts = <GIC_SPI 121 IRQ_TYPE_NONE>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + clock-frequency = <100000>;
>
> Should not this have a:
> status = "disabled"
>
> property by default such that boards can override this by phandle when
> there are actual I2C devices available?
Your comment appears to match the way it is being done in most device trees.
NSP implementation is identical to this. So, if I should change it
here, I should make there too.
Thanks,
Jon
>> + };
>> +
>> lcpll0: lcpll0@1800c100 {
>> #clock-cells = <1>;
>> compatible = "brcm,nsp-lcpll0";
>>
>
> --
> Florian
^ permalink raw reply [flat|nested] 20+ messages in thread