All of lore.kernel.org
 help / color / mirror / Atom feed
* [GIT PULL FOR 5.17] V4L2 patches
@ 2021-12-13 13:49 Sakari Ailus
  2021-12-13 14:27 ` [GIT PULL FOR 5.17] V4L2 patches (#79352) Jenkins
  2021-12-14 10:21 ` [GIT PULL FOR 5.17] V4L2 patches Mauro Carvalho Chehab
  0 siblings, 2 replies; 6+ messages in thread
From: Sakari Ailus @ 2021-12-13 13:49 UTC (permalink / raw)
  To: linux-media

Hi Mauro,

Here's a few patches for 5.17. In particular, included are isl79987 driver
from Michale Tretter and Marek Vasut, and a dependency fix for the recently
added max96712 driver.

Please pull.


The following changes since commit 4383cfa18c5bbc5b9b6a9e77adc12aec1c20b72d:

  Merge tag 'v5.16-rc4' into media_tree (2021-12-07 11:29:41 +0100)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.17-2-signed

for you to fetch changes up to 0541ae639fb1165fb309ebf58605f901146c2535:

  media: i2c: isl7998x: Add driver for Intersil ISL7998x (2021-12-08 09:51:54 +0200)

----------------------------------------------------------------
V4L2 patches for 5.17

----------------------------------------------------------------
Andy Shevchenko (1):
      media: i2c: max9286: Get rid of duplicate of_node assignment

Marek Vasut (2):
      media: dt-bindings: Add Intersil ISL79987 DT bindings
      media: i2c: isl7998x: Add driver for Intersil ISL7998x

Michael Tretter (1):
      media: imx6-mipi-csi2: use pre_streamon callback to set sensor into LP11

Sakari Ailus (1):
      max96712: Depend on VIDEO_V4L2

 .../bindings/media/i2c/isil,isl79987.yaml          |  113 ++
 MAINTAINERS                                        |    8 +
 drivers/media/i2c/Kconfig                          |   10 +
 drivers/media/i2c/Makefile                         |    1 +
 drivers/media/i2c/isl7998x.c                       | 1543 ++++++++++++++++++++
 drivers/media/i2c/max9286.c                        |    1 -
 drivers/staging/media/imx/imx6-mipi-csi2.c         |    9 +-
 drivers/staging/media/max96712/Kconfig             |    1 +
 include/uapi/linux/v4l2-controls.h                 |    6 +
 9 files changed, 1690 insertions(+), 2 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/isil,isl79987.yaml
 create mode 100644 drivers/media/i2c/isl7998x.c

-- 
Sakari Ailus

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL FOR 5.17] V4L2 patches (#79352)
  2021-12-13 13:49 [GIT PULL FOR 5.17] V4L2 patches Sakari Ailus
@ 2021-12-13 14:27 ` Jenkins
  2021-12-14 10:21 ` [GIT PULL FOR 5.17] V4L2 patches Mauro Carvalho Chehab
  1 sibling, 0 replies; 6+ messages in thread
From: Jenkins @ 2021-12-13 14:27 UTC (permalink / raw)
  To: mchehab+samsung, linux-media; +Cc: builder

From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YbdPdh/ivnjZKIOA@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/165075/
Build time: 00:35:23
Link: https://lore.kernel.org/linux-media/YbdPdh/ivnjZKIOA@valkosipuli.retiisi.eu

gpg: Signature made Mon 13 Dec 2021 12:05:48 PM UTC
gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Good signature from "Sakari Ailus <sakari.ailus@linux.intel.com>" [unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: F0D0 377A 0D4F 25A7 9238  EFE5 6D40 361B 6E28 C193
     Subkey fingerprint: 53AC 58A5 F594 8636 C04A  1BF8 141D FA54 A1EC 8DEA

Summary: got 3/5 patches with issues, being 2 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-max96712-Depend-on-VIDEO_V4L2.patch:

    allyesconfig: return code #0:
	../scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr]
	../scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr]
	../scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples

    allyesconfig: return code #0:
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	SPARSE:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: warning: array of flexible structures
	../drivers/media/rc/meson-ir-tx.c:22: warning: expecting prototype for meson(). Prototype was for DEVICE_NAME() instead
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2625 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1981 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:658 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 654)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2874 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0004-media-dt-bindings-Add-Intersil-ISL79987-DT-bindings.patch:

   checkpatch.pl:
	$ cat patches/0004-media-dt-bindings-Add-Intersil-ISL79987-DT-bindings.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:23: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0005-media-i2c-isl7998x-Add-driver-for-Intersil-ISL7998x.patch:

    allyesconfig: return code #0:
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2625 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:658 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 654)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2864 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

   checkpatch.pl:
	$ cat patches/0005-media-i2c-isl7998x-Add-driver-for-Intersil-ISL7998x.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:59: WARNING: please write a paragraph that describes the config symbol fully
	-:879: CHECK: Alignment should match open parenthesis
	-:1165: ERROR: Macros with complex values should be enclosed in parentheses
	-:1165: CHECK: Macro argument reuse 'page' - possible side-effects?
	-:1193: ERROR: Macros with complex values should be enclosed in parentheses
	-:1193: CHECK: Macro argument reuse 'page' - possible side-effects?
	-:1227: ERROR: Macros with complex values should be enclosed in parentheses
	-:1227: CHECK: Macro argument reuse 'page' - possible side-effects?


Error #512 when building PDF docs


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL FOR 5.17] V4L2 patches
  2021-12-13 13:49 [GIT PULL FOR 5.17] V4L2 patches Sakari Ailus
  2021-12-13 14:27 ` [GIT PULL FOR 5.17] V4L2 patches (#79352) Jenkins
@ 2021-12-14 10:21 ` Mauro Carvalho Chehab
  2021-12-14 10:33   ` Sakari Ailus
  1 sibling, 1 reply; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2021-12-14 10:21 UTC (permalink / raw)
  To: Sakari Ailus; +Cc: linux-media

Hi Sakari,

Em Mon, 13 Dec 2021 15:49:42 +0200
Sakari Ailus <sakari.ailus@iki.fi> escreveu:

> Hi Mauro,
> 
> Here's a few patches for 5.17. In particular, included are isl79987 driver
> from Michale Tretter and Marek Vasut, and a dependency fix for the recently
> added max96712 driver.
> 
> Please pull.
> 
> 
> The following changes since commit 4383cfa18c5bbc5b9b6a9e77adc12aec1c20b72d:
> 
>   Merge tag 'v5.16-rc4' into media_tree (2021-12-07 11:29:41 +0100)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/sailus/media_tree.git tags/for-5.17-2-signed
> 
> for you to fetch changes up to 0541ae639fb1165fb309ebf58605f901146c2535:
> 
>   media: i2c: isl7998x: Add driver for Intersil ISL7998x (2021-12-08 09:51:54 +0200)
> 
> ----------------------------------------------------------------
> V4L2 patches for 5.17
> 
> ----------------------------------------------------------------
> Andy Shevchenko (1):
>       media: i2c: max9286: Get rid of duplicate of_node assignment
> 
> Marek Vasut (2):
>       media: dt-bindings: Add Intersil ISL79987 DT bindings
>       media: i2c: isl7998x: Add driver for Intersil ISL7998x

There are a couple of issues at the STD decoding logic, plus some coding
style issues at the isl7998x driver. So, I'm not applying those two
patches. Those are easily fixable, but we'll be closing the merge window
for 5.17 this week. If it were just the coding styles, I would have just
fixed and applied, but the best here would be for Marek to fix the issues
I point, re-test and submit asap, in order to be on this merge window.

> 
> Michael Tretter (1):
>       media: imx6-mipi-csi2: use pre_streamon callback to set sensor into LP11
> 
> Sakari Ailus (1):
>       max96712: Depend on VIDEO_V4L2
> 
>  .../bindings/media/i2c/isil,isl79987.yaml          |  113 ++
>  MAINTAINERS                                        |    8 +
>  drivers/media/i2c/Kconfig                          |   10 +
>  drivers/media/i2c/Makefile                         |    1 +
>  drivers/media/i2c/isl7998x.c                       | 1543 ++++++++++++++++++++
>  drivers/media/i2c/max9286.c                        |    1 -
>  drivers/staging/media/imx/imx6-mipi-csi2.c         |    9 +-
>  drivers/staging/media/max96712/Kconfig             |    1 +
>  include/uapi/linux/v4l2-controls.h                 |    6 +
>  9 files changed, 1690 insertions(+), 2 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/media/i2c/isil,isl79987.yaml
>  create mode 100644 drivers/media/i2c/isl7998x.c
> 



Thanks,
Mauro

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL FOR 5.17] V4L2 patches
  2021-12-14 10:21 ` [GIT PULL FOR 5.17] V4L2 patches Mauro Carvalho Chehab
@ 2021-12-14 10:33   ` Sakari Ailus
  0 siblings, 0 replies; 6+ messages in thread
From: Sakari Ailus @ 2021-12-14 10:33 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: linux-media

On Tue, Dec 14, 2021 at 11:21:31AM +0100, Mauro Carvalho Chehab wrote:
> Hi Sakari,
> 
> Em Mon, 13 Dec 2021 15:49:42 +0200
> Sakari Ailus <sakari.ailus@iki.fi> escreveu:
> 
> > Hi Mauro,
> > 
> > Here's a few patches for 5.17. In particular, included are isl79987 driver
> > from Michale Tretter and Marek Vasut, and a dependency fix for the recently
> > added max96712 driver.
> > 
> > Please pull.
> > 
> > 
> > The following changes since commit 4383cfa18c5bbc5b9b6a9e77adc12aec1c20b72d:
> > 
> >   Merge tag 'v5.16-rc4' into media_tree (2021-12-07 11:29:41 +0100)
> > 
> > are available in the Git repository at:
> > 
> >   git://linuxtv.org/sailus/media_tree.git tags/for-5.17-2-signed
> > 
> > for you to fetch changes up to 0541ae639fb1165fb309ebf58605f901146c2535:
> > 
> >   media: i2c: isl7998x: Add driver for Intersil ISL7998x (2021-12-08 09:51:54 +0200)
> > 
> > ----------------------------------------------------------------
> > V4L2 patches for 5.17
> > 
> > ----------------------------------------------------------------
> > Andy Shevchenko (1):
> >       media: i2c: max9286: Get rid of duplicate of_node assignment
> > 
> > Marek Vasut (2):
> >       media: dt-bindings: Add Intersil ISL79987 DT bindings
> >       media: i2c: isl7998x: Add driver for Intersil ISL7998x
> 
> There are a couple of issues at the STD decoding logic, plus some coding
> style issues at the isl7998x driver. So, I'm not applying those two
> patches. Those are easily fixable, but we'll be closing the merge window
> for 5.17 this week. If it were just the coding styles, I would have just
> fixed and applied, but the best here would be for Marek to fix the issues
> I point, re-test and submit asap, in order to be on this merge window.

Ack, I see you've already replied Michael. Thanks.

-- 
Sakari Ailus

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [GIT PULL FOR 5.17] V4L2 patches
@ 2021-12-16  0:11 Sakari Ailus
  0 siblings, 0 replies; 6+ messages in thread
From: Sakari Ailus @ 2021-12-16  0:11 UTC (permalink / raw)
  To: linux-media

Hi Mauro,

Here's a few more V4L2 patches for 5.17.

There's a dependency fix for the max96712 driver, various patches for
supporting sensor driver probing in non-D0 ACPI power states, a few random
fixes and imx6-mipi-csi2 support for pre- and post-streamon ops.

Note that the checkpatch.pl complaint about Martin's From: and Sob:
originate from .mailmap.

Please pull.


The following changes since commit 9b4d7b5c81a2578e080da33b5cddc3149fa611aa:

  media: bttv: use DEVICE_ATTR_RO() helper macro (2021-12-07 11:29:58 +0100)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.17-3-signed

for you to fetch changes up to 8d08471daa49e47671e3b64a9c89bc3cbdac5438:

  media: i2c: hi846: use pm_runtime_force_suspend/resume for system suspend (2021-12-16 01:27:18 +0200)

----------------------------------------------------------------
V4L2 patches for 5.17

----------------------------------------------------------------
Andy Shevchenko (1):
      media: i2c: max9286: Get rid of duplicate of_node assignment

Bingbu Cao (5):
      media: ov8856: support device probe in non-zero ACPI D state
      media: ov2740: support device probe in non-zero ACPI D state
      media: imx208: Support device probe in non-zero ACPI D state
      media: ov5675: Support device probe in non-zero ACPI D state
      media: hi556: Support device probe in non-zero ACPI D state

Jean-Michel Hautbois (1):
      media: staging: ipu3-imgu: add the AWB memory layout

Martin Kepplinger (2):
      media: i2c: hi846: check return value of regulator_bulk_disable()
      media: i2c: hi846: use pm_runtime_force_suspend/resume for system suspend

Michael Tretter (1):
      media: imx6-mipi-csi2: use pre_streamon callback to set sensor into LP11

Sakari Ailus (4):
      max96712: Depend on VIDEO_V4L2
      Update Intel-submitted camera sensor driver contacts
      ov8865: Disable only enabled regulators on error path
      media: ov5670: Support device probe in non-zero ACPI D state

 MAINTAINERS                                        |   1 -
 drivers/media/i2c/hi556.c                          |  70 +++++----
 drivers/media/i2c/hi846.c                          |  14 +-
 drivers/media/i2c/imx208.c                         |  82 +++++++----
 drivers/media/i2c/imx319.c                         |   2 +-
 drivers/media/i2c/imx355.c                         |   2 +-
 drivers/media/i2c/max9286.c                        |   1 -
 drivers/media/i2c/ov13858.c                        |   2 +-
 drivers/media/i2c/ov2740.c                         |  69 ++++++---
 drivers/media/i2c/ov5670.c                         |  80 +++++-----
 drivers/media/i2c/ov5675.c                         |  71 +++++----
 drivers/media/i2c/ov8856.c                         | 162 +++++++++++----------
 drivers/media/i2c/ov8865.c                         |  12 +-
 drivers/staging/media/imx/imx6-mipi-csi2.c         |   9 +-
 .../staging/media/ipu3/include/uapi/intel-ipu3.h   |  32 +++-
 drivers/staging/media/max96712/Kconfig             |   1 +
 16 files changed, 379 insertions(+), 231 deletions(-)

-- 
Sakari Ailus

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [GIT PULL FOR 5.17] V4L2 patches
@ 2021-11-24 15:11 Sakari Ailus
  0 siblings, 0 replies; 6+ messages in thread
From: Sakari Ailus @ 2021-11-24 15:11 UTC (permalink / raw)
  To: linux-media

Hi Mauro,

Here are a bunch of V4L2 patches for 5.17.

In particular, this includes support for the ov5693 sensor, ACPI support
for the ov8865 driver as well as improvements to the imx274 driver.

Please pull.


The following changes since commit 0d2517b3765aa331aee0a95f9b8072062d6bb7e5:

  media: hantro: Support NV12 on the G2 core (2021-11-22 07:53:37 +0000)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.17-1-signed

for you to fetch changes up to bd697d6a91d20cd05860f14f672bd7e8237d4541:

  dt-bindings: media: renesas,jpu: Convert to json-schema (2021-11-24 11:15:14 +0200)

----------------------------------------------------------------
V4L2 patches for 5.17

----------------------------------------------------------------
Daniel Scally (19):
      media: ipu3-cio2: Toggle sensor streaming in pm runtime ops
      media: i2c: Add support for ov5693 sensor
      media: ipu3-cio2: Add link freq for INT33BE entry
      media: i2c: Re-order runtime pm initialisation
      media: i2c: Add ACPI support to ov8865
      media: i2c: Fix incorrect value in comment
      media: i2c: Defer probe if not endpoint found
      media: i2c: Support 19.2MHz input clock in ov8865
      media: i2c: Add .get_selection() support to ov8865
      media: i2c: Switch control to V4L2_CID_ANALOGUE_GAIN
      media: i2c: Add vblank control to ov8865
      media: i2c: Add hblank control to ov8865
      media: i2c: Update HTS values in ov8865
      media: i2c: cap exposure at height + vblank in ov8865
      media: i2c: Add controls from fwnode to ov8865
      media: i2c: Switch exposure control unit to lines
      media: i2c: Use dev_err_probe() in ov8865
      media: i2c: Fix max gain in ov8865
      media: ipu3-cio2: Add INT347A to cio2-bridge

Eugen Hristev (6):
      media: i2c: imx274: fix s_frame_interval runtime resume not requested
      media: i2c: imx274: fix trivial typo expsoure/exposure
      media: i2c: imx274: fix trivial typo obainted/obtained
      media: i2c: imx274: implement enum_mbus_code
      media: i2c: imx274: simplify probe function by adding local variable dev
      media: i2c: imx274: implement fwnode parsing

Geert Uytterhoeven (1):
      dt-bindings: media: renesas,jpu: Convert to json-schema

Hans de Goede (1):
      media: i2c: ov8865: Fix lockdep error

Kieran Bingham (1):
      media: i2c: max9286: Depend on VIDEO_V4L2

 .../devicetree/bindings/media/renesas,jpu.txt      |   25 -
 .../devicetree/bindings/media/renesas,jpu.yaml     |   65 +
 MAINTAINERS                                        |    7 +
 drivers/media/i2c/Kconfig                          |   12 +
 drivers/media/i2c/Makefile                         |    1 +
 drivers/media/i2c/imx274.c                         |  102 +-
 drivers/media/i2c/ov5693.c                         | 1537 ++++++++++++++++++++
 drivers/media/i2c/ov8865.c                         |  466 ++++--
 drivers/media/pci/intel/ipu3/cio2-bridge.c         |    4 +-
 drivers/media/pci/intel/ipu3/ipu3-cio2-main.c      |   17 +-
 10 files changed, 2052 insertions(+), 184 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/renesas,jpu.txt
 create mode 100644 Documentation/devicetree/bindings/media/renesas,jpu.yaml
 create mode 100644 drivers/media/i2c/ov5693.c

-- 
Kind regards,

Sakari Ailus

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-12-16  0:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-13 13:49 [GIT PULL FOR 5.17] V4L2 patches Sakari Ailus
2021-12-13 14:27 ` [GIT PULL FOR 5.17] V4L2 patches (#79352) Jenkins
2021-12-14 10:21 ` [GIT PULL FOR 5.17] V4L2 patches Mauro Carvalho Chehab
2021-12-14 10:33   ` Sakari Ailus
  -- strict thread matches above, loose matches on Subject: below --
2021-12-16  0:11 Sakari Ailus
2021-11-24 15:11 Sakari Ailus

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.