From mboxrd@z Thu Jan  1 00:00:00 1970
From: Aravind Gopalakrishnan <aravind.gopalakrishnan@amd.com>
Subject: Re: [PATCH V2] x86,
	mce: Consolidate AMD mcheck initialization
Date: Wed, 28 May 2014 10:18:12 -0500
Message-ID: <5385FE34.5040902@amd.com>
References: <1401203677-16497-1-git-send-email-aravind.gopalakrishnan@amd.com>
	<5385B41B.8000309@amazon.de>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <5385B41B.8000309@amazon.de>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: "Egger, Christoph" <chegger@amazon.de>, jinsong.liu@alibaba-inc.com, xen-devel@lists.xen.org, JBeulich@suse.com
List-Id: xen-devel@lists.xenproject.org

On 5/28/2014 5:02 AM, Egger, Christoph wrote:
> On 27.05.14 17:14, Aravind Gopalakrishnan wrote:
>> amd_k8.c did a lot of common work and very little K8
>> specific work. So merge init functions of amd_f10.c and
>> amd_k8.c and move it into the common amd_mcheck_init
>> handler. With that done, there is not much left in either
>> files, so fold all code into just one file - mce_amd.c
>>
>> While at it, update the comments regarding documentation
>> with correct URL's and revision numbers.
>>
>> Also, update copyright info.
>>
>> Changes in V2:
>>      - Fix coding style issues
>  From the functional perspective this patch is fine.
> Regarding coding style, make Jan happy, then you have my
>
> Acked-by: Christoph Egger <chegger@amazon.de>
>

Thanks.

I'll address the issues Jan pointed out in the earlier reply and re-send.
-Aravind.