Messages in this thread | | | Date | Tue, 27 Aug 2019 09:53:36 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH v5 2/2] EDAC: add EDAC driver for DMC520 |
| |
On Tue, Aug 27, 2019 at 01:49:29AM +0000, Lei Wang wrote: > Yes, this is to help open source developers who potentially might want > to expand this driver, most likely colleagues, and possibly other > developers.
Then please put that info in the comment at the beginning of the driver - not some random single sentences about what can be done, interspersed throughout the code. It needs to be visible at a first glance.
Also, if you want this to be a longer doc, I wouldn't mind at all having it in Documentation/edac/ (which doesn't exist yet) and then point people to it from the comment at the beginning of the driver.
Thx.
-- Regards/Gruss, Boris.
Good mailing practices for 400: avoid top-posting and trim the reply.
| |