From: Andy Shevchenko <andy@kernel.org>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Tsuchiya Yuto <kitakar@gmail.com>,
Yury Luneff <yury.lunev@gmail.com>,
Nable <nable.maininbox@googlemail.com>,
andrey.i.trufanov@gmail.com, Fabio Aiuto <fabioaiuto83@gmail.com>,
linux-media@vger.kernel.org, linux-staging@lists.linux.dev
Subject: Re: [PATCH 00/20] media: atomisp: Misc. cleanups / fixes
Date: Mon, 21 Nov 2022 11:20:45 +0200 [thread overview]
Message-ID: <Y3tC7Vme5dpGrO+A@smile.fi.intel.com> (raw)
In-Reply-To: <20221120224101.746199-1-hdegoede@redhat.com>
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>
> 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
>
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2022-11-21 9:21 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-20 22:40 [PATCH 00/20] media: atomisp: Misc. cleanups / fixes Hans de Goede
2022-11-20 22:40 ` [PATCH 01/20] media: atomisp: Silence: 'atomisp_q_one_s3a_buffer: drop one s3a stat which has exp_id xx' log messages Hans de Goede
2022-11-21 8:54 ` Andy Shevchenko
2022-11-21 12:27 ` Hans de Goede
2022-11-20 22:40 ` [PATCH 02/20] media: atomisp: Remove accelerator pipe creation code Hans de Goede
2022-11-20 22:40 ` [PATCH 03/20] media: atomisp: Remove unused QOS defines / structure member Hans de Goede
2022-11-20 22:40 ` [PATCH 04/20] media: atomisp: Flush queue on atomisp_css_start() error Hans de Goede
2022-11-20 22:40 ` [PATCH 05/20] media: atomisp: Log an error on failing to alloc private-mem Hans de Goede
2022-11-21 8:58 ` Andy Shevchenko
2022-11-21 12:29 ` Hans de Goede
2022-11-20 22:40 ` [PATCH 06/20] media: atomisp: Fix deadlock when the /dev/video# node is closed while still streaming Hans de Goede
2022-11-20 22:40 ` [PATCH 07/20] media: atomisp: Remove 2 unused accelerator mode related functions Hans de Goede
2022-11-20 22:40 ` [PATCH 08/20] media: atomisp: Remove atomisp_css_yuvpp_configure_viewfinder() function Hans de Goede
2022-11-20 22:40 ` [PATCH 09/20] media: atomisp: Remove unused ia_css_frame_*() functions Hans de Goede
2022-11-20 22:40 ` [PATCH 10/20] media: atomisp: Drop userptr support from hmm Hans de Goede
2022-11-21 9:01 ` Andy Shevchenko
2022-11-21 12:31 ` Hans de Goede
2022-11-20 22:40 ` [PATCH 11/20] media: atomisp: Remove double atomisp_mrfld_power_down()/_up() calls from atomisp_reset() Hans de Goede
2022-11-20 22:40 ` [PATCH 12/20] media: atomisp: Remove atomisp_mrfld_power_down()/_up() Hans de Goede
2022-11-20 22:40 ` [PATCH 13/20] media: atomisp: Remove clearing of config from atomisp_css_uninit() Hans de Goede
2022-11-20 22:40 ` [PATCH 14/20] media: atomisp: Remove atomisp_css_suspend()/_resume() Hans de Goede
2022-11-20 22:40 ` [PATCH 15/20] media: atomisp: Remove sw_contex.power_state checks Hans de Goede
2022-11-21 9:09 ` Andy Shevchenko
2022-11-21 9:16 ` Andy Shevchenko
2022-11-20 22:40 ` [PATCH 16/20] media: atomisp: Remove duplication between runtime-pm and normal-pm code Hans de Goede
2022-11-20 22:40 ` [PATCH 17/20] media: atomisp: Move calling of css_[un]init() to power_on()/_off() Hans de Goede
2022-11-21 9:14 ` Andy Shevchenko
2022-11-21 12:33 ` Hans de Goede
2022-11-20 22:40 ` [PATCH 18/20] media: atomisp: Remove atomisp_ospm_dphy_down() call from probe error path Hans de Goede
2022-11-20 22:41 ` [PATCH 19/20] media: atomisp: Remove atomisp_ospm_dphy_up()/_down() functions Hans de Goede
2022-11-20 22:41 ` [PATCH 20/20] media: atomisp_ov2680: Fix 1280x720 -> 1296x736 resolution Hans de Goede
2022-11-21 9:20 ` Andy Shevchenko [this message]
2022-11-21 12:40 ` [PATCH 00/20] media: atomisp: Misc. cleanups / fixes Hans de Goede
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y3tC7Vme5dpGrO+A@smile.fi.intel.com \
--to=andy@kernel.org \
--cc=andrey.i.trufanov@gmail.com \
--cc=fabioaiuto83@gmail.com \
--cc=hdegoede@redhat.com \
--cc=kitakar@gmail.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=mchehab@kernel.org \
--cc=nable.maininbox@googlemail.com \
--cc=sakari.ailus@linux.intel.com \
--cc=yury.lunev@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.