Hi, Shu-hsiang: On Wed, 2024-10-09 at 19:15 +0800, Shu-hsiang Yang wrote: > Introduces state management and debugging mechanisms for the MediaTek > ISP7.x camsys platform. State management establishes control over ISP > operations and events, defining distinct states for request handling, > sensor control, and frame synchronization, ensuring event processing. > The debugging mechanism ensures stable operation and timely data > collection during anomalies. > > Signed-off-by: Shu-hsiang Yang <Shu-hsiang.Yang@xxxxxxxxxxxx> > --- [snip] > + > +enum MTK_CAMSYS_ENGINE_TYPE { > + CAMSYS_ENGINE_RAW, > + CAMSYS_ENGINE_MRAW, > + CAMSYS_ENGINE_CAMSV, > + CAMSYS_ENGINE_SENINF, Only CAMSYS_ENGINE_RAW is used, so it's not necessary to define MTK_CAMSYS_ENGINE_TYPE because only one type. CAMSYS_ENGINE_RAW could also be dropped because only this type so it's not necessary to define it. Regards, CK > +}; > +