Re: [PATCH V5 2/3] arm64: dts: qcom: monaco-evk: Enable SDHCI for SD Card via overlay

[Monish Chunara <monish.chunara@oss.qualcomm.com>]

On 02-09-2026 21:59, Konrad Dybcio wrote:
> On 8/26/26 1:24 PM, Monish Chunara wrote:
>> The monaco EVK board supports either eMMC or SD-card, but only one can
>> be active at a time.
>>
>> Make the base monaco-evk.dtsi storage-agnostic by removing eMMC-specific
>> configuration from it. This allows storage media selection to be handled
>> through device tree overlays.
>>
>> Introduce an SD card overlay that enables the SD Host Controller
>> Interface (SDHCI) for SD-based storage on the monaco EVK board.
>>
>> Signed-off-by: Monish Chunara <monish.chunara@oss.qualcomm.com>
>> ---
> 
> [...]
> 
>> +&sdhc1_opp_table {
>> +	opp-100000000 {
>> +		opp-hz = /bits/ 64 <100000000>;
>> +		required-opps = <&rpmhpd_opp_low_svs>;
>> +	};
>> +
>> +	opp-202000000 {
>> +		opp-hz = /bits/ 64 <202000000>;
>> +		required-opps = <&rpmhpd_opp_svs_l1>;
>> +	};
>> +};
> 
> This only adds to the existing table, not removing the existing
> entries
> 
> Based on the previous discussions at
> 
> https://lore.kernel.org/linux-arm-msm/de10ed05-a0fc-4f84-83ea-8810270ff2d6@oss.qualcomm.com/
> 
> I'm not sure this is intended
> 
> Konrad

Yes, this is intentional. SD card only uses these 2 frequency options.
The remaining ones in the base files are for eMMC and woouldn't hinder
the SD card functionality.

Instead of maintaining two separate tables, the common one can be 
utilized.

Regards,
Monish