|
From: Andy S. <and...@gm...> - 2025-09-20 10:49:41
|
On Fri, Sep 19, 2025 at 12:57 PM Jai Luthra <jai...@id...> wrote: > > Hi, > > This patch series introduces state management for video devices. > > Currently, V4L2 subdevices have a well-established state management > system [1] that allows the framework to store the subdevice state > at a central location. > > Video devices however lack this, leading to inconsistent state handling > across drivers and making it difficult to implement features like > hardware multiplexing in complex media pipelines [2]. > > The series is composed of three parts: > > - Core Infrastructure (PATCH 1-4) > Introduces the basic state structure, try state support, > initialization callbacks, and driver helpers > - Framework Integration (PATCH 5-7) > Integrates state passing through the ioctl wrappers and driver > implementations > - Driver Examples (PATCH 8-10) > Use the state to store formats in TI J721E CSI2RX and Rockchip > RKISP1 drivers > > This should also provide a foundation for drivers to extend or subclass > the state structure with device-specific variables in future. > > I have tested capture and format negotiation with the converted drivers > on SK AM62A (j721e-csi2rx) and Debix Model A (rkisp1). > > [1]: https://lore.kernel.org/linux-media/202...@id.../ > [2]: https://lore.kernel.org/linux-media/202...@id.../ When you add a Cc list., do it after the '---' (three minus signs) line, otherwise that huge unneeded noise will become part of the commit message. -- With Best Regards, Andy Shevchenko |