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

[Aravind Gopalakrishnan <aravind.gopalakrishnan@amd.com>]

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@alien8.de>:
>>    *		- misc fixes and code cleanups
>>    *
>> + *	Changes by Aravind Gopalakrishnan <aravind.gopalakrishnan@amd.com>
>> + *		- 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.