On Fri, Jun 2, 2023 at 12:03 PM Marek Vasut <ma...@denx.de> wrote: > > On 6/2/23 08:28, Jagan Teki wrote: > > On Tue, May 30, 2023 at 7:07 PM Marek Vasut <ma...@denx.de> wrote: > >> > >> On 5/30/23 14:23, Michal Simek wrote: > >>> > >>> > >>> On 5/16/23 13:52, Ashok Reddy Soma wrote: > >>>> For sf commands, when '0' length is passed for erase, update, write or > >>>> read, there might be undesired results. Ideally '0' length means > >>>> nothing to > >>>> do. > >>>> > >>>> So print 'ERROR: Invalid size 0' and return cmd failure when length > >>>> '0' is > >>>> passed to sf commands. Same thing applies for nand commands also. > >>>> > >>>> Example: > >>>> > >>>> ZynqMP> sf erase 0 0 > >>>> ERROR: Invalid size 0 > >>>> ZynqMP> sf write 10000 0 0 > >>>> ERROR: Invalid size 0 > >>>> ZynqMP> sf read 10000 0 0 > >>>> ERROR: Invalid size 0 > >>>> ZynqMP> sf update 1000 10000 0 > >>>> ERROR: Invalid size 0 > >>>> ZynqMP> > >>>> > >>>> Signed-off-by: Ashok Reddy Soma <ashok.reddy.s...@amd.com> > >>>> --- > >>>> > >>> > >>> Applied it to my queue. If someone else wants to take please let me know > >>> and I will drop it from my queue. > >> > >> I believe Jagan should be picking SF patches ? > > > > Sorry, I missed this during a couple of travels last week. > > Time to prepare a late MR against u-boot/next then ? > I think there are also a few other patches pending .
Yes, Will wind up all. thanks. Jagan.