linux-kernel - Re: [PATCH 3/5] arm64: dts: qcom: lemans-evk: Extend peripheral and subsystem support

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] [thread-next>] [day] [month] [year] [list]
Message-ID: <aLBwS6HgYVsIPndP@hu-mchunara-hyd.qualcomm.com>
Date: Thu, 28 Aug 2025 20:35:47 +0530
From: Monish Chunara <quic_mchunara@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
CC: Wasim Nazir <wasim.nazir@....qualcomm.com>,
        Ulf Hansson
	<ulf.hansson@...aro.org>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski
	<krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Bjorn Andersson
	<andersson@...nel.org>,
        Konrad Dybcio <konradybcio@...nel.org>,
        "Richard
 Cochran" <richardcochran@...il.com>,
        <kernel@....qualcomm.com>, <linux-mmc@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-arm-msm@...r.kernel.org>, <netdev@...r.kernel.org>,
        Viken Dadhaniya <viken.dadhaniya@....qualcomm.com>,
        Sushrut Shree Trivedi <quic_sushruts@...cinc.com>,
        Nirmesh Kumar Singh
	<quic_nkumarsi@...cinc.com>,
        Krishna Kurapati
	<krishna.kurapati@....qualcomm.com>,
        Mohd Ayaan Anwar
	<quic_mohdayaa@...cinc.com>,
        Dikshita Agarwal <quic_dikshita@...cinc.com>,
        Vishal Kumar Pal <quic_vispal@...cinc.com>
Subject: Re: [PATCH 3/5] arm64: dts: qcom: lemans-evk: Extend peripheral and
 subsystem support

On Wed, Aug 27, 2025 at 04:35:59AM +0300, Dmitry Baryshkov wrote:
> On Tue, Aug 26, 2025 at 11:51:02PM +0530, Wasim Nazir wrote:
> > Enhance the Qualcomm Lemans EVK board file to support essential
> > peripherals and improve overall hardware capabilities, as
> > outlined below:
> >   - Enable GPI (Generic Peripheral Interface) DMA-0/1/2 and QUPv3-0/2
> >     controllers to facilitate DMA and peripheral communication.
> >   - Add support for PCIe-0/1, including required regulators and PHYs,
> >     to enable high-speed external device connectivity.
> >   - Integrate the TCA9534 I/O expander via I2C to provide 8 additional
> >     GPIO lines for extended I/O functionality.
> >   - Enable the USB0 controller in device mode to support USB peripheral
> >     operations.
> >   - Activate remoteproc subsystems for supported DSPs such as Audio DSP,
> >     Compute DSP-0/1 and Generic DSP-0/1, along with their corresponding
> >     firmware.
> >   - Configure nvmem-layout on the I2C EEPROM to store data for Ethernet
> >     and other consumers.
> >   - Enable the QCA8081 2.5G Ethernet PHY on port-0 and expose the
> >     Ethernet MAC address via nvmem for network configuration.
> >     It depends on CONFIG_QCA808X_PHY to use QCA8081 PHY.
> >   - Add support for the Iris video decoder, including the required
> >     firmware, to enable video decoding capabilities.
> >   - Enable SD-card slot on SDHC.
> > 
> > Co-developed-by: Viken Dadhaniya <viken.dadhaniya@....qualcomm.com>
> > Signed-off-by: Viken Dadhaniya <viken.dadhaniya@....qualcomm.com>
> > Co-developed-by: Sushrut Shree Trivedi <quic_sushruts@...cinc.com>
> > Signed-off-by: Sushrut Shree Trivedi <quic_sushruts@...cinc.com>
> > Co-developed-by: Nirmesh Kumar Singh <quic_nkumarsi@...cinc.com>
> > Signed-off-by: Nirmesh Kumar Singh <quic_nkumarsi@...cinc.com>
> > Co-developed-by: Krishna Kurapati <krishna.kurapati@....qualcomm.com>
> > Signed-off-by: Krishna Kurapati <krishna.kurapati@....qualcomm.com>
> > Co-developed-by: Mohd Ayaan Anwar <quic_mohdayaa@...cinc.com>
> > Signed-off-by: Mohd Ayaan Anwar <quic_mohdayaa@...cinc.com>
> > Co-developed-by: Dikshita Agarwal <quic_dikshita@...cinc.com>
> > Signed-off-by: Dikshita Agarwal <quic_dikshita@...cinc.com>
> > Co-developed-by: Monish Chunara <quic_mchunara@...cinc.com>
> > Signed-off-by: Monish Chunara <quic_mchunara@...cinc.com>
> > Co-developed-by: Vishal Kumar Pal <quic_vispal@...cinc.com>
> > Signed-off-by: Vishal Kumar Pal <quic_vispal@...cinc.com>
> > Signed-off-by: Wasim Nazir <wasim.nazir@....qualcomm.com>
> > ---
> >  arch/arm64/boot/dts/qcom/lemans-evk.dts | 387 ++++++++++++++++++++++++++++++++
> >  1 file changed, 387 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/qcom/lemans-evk.dts b/arch/arm64/boot/dts/qcom/lemans-evk.dts
> > index 9e415012140b..642b66c4ad1e 100644
> > --- a/arch/arm64/boot/dts/qcom/lemans-evk.dts
> > +++ b/arch/arm64/boot/dts/qcom/lemans-evk.dts
> > @@ -16,7 +16,10 @@ / {
> >  	compatible = "qcom,lemans-evk", "qcom,qcs9100", "qcom,sa8775p";
> >  
> >  	aliases {
> > +		ethernet0 = &ethernet0;
> > +		mmc1 = &sdhc;
> >  		serial0 = &uart10;
> > +		serial1 = &uart17;
> >  	};
> >  
> >  	chosen {
> > @@ -46,6 +49,30 @@ edp1_connector_in: endpoint {
> >  			};
> >  		};
> >  	};
> > +
> > +	vmmc_sdc: regulator-vmmc-sdc {
> > +		compatible = "regulator-fixed";
> > +		regulator-name = "vmmc_sdc";
> 
> Non-switchable, always enabled?

According to the hardware schematic, the VMMC supply to the SD card is connected
to an always-on rail. Therefore, it remains continuously enabled and cannot be
turned off via software.

> 
> > +
> > +		regulator-min-microvolt = <2950000>;
> > +		regulator-max-microvolt = <2950000>;
> > +	};
> > +
> > +	vreg_sdc: regulator-vreg-sdc {
> > +		compatible = "regulator-gpio";
> > +
> > +		regulator-min-microvolt = <1800000>;
> > +		regulator-max-microvolt = <2950000>;
> > +		regulator-name = "vreg_sdc";
> > +		regulator-type = "voltage";
> 
> This one also can not be disabled?

This is a voltage translator regulator for the SD Card IO lines (vqmmc), so it
can be at either of the 2 mentioned voltage levels : 1.8 V or 2.95 V, and cannot
be disabled.

> 
> > +
> > +		startup-delay-us = <100>;
> > +
> > +		gpios = <&expander1 7 GPIO_ACTIVE_HIGH>;
> > +
> > +		states = <1800000 0x1
> > +			  2950000 0x0>;
> > +	};
> >  };
> >  

Regards,
Monish