[PATCH] arm64: dts: rockchip: Add cif test clocks for rk3399

Brian Norris briannorris at chromium.org
Mon Jan 8 17:56:13 PST 2018


On Tue, Jan 09, 2018 at 09:45:05AM +0800, Shunqian Zheng wrote:
> On 2018年01月09日 03:17, Brian Norris wrote:
> >On Mon, Jan 08, 2018 at 01:48:51PM +0800, Shunqian Zheng wrote:
> >>+			test_clkout2: test-clkout2 {
> >>+				rockchip,pins =
> >>+					<0 8 RK_FUNC_3 &pcfg_pull_none>;
> >Your function indexing is a little confusing to me, but one or more of
> >your datasheet, TRM, or patch are incorrect here. The datasheet says
> >"Func 3" (which is 1-indexed, so really means RK_FUNC_2) should be
> >TEST_CLKOUT2, but your TRM agrees with the patch, saying
> >2'b11=test_clkout2. So I think your patch is correct, but the datasheet
> >needs updated?
> Yeah, please follow the TRM.

Can you make sure the datasheet gets updated?

Thanks,
Brian



More information about the Linux-rockchip mailing list