* Re: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
@ 2015-06-17 12:13 ` Maxime Coquelin
0 siblings, 0 replies; 12+ messages in thread
From: Maxime Coquelin @ 2015-06-17 12:13 UTC (permalink / raw)
To: Fabien Dessenne, Srinivas Kandagatla, Patrice Chotard,
Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
Russell King, linux-arm-kernel, kernel, devicetree, linux-kernel
Cc: Benjamin Gaignard
Hi Fabien,
On 06/16/2015 03:30 PM, Fabien Dessenne wrote:
> Add the bdisp (2D blitter for STMicroelectronics SoC) dt nodes for the
> first of the two bdisp devices, defining register address, interrupt and
> clock.
>
> Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
> ---
> arch/arm/boot/dts/stih410.dtsi | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
> index 208b5e8..6f40bc9 100644
> --- a/arch/arm/boot/dts/stih410.dtsi
> +++ b/arch/arm/boot/dts/stih410.dtsi
> @@ -10,6 +10,10 @@
> #include "stih407-family.dtsi"
> #include "stih410-pinctrl.dtsi"
> / {
> + aliases {
> + bdisp0 = &bdisp0;
> + };
> +
> soc {
> usb2_picophy1: phy2 {
> compatible = "st,stih407-usb2-phy";
> @@ -218,5 +222,13 @@
> };
> };
> };
> +
> + bdisp0:bdisp@9f10000 {
> + compatible = "st,stih407-bdisp";
> + reg = <0x9f10000 0x1000>;
> + interrupts = <GIC_SPI 38 IRQ_TYPE_NONE>;
> + clock-names = "bdisp";
> + clocks = <&clk_s_c0_flexgen CLK_IC_BDISP_0>;
> + };
Isn't the bdisp also present on STiH407 SoC?
If so, could you please move this node to stih407-family file?
Thanks,
Maxime
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
@ 2015-06-17 12:13 ` Maxime Coquelin
0 siblings, 0 replies; 12+ messages in thread
From: Maxime Coquelin @ 2015-06-17 12:13 UTC (permalink / raw)
To: Fabien Dessenne, Srinivas Kandagatla, Patrice Chotard,
Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
Russell King, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
kernel-F5mvAk5X5gdBDgjK7y7TUQ, devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Benjamin Gaignard
Hi Fabien,
On 06/16/2015 03:30 PM, Fabien Dessenne wrote:
> Add the bdisp (2D blitter for STMicroelectronics SoC) dt nodes for the
> first of the two bdisp devices, defining register address, interrupt and
> clock.
>
> Signed-off-by: Fabien Dessenne <fabien.dessenne-qxv4g6HH51o@public.gmane.org>
> ---
> arch/arm/boot/dts/stih410.dtsi | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
> index 208b5e8..6f40bc9 100644
> --- a/arch/arm/boot/dts/stih410.dtsi
> +++ b/arch/arm/boot/dts/stih410.dtsi
> @@ -10,6 +10,10 @@
> #include "stih407-family.dtsi"
> #include "stih410-pinctrl.dtsi"
> / {
> + aliases {
> + bdisp0 = &bdisp0;
> + };
> +
> soc {
> usb2_picophy1: phy2 {
> compatible = "st,stih407-usb2-phy";
> @@ -218,5 +222,13 @@
> };
> };
> };
> +
> + bdisp0:bdisp@9f10000 {
> + compatible = "st,stih407-bdisp";
> + reg = <0x9f10000 0x1000>;
> + interrupts = <GIC_SPI 38 IRQ_TYPE_NONE>;
> + clock-names = "bdisp";
> + clocks = <&clk_s_c0_flexgen CLK_IC_BDISP_0>;
> + };
Isn't the bdisp also present on STiH407 SoC?
If so, could you please move this node to stih407-family file?
Thanks,
Maxime
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
@ 2015-06-17 13:52 ` Fabien DESSENNE
0 siblings, 0 replies; 12+ messages in thread
From: Fabien DESSENNE @ 2015-06-17 13:52 UTC (permalink / raw)
To: linux-arm-kernel
Hi Maxime,
> -----Original Message-----
> From: Maxime COQUELIN
> Sent: mercredi 17 juin 2015 14:13
> To: Fabien DESSENNE; Srinivas Kandagatla; Patrice CHOTARD; Rob Herring;
> Pawel Moll; Mark Rutland; Ian Campbell; Kumar Gala; Russell King; linux-arm-
> kernel at lists.infradead.org; kernel at stlinux.com;
> devicetree at vger.kernel.org; linux-kernel at vger.kernel.org
> Cc: Benjamin Gaignard
> Subject: Re: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
>
> Hi Fabien,
>
> On 06/16/2015 03:30 PM, Fabien Dessenne wrote:
> > Add the bdisp (2D blitter for STMicroelectronics SoC) dt nodes for the
> > first of the two bdisp devices, defining register address, interrupt
> > and clock.
> >
> > Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
> > ---
> > arch/arm/boot/dts/stih410.dtsi | 12 ++++++++++++
> > 1 file changed, 12 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/stih410.dtsi
> > b/arch/arm/boot/dts/stih410.dtsi index 208b5e8..6f40bc9 100644
> > --- a/arch/arm/boot/dts/stih410.dtsi
> > +++ b/arch/arm/boot/dts/stih410.dtsi
> > @@ -10,6 +10,10 @@
> > #include "stih407-family.dtsi"
> > #include "stih410-pinctrl.dtsi"
> > / {
> > + aliases {
> > + bdisp0 = &bdisp0;
> > + };
> > +
> > soc {
> > usb2_picophy1: phy2 {
> > compatible = "st,stih407-usb2-phy"; @@ -218,5
> +222,13 @@
> > };
> > };
> > };
> > +
> > + bdisp0:bdisp at 9f10000 {
> > + compatible = "st,stih407-bdisp";
> > + reg = <0x9f10000 0x1000>;
> > + interrupts = <GIC_SPI 38 IRQ_TYPE_NONE>;
> > + clock-names = "bdisp";
> > + clocks = <&clk_s_c0_flexgen CLK_IC_BDISP_0>;
> > + };
>
> Isn't the bdisp also present on STiH407 SoC?
> If so, could you please move this node to stih407-family file?
>
> Thanks,
> Maxime
For the time being, it is not planned to be supported on STiH407. Maybe one day...
Fabien
^ permalink raw reply [flat|nested] 12+ messages in thread* RE: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
@ 2015-06-17 13:52 ` Fabien DESSENNE
0 siblings, 0 replies; 12+ messages in thread
From: Fabien DESSENNE @ 2015-06-17 13:52 UTC (permalink / raw)
To: Maxime COQUELIN, Srinivas Kandagatla, Patrice CHOTARD,
Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
Russell King, linux-arm-kernel@lists.infradead.org,
kernel@stlinux.com, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Cc: Benjamin Gaignard
Hi Maxime,
> -----Original Message-----
> From: Maxime COQUELIN
> Sent: mercredi 17 juin 2015 14:13
> To: Fabien DESSENNE; Srinivas Kandagatla; Patrice CHOTARD; Rob Herring;
> Pawel Moll; Mark Rutland; Ian Campbell; Kumar Gala; Russell King; linux-arm-
> kernel@lists.infradead.org; kernel@stlinux.com;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: Benjamin Gaignard
> Subject: Re: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
>
> Hi Fabien,
>
> On 06/16/2015 03:30 PM, Fabien Dessenne wrote:
> > Add the bdisp (2D blitter for STMicroelectronics SoC) dt nodes for the
> > first of the two bdisp devices, defining register address, interrupt
> > and clock.
> >
> > Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
> > ---
> > arch/arm/boot/dts/stih410.dtsi | 12 ++++++++++++
> > 1 file changed, 12 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/stih410.dtsi
> > b/arch/arm/boot/dts/stih410.dtsi index 208b5e8..6f40bc9 100644
> > --- a/arch/arm/boot/dts/stih410.dtsi
> > +++ b/arch/arm/boot/dts/stih410.dtsi
> > @@ -10,6 +10,10 @@
> > #include "stih407-family.dtsi"
> > #include "stih410-pinctrl.dtsi"
> > / {
> > + aliases {
> > + bdisp0 = &bdisp0;
> > + };
> > +
> > soc {
> > usb2_picophy1: phy2 {
> > compatible = "st,stih407-usb2-phy"; @@ -218,5
> +222,13 @@
> > };
> > };
> > };
> > +
> > + bdisp0:bdisp@9f10000 {
> > + compatible = "st,stih407-bdisp";
> > + reg = <0x9f10000 0x1000>;
> > + interrupts = <GIC_SPI 38 IRQ_TYPE_NONE>;
> > + clock-names = "bdisp";
> > + clocks = <&clk_s_c0_flexgen CLK_IC_BDISP_0>;
> > + };
>
> Isn't the bdisp also present on STiH407 SoC?
> If so, could you please move this node to stih407-family file?
>
> Thanks,
> Maxime
For the time being, it is not planned to be supported on STiH407. Maybe one day...
Fabien
^ permalink raw reply [flat|nested] 12+ messages in thread* RE: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
@ 2015-06-17 13:52 ` Fabien DESSENNE
0 siblings, 0 replies; 12+ messages in thread
From: Fabien DESSENNE @ 2015-06-17 13:52 UTC (permalink / raw)
To: Maxime COQUELIN, Srinivas Kandagatla, Patrice CHOTARD,
Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
Russell King,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: Benjamin Gaignard
Hi Maxime,
> -----Original Message-----
> From: Maxime COQUELIN
> Sent: mercredi 17 juin 2015 14:13
> To: Fabien DESSENNE; Srinivas Kandagatla; Patrice CHOTARD; Rob Herring;
> Pawel Moll; Mark Rutland; Ian Campbell; Kumar Gala; Russell King; linux-arm-
> kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org;
> devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Cc: Benjamin Gaignard
> Subject: Re: [PATCH] ARM: DT: STiH410: Add bdisp dt nodes
>
> Hi Fabien,
>
> On 06/16/2015 03:30 PM, Fabien Dessenne wrote:
> > Add the bdisp (2D blitter for STMicroelectronics SoC) dt nodes for the
> > first of the two bdisp devices, defining register address, interrupt
> > and clock.
> >
> > Signed-off-by: Fabien Dessenne <fabien.dessenne-qxv4g6HH51o@public.gmane.org>
> > ---
> > arch/arm/boot/dts/stih410.dtsi | 12 ++++++++++++
> > 1 file changed, 12 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/stih410.dtsi
> > b/arch/arm/boot/dts/stih410.dtsi index 208b5e8..6f40bc9 100644
> > --- a/arch/arm/boot/dts/stih410.dtsi
> > +++ b/arch/arm/boot/dts/stih410.dtsi
> > @@ -10,6 +10,10 @@
> > #include "stih407-family.dtsi"
> > #include "stih410-pinctrl.dtsi"
> > / {
> > + aliases {
> > + bdisp0 = &bdisp0;
> > + };
> > +
> > soc {
> > usb2_picophy1: phy2 {
> > compatible = "st,stih407-usb2-phy"; @@ -218,5
> +222,13 @@
> > };
> > };
> > };
> > +
> > + bdisp0:bdisp@9f10000 {
> > + compatible = "st,stih407-bdisp";
> > + reg = <0x9f10000 0x1000>;
> > + interrupts = <GIC_SPI 38 IRQ_TYPE_NONE>;
> > + clock-names = "bdisp";
> > + clocks = <&clk_s_c0_flexgen CLK_IC_BDISP_0>;
> > + };
>
> Isn't the bdisp also present on STiH407 SoC?
> If so, could you please move this node to stih407-family file?
>
> Thanks,
> Maxime
For the time being, it is not planned to be supported on STiH407. Maybe one day...
Fabien
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread