Re: [PATCH v2 0/5] media: iris: add support for purwa platform

[Wangao Wang]

On 2026/3/7 21:12, Krzysztof Kozlowski wrote:
> On Fri, Mar 06, 2026 at 04:44:28PM +0800, Wangao Wang wrote:
> > This series enables the Iris video codec on purwa, allowing purwa to
> > use hardware‑accelerated video encoding and decoding.
> >
> > The Iris codec on purwa is nearly identical to the one on hamoa(X1E),
> > except that it requires one additional clock and uses a different OPP
> > table.
> >
> > Therefore, purwa can reuse the Iris node from hamoa, but the clocks
> > and OPP table need to be redefined.
> >
> > Dependencies:
> > https://lore.kernel.org/all/20260202-purwa-v5-0-1f5a93578802@xxxxxxxxxxxxxxxx/
> > https://lore.kernel.org/all/20260304-purwa-videocc-camcc-v2-0-dbbd2d258bd6@xxxxxxxxxxxxxxxx/
>
> I don't understand why you coupled them and added these as dependencies.
> This only makes it difficult to test and technically your media patches
> cannot be applied.
>
> And I don't even see what is needed from these patchsets for the media
> bits.
>
> Best regards,
> Krzysztof

Thanks for the feedback.

You are right — coupling these patchsets as dependencies was not 
necessary and indeed makes the series harder to test and apply. The 
media/iris changes should be self-contained and not depend on the Purwa 
base DT/videocc series.

The dependency note was added mainly for bring-up convenience. However, 
that should not be expressed as a hard dependency for the media patches.

Should I send the DT patches separately from the driver patches?

--
Best Regards,
Wangao