Messages in this thread |  | | Date | Thu, 5 Dec 2013 11:30:16 +0000 | From | Matt Fleming <> | Subject | Re: [PATCH 1/3] Introduce FW_INFO* functions and messages |
| |
On Wed, 04 Dec, at 07:22:57PM, Arnd Bergmann wrote: > The other part I noticed about this particular patchset is that it's > not really "firmware" as such, but specifically PC wiht ACPI that > gets covered here. So rather than generalizing the code, another > option would be to narrow down the scope and make it > acpi_{warn,info,dbg} instead.
Making this specific to ACPI runs the risk of people introducing a multitude of new logging functions for every subsystem, e.g. efi_{warn,info,dbg}.
FWIW, I'd be interested in using something like this patch series to properly log EFI implementation bugs. The logging for EFI is currently done fairly haphazardly.
-- Matt Fleming, Intel Open Source Technology Center
|  |