Re: [PATCH 3/3] EDAC, amd64_edac: Update copyright and documentation info

[Aravind Gopalakrishnan]

On 9/17/15 2:43 AM, Borislav Petkov wrote:
> On Wed, Sep 16, 2015 at 03:53:31PM -0500, Aravind Gopalakrishnan wrote:
> >    *	Changes/Fixes by Borislav Petkov <bp@xxxxxxxxx>:
> >    *		- misc fixes and code cleanups
> >    *
> > + *	Changes by Aravind Gopalakrishnan <aravind.gopalakrishnan@xxxxxxx>
> > + *		- Add support for-
> > + *		   - Fam15h Models (30-3fh; 60-6fh)
> > + *		   - Fam16h Models (00-0fh; 30-3fh)
> > + *
> Actually, we have git which can give you all the exact changelog
> information you'd need. So instead of adding more to the header, simply
> remove all changelog info from there. It is a thing from the past anyway.

Okay, will do.

> >    *
> > + *	Title: BIOS and Kernel Developer's Guide (BKDG) for AMD Family 15h
> > + *	       Models 30h-3Fh Processors
> > + *	AMD publication #: 49125
> > + *	Revision: 3.06
> > + *	Issue Date: 2/12/2015 (latest release)
> > + *	Link: http://support.amd.com/TechDocs/49125_15h_Models_30h-3Fh_BKDG.pdf
> > + *
> > + *	Title: BIOS and Kernel Developer's Guide (BKDG) for AMD Family 15h
> > + *	       Models 60h-6Fh Processors
> > + *	AMD publication #: 50742
> > + *	Revision: 3.01
> > + *	Issue Date: 7/23/2015 (latest release)
> > + *	Link: http://support.amd.com/TechDocs/50742_15h_Models_60h-6Fh_BKDG.pdf
> > + *
> > + *	Title: BIOS and Kernel Developer's Guide (BKDG) for AMD Family 16h
> > + *	       Models 00h-0Fh Processors
> > + *	AMD publication #: 48751
> > + *	Revision: 3.03
> > + *	Issue Date: 2/23/2015 (latest release)
> > + *	Link: http://support.amd.com/TechDocs/48751_16h_bkdg.pdf
> > + *
> You can add those to an amd64_edac section of Documentation/edac.txt.

Okay, I'll look into that.

> I want to keep header files as clean as possible. Copyright updates are
> fine but everything history-like and unrelated to functionality should
> go somewhere else, if at all.
Shall I send a V2 of this patch to update copyrights and remove the 
changelogs alone?
(I can meld it with patch 2/3 too if you prefer..)

And I can take up the Documentation work as a separate patch..

Thanks,
-Aravind.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/