Re: [PATCH] arm64: defconfig: Change CONFIG_SM_TCSRCC_8750 from m to y

[Krzysztof Kozlowski <krzk@kernel.org>]

On 17/10/2025 10:15, Konrad Dybcio wrote:
> On 10/17/25 8:54 AM, Krzysztof Kozlowski wrote:
>> On 17/10/2025 07:56, Krzysztof Kozlowski wrote:
>>> On 17/10/2025 07:49, Taniya Das wrote:
>>>>
>>>>
>>>> On 10/17/2025 10:51 AM, Krzysztof Kozlowski wrote:
>>>>> On 17/10/2025 07:16, Taniya Das wrote:
> 
> [...]
> 
>>>> We have tried booting up recently and and that is what we observed. The
>>>> patch from 'm' to 'y' helps the UFS probe is successful and the rootfs
>>>> is picked from ufs partitions. I will add these fail & success log
>>>> snippets in the commit text.
>>>
>>> That's not enough. You need to explain why UFS fails. After explaining
>>> this, I guess bug in UFS would be exposed thus that one should be fixed.
>>> You just provided band-aid without fixing the real problem.
>>>
>>> NAK
>>
>>
>> ... and to prove your analysis is wrong (because your setup is likely
>> having issues) I even tested now linux next with defconfig. Works all
>> fine on next-20251013. You did not share which kernel even has this
>> issue, maybe some downstream tree?
> 
> Is there a chance you have the TCSR module packed in initramfs and
> Taniya doesn't?

Of course that's a chance, since I have all modules packed. Why would
someone pack UFS to initramfs but not its dependencies? It would never
work, actually, nothing would work...

If initramfs does not have necessary modules, way to fix it is to add
these modules, not make the symbol enabled.

Best regards,
Krzysztof