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

[Neha Malcom Francis]

Hi Neil

On 18/09/24 13:36, 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 ?
> 

For the K3 platforms (j721s2, j721e, j7200, j784s4) I have already done a sanity 
test with this series on j784s4 with no regressions seen, so should be fine.

> It seems the only user of UFS_PCI is qemu-riscv, is there other users ?
> 
> Thanks,
> Neil
> 
>>
>> Signed-off-by: Neil Armstrong <neil.armstrong at linaro.org>
>> ---
>> Bhupesh Sharma (5):
>>        ufs/ufs.h: Add definition of 'ufshcd_rmwl()'
>>        ufs: Clear UECPA once due to LINERESET has happened during LINK_STARTUP
>>        ufs: Sync possible UFS Quirks with Linux UFS driver
>>        ufs: Add missing memory barriers
>>        ufs: Fix debug message in 'ufs_start'
>>
>> Marek Vasut (2):
>>        ufs: Add UFSHCD_QUIRK_BROKEN_64BIT_ADDRESS
>>        ufs: Add UFSHCD_QUIRK_HIBERN_FASTAUTO
>>
>> Neil Armstrong (6):
>>        ufs: allocate descriptors with size aligned with DMA_MINALIGN
>>        ufs: fix dcache flush and invalidate range calculation
>>        ufs: split flush and invalidate to only invalidate when required
>>        ufs: use dcache helpers for scsi_cmd data and only invalidate if necessary
>>        ufs: core: remove link_startup_again logic
>>        MAINTAINERS: Add myself to the list of UFS maintainers
>>
>>   MAINTAINERS       |   1 +
>>   drivers/ufs/ufs.c |  98 ++++++++++++++++++++--------------
>>   drivers/ufs/ufs.h | 157 +++++++++++++++++++++++++++++++++++++++++++++++-------
>>   3 files changed, 197 insertions(+), 59 deletions(-)
>> ---
>> base-commit: ca55cf8104c0dd78aae45fa66dd8400ef1b3d0ac
>> change-id: 20240910-topic-ufs-enhancements-fe8ef9ce39d8
>>
>> Best regards,
> 

-- 
Thanking You
Neha Malcom Francis