Re: [PATCH v2] nfit: Continue init even if ARS commands are unimplemented
On Fri, 2016-03-04 at 11:34 +0800, Zhang, Haozhong wrote:
On 03/04/16 11:28, Xiao Guangrong wrote:
>
>
> CCed: Haozhong.
>
> On 03/04/2016 06:39 AM, Vishal Verma wrote:
> >
> > If firmware doesn't implement any of the ARS commands, take that
> > to
> > mean that ARS is unsupported, and continue to initialize regions
> > without
> > bad block lists. We cannot make the assumption that ARS commands
> > will be
> > unconditionally supported on all NVDIMMs.
> >
> > Cc: Dan Williams <dan.j.williams(a)intel.com>
> > Reported-by: Xiao Guangrong <guangrong.xiao(a)linux.intel.com>
> Thank you for the fix, Vishal! it looks good to me.
>
> However, It is haozhong's credit as he is the one reported the bug
> not
> me. :)
>
Thanks! I have tested on QEMU which has not implemented ARS yet and
this patch does work.
Tested-by: Haozhong Zhang <haozhong.zhang(a)intel.com>
Thanks, both Haozhong and Guangrong. Dan, can you please fixup the
Reported-by tag when you apply.
-Vishal
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm(a)lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm