[PATCH 00/13] ufs: enhancements to support Qualcomm UFS controllers

[neil.armstrong at linaro.org]

On 18/09/2024 12:47, Michal Simek wrote:
> 
> 
> On 9/18/24 12:40, neil.armstrong at linaro.org wrote:
>> Hi,
>>
>> On 18/09/2024 11:03, Michal Simek wrote:
>>> Hi Neil,
>>>
>>> On 9/18/24 10:06, Neil Armstrong wrote:
>>>> Hi Marek, Manorit, Tom, Michal,
>>>>
>>>> On 10/09/2024 11:20, Neil Armstrong wrote:
>>>>> This serie regroups all the fixes and base enhancements required to
>>>>> support the Qualcomm UFS controllers in U-Boot.
>>>>>
>>>>> This syncs headers & defines from Linux, and includes 2 set of
>>>>> fixes that were sent separately:
>>>>> - ufs: core: remove link_startup_again logic
>>>>> - ufs: properly fix cache operations
>>>>>
>>>>> Without those 2 sets, UFS cannot initialize on Qualcomm controlers
>>>>> since v5, and a numerous of Cache issues makes any UFS controller
>>>>> fail to initialize.
>>>>>
>>>>> Since UFS core hasn't changed for a while, and since UFS is core
>>>>> technology for the Qualcomm SoCs, I volunteer maintaininig the
>>>>> UFS subsystem if Bhupesh & Neha Malcom Francis are ok with that.
>>>>
>>>> Could you run this serie on the r8a779f0_spider, j721s2_evm_a72, j721e_evm_a72,
>>>> j7200_evm_a72, amd_versal2_virt and qemu-riscv and check for possible regressions ?
>>>>
>>>> It seems the only user of UFS_PCI is qemu-riscv, is there other users ?
>>>
>>>
>>> Good timing. We are close to send some updates to UFS which we require to get DWC version to work inside U-Boot.
>>>
>>> How far are that core changes from Linux? I see 9/13 is sync with Linux but when this is done you should also say which Linux version was used for sync.
>>
>> I think we are still quite far from Linux, we just took the basic required bits and pieces
>> to at least make the Qcom UFS controller driver build work, but Linux driver is overly complicated
>> with power management code we do not really need in U-Boot, so it's quite hard to sync honestly.
>>
>> But the plan is to align as much as possible once we get the basic Qcom UFS mainline.
> 
> That's understandable. But even when you say sync with Linux for some headers, etc it is good to say that it is sync with 6.11 for example.

Oh yes exact, I forgot to add this info, I planned to... I'll fix this in a v2.

Thanks,
Neil

> 
> Thanks,
> Michal
>