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

[Michal Simek]

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.

Thanks,
Michal