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

[Abbarapu, Venkatesh]

Hi,

> -----Original Message-----
> From: neil.armstrong at linaro.org <neil.armstrong at linaro.org>
> Sent: Thursday, September 19, 2024 1:22 PM
> To: Simek, Michal <michal.simek at amd.com>; 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
> 
> 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
> >
These changes are verified on AMD platform (amd_versal2_virt_defconfig) and no issues observed with this series.
Tested-by: Venkatesh Yadav Abbarapu <venkatesh.abbarapu at amd.com>

Thanks
Venkatesh