Staging/max77978

[sean-park-pub]

PR Description

• Please replace this comment with a summary of your changes, and add any context
  necessary to understand them. List any dependencies required for this change.
• To check the checkboxes below, insert a 'x' between square brackets (without
  any space), or simply check them after publishing the PR.
• If you changes include a breaking change, please specify dependent PRs in the
  description and try to push all related PRs simultaneously.

PR Type

• Bug fix (a change that fixes an issue)
• New feature (a change that adds new functionality)
• Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

• I have conducted a self-review of my own code changes
• I have compiled my changes, including the documentation
• I have tested the changes on the relevant hardware
• I have updated the documentation outside this repo accordingly
• I have provided links for the relevant upstream lore

[gastmaier]

Hi. Changed the merge target to mirror_ci/jic23/iio/testing.
Please ignore the Azure pipelines results.
Only consider the GitHub Actions output

There a lot of direct actionable auto-checker that needs to be addressed
https://github.com/analogdevicesinc/linux/actions/runs/28950047520/job/85894059869?pr=3424
starting from stylistic issues:

#2008: FILE: drivers/usb/typec/maxim/Kconfig:131:
Error: checkpatch: trailing whitespace
+^I  Say yes here to add support for Maxim Integrated MAX77958. $

Some annotations shows in the File changed tab too; but is limited to the N-first warnings, are there are more. check the ci/cd output

[github-actions]

LLM review

This series adds the MAX77978 USB Type-C CC/PD controller driver under drivers/usb/typec/maxim/, including core I2C, IRQ, CC, PD, BC1.2, alternate-mode and debug sub-drivers.

run: 28953806165

c50f7f0 - max77978 upstream for v1.2.1

Critical build failure — missing MAX77775 source files:

drivers/usb/typec/maxim/Kconfig introduces config CCIC_MAX77775 and the Makefile references max77775_cc.o, max77775_pd.o, max77775_usbc.o, max77775_alternate.o — none of which exist in the tree. auto_set_kconfig enables CONFIG_CCIC_MAX77775=y because the Kconfig block was touched, causing every build job to fail:

make[7]: *** No rule to make target 'drivers/usb/typec/maxim/max77775_cc.o',
         needed by 'drivers/usb/typec/maxim/built-in.a'.  Stop.

This was confirmed by reproducing the build for gcc_aarch64 and gcc_x86. The MAX77775 Kconfig/Makefile entries must be removed (or the corresponding source files must be included) to unblock CI.

Wrong architecture defconfig:

arch/arm64/configs/my_defconfig is stored under arm64/configs/ but its header reads # Linux/x86 6.12.30 Kernel Configuration and enables CONFIG_X86_32=y. This is an x86 config misplaced in the arm64 tree.

Checkpatch errors (strict):

Three lines in Kconfig have trailing whitespace (lines 123, 131, 142) causing checkpatch ERROR: trailing whitespace failures.

Coccicheck warnings:

18 request_threaded_irq() calls in max77978-cc.c (7), max77978-pd.c (3), max77978-bc12.c (6), and max77978-usbc.c (2) pass flags=0 without IRQF_ONESHOT. These are nested IRQs (registered with irq_set_nested_thread()) so the code is technically correct, but adding IRQF_ONESHOT makes intent explicit and silences coccicheck.

Additional issues (not CI-blocking but must be resolved before upstream):

• CONFIG_ADIENV is referenced in multiple files but never defined in any Kconfig — dead code in upstream builds
• Multiple typedefs for structs/enums violate kernel coding style
• Global singleton g_usbc_data prevents multi-instance use
• No DT binding YAML for adi,max77978 compatible string
• MAINTAINERS not updated
• create_singlethread_workqueue() should be alloc_ordered_workqueue()
• sprintf() with unbounded caller name in max77978_i2c_bulk() (should be snprintf)
• container_of(dev, struct i2c_client, dev) should use to_i2c_client()

CI warnings

The checkpatch warnings for Kconfig help paragraphs being shorter than 4 lines affect the MAX77775 entries; removing those entries (required to fix the build) will also resolve these warnings.

Verification data

• Build failures reproduced locally with set_arch gcc_aarch64 + auto_set_kconfig + make drivers/usb/typec/maxim/ — confirmed max77775_cc.o missing
• Build passes after disabling CONFIG_CCIC_MAX77775 and applying the two fixup patches
• perl scripts/checkpatch.pl --no-tree -f drivers/usb/typec/maxim/Kconfig confirmed trailing whitespace errors and help-length warnings
• irq_set_nested_thread() verified in max77978-irq.c:293 — sub-IRQs are nested threads

Suggested patches

Apply the suggested patches with:

cd path/to/repository
export GITHUB_TOKEN=ghp_***
apply-patches --repo=analogdevicesinc/linux 28953806165

Install instructions

The following one-liner installs the script if not present already:

curl -fSsL "https://raw.githubusercontent.com/analogdevicesinc/doctools/refs/heads/main/ci/scripts/apply-patches.sh"      -o ~/.local/bin/apply-patches.sh &&   grep -q "/apply-patches.sh" ~/.bashrc || echo "source ~/.local/bin/apply-patches.sh" >> $_ ; . $_

More information at AI Usage.