Hi, On 20-Dec-24 4:15 PM, Bryan O'Donoghue wrote: > On 20/12/2024 14:41, Hans de Goede wrote: >> Hi All, >> >> Here is v2 of my ov08x40 series with various ov08x40 improvements >> aimed at getting the ov08x40 driver to work on x86_64/ACPI platforms. >> >> Changes in v2: >> - Fix the brownpaper bag bug swapping the suspend/resume arguments >> to the DEFINE_RUNTIME_DEV_PM_OPS() macro. Many thanks to Bryan for >> catching this >> - Add Bryan's Tested-by to patches 1-8 >> - Add 2 logging improvement patches which I posted as follow up >> patches to v1, so these now can run through CI too >> >> Here is v1 of the series: >> https://lore.kernel.org/linux-media/20241219134940.15472-1-hdegoede@xxxxxxxxxx/ >> >> Regards, >> >> Hans > > How would you feel about picking up a Fixes: appended to the front of this series ? > > I noticed the reset pin logic and timings were out of spec when I was testing out v1. > > https://git.codelinaro.org/bryan.odonoghue/kernel/-/commits/arm-laptop/wip/x1e80100-6.13-rc2+camss?ref_type=heads > > If this commit goes first, then it can go to -stable: > > https://git.codelinaro.org/bryan.odonoghue/kernel/-/commit/f10539a3cbe5c072829b082683f0d1393a7e50dc I'm fine with merging this one before this series, AFAICT it should not cause any conflicts. Note Patch 1/10 of this series also has a Fixes tag. Note last work email from me before the Christmas break, see you all January 6th. Regard, Hans