From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1759816AbcBTXlC (ORCPT <rfc822;w@1wt.eu>);
	Sat, 20 Feb 2016 18:41:02 -0500
Received: from mail.linuxfoundation.org ([140.211.169.12]:48397 "EHLO
	mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1752683AbcBTXlA (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Sat, 20 Feb 2016 18:41:00 -0500
Date: Sat, 20 Feb 2016 15:40:59 -0800
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Olof Johansson <olof@lixom.net>
Cc: Nicolas Boichat <drinkcat@chromium.org>,
        Arve =?iso-8859-1?B?SGr4bm5lduVn?= <arve@android.com>,
        Riley Andrews <riandrews@android.com>,
        "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Olof Johansson <olofj@chromium.org>, Lisa Du <cldu@marvell.com>
Subject: Re: [RESEND PATCH] drivers: android: correct the size of struct
 binder_uintptr_t for BC_DEAD_BINDER_DONE
Message-ID: <20160220234059.GA28573@kroah.com>
References: <1455672772-7646-1-git-send-email-drinkcat@chromium.org>
 <CAOesGMg7O8tPY5imeK+4d7x85ps2NXUckYzmQ4XWJ=TeRrD+Wg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAOesGMg7O8tPY5imeK+4d7x85ps2NXUckYzmQ4XWJ=TeRrD+Wg@mail.gmail.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Feb 19, 2016 at 07:08:40AM -0800, Olof Johansson wrote:
> Hi,
> 
> On Tue, Feb 16, 2016 at 5:32 PM, Nicolas Boichat <drinkcat@chromium.org> wrote:
> > From: Lisa Du <cldu@marvell.com>
> >
> > There's one point was missed in the patch commit da49889deb34 ("staging:
> > binder: Support concurrent 32 bit and 64 bit processes."). When configure
> > BINDER_IPC_32BIT, the size of binder_uintptr_t was 32bits, but size of
> > void * is 64bit on 64bit system. Correct it here.
> >
> > Signed-off-by: Lisa Du <cldu@marvell.com>
> > Signed-off-by: Nicolas Boichat <drinkcat@chromium.org>
> 
> Fixes: da49889deb34 ("staging: binder: Support concurrent 32 bit and
> 64 bit processes.")
> Cc: <stable@vger.kernel.org>
> Acked-by: Olof Johansson <olof@lixom.net>
> 
> (Hopefully Greg's script adds all 3 tags when he applies the patch. :)

Now added, thanks.

greg k-h