On Wed, Feb 24, 2021 at 3:48 PM Tsaur, Erwin <erwin.tsaur(a)intel.com> wrote:
The motivation is to allow access to ACPI defined NVDIMM Root Device _DSM Function Index
5(Translate SPA). The rest of the _DSM functions, which are mostly ARS related, are
For future reference you'll want to fix your editor to make sure it
wraps at 72 columns for commit messages. If you're a vim user, I have
the following in my .vimrc
...and then you can use the 'gq' command to fixup line wrapping after the fact.
Otherwise, looks good to me. I was worried that you needed to double
check that the bus argument actually is an nfit bus, but
bus_has_translate_spa() already takes care of that, so I think we're
good to go. Longer term if other buses grow the ability to return the
DPA we can add a proper generic wrapper for that, to date I have not
seen much support for RAS brewing in other buses.
Reviewed-by: Dan Williams <dan.j.williams(a)intel.com>