On 10/15/26 10:39 AM, Dmitry Baryshkov wrote:
> Add device tree for the Qualcomm / Lantronix SDM845 HDK. It is the
> development platform using the modem-less (SDA845) SoC, optional onboard
> DSI panel and a rich set of connectors.

[...]

> +     gpio-keys {
> +             compatible = "gpio-keys";
> +             autorepeat;
> +
> +             pinctrl-names = "default";
> +             pinctrl-0 = <&home_pin_a>, <&vol_up_pin_a>;

property-n
property-names

please

[...]

> +&cluster_sleep_0 {
> +     /* default, 0x4100c244, kills the board */
> +     arm,psci-suspend-param = <0x41008244>;
> +};

Really?

[...]

> +&mdss_dsi0 {
> +     status = "okay";

status last, with a \n before it, please

[...]

> +&pmi8998_wled {
> +     status = "disabled";
> +};

Is there a problem with leaving it as-is?


[...]

> +&tlmm {
> +     gpio-reserved-ranges = <0 4>, /* SPI (eSE - embedded Secure Element) */
> +                            <81 4>; /* SPI (fingerprint reader) */
> +
> +     lt9611_irq_pin: lt9611-irq-state {
> +             pins = "gpio113";
> +             function = "gpio";
> +             bias-disable;
> +     };

Sort by pin #, please

[...]

> +&venus {
> +     status = "okay";

firmware-name, please

otherwise generally lg

Konrad

Reply via email to