Linux Sound subsystem development
 help / color / mirror / Atom feed
From: "Rob Herring (Arm)" <robh@kernel.org>
To: Dzmitry Sankouski <dsankouski@gmail.com>
Cc: Conor Dooley <conor+dt@kernel.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	 Krzysztof Kozlowski <krzk+dt@kernel.org>,
	 cros-qcom-dts-watchers@chromium.org,
	Mark Brown <broonie@kernel.org>,
	 devicetree@vger.kernel.org, Takashi Iwai <tiwai@suse.com>,
	 linux-arm-msm@vger.kernel.org, Jaroslav Kysela <perex@perex.cz>,
	 Bjorn Andersson <andersson@kernel.org>,
	linux-sound@vger.kernel.org,  linux-kernel@vger.kernel.org,
	Konrad Dybcio <konradybcio@kernel.org>,
	 Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
	 Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Subject: Re: [PATCH v9 00/12] This is continued work on Samsung S9(SM-9600) starqltechn
Date: Wed, 26 Feb 2025 09:36:49 -0600	[thread overview]
Message-ID: <174058387176.2466803.5224190356165061330.robh@kernel.org> (raw)
In-Reply-To: <20250225-starqltechn_integration_upstream-v9-0-a5d80375cb66@gmail.com>


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@gitolite.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@gmail.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@gmail.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@gmail.com:

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






  parent reply	other threads:[~2025-02-26 15:36 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-25 16:38 [PATCH v9 00/12] This is continued work on Samsung S9(SM-9600) starqltechn Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 01/12] arm64: dts: qcom: sdm845: enable gmu Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 02/12] arm64: dts: qcom: sdm845-starqltechn: remove wifi Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 03/12] arm64: dts: qcom: sdm845-starqltechn: fix usb regulator mistake Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 04/12] arm64: dts: qcom: sdm845-starqltechn: refactor node order Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 05/12] arm64: dts: qcom: sdm845-starqltechn: remove excess reserved gpios Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 06/12] arm64: dts: qcom: sdm845-starqltechn: add gpio keys Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 07/12] arm64: dts: qcom: sdm845-starqltechn: add max77705 PMIC Dzmitry Sankouski
2025-02-25 16:38 ` [PATCH v9 08/12] arm64: dts: qcom: sdm845-starqltechn: add display PMIC Dzmitry Sankouski
2025-02-25 16:39 ` [PATCH v9 09/12] arm64: dts: qcom: sdm845-starqltechn: add touchscreen support Dzmitry Sankouski
2025-02-25 16:39 ` [PATCH v9 10/12] arm64: dts: qcom: sdm845-starqltechn: add initial sound support Dzmitry Sankouski
2025-02-25 16:39 ` [PATCH v9 11/12] arm64: dts: qcom: sdm845-starqltechn: add graphics support Dzmitry Sankouski
2025-02-25 16:39 ` [PATCH v9 12/12] arm64: dts: qcom: sdm845-starqltechn: add modem support Dzmitry Sankouski
2025-02-26 15:36 ` Rob Herring (Arm) [this message]
2025-03-14 20:00 ` [PATCH v9 00/12] This is continued work on Samsung S9(SM-9600) starqltechn Bjorn Andersson
2025-05-20  2:14 ` (subset) " Bjorn Andersson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=174058387176.2466803.5224190356165061330.robh@kernel.org \
    --to=robh@kernel.org \
    --cc=andersson@kernel.org \
    --cc=broonie@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=cros-qcom-dts-watchers@chromium.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dsankouski@gmail.com \
    --cc=konrad.dybcio@oss.qualcomm.com \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=srinivas.kandagatla@linaro.org \
    --cc=tiwai@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox