Re: [PATCH v2] arm64: dts: qcom: sm8550: add SDHC4 controller node

[Konrad Dybcio]

On 6/30/26 2:41 PM, Vladimir Zapolskiy wrote:
> On 6/30/26 14:09, Konrad Dybcio wrote:
>> On 6/12/26 11:11 AM, Vladimir Zapolskiy wrote:
>>> On 6/12/26 11:47, William Bright wrote:
>>>> On Thu, Jun 11, 2026 at 10:48:34AM +0300, Vladimir Zapolskiy wrote:
>>>>> Looks like the SDHC driver behaves expectedly then. For me it's hard to say
>>>>> what may be the rootcause, I believe the lower bus frequency should be fine,
>>>>> so it sounds like a hardware issue, but could it be PCB/board specific one?
>>>>>
>>>>> If you find a chance to copy the SDHC driver (and its small dependencies)
>>>>> from Android and test it on your board, and if it also fails, then it might
>>>>> be well concluded that something is wrong with hardware, still it won't be
>>>>> quite convincing that the SoC SDHC is to blame here.
>>>>>
>>>>> Hope it helps.
>>>>>
>>>> My colleague Tendai (<tendai.makumire@xxxxxxxxxxx>) had the same issue
>>>> with dll-tuning failing in SDR50 when working on the downstream 5.15 msm
>>>> kernel [1].
>>
>> I have a "feature table" and it says SM8550->SDC4->DLL [Y/N] -> No
>>
>> no wonder it fails the tuning if it's not present (or that's at least
>> my interpretation of this doc..)
> 
> So, is it a hardware problem of the SDHC4 controller, which reports "SDR50 mode
> requires tuning" bit in CAPS1 (see SDHCI_USE_SDR50_TUNING from sdhci.h)?
> 
> https://lore.kernel.org/linux-arm-msm/aik1ZYUT-cnpfdQn@will-Legion-Slim-5-16APH8/
> 
>> There's some notes about the frequency being limited to 75 MHz in
>> SDR50 and to 37.5 in DDR50.
>>
>> The supported modes are DS, HS, SDR12, SDR25, SDR50 and DDR50 (with
>> the caveat above).

I found another doc now and it says that on 8550 specifically the bit
you mentioned above is recommended to be "overwritten to 0". I don't
know if it's actually r/w, but the bottom line is that tuning should
not take place, per my understanding.

Konrad