Re: [PATCH 00/15] media: atomisp: NULL pointer deref + missing firmware fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Dec 31, 2023 at 12:31 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
>
> Hi All,
>
> Here is a set of atomisp fixes I have been working on:
>
> - The first commit in this series fixes the driver no longer
>   working due to v4l2_subdev_state handling in media_stage/master
>
> - Gracefully handle missing firmware:
>   - Fix the driver crashing the system when the firmware is missing
>   - Make the driver bind in power-managent-only mode when the firmware
>     is missing so that the ISP is still properly turned off. This
>     requires custom handling, not just standard PCI power-management.
>     This allows the system to properly enter S0i3 with missing ISP
>     firmware, allowing the atomisp driver to be used as a replacement
>     for the pm-only drivers/platform/x86/atomisp/pm.c driver.
>
> I am going to send out a pull-request to Mauro for this almost directly
> after sending these patches to the list, leaving pretty much zero time
> for review. Sorry about that, with the holidays the timing did not work
> out very well.
>
> If review finds any significant issues I'll do a set of follow up
> patches to fix those.

One which would be nice to fixup is based on the sysfs patch.
Another, but not so hurried, is to have cleanup.h management for v4l2 calls.

Side note: Since we have many headers to be split from kernel.h
(array_size.h, container_of.h, hex.h, kstrtox.h, sprintf.h) consider
replacing "proxy" headers in the code. Especially when headers in the
driver include kernel.h for peanuts.

-- 
With Best Regards,
Andy Shevchenko





[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux