mbox series

[GIT,PULL] Immutable branch with INT3472 ACPI/i2c + pdx86 patches

Message ID 20211215215212.321235-1-hdegoede@redhat.com (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL] Immutable branch with INT3472 ACPI/i2c + pdx86 patches | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git tags/platform-drivers-x86-int3472-1

Message

Hans de Goede Dec. 15, 2021, 9:52 p.m. UTC
Hello clk, regulator and media maintainers,

As discussed here is a pull-req from an immutable branch based on 5.16-rc1
with the ACPI/i2c, tps68470 and pdx86/INT3472 changes necessary as a base
to merge the remaining clk / regulator / media changes from my
[PATCH v7 00/14] Add support for X86/ACPI camera sensor/PMIC setup with clk and regulator platform data
series. The remaining changes are:

[PATCH v7 05/14] regulator: Introduce tps68470-regulator driver
[PATCH v7 06/14] clk: Introduce clk-tps68470 driver
[PATCH v7 12/14] media: ipu3-cio2: Defer probing until the PMIC is fully setup
[PATCH v7 13/14] media: ipu3-cio2: Call cio2_bridge_init() before anything else
[PATCH v7 14/14] media: ipu3-cio2: Add support for instantiating i2c-clients for VCMs

Please pull this immutable branch into your tree and then merge the
remaining patches relevant for your tree on top.

Regards,

Hans


The following changes since commit fa55b7dcdc43c1aa1ba12bca9d2dd4318c2a0dbf:

  Linux 5.16-rc1 (2021-11-14 13:56:52 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git tags/platform-drivers-x86-int3472-1

for you to fetch changes up to 97c2259ec7757ec24a90f0ef8fc5ea7fa1c6acca:

  platform/x86: int3472: Deal with probe ordering issues (2021-12-13 11:44:51 +0100)

----------------------------------------------------------------
Signed tag for the immutable platform-drivers-x86-int3472 branch

This branch contains 5.16-rc1 + the pending ACPI/i2c, tps68570 platform_data
and INT3472 driver patches.

----------------------------------------------------------------
Hans de Goede (9):
      ACPI: delay enumeration of devices with a _DEP pointing to an INT3472 device
      i2c: acpi: Use acpi_dev_ready_for_enumeration() helper
      i2c: acpi: Add i2c_acpi_new_device_by_fwnode() function
      platform_data: Add linux/platform_data/tps68470.h file
      platform/x86: int3472: Split into 2 drivers
      platform/x86: int3472: Add get_sensor_adev_and_name() helper
      platform/x86: int3472: Pass tps68470_clk_platform_data to the tps68470-regulator MFD-cell
      platform/x86: int3472: Pass tps68470_regulator_platform_data to the tps68470-regulator MFD-cell
      platform/x86: int3472: Deal with probe ordering issues

 drivers/acpi/scan.c                                |  37 +++++-
 drivers/i2c/i2c-core-acpi.c                        |  22 +++-
 drivers/platform/x86/intel/int3472/Makefile        |   9 +-
 ...472_clk_and_regulator.c => clk_and_regulator.c} |   2 +-
 drivers/platform/x86/intel/int3472/common.c        |  82 ++++++++++++
 .../{intel_skl_int3472_common.h => common.h}       |   6 +-
 .../{intel_skl_int3472_discrete.c => discrete.c}   |  51 +++++---
 .../x86/intel/int3472/intel_skl_int3472_common.c   | 106 ---------------
 .../{intel_skl_int3472_tps68470.c => tps68470.c}   |  92 +++++++++++--
 drivers/platform/x86/intel/int3472/tps68470.h      |  25 ++++
 .../x86/intel/int3472/tps68470_board_data.c        | 145 +++++++++++++++++++++
 include/acpi/acpi_bus.h                            |   5 +-
 include/linux/i2c.h                                |  17 ++-
 include/linux/platform_data/tps68470.h             |  35 +++++
 14 files changed, 473 insertions(+), 161 deletions(-)
 rename drivers/platform/x86/intel/int3472/{intel_skl_int3472_clk_and_regulator.c => clk_and_regulator.c} (99%)
 create mode 100644 drivers/platform/x86/intel/int3472/common.c
 rename drivers/platform/x86/intel/int3472/{intel_skl_int3472_common.h => common.h} (94%)
 rename drivers/platform/x86/intel/int3472/{intel_skl_int3472_discrete.c => discrete.c} (91%)
 delete mode 100644 drivers/platform/x86/intel/int3472/intel_skl_int3472_common.c
 rename drivers/platform/x86/intel/int3472/{intel_skl_int3472_tps68470.c => tps68470.c} (56%)
 create mode 100644 drivers/platform/x86/intel/int3472/tps68470.h
 create mode 100644 drivers/platform/x86/intel/int3472/tps68470_board_data.c
 create mode 100644 include/linux/platform_data/tps68470.h

Comments

Jenkins Dec. 15, 2021, 10:44 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20211215215212.321235-1-hdegoede@redhat.com/
Build log: https://builder.linuxtv.org/job/patchwork/165742/
Build time: 00:43:45
Link: https://lore.kernel.org/linux-media/20211215215212.321235-1-hdegoede@redhat.com

gpg: Signature made Wed 15 Dec 2021 09:40:04 PM UTC
gpg:                using RSA key BAF03B5D2718411A5E9E177E92EC4779440327DC
gpg:                issuer "hdegoede@redhat.com"
gpg: Good signature from "Hans de Goede <hdegoede@redhat.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: A1EA 0673 EAD8 B74F 17D2  B9E1 7C31 E21A 98D2 1E0D
     Subkey fingerprint: BAF0 3B5D 2718 411A 5E9E  177E 92EC 4779 4403 27DC

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

Error/warnings:

patches/0001-ACPI-delay-enumeration-of-devices-with-a-_DEP-pointi.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:
	../drivers/media/rc/meson-ir-tx.c:22: warning: expecting prototype for meson(). Prototype was for DEVICE_NAME() instead
	../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
	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/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2868 em28xx_v4l2_init() 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)

patches/0003-i2c-acpi-Add-i2c_acpi_new_device_by_fwnode-function.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:2894 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

   checkpatch.pl:
	$ cat patches/0003-i2c-acpi-Add-i2c_acpi_new_device_by_fwnode-function.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:95: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:95: CHECK: Lines should not end with a '('

patches/0004-platform_data-Add-linux-platform_data-tps68470.h-fil.patch:

   checkpatch.pl:
	$ cat patches/0004-platform_data-Add-linux-platform_data-tps68470.h-fil.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:32: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0005-platform-x86-int3472-Split-into-2-drivers.patch:

   checkpatch.pl:
	$ cat patches/0005-platform-x86-int3472-Split-into-2-drivers.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:33: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0008-platform-x86-int3472-Pass-tps68470_regulator_platfor.patch:

   checkpatch.pl:
	$ cat patches/0008-platform-x86-int3472-Pass-tps68470_regulator_platfor.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:75: WARNING: line length of 108 exceeds 100 columns
	-:128: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?


Error #512 when building PDF docs