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. Jagan.