Hi, > -----Original Message----- > From: [email protected] <[email protected]> > Sent: Thursday, September 19, 2024 1:22 PM > To: Simek, Michal <[email protected]>; Marek Vasut > <[email protected]>; Neha Malcom Francis <[email protected]>; > Manorit Chawdhry <[email protected]>; Tom Rini <[email protected]>; > [email protected]; Abbarapu, Venkatesh <[email protected]> > Cc: [email protected]; [email protected]; Bhupesh Sharma > <[email protected]> > 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, [email protected] 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 <[email protected]>
Thanks Venkatesh
