[GIT FIXE FOR 6.17] Drop redundant pm_runtime_mark_last_busy() and lt6911uxe fix

[GIT FIXE FOR 6.17] Drop redundant pm_runtime_mark_last_busy() and lt6911uxe fix

[Sakari Ailus]

Hi Hans, Mauro,

Here are the patches to drop the redundant pm_runtime_mark_last_busy()
calls in drivers and an lt6911uxe fix that should also be merged to 6.16.

Please pull.


The following changes since commit 8f5ae30d69d7543eee0d70083daf4de8fe15d585:

  Linux 6.17-rc1 (2025-08-10 19:41:16 +0300)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/fixes-6.17-1-signed

for you to fetch changes up to 7947309bafb07bc1b2eb63c69aae13003b53dad1:

  media: Remove redundant pm_runtime_mark_last_busy() calls (2025-08-11 15:43:44 +0300)

----------------------------------------------------------------
V4L2 fixes for 6.17

----------------------------------------------------------------
Dongcheng Yan (1):
      platform/x86: int3472: add hpd pin support

Sakari Ailus (1):
      media: Remove redundant pm_runtime_mark_last_busy() calls

 drivers/media/i2c/alvium-csi2.c                          | 1 -
 drivers/media/i2c/ccs/ccs-core.c                         | 7 +------
 drivers/media/i2c/dw9768.c                               | 1 -
 drivers/media/i2c/gc0308.c                               | 3 ---
 drivers/media/i2c/gc2145.c                               | 3 ---
 drivers/media/i2c/imx219.c                               | 2 --
 drivers/media/i2c/imx283.c                               | 3 ---
 drivers/media/i2c/imx290.c                               | 3 ---
 drivers/media/i2c/imx296.c                               | 1 -
 drivers/media/i2c/imx415.c                               | 1 -
 drivers/media/i2c/mt9m114.c                              | 6 ------
 drivers/media/i2c/ov4689.c                               | 3 ---
 drivers/media/i2c/ov5640.c                               | 4 ----
 drivers/media/i2c/ov5645.c                               | 3 ---
 drivers/media/i2c/ov64a40.c                              | 7 +------
 drivers/media/i2c/ov8858.c                               | 2 --
 drivers/media/i2c/st-mipid02.c                           | 2 --
 drivers/media/i2c/tc358746.c                             | 5 -----
 drivers/media/i2c/thp7312.c                              | 4 ----
 drivers/media/i2c/vd55g1.c                               | 4 ----
 drivers/media/i2c/vd56g3.c                               | 4 ----
 drivers/media/i2c/video-i2c.c                            | 4 ----
 drivers/media/platform/chips-media/wave5/wave5-vpu-dec.c | 4 ----
 drivers/media/platform/chips-media/wave5/wave5-vpu-enc.c | 5 -----
 drivers/media/platform/nvidia/tegra-vde/h264.c           | 2 --
 drivers/media/platform/qcom/iris/iris_hfi_queue.c        | 1 -
 drivers/media/platform/raspberrypi/pisp_be/pisp_be.c     | 2 --
 drivers/media/platform/verisilicon/hantro_drv.c          | 1 -
 drivers/media/rc/gpio-ir-recv.c                          | 4 +---
 drivers/platform/x86/intel/int3472/discrete.c            | 6 ++++++
 include/linux/platform_data/x86/int3472.h                | 1 +
 31 files changed, 10 insertions(+), 89 deletions(-)

-- 
Kind regards,

Sakari Ailus

Re: [GIT FIXE FOR 6.17] Drop redundant pm_runtime_mark_last_busy() and lt6911uxe fix

[Mauro Carvalho Chehab]

Em Wed, 13 Aug 2025 09:04:23 +0000
Sakari Ailus <sakari.ailus@iki.fi> escreveu:

> Hi Hans, Mauro,
> 
> Here are the patches to drop the redundant pm_runtime_mark_last_busy()
> calls in drivers and an lt6911uxe fix that should also be merged to 6.16.

Please fix your description. The above is not clear and doesn't tell
the hole history.

In this specific case, I would expect the description to be
similar to this one:


	Hi Hans, Mauro,

	Please pull for:

	  - drop the redundant pm_runtime_mark_last_busy() on rkvdec driver;
	  - fix an error handling in probe at rkvdec driver;
	  - fix an issue affecting lt6911uxe/lt6911uxc related to CSI-2
	    GPIO pins at int3472.

	The fix for lt6911uxe/lt6911uxc is against drivers/platform/x86/intel,
	and contains an A-B from one of X86 PLATFORM DRIVERS maintainers.

We need to have those at the merge requests as well, as we're using them
to generate PR summaries.

Regards,
Mauro

Re: [GIT FIXE FOR 6.17] Drop redundant pm_runtime_mark_last_busy() and lt6911uxe fix

[Sakari Ailus]

Hi Mauro,

On Wed, Aug 27, 2025 at 11:05:28AM +0200, Mauro Carvalho Chehab wrote:
> Em Wed, 13 Aug 2025 09:04:23 +0000
> Sakari Ailus <sakari.ailus@iki.fi> escreveu:
> 
> > Hi Hans, Mauro,
> > 
> > Here are the patches to drop the redundant pm_runtime_mark_last_busy()
> > calls in drivers and an lt6911uxe fix that should also be merged to 6.16.
> 
> Please fix your description. The above is not clear and doesn't tell
> the hole history.
> 
> In this specific case, I would expect the description to be
> similar to this one:
> 
> 
> 	Hi Hans, Mauro,
> 
> 	Please pull for:
> 
> 	  - drop the redundant pm_runtime_mark_last_busy() on rkvdec driver;
> 	  - fix an error handling in probe at rkvdec driver;
> 	  - fix an issue affecting lt6911uxe/lt6911uxc related to CSI-2
> 	    GPIO pins at int3472.
> 
> 	The fix for lt6911uxe/lt6911uxc is against drivers/platform/x86/intel,
> 	and contains an A-B from one of X86 PLATFORM DRIVERS maintainers.
> 
> We need to have those at the merge requests as well, as we're using them
> to generate PR summaries.

That sounds entirely reasonable. Still, isn't this kind of more detailed
information already available on the patch subject lines? Could those be
used instead? The message in the PR usually describes the patches with less
detail.

-- 
Kind regards,

Sakari Ailus

Re: [GIT FIXE FOR 6.17] Drop redundant pm_runtime_mark_last_busy() and lt6911uxe fix

[Mauro Carvalho Chehab]

Em Wed, 27 Aug 2025 13:14:18 +0300
Sakari Ailus <sakari.ailus@iki.fi> escreveu:

> Hi Mauro,
> 
> On Wed, Aug 27, 2025 at 11:05:28AM +0200, Mauro Carvalho Chehab wrote:
> > Em Wed, 13 Aug 2025 09:04:23 +0000
> > Sakari Ailus <sakari.ailus@iki.fi> escreveu:
> >   
> > > Hi Hans, Mauro,
> > > 
> > > Here are the patches to drop the redundant pm_runtime_mark_last_busy()
> > > calls in drivers and an lt6911uxe fix that should also be merged to 6.16.  
> > 
> > Please fix your description. The above is not clear and doesn't tell
> > the hole history.
> > 
> > In this specific case, I would expect the description to be
> > similar to this one:
> > 
> > 
> > 	Hi Hans, Mauro,
> > 
> > 	Please pull for:
> > 
> > 	  - drop the redundant pm_runtime_mark_last_busy() on rkvdec driver;
> > 	  - fix an error handling in probe at rkvdec driver;
> > 	  - fix an issue affecting lt6911uxe/lt6911uxc related to CSI-2
> > 	    GPIO pins at int3472.
> > 
> > 	The fix for lt6911uxe/lt6911uxc is against drivers/platform/x86/intel,
> > 	and contains an A-B from one of X86 PLATFORM DRIVERS maintainers.
> > 
> > We need to have those at the merge requests as well, as we're using them
> > to generate PR summaries.  
> 
> That sounds entirely reasonable. Still, isn't this kind of more detailed
> information already available on the patch subject lines? Could those be
> used instead? The message in the PR usually describes the patches with less
> detail.

It is available, but:

1. a short description helps reviewing the patchset;
2. If anyone uses the affected devices and have those bugs, the PR 
   summary is the only e-mail he needs to check before deciding to
   pick or not the series;
3. it is also used when maintainers send upstream PRs;
4. the upstream PR summary can also be used by distro packagers to
   detect if a sent patch fixes or not existing open bugs.

By not doing it, it means that people that could be using the hardware
will have more problems to get patches to addres their issues. 

It also means that maintainers need to open all the patches in
the series before sending a patches upstream to generate the PR description,
which doesn't scale, specially on a multi-committers scenario.

On this particular case, there was not much efforts, as this was a 
small series, but when sending a PR with hundreds of patches, not 
having proper descriptions is a nightmare for maintainers.

Thanks,
Mauro