Hi Hans, On Thu, Jan 09, 2025 at 10:23:41PM +0100, Hans de Goede wrote: > Hi, > > On 9-Jan-25 5:37 PM, Sakari Ailus wrote: > > Hi Hans, > > > > On Fri, Dec 20, 2024 at 03:41:28PM +0100, Hans de Goede wrote: > >> The driver might skip the ov08x40_identify_module() on probe() based on > >> the acpi_dev_state_d0() check done in probe(). > >> > >> If the ov08x40_identify_module() call is skipped on probe() it should > >> be done on the first stream start. Add the missing call. > >> > >> Note ov08x40_identify_module() will only do something on its first call, > >> subsequent calls are no-ops. > >> > >> Tested-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> > >> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > > > > I'll add: > > > > Fixes: b1a42fde6e07 ("media: ov08x40: Avoid sensor probing in D0 state") > > Cc: stable@xxxxxxxxxxxxxxx > > Sounds good, thank you. > > That is when you say "I'll add", you mean you'll add those tags > while merging this series, right ? > > Or do you want me to the tags in a v3 series? No need for v3. Thanks! Are there other patches not merged for the driver, besides this set? No? -- Regards, Sakari Ailus