Hi, Le mardi 03 novembre 2026 à 02:55 +0200, Igor Velkov a écrit : > On Mon, Nov 02, 2026, Laurent Pinchart wrote: > > > ARCH_ROCKCHIP and arm64 ARCH_MXC select PM, > > > > Why do we need this patch then ? > > Only COMPILE_TEST builds reach PM=n, and there the driver still binds and > probe reads RKISP1_CIF_VI_ID with the clocks off. Nicolas asked to close > the PM=n case per driver [1]; rkcif already depends on PM. If you'd rather > leave COMPILE_TEST-only configs alone, I'll drop it. > > [1] https://lore.kernel.org/r/8aafdbbf04a0d939c6b4d7d5577d97bb58d1c439.camel@collabora.com The rkvdec coded was using macros capable of supporting both, so it was logical to make that functional, though if non of the Rockchip ARCH (32bit and 64bit) can be built without PM, then indeed we can drop. Nicolas