From patchwork Thu Jun 1 15:59:55 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: SF Markus Elfring X-Patchwork-Id: 9760061 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id B1070602BF for ; Thu, 1 Jun 2017 16:00:41 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9E337284BF for ; Thu, 1 Jun 2017 16:00:41 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 92D60284F9; Thu, 1 Jun 2017 16:00:41 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 2D07F284BF for ; Thu, 1 Jun 2017 16:00:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:References:To:From:Subject:Reply-To:Content-ID:Content-Description :Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=7/FUKnNFzXDgweYNPmEqZ9koAgFlUdjV+eoMw27bCGw=; b=LFrpThg/nkR439 ueIWNJmjOCSLl+JuhSJBFaGJsIlYW4FyaUUc6CixRacLFMK6q0u2zEZwuxY5upC/Hbv56AOfY01Pd 9BWfYSOmkP5dWxWwj+Fjvmy9jtZffhOeH+aJlkCmCMDxLWQmbmwNdqRA7IZgkMIYEB54hg7ynec6j wo+MfNLUXq9jMylKlQOsfFJTWgxUJ7108/TYB7TZ6d5PPNw+pbOEyl1sIbEQeZcyTCYbEaJTd60Zr HCgyWKSpqQhevfW3m+/jKLTZpBX8WYJj1tbWGhk0wUeoi6UiCCzKiCZi0u6mEmHtJJRL8AeJ/zRx4 KpHsuoOhxO87rBE+QhZQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dGSWe-0000qP-9N; Thu, 01 Jun 2017 16:00:40 +0000 Received: from mout.web.de ([212.227.15.4]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dGSWb-0000nM-Cg for linux-arm-kernel@lists.infradead.org; Thu, 01 Jun 2017 16:00:39 +0000 Received: from [192.168.1.2] ([77.181.221.49]) by smtp.web.de (mrweb003 [213.165.67.108]) with ESMTPSA (Nemesis) id 0Lj2Cs-1dtdcc0Zri-00dJXi; Thu, 01 Jun 2017 18:00:00 +0200 Subject: [PATCH 1/3] ARM: dmabounce: Improve a size determination in two functions From: SF Markus Elfring To: linux-arm-kernel@lists.infradead.org, Alexander Duyck , Andrew Morton , Bart Van Assche , =?UTF-8?B?SsO2cmcgUsO2ZGVs?= , Krzysztof Kozlowski , Russell King , Vineet Gupta References: <697e7810-d280-08f3-2a24-026ff41e15a3@users.sourceforge.net> Message-ID: Date: Thu, 1 Jun 2017 17:59:55 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <697e7810-d280-08f3-2a24-026ff41e15a3@users.sourceforge.net> Content-Language: en-GB X-Provags-ID: V03:K0:0GToeDLj6xo+twoJPXytpbzxc7zjm/5EW8CdrokahcxvWZppqv4 2U+TEG6F25Z/c277xryre2s5RDVDKesLuw/eSEvgAakpfjww/ekqmlZy1midb4eoa3OGRxw Ez2DmkJOpd2VbasFD3M0D7WiTggCCAii9qsF1KUjXtbpa0rHVCERxJYryQPsjfaaWHcbMYu lvABw1BV3t2jzELb/psQQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:HtdBDJXiFFw=:+UZnyyQKWGWnRT10X4Bc8r asZhnrJGWanI7KZNFTlJQIcCx/zXfIbKJwbkd0QQOv5kV3uEi+ELD+6LA4VGBwcElqS6P+jwW B9zphrSOHpP3vRmO3oxYHwECfU/XWTO/0ekXGKdyqarA6JjD+uUWCBvNrb54JZjyfLEijxp0Q 75BwHmfhLOtzjcXs+W4ngk+kJ0didjQzirUj+GoCDjhCHKcCVODGwKxeo9VAWWMkUa0FROQ6+ /VEu4GqbtBcKLhXAQeIoRvuSvbKngnsAwibpfiGotciBZ424Z0Y8ujuUYqdF+XE9i23TbKCxZ 4RTLjLadnA/kgYAqiJyShL8otB6BkdX9Cip/cyIBj5HHSBuV0hB7QMtN4Vku8/x0YHbxKG30A Zl9Q88EIHSRwmGzLXlExdPqP548G3kJAunSLcTa9tBloprxNYPVTlLmMUcZ2WU5yfb5uITr9D hGJ03YkRup7IZj+y1r7EDLObir6oVJlCCwVXUgdfmUMMcazfjQyYZRnRNs6PlSRK2UCwNrHt6 2DZA81jMAeDyqBClNioSyX4E42zwzS0/CIlRfNG1Y+NzHuyi5Jtmhnyv1eVCJh/R8PUXllylQ 7p6+ivCuwrTfb+oYiME8lioq0j+RlzO8J9/9+n4IltUEBfqWRknwK479rP2Uh92diilBeWFT9 vafiVBYthk1+UEuRdW/Rnpo4ai7pZIq5hVnDrHsi+roYALEpInKGm/8BTWYpQnUH250Grj3Km kzj2PU7Un1qPxYe740gnxLRBILk/zNBR7coOgV8iCf6Hpltg3YA0rl6MV/f1JRCl9k/f2L+SV mLxfyWJ X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170601_090037_768081_250295EF X-CRM114-Status: GOOD ( 10.34 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kernel-janitors@vger.kernel.org, LKML Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP From: Markus Elfring Date: Thu, 1 Jun 2017 17:00:17 +0200 Replace the specification of two data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus Elfring --- arch/arm/common/dmabounce.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/common/dmabounce.c b/arch/arm/common/dmabounce.c index 9b1b7be2ec0e..fcdecb2de242 100644 --- a/arch/arm/common/dmabounce.c +++ b/arch/arm/common/dmabounce.c @@ -123,5 +123,5 @@ alloc_safe_buffer(struct dmabounce_device_info *device_info, void *ptr, pool = NULL; } - buf = kmalloc(sizeof(struct safe_buffer), GFP_ATOMIC); + buf = kmalloc(sizeof(*buf), GFP_ATOMIC); if (buf == NULL) { @@ -487,5 +487,5 @@ int dmabounce_register_dev(struct device *dev, unsigned long small_buffer_size, struct dmabounce_device_info *device_info; int ret; - device_info = kmalloc(sizeof(struct dmabounce_device_info), GFP_ATOMIC); + device_info = kmalloc(sizeof(*device_info), GFP_ATOMIC); if (!device_info) {