On Wed, Feb 24, 2021 at 3:48 PM Tsaur, Erwin <erwin.ts...@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 already public.
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 set tw=72 ...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.willi...@intel.com> _______________________________________________ Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org To unsubscribe send an email to linux-nvdimm-le...@lists.01.org