Thank you for your contribution! Sashiko AI review found 3 potential issue(s) to consider: - [High] Missing `vdd-micb-supply` in `lpass_vamacro` node breaks DMIC capture. - [Low] Missing `key-volume-up` subnode in `gpio-keys` despite claiming its pinctrl. - [Low] The `nvme_sec_reg_en` pinctrl node is defined but never referenced. --
commit cf25549a8fd93e158d3d638230deeb2a24306da5 Author: Konrad Dybcio <[email protected]> arm64: dts: qcom: Add Glymur-based Lenovo Yoga Slim 7x Gen11 This commit introduces the device tree for the Lenovo Yoga Slim 7x Gen11 laptop. It defines nodes and configurations for several supported hardware features such as audio, GPU, keyboard, and networking. > diff --git a/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts > b/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts > new file mode 100644 > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/glymur-lenovo-yoga-slim7x.dts [ ... ] > + gpio-keys { > + compatible = "gpio-keys"; > + > + pinctrl-0 = <&key_vol_up_default>, <&hall_int_n_default>; > + pinctrl-names = "default"; > + > + switch-lid { > + label = "lid"; > + gpios = <&tlmm 92 GPIO_ACTIVE_LOW>; > + linux,input-type = <EV_SW>; > + linux,code = <SW_LID>; > + wakeup-source; > + wakeup-event-action = <EV_ACT_DEASSERTED>; > + }; > + }; [Severity: Low] Does this code miss a subnode for the volume up key? The pinctrl state for key_vol_up_default is claimed by the gpio-keys node here, but without a corresponding child node, the input subsystem will not register the volume up button. [ ... ] > +&lpass_vamacro { > + pinctrl-0 = <&dmic01_default>, <&dmic23_default>; > + pinctrl-names = "default"; > + qcom,dmic-sample-rate = <4800000>; > +}; [Severity: High] Should this node include a vdd-micb-supply property pointing to vreg_l1b_e0? The audio routing connects the digital microphones to the vdd-micb DAPM widget. Without a vdd-micb-supply property here, the lpass-va-macro driver falls back to a dummy regulator. Since the physical supply (vreg_l1b_e0) has no consumers and lacks a boot-on or always-on property, the kernel regulator core will disable it. Will this leave the microphones without bias power and result in silent audio capture? [ ... ] > +&pmh0110_f_e1_gpios { > + nvme_sec_reg_en: nvme-reg-en-state { > + pins = "gpio14"; > + function = "normal"; > + bias-disable; > + }; > +}; [Severity: Low] Is this pinctrl state needed? It is defined here but does not appear to be referenced by any device node's pinctrl properties. Could this be leftover dead code from a reference design? -- Sashiko AI review ยท https://sashiko.dev/#/patchset/20260604-topic-yoga_submission-v1-0-57c70c23d...@oss.qualcomm.com?part=3
