linux-kernel - Re: [PATCH v9 00/12] This is continued work on Samsung S9(SM-9600) starqltechn

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <174058387176.2466803.5224190356165061330.robh@kernel.org>
Date: Wed, 26 Feb 2025 09:36:49 -0600
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Dzmitry Sankouski <dsankouski@...il.com>
Cc: Conor Dooley <conor+dt@...nel.org>, Liam Girdwood <lgirdwood@...il.com>, 
 Krzysztof Kozlowski <krzk+dt@...nel.org>, 
 cros-qcom-dts-watchers@...omium.org, Mark Brown <broonie@...nel.org>, 
 devicetree@...r.kernel.org, Takashi Iwai <tiwai@...e.com>, 
 linux-arm-msm@...r.kernel.org, Jaroslav Kysela <perex@...ex.cz>, 
 Bjorn Andersson <andersson@...nel.org>, linux-sound@...r.kernel.org, 
 linux-kernel@...r.kernel.org, Konrad Dybcio <konradybcio@...nel.org>, 
 Srinivas Kandagatla <srinivas.kandagatla@...aro.org>, 
 Konrad Dybcio <konrad.dybcio@....qualcomm.com>
Subject: Re: [PATCH v9 00/12] This is continued work on Samsung S9(SM-9600)
 starqltechn


On Tue, 25 Feb 2025 19:38:51 +0300, Dzmitry Sankouski wrote:
> Contains starqltechn device tree changes.
> Add support for new features:
> - sound (headphones and mics only)
> - gpu
> - panel
> - buttons
> - MAX77705 MFD:
>   - charger
>   - fuelgauge
>   - haptic
>   - led
> 
> Binding Dependencies:
> - s2dos05: https://lore.kernel.org/r/20241007-starqltechn_integration_upstream-v6-0-264309aa66de@gmail.com
>   - This series was applied to krzk/linux.git (for-next)
> 
> - max77705: https://lore.kernel.org/r/20241209-starqltechn_integration_upstream-v11-4-dc0598828e01@gmail.com
>   - applied to
>     git@...olite.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git
> 
> - s6e3ha8 panel: https://lore.kernel.org/r/20241006-starqltechn_integration_upstream-v6-0-8336b9cd6c34@gmail.com
>   - applied to https://gitlab.freedesktop.org/drm/misc/kernel.git (drm-misc-next)
> 
> Runtime Dependencies:
> - gcc845 gp clock: https://lore.kernel.org/r/20241007-starqltechn_integration_upstream-v6-0-dd75c06c708d@gmail.com
>   - applied to clk-next
> 
> Signed-off-by: Dzmitry Sankouski <dsankouski@...il.com>
> ---
> Changes in v9:
> - update applied dependency patchsets
> - Link to v8: https://lore.kernel.org/r/20241209-starqltechn_integration_upstream-v8-0-ec604481d691@gmail.com
> 
> Changes in v8:
> - add reviewed tags
> - minor fixes
> - remove 'reg' property in fuel-gauge to comply with v11 max77705 mfd
> - Link to v7: https://lore.kernel.org/r/20241205-starqltechn_integration_upstream-v7-0-84f9a3547803@gmail.com
> 
> Changes in v7:
> - review fixes.
> - new patch with dt-binding header for wcd934x
> - Link to v6: https://lore.kernel.org/r/20241008-starqltechn_integration_upstream-v6-0-5445365d3052@gmail.com
> 
> Changes in v6:
> - refactor: no space between tags in commit message
> - rename starqltechn to sdm845-starqltechn in commit summaries
> - Link to v5: https://lore.kernel.org/r/20240926-starqltechn_integration_upstream-v5-0-d2084672ff2f@gmail.com
> 
> Changes in v5:
> - Split patchset per subsystem
> - Add links to subsystem patchsets in description
> - Link to v4: https://lore.kernel.org/r/20240913-starqltechn_integration_upstream-v4-0-2d2efd5c5877@gmail.com
> 
> Changes in v4:
> - Rewrite max77705, max77705_charger, max77705_fuel_gauge from scratch
> - Reorder patches:
>   - squash max77705 subdevice bindings in core file because
>     no resources there
>   - split device tree changes
> - Use _ as space for filenames in power/supply like the majority
> - Replace gcc-845 freq_tbl frequencies patch with new approach,
>   based on automatic m/n/pre_div value generation
> - Link to v3: https://lore.kernel.org/r/20240618-starqltechn_integration_upstream-v3-0-e3f6662017ac@gmail.com
> 
> Changes in version 3:
> - disable crypto patch removed(disabled on distro level)
> - dts framebuffer node along with related patches removed,
> because panel driver added
> - fix 'make O=.output_arm64 CHECK_DTBS=y qcom/sdm845-samsung-starqltechn.dtb'
> errors, but it still complains on 'monitored-battery' and
> 'power-supplies' though I have 'power-supply.yaml' link in charger
> and fuel gauge bindings.
> 
> ---
> Dzmitry Sankouski (12):
>       arm64: dts: qcom: sdm845: enable gmu
>       arm64: dts: qcom: sdm845-starqltechn: remove wifi
>       arm64: dts: qcom: sdm845-starqltechn: fix usb regulator mistake
>       arm64: dts: qcom: sdm845-starqltechn: refactor node order
>       arm64: dts: qcom: sdm845-starqltechn: remove excess reserved gpios
>       arm64: dts: qcom: sdm845-starqltechn: add gpio keys
>       arm64: dts: qcom: sdm845-starqltechn: add max77705 PMIC
>       arm64: dts: qcom: sdm845-starqltechn: add display PMIC
>       arm64: dts: qcom: sdm845-starqltechn: add touchscreen support
>       arm64: dts: qcom: sdm845-starqltechn: add initial sound support
>       arm64: dts: qcom: sdm845-starqltechn: add graphics support
>       arm64: dts: qcom: sdm845-starqltechn: add modem support
> 
>  arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi                   |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-db845c.dts                   |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-mtp.dts                      |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi          |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts      | 600 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----
>  arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts            |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi        |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts           |   4 ---
>  arch/arm64/boot/dts/qcom/sdm845.dtsi                         |   2 --
>  arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts         |   4 ---
>  11 files changed, 591 insertions(+), 47 deletions(-)
> ---
> base-commit: d4b0fd87ff0d4338b259dc79b2b3c6f7e70e8afa
> change-id: 20240617-starqltechn_integration_upstream-bc86850b2fe3
> 
> Best regards,
> --
> Dzmitry Sankouski <dsankouski@...il.com>
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/qcom/' for 20250225-starqltechn_integration_upstream-v9-0-a5d80375cb66@...il.com:

arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dtb: /soc@...eniqup@...000/i2c@...000/pmic@66: failed to match any schema with compatible: ['maxim,max77705']
arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dtb: /soc@...eniqup@...000/i2c@...000/pmic@...leds: failed to match any schema with compatible: ['maxim,max77705-rgb']
arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dtb: /soc@...eniqup@...000/i2c@...000/pmic@...haptic: failed to match any schema with compatible: ['maxim,max77705-haptic']
arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dtb: /soc@...eniqup@...000/i2c@...000/charger@69: failed to match any schema with compatible: ['maxim,max77705-charger']