1;4601;0c
On Fri, May 05, 2017 at 08:34:16PM +0800, Icenowy Zheng wrote:

于 2017年5月5日 GMT+08:00 下午8:30:35, Maxime Ripard [off-list ref] 写到:

quoted

On Fri, May 05, 2017 at 04:53:43PM +0800, icenowy-h8G6r0blFSE@public.gmane.org wrote:

quoted

quoted

quoted

+               de2_clocks: clock@1000000 {
+                       compatible =

"allwinner,sun50i-h5-de2-clk";

quoted

quoted

I am a bit skeptical about this. Since the V3S only has one mixer,

do

quoted

quoted

the clocks
for the second one even exist?

It's described in the de_clock.c in the BSP source code, and in

hardware

quoted

these bits can be really set (although without clock output).

So I use this compatible which has still the extra clocks.

If it's not usable, then it shouldn't be in the code, it's basically
dead code.

Thus should we have one more DE2 CCU compatible without mixer1
clocks for V3s?

If those clocks don't exist on v3s, then yes.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com