On Mon, Feb 19, 2018 at 2:46 PM, Heiko Stuebner <heiko@xxxxxxxxx> wrote: > Hi Rob, > > Am Montag, 19. Februar 2018, 21:28:53 CET schrieb Rob Herring: >> On Fri, Feb 16, 2018 at 09:41:58PM +0100, Heiko Stuebner wrote: >> > The rk3328 uses an external hdmi phy from Innosilicon which uses >> > the generic phy framework for access. Add the necessary data >> > and the compatible for it. >> > >> > Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx> >> > --- >> > .../bindings/display/rockchip/dw_hdmi-rockchip.txt | 1 + >> > drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 106 +++++++++++++++++++++ >> > 2 files changed, 107 insertions(+) >> > >> > diff --git a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> > index 937bfb472e1d..39143424a474 100644 >> > --- a/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> > +++ b/Documentation/devicetree/bindings/display/rockchip/dw_hdmi-rockchip.txt >> > @@ -13,6 +13,7 @@ Required properties: >> > >> > - compatible: should be one of the following: >> > "rockchip,rk3288-dw-hdmi" >> > + "rockchip,rk3328-dw-hdmi" >> >> Is Designware (Synopsys) now Innosilicon? > > Nope :-) . > > The rk3328 just combines the Designware dw-hdmi with an (external) > hdmi-phy from Innosilicon, where the hdmi-phy otherwise was was > integrated into the hdmi ip-block. Okay, the commit msg is misleading because it talks about the phy and adding the "compatible for it". Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html