* [PATCH 1/8] ARM: shmobile: r8a7790: Add PCIe clock device tree nodes
@ 2014-05-28 7:23 Phil Edworthy
2014-06-09 10:29 ` Laurent Pinchart
2014-06-10 8:01 ` Phil Edworthy
0 siblings, 2 replies; 3+ messages in thread
From: Phil Edworthy @ 2014-05-28 7:23 UTC (permalink / raw)
To: linux-sh
This patch adds the device tree clock nodes for PCIe
Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
---
arch/arm/boot/dts/r8a7790.dtsi | 6 +++---
include/dt-bindings/clock/r8a7790-clock.h | 1 +
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index 7ff2960..ae7d3da 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -754,17 +754,17 @@
reg = <0 0xe615013c 0 4>, <0 0xe6150048 0 4>;
clocks = <&hp_clk>, <&cp_clk>, <&mmc1_clk>, <&sd3_clk>,
<&sd2_clk>, <&cpg_clocks R8A7790_CLK_SD1>, <&cpg_clocks R8A7790_CLK_SD0>, <&mmc0_clk>,
- <&hp_clk>, <&hp_clk>, <&rclk_clk>;
+ <&hp_clk>, <&hp_clk>, <&rclk_clk>, <&mp_clk>;
#clock-cells = <1>;
renesas,clock-indices = <
R8A7790_CLK_IIC2 R8A7790_CLK_TPU0 R8A7790_CLK_MMCIF1 R8A7790_CLK_SDHI3
R8A7790_CLK_SDHI2 R8A7790_CLK_SDHI1 R8A7790_CLK_SDHI0 R8A7790_CLK_MMCIF0
- R8A7790_CLK_IIC0 R8A7790_CLK_IIC1 R8A7790_CLK_CMT1
+ R8A7790_CLK_IIC0 R8A7790_CLK_IIC1 R8A7790_CLK_CMT1 R8A7790_CLK_PCIE
>;
clock-output-names "iic2", "tpu0", "mmcif1", "sdhi3",
"sdhi2", "sdhi1", "sdhi0", "mmcif0",
- "iic0", "iic1", "cmt1";
+ "iic0", "iic1", "cmt1", "pcie";
};
mstp5_clks: mstp5_clks@e6150144 {
compatible = "renesas,r8a7790-mstp-clocks", "renesas,cpg-mstp-clocks";
diff --git a/include/dt-bindings/clock/r8a7790-clock.h b/include/dt-bindings/clock/r8a7790-clock.h
index 1118f7a..e0a6e38 100644
--- a/include/dt-bindings/clock/r8a7790-clock.h
+++ b/include/dt-bindings/clock/r8a7790-clock.h
@@ -59,6 +59,7 @@
#define R8A7790_CLK_SDHI0 14
#define R8A7790_CLK_MMCIF0 15
#define R8A7790_CLK_IIC0 18
+#define R8A7790_CLK_PCIE 19
#define R8A7790_CLK_IIC1 23
#define R8A7790_CLK_SSUSB 28
#define R8A7790_CLK_CMT1 29
--
1.9.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/8] ARM: shmobile: r8a7790: Add PCIe clock device tree nodes
2014-05-28 7:23 [PATCH 1/8] ARM: shmobile: r8a7790: Add PCIe clock device tree nodes Phil Edworthy
@ 2014-06-09 10:29 ` Laurent Pinchart
2014-06-10 8:01 ` Phil Edworthy
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Pinchart @ 2014-06-09 10:29 UTC (permalink / raw)
To: linux-sh
Hi Phil,
Thank you for the patch.
On Wednesday 28 May 2014 08:23:56 Phil Edworthy wrote:
> This patch adds the device tree clock nodes for PCIe
>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> ---
> arch/arm/boot/dts/r8a7790.dtsi | 6 +++---
> include/dt-bindings/clock/r8a7790-clock.h | 1 +
> 2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index 7ff2960..ae7d3da 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -754,17 +754,17 @@
> reg = <0 0xe615013c 0 4>, <0 0xe6150048 0 4>;
> clocks = <&hp_clk>, <&cp_clk>, <&mmc1_clk>, <&sd3_clk>,
> <&sd2_clk>, <&cpg_clocks R8A7790_CLK_SD1>, <&cpg_clocks
> R8A7790_CLK_SD0>, <&mmc0_clk>,
> - <&hp_clk>, <&hp_clk>, <&rclk_clk>;
> + <&hp_clk>, <&hp_clk>, <&rclk_clk>, <&mp_clk>;
> #clock-cells = <1>;
> renesas,clock-indices = <
> R8A7790_CLK_IIC2 R8A7790_CLK_TPU0 R8A7790_CLK_MMCIF1
> R8A7790_CLK_SDHI3
> R8A7790_CLK_SDHI2 R8A7790_CLK_SDHI1 R8A7790_CLK_SDHI0
> R8A7790_CLK_MMCIF0
> - R8A7790_CLK_IIC0 R8A7790_CLK_IIC1 R8A7790_CLK_CMT1
> + R8A7790_CLK_IIC0 R8A7790_CLK_IIC1 R8A7790_CLK_CMT1
> R8A7790_CLK_PCIE
We try to keep the entries sorted by bit number. Could you this please insert
the PCIE clock at the right position (between IIC0 and IIC1) ? Don't forget to
update the clocks and clock-output-names properties accordingly.
> >;
>
> clock-output-names > "iic2", "tpu0", "mmcif1", "sdhi3",
> "sdhi2", "sdhi1", "sdhi0", "mmcif0",
> - "iic0", "iic1", "cmt1";
> + "iic0", "iic1", "cmt1", "pcie";
> };
> mstp5_clks: mstp5_clks@e6150144 {
> compatible = "renesas,r8a7790-mstp-clocks", "renesas,cpg-mstp-
clocks";
> diff --git a/include/dt-bindings/clock/r8a7790-clock.h
> b/include/dt-bindings/clock/r8a7790-clock.h index 1118f7a..e0a6e38 100644
> --- a/include/dt-bindings/clock/r8a7790-clock.h
> +++ b/include/dt-bindings/clock/r8a7790-clock.h
> @@ -59,6 +59,7 @@
> #define R8A7790_CLK_SDHI0 14
> #define R8A7790_CLK_MMCIF0 15
> #define R8A7790_CLK_IIC0 18
> +#define R8A7790_CLK_PCIE 19
The datasheet documents the MSTP bit as "PCIEC". Would it make sense to use
the same name here, and in the clock-output-names property ?
The same comments apply to patch 2/8.
> #define R8A7790_CLK_IIC1 23
> #define R8A7790_CLK_SSUSB 28
> #define R8A7790_CLK_CMT1 29
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH 1/8] ARM: shmobile: r8a7790: Add PCIe clock device tree nodes
2014-05-28 7:23 [PATCH 1/8] ARM: shmobile: r8a7790: Add PCIe clock device tree nodes Phil Edworthy
2014-06-09 10:29 ` Laurent Pinchart
@ 2014-06-10 8:01 ` Phil Edworthy
1 sibling, 0 replies; 3+ messages in thread
From: Phil Edworthy @ 2014-06-10 8:01 UTC (permalink / raw)
To: linux-sh
Hi Laurent,
On 09 June 2014 11:29, Laurent wrote:
> On Wednesday 28 May 2014 08:23:56 Phil Edworthy wrote:
> > This patch adds the device tree clock nodes for PCIe
> >
> > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> > ---
> > arch/arm/boot/dts/r8a7790.dtsi | 6 +++---
> > include/dt-bindings/clock/r8a7790-clock.h | 1 +
> > 2 files changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/r8a7790.dtsi
> b/arch/arm/boot/dts/r8a7790.dtsi
> > index 7ff2960..ae7d3da 100644
> > --- a/arch/arm/boot/dts/r8a7790.dtsi
> > +++ b/arch/arm/boot/dts/r8a7790.dtsi
> > @@ -754,17 +754,17 @@
> > reg = <0 0xe615013c 0 4>, <0 0xe6150048 0 4>;
> > clocks = <&hp_clk>, <&cp_clk>, <&mmc1_clk>,
> <&sd3_clk>,
> > <&sd2_clk>, <&cpg_clocks
> R8A7790_CLK_SD1>, <&cpg_clocks
> > R8A7790_CLK_SD0>, <&mmc0_clk>,
> > - <&hp_clk>, <&hp_clk>, <&rclk_clk>;
> > + <&hp_clk>, <&hp_clk>, <&rclk_clk>,
> <&mp_clk>;
> > #clock-cells = <1>;
> > renesas,clock-indices = <
> > R8A7790_CLK_IIC2 R8A7790_CLK_TPU0
> R8A7790_CLK_MMCIF1
> > R8A7790_CLK_SDHI3
> > R8A7790_CLK_SDHI2 R8A7790_CLK_SDHI1
> R8A7790_CLK_SDHI0
> > R8A7790_CLK_MMCIF0
> > - R8A7790_CLK_IIC0 R8A7790_CLK_IIC1
> R8A7790_CLK_CMT1
> > + R8A7790_CLK_IIC0 R8A7790_CLK_IIC1
> R8A7790_CLK_CMT1
> > R8A7790_CLK_PCIE
>
> We try to keep the entries sorted by bit number. Could you this please insert
> the PCIE clock at the right position (between IIC0 and IIC1) ? Don't forget to
> update the clocks and clock-output-names properties accordingly.
No problem, will do.
> > >;
> >
> > clock-output-names > > "iic2", "tpu0", "mmcif1", "sdhi3",
> > "sdhi2", "sdhi1", "sdhi0", "mmcif0",
> > - "iic0", "iic1", "cmt1";
> > + "iic0", "iic1", "cmt1", "pcie";
> > };
> > mstp5_clks: mstp5_clks@e6150144 {
> > compatible = "renesas,r8a7790-mstp-clocks",
> "renesas,cpg-mstp-
> clocks";
> > diff --git a/include/dt-bindings/clock/r8a7790-clock.h
> > b/include/dt-bindings/clock/r8a7790-clock.h index 1118f7a..e0a6e38
> 100644
> > --- a/include/dt-bindings/clock/r8a7790-clock.h
> > +++ b/include/dt-bindings/clock/r8a7790-clock.h
> > @@ -59,6 +59,7 @@
> > #define R8A7790_CLK_SDHI0 14
> > #define R8A7790_CLK_MMCIF0 15
> > #define R8A7790_CLK_IIC0 18
> > +#define R8A7790_CLK_PCIE 19
>
> The datasheet documents the MSTP bit as "PCIEC". Would it make sense to
> use
> the same name here, and in the clock-output-names property ?
>
> The same comments apply to patch 2/8.
Sure.
> > #define R8A7790_CLK_IIC1 23
> > #define R8A7790_CLK_SSUSB 28
> > #define R8A7790_CLK_CMT1 29
Thanks
Phil
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-06-10 8:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-28 7:23 [PATCH 1/8] ARM: shmobile: r8a7790: Add PCIe clock device tree nodes Phil Edworthy
2014-06-09 10:29 ` Laurent Pinchart
2014-06-10 8:01 ` Phil Edworthy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox