Re: [PATCH] ARM: dts: rockchip: add display nodes for rk322x

[Justin Swartz]

Hi Heiko,

On 2019-06-14 11:15, Heiko Stuebner wrote:

> Hi Justin,
>
> Am Donnerstag, 13. Juni 2019, 12:13:04 CEST schrieb Justin Swartz:
>
> > Add display_subsystem, hdmi_phy, vop, and hdmi device nodes plus
> > a few hdmi pinctrl entries to allow for HDMI output.
> >
> > Signed-off-by: Justin Swartz <justin.swartz@xxxxxxxxxxxxxxxx>
>
> Overall looks good, but in combination with the clock-patch you posted,
> I'd really prefer if we could try a slightly different approach.
>
> Hard register-level settings in the clock driver look bad and tend to
> cause problems later on, so I've adapted things a bit in [0] (untested)
> and would be glad if you could give it a try on actual hardware.

I can confirm that your approach works properly here.

> The hdmiphy itself is a clock-provider for its pll and therefore the
> assigned-clock* properties into the hdmi controller, as the phy needs
> to probe before trying to set clocks.
> But in theory this should achieve the same result of reparenting the
> system's hdmiphy clock to the actual output of the phy-pll.
>
> I've also moved the iommu-cells fix to a separate commit.
>
> Please test, thanks
> Heiko
>
> [0] https://github.com/mmind/linux-rockchip/commits/wip/rk3229-hdmi

Thanks
Justin