Re: [PATCH v4 3/9] media: qcom: venus: Add msm8939 resource struct

[Bryan O'Donoghue]

On 05/05/2026 22:44, Erikas Bitovtas wrote:
> +	.vcodec_clks = { "core0", "core1" },
> +	.vcodec_clks_num = 2,

This smells a bit dodgy to me.

You already have

struct clk *vcodec0_clks[VIDC_VCODEC_CLKS_NUM_MAX];
struct clk *vcodec1_clks[VIDC_VCODEC_CLKS_NUM_MAX];

so why add vcodec_clks and put core0 and core1 into a new array. 
vcodec0_clks and vcodec1_clks seem like a very natural place for core0 
and core1 clocks to live ?

---
bod