| 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 |
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
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