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

[Abbarapu, Venkatesh]

Hi Michal,

> -----Original Message-----
> From: Simek, Michal <michal.simek at amd.com>
> Sent: Wednesday, September 18, 2024 2:33 PM
> To: neil.armstrong at linaro.org; Marek Vasut <marek.vasut+renesas at mailbox.org>;
> Neha Malcom Francis <n-francis at ti.com>; Manorit Chawdhry <m-
> chawdhry at ti.com>; Tom Rini <trini at konsulko.com>; bmeng.cn at gmail.com;
> Abbarapu, Venkatesh <venkatesh.abbarapu at amd.com>
> Cc: u-boot at lists.denx.de; u-boot-qcom at groups.io; Bhupesh Sharma
> <bhupesh.linux at gmail.com>
> Subject: Re: [PATCH 00/13] ufs: enhancements to support Qualcomm UFS
> controllers
> 
> 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 see that there are definitely some kernel-doc issues
> 
> Also s/U-boot/U-Boot/g
> 
> Venkatesh: Can you please look at this patch series and validate it on our platform?
> Also make sense to send our changes to make sure that they won't break others.
I will pull these changes and try on our platform.

Thanks
Venkatesh
> 
> Thanks,
> Michal