On Tue, Apr 3, 2018 at 8:29 AM, Dave Jiang <dave.ji...@intel.com> wrote:
>
[..]
>> + if (test_and_set_bit(ARS_REQ, &nfit_spa->ars_state))
>> + busy++;
>> + else {
>> + if (flags == ND_ARS_RETURN_PREV_DATA)
>
>
> if (flags & ND_ARS_RETURN_PREV_DATA)?
>
> I think the spec allows other bits set in the flag byte.
>
True, and it might be better to pass in the kernel's ARS_SHORT flag
rather than the firmware interface value.
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
