Re: [PATCH 00/20] media: atomisp: Misc. cleanups / fixes

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 11/21/22 10:20, Andy Shevchenko wrote:
> On Sun, Nov 20, 2022 at 11:40:41PM +0100, Hans de Goede wrote:
>> Hi All,
>>
>> Here is another round of atomisp cleanups / fixes + preparation work
>> for getting rid of the ugly PCI power-management errors in dmesg
>> when doing (runtime)suspend/resume.
> 
> As usual, for non-commented (by me) patches:
> Reviewed-by: Andy Shevchenko <andy@kernel.org>

Thank you for all the reviews!

The patches with your suggested fixes + Reviewed-by have all
been updated in my media-atomisp branch.

Regards,

Hans

>> Hans de Goede (20):
>>   media: atomisp: Silence: 'atomisp_q_one_s3a_buffer: drop one s3a stat
>>     which has exp_id xx' log messages
>>   media: atomisp: Remove accelerator pipe creation code
>>   media: atomisp: Remove unused QOS defines / structure member
>>   media: atomisp: Flush queue on atomisp_css_start() error
>>   media: atomisp: Log an error on failing to alloc private-mem
>>   media: atomisp: Fix deadlock when the /dev/video# node is closed while
>>     still streaming
>>   media: atomisp: Remove 2 unused accelerator mode related functions
>>   media: atomisp: Remove atomisp_css_yuvpp_configure_viewfinder()
>>     function
>>   media: atomisp: Remove unused ia_css_frame_*() functions
>>   media: atomisp: Drop userptr support from hmm
>>   media: atomisp: Remove double atomisp_mrfld_power_down()/_up() calls
>>     from atomisp_reset()
>>   media: atomisp: Remove atomisp_mrfld_power_down()/_up()
>>   media: atomisp: Remove clearing of config from atomisp_css_uninit()
>>   media: atomisp: Remove atomisp_css_suspend()/_resume()
>>   media: atomisp: Remove sw_contex.power_state checks
>>   media: atomisp: Remove duplication between runtime-pm and normal-pm
>>     code
>>   media: atomisp: Move calling of css_[un]init() to power_on()/_off()
>>   media: atomisp: Remove atomisp_ospm_dphy_down() call from probe error
>>     path
>>   media: atomisp: Remove atomisp_ospm_dphy_up()/_down() functions
>>   media: atomisp_ov2680: Fix 1280x720 -> 1296x736 resolution
>>
>>  drivers/staging/media/atomisp/i2c/ov2680.h    |  46 +-
>>  .../staging/media/atomisp/include/hmm/hmm.h   |   1 -
>>  .../media/atomisp/include/hmm/hmm_bo.h        |   2 -
>>  .../staging/media/atomisp/pci/atomisp_cmd.c   |  76 +--
>>  .../staging/media/atomisp/pci/atomisp_cmd.h   |   8 +-
>>  .../media/atomisp/pci/atomisp_compat.h        |  11 -
>>  .../media/atomisp/pci/atomisp_compat_css20.c  |  88 +--
>>  .../staging/media/atomisp/pci/atomisp_fops.c  |  34 +-
>>  .../media/atomisp/pci/atomisp_internal.h      |   1 -
>>  .../staging/media/atomisp/pci/atomisp_ioctl.c |  80 ++-
>>  .../staging/media/atomisp/pci/atomisp_v4l2.c  | 104 +--
>>  .../pipe/interface/ia_css_pipe_stagedesc.h    |   5 -
>>  .../pci/camera/pipe/src/pipe_stagedesc.c      |  21 -
>>  drivers/staging/media/atomisp/pci/hmm/hmm.c   |  19 +-
>>  .../staging/media/atomisp/pci/hmm/hmm_bo.c    |  54 +-
>>  .../media/atomisp/pci/ia_css_frame_public.h   |  74 ---
>>  .../staging/media/atomisp/pci/ia_css_pipe.h   |   3 +-
>>  .../media/atomisp/pci/ia_css_pipe_public.h    |  69 --
>>  .../pci/runtime/debug/src/ia_css_debug.c      |   7 -
>>  .../atomisp/pci/runtime/frame/src/frame.c     | 121 ----
>>  .../pipeline/interface/ia_css_pipeline.h      |   2 -
>>  .../pci/runtime/pipeline/src/pipeline.c       |   8 -
>>  drivers/staging/media/atomisp/pci/sh_css.c    | 608 +-----------------
>>  .../media/atomisp/pci/sh_css_internal.h       |  13 -
>>  .../staging/media/atomisp/pci/sh_css_legacy.h |   1 -
>>  drivers/staging/media/atomisp/pci/sh_css_sp.c |   2 +-
>>  26 files changed, 121 insertions(+), 1337 deletions(-)
>>
>> -- 
>> 2.38.1
>>
>