From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S969478AbdEYNly (ORCPT ); Thu, 25 May 2017 09:41:54 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:43164 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965603AbdEYNlx (ORCPT ); Thu, 25 May 2017 09:41:53 -0400 Date: Thu, 25 May 2017 15:41:42 +0200 From: Greg Kroah-Hartman To: Dmitry Torokhov Cc: Wei-Ning Huang , Julius Werner , Guenter Roeck , linux-kernel@vger.kernel.org Subject: Re: [PATCH 8/8] firmware: vpd: fix confusion between memremap and iounmap Message-ID: <20170525134142.GA12184@kroah.com> References: <20170524000748.39310-1-dmitry.torokhov@gmail.com> <20170524000748.39310-8-dmitry.torokhov@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170524000748.39310-8-dmitry.torokhov@gmail.com> User-Agent: Mutt/1.8.2 (2017-04-18) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 23, 2017 at 05:07:48PM -0700, Dmitry Torokhov wrote: > We should not be using iounmap to unmap memory mapped with memremap. > > This fixes following warnings generated by sparse in response to > incorrect type annotations: > > CHECK drivers/firmware/google/vpd.c > drivers/firmware/google/vpd.c:235:20: warning: incorrect type in argument 1 (different address spaces) > drivers/firmware/google/vpd.c:235:20: expected void volatile [noderef] *addr > drivers/firmware/google/vpd.c:235:20: got char *baseaddr > drivers/firmware/google/vpd.c:246:28: warning: incorrect type in argument 1 (different address spaces) > drivers/firmware/google/vpd.c:246:28: expected void volatile [noderef] *addr > drivers/firmware/google/vpd.c:246:28: got char *baseaddr > drivers/firmware/google/vpd.c:258:14: warning: incorrect type in assignment (different address spaces) > drivers/firmware/google/vpd.c:258:14: expected struct vpd_cbmem [noderef] *temp > drivers/firmware/google/vpd.c:258:14: got void * > > Fixes: 049a59db34eb ("firmware: Google VPD sysfs driver") > Signed-off-by: Dmitry Torokhov > Reviewed-by: Guenter Roeck This patch doesn't seem to apply to my tree :(