mbox series

[GIT,FIXES,FOR,5.9] Imx274 DT binding fix

Message ID 20200917144416.GN834@valkosipuli.retiisi.org.uk (mailing list archive)
State New, archived
Headers show
Series [GIT,FIXES,FOR,5.9] Imx274 DT binding fix | expand

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/fixes-5.9-2-signed

Message

Sakari Ailus Sept. 17, 2020, 2:44 p.m. UTC 
Hi Mauro,

Here's a single DT binding fix for 5.9. The patch converts the bindings to
YAML but at the same time address requested regulator name change.

Please pull.


The following changes since commit 129134e5415d46f38b9978b3809af94ed649b57d:

  media: media/v4l2: remove V4L2_FLAG_MEMORY_NON_CONSISTENT flag (2020-09-14 15:28:06 +0200)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/fixes-5.9-2-signed

for you to fetch changes up to 3be32303b83fd56708fd86d27590ecb6377e36ea:

  dt-bindings: media: imx274: Convert to json-schema (2020-09-17 16:58:05 +0300)

----------------------------------------------------------------
V4L2 fixes for 5.9

----------------------------------------------------------------
Jacopo Mondi (1):
      dt-bindings: media: imx274: Convert to json-schema

 .../devicetree/bindings/media/i2c/imx274.txt       | 38 -----------
 .../devicetree/bindings/media/i2c/sony,imx274.yaml | 76 ++++++++++++++++++++++
 MAINTAINERS                                        |  2 +-
 3 files changed, 77 insertions(+), 39 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/i2c/imx274.txt
 create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml

Comments

Jenkins Sept. 17, 2020, 8:17 p.m. UTC | #1 
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20200917144416.GN834@valkosipuli.retiisi.org.uk/
Build log: https://builder.linuxtv.org/job/patchwork/68754/
Build time: 00:14:48
Link: https://lore.kernel.org/linux-media/20200917144416.GN834@valkosipuli.retiisi.org.uk

gpg: Signature made Thu 17 Sep 2020 02:02:52 PM UTC
gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Can't check signature: No public key

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

Error/warnings:

patches/0001-dt-bindings-media-imx274-Convert-to-json-schema.patch:

    allyesconfig: return code #0:
	../drivers/staging/media/atomisp/pci/sh_css.c:1685:16: warning: assignment left-hand side might be a candidate for a format attribute [-Wsuggest-attribute=format]
	../drivers/staging/media/atomisp/pci/atomisp_compat_css20.c: ../drivers/staging/media/atomisp/pci/atomisp_compat_css20.c:703 is_pipe_valid_to_current_run_mode() warn: ignoring unreachable code.

    allyesconfig: return code #0:
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1909 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2856 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

   checkpatch.pl:
	$ cat patches/0001-dt-bindings-media-imx274-Convert-to-json-schema.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:26: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:66: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst


Error #512 when building PDF docs
Sakari Ailus Sept. 22, 2020, 7:46 p.m. UTC | #2 
Hi Mauro,

On Thu, Sep 17, 2020 at 08:17:02PM +0000, Jenkins wrote:
> From: builder@linuxtv.org
> 
> Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20200917144416.GN834@valkosipuli.retiisi.org.uk/
> Build log: https://builder.linuxtv.org/job/patchwork/68754/
> Build time: 00:14:48
> Link: https://lore.kernel.org/linux-media/20200917144416.GN834@valkosipuli.retiisi.org.uk
> 
> gpg: Signature made Thu 17 Sep 2020 02:02:52 PM UTC
> gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
> gpg:                issuer "sakari.ailus@linux.intel.com"
> gpg: Can't check signature: No public key

I think I've added a subkey for signing recently. It's in pgpkeys
repository:

<URL:https://git.kernel.org/pub/scm/docs/kernel/pgpkeys.git/>

I thought I had added an explanation to the pull request. It seems I did
not.

Feel free to use this one:

This pull request converts IMX274 DT bindings to YAML format. Also
regulators are converted to lower case. This needs to be done now (or
alternatively never) as the regulators were added to IMX274 bindings during
5.9 cycle in commit 74ea3273d24b ("dt-bindings: media: imx274: Add optional
input clock and supplies").