| Message ID | e68f9647bff12fda5c70b3f068d3feb3911bca91.1425656120.git.jsarha@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
* Jyri Sarha <jsarha@ti.com> [150306 07:37]: > Use new binding for the external tda19988 HDMI encoder. Can this be merged separately after the driver code is merged with things still working? Otherwise we can have glitches with the output working and have dependencies between kernel branches. Regards, Tony > Signed-off-by: Jyri Sarha <jsarha@ti.com> > --- > arch/arm/boot/dts/am335x-boneblack.dts | 20 +++++++++++++++----- > 1 file changed, 15 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/boot/dts/am335x-boneblack.dts b/arch/arm/boot/dts/am335x-boneblack.dts > index 5c42d25..eadbba3 100644 > --- a/arch/arm/boot/dts/am335x-boneblack.dts > +++ b/arch/arm/boot/dts/am335x-boneblack.dts > @@ -68,16 +68,26 @@ > > &lcdc { > status = "okay"; > + port { > + lcdc_0: endpoint@0 { > + remote-endpoint = <&hdmi_0>; > + }; > + }; > }; > > -/ { > - hdmi { > - compatible = "ti,tilcdc,slave"; > - i2c = <&i2c0>; > +&i2c0 { > + tda19988 { > + compatible = "nxp,tda998x"; > + reg = <0x70>; > pinctrl-names = "default", "off"; > pinctrl-0 = <&nxp_hdmi_bonelt_pins>; > pinctrl-1 = <&nxp_hdmi_bonelt_off_pins>; > - status = "okay"; > + > + port { > + hdmi_0: endpoint@0 { > + remote-endpoint = <&lcdc_0>; > + }; > + }; > }; > }; > > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 03/06/15 17:36, Tony Lindgren wrote: > * Jyri Sarha <jsarha@ti.com> [150306 07:37]: >> Use new binding for the external tda19988 HDMI encoder. > > Can this be merged separately after the driver code is > merged with things still working? > Yes absolutely. That is why DRM_TILCDC_SLAVE_COMPAT is there for. Cheers, Jyri -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Jyri Sarha <jsarha@ti.com> [150306 07:57]: > On 03/06/15 17:36, Tony Lindgren wrote: > >* Jyri Sarha <jsarha@ti.com> [150306 07:37]: > >>Use new binding for the external tda19988 HDMI encoder. > > > >Can this be merged separately after the driver code is > >merged with things still working? > > > > Yes absolutely. That is why DRM_TILCDC_SLAVE_COMPAT is there for. Great, thanks for sorting it out properly. Please ping me to apply this patch when the driver changes are in Linux next. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/am335x-boneblack.dts b/arch/arm/boot/dts/am335x-boneblack.dts index 5c42d25..eadbba3 100644 --- a/arch/arm/boot/dts/am335x-boneblack.dts +++ b/arch/arm/boot/dts/am335x-boneblack.dts @@ -68,16 +68,26 @@ &lcdc { status = "okay"; + port { + lcdc_0: endpoint@0 { + remote-endpoint = <&hdmi_0>; + }; + }; }; -/ { - hdmi { - compatible = "ti,tilcdc,slave"; - i2c = <&i2c0>; +&i2c0 { + tda19988 { + compatible = "nxp,tda998x"; + reg = <0x70>; pinctrl-names = "default", "off"; pinctrl-0 = <&nxp_hdmi_bonelt_pins>; pinctrl-1 = <&nxp_hdmi_bonelt_off_pins>; - status = "okay"; + + port { + hdmi_0: endpoint@0 { + remote-endpoint = <&lcdc_0>; + }; + }; }; };
Use new binding for the external tda19988 HDMI encoder. Signed-off-by: Jyri Sarha <jsarha@ti.com> --- arch/arm/boot/dts/am335x-boneblack.dts | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-)