2023-08-04 21:41:55

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH 6/6] media: venus: core: Add SC8280XP resource struct

On 4.08.2023 23:10, Bryan O'Donoghue wrote:
> On 04/08/2023 21:09, Konrad Dybcio wrote:
>> +    .freq_tbl = sc8280xp_freq_table,
>> +    .freq_tbl_size = ARRAY_SIZE(sc8280xp_freq_table),
>
> Would it not be more legitimate and logical to have 8350 use 8280xp's frequency table, instead of 8250 ?
top freq is higher on 8280

Konrad


2023-08-04 22:41:45

by Bryan O'Donoghue

[permalink] [raw]
Subject: Re: [PATCH 6/6] media: venus: core: Add SC8280XP resource struct

On 04/08/2023 22:10, Konrad Dybcio wrote:
>> Would it not be more legitimate and logical to have 8350 use 8280xp's frequency table, instead of 8250 ?
> top freq is higher on 8280

Still though its a bit suspicious 8350 doesn't have its own table.

Are we missing the downstream reference ?

---
bod