| Message ID | YbtkGLF1cNN6d2cC@valkosipuli.retiisi.eu (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,5.17] Ipu3-cio2 VCM on Windows BIOS | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YbtkGLF1cNN6d2cC@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/165936/
Build time: 00:30:10
Link: https://lore.kernel.org/linux-media/YbtkGLF1cNN6d2cC@valkosipuli.retiisi.eu
gpg: Signature made Thu 16 Dec 2021 03:43:38 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 6/12 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:
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/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2843 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)
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:2843 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:2874 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?
patches/0012-media-ipu3-cio2-Add-support-for-instantiating-i2c-cl.patch:
checkpatch.pl:
$ cat patches/0012-media-ipu3-cio2-Add-support-for-instantiating-i2c-cl.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:151: ERROR: Macros with complex values should be enclosed in parentheses
Error #512 when building PDF docs
Hi Mauro, Here are ipu3-cio2 patches for supporting VCM on those laptops shipped with Windows from Hans de Goede, on top of an immutable branch on rc1. I pulled Hans's branch on the media tree master and applied the media patches there. I hope this is fine. 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-hans-signed for you to fetch changes up to e41aa17756846f7796771566c4baab5d96117ae1: media: ipu3-cio2: Add support for instantiating i2c-clients for VCMs (2021-12-16 16:28:59 +0200) ---------------------------------------------------------------- Hans's ipu3-cio2 patches on top of INT3472 stuff ---------------------------------------------------------------- Hans de Goede (12): 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 media: ipu3-cio2: Defer probing until the PMIC is fully setup media: ipu3-cio2: Call cio2_bridge_init() before anything else media: ipu3-cio2: Add support for instantiating i2c-clients for VCMs Sakari Ailus (1): Merge tag 'platform-drivers-x86-int3472-1' of https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86 into for-5.17-stable drivers/acpi/scan.c | 37 +++++- drivers/i2c/i2c-core-acpi.c | 22 +++- drivers/media/pci/intel/ipu3/cio2-bridge.c | 92 +++++++++++++ drivers/media/pci/intel/ipu3/cio2-bridge.h | 16 ++- drivers/media/pci/intel/ipu3/ipu3-cio2-main.c | 10 +- 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 +++++ 17 files changed, 584 insertions(+), 168 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