[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Fabio Estevam]

From: Fabio Estevam <fabio.estevam@freescale.com>

Add HDMI support. As the DDC I2C pins are not connected due to
pin conflicts, we do not pass the 'ddc-i2c-bus' property in 
this case.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
index 46b2fed..48ccfee 100644
--- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
@@ -76,6 +76,10 @@
 	status = "okay";
 };
 
+&hdmi {
+	status = "okay";
+};
+
 &i2c2 {
 	clock-frequency = <100000>;
 	pinctrl-names = "default";
-- 
1.9.1

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Shawn Guo]

On Tue, Jun 02, 2015 at 05:41:33PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Add HDMI support. As the DDC I2C pins are not connected due to
> pin conflicts, we do not pass the 'ddc-i2c-bus' property in 
> this case.

HDMI works without 'ddc-i2c-bus' property and I2C pins?  If not, what's
point of this patch.

Shawn

> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> index 46b2fed..48ccfee 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> @@ -76,6 +76,10 @@
>  	status = "okay";
>  };
>  
> +&hdmi {
> +	status = "okay";
> +};
> +
>  &i2c2 {
>  	clock-frequency = <100000>;
>  	pinctrl-names = "default";
> -- 
> 1.9.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Fabio Estevam]

On Mon, Jun 29, 2015 at 8:47 AM, Shawn Guo <shawnguo@kernel.org> wrote:
> On Tue, Jun 02, 2015 at 05:41:33PM -0300, Fabio Estevam wrote:
>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>
>> Add HDMI support. As the DDC I2C pins are not connected due to
>> pin conflicts, we do not pass the 'ddc-i2c-bus' property in
>> this case.
>
> HDMI works without 'ddc-i2c-bus' property and I2C pins?  If not, what's
> point of this patch.

Yes, HDMI works fine without 'ddc-i2c-bus' property.

'ddc-i2c-bus' property is not mandatory to get HDMI functional.

Regards,

Fabio Estevam

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Fabio Estevam]

On Mon, Jun 29, 2015 at 8:55 AM, Fabio Estevam <festevam@gmail.com> wrote:
> On Mon, Jun 29, 2015 at 8:47 AM, Shawn Guo <shawnguo@kernel.org> wrote:
>> On Tue, Jun 02, 2015 at 05:41:33PM -0300, Fabio Estevam wrote:
>>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>>
>>> Add HDMI support. As the DDC I2C pins are not connected due to
>>> pin conflicts, we do not pass the 'ddc-i2c-bus' property in
>>> this case.
>>
>> HDMI works without 'ddc-i2c-bus' property and I2C pins?  If not, what's
>> point of this patch.
>
> Yes, HDMI works fine without 'ddc-i2c-bus' property.
>
> 'ddc-i2c-bus' property is not mandatory to get HDMI functional.

You can also refer to
Documentation/devicetree/bindings/drm/imx/hdmi.txt, which states that
ddc-i2c-bus is an optional property.

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Shawn Guo]

On Mon, Jun 29, 2015 at 08:55:57AM -0300, Fabio Estevam wrote:
> On Mon, Jun 29, 2015 at 8:47 AM, Shawn Guo <shawnguo@kernel.org> wrote:
> > On Tue, Jun 02, 2015 at 05:41:33PM -0300, Fabio Estevam wrote:
> >> From: Fabio Estevam <fabio.estevam@freescale.com>
> >>
> >> Add HDMI support. As the DDC I2C pins are not connected due to
> >> pin conflicts, we do not pass the 'ddc-i2c-bus' property in
> >> this case.
> >
> > HDMI works without 'ddc-i2c-bus' property and I2C pins?  If not, what's
> > point of this patch.
> 
> Yes, HDMI works fine without 'ddc-i2c-bus' property.
> 
> 'ddc-i2c-bus' property is not mandatory to get HDMI functional.

Okay.  Patch applied.

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Fabio Estevam]

On Mon, Jun 29, 2015 at 9:54 AM, Shawn Guo <shawnguo@kernel.org> wrote:

>> 'ddc-i2c-bus' property is not mandatory to get HDMI functional.
>
> Okay.  Patch applied.

Applied into which tree, please?

Can't find it at:
https://git.kernel.org/cgit/linux/kernel/git/shawnguo/linux.git/

[PATCH] ARM: dts: imx6qdl-sabreauto: Add HDMI support

[Shawn Guo]

On Mon, Jul 06, 2015 at 10:05:02PM -0300, Fabio Estevam wrote:
> On Mon, Jun 29, 2015 at 9:54 AM, Shawn Guo <shawnguo@kernel.org> wrote:
> 
> >> 'ddc-i2c-bus' property is not mandatory to get HDMI functional.
> >
> > Okay.  Patch applied.
> 
> Applied into which tree, please?
> 
> Can't find it at:
> https://git.kernel.org/cgit/linux/kernel/git/shawnguo/linux.git/

I'm rebasing the branches to v4.2-rc1, and will push them out soon.

Shawn