From patchwork Fri Feb 2 15:02:12 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: SF Markus Elfring X-Patchwork-Id: 868612 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-i2c-owner@vger.kernel.org; receiver=) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3zY0Zk3Yx5z9t0m for ; Sat, 3 Feb 2018 02:02:22 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752204AbeBBPCU (ORCPT ); Fri, 2 Feb 2018 10:02:20 -0500 Received: from mout.web.de ([212.227.15.14]:63061 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752180AbeBBPCT (ORCPT ); Fri, 2 Feb 2018 10:02:19 -0500 Received: from [192.168.1.2] ([92.228.169.238]) by smtp.web.de (mrweb001 [213.165.67.108]) with ESMTPSA (Nemesis) id 0Lo0Hw-1fEjgE2pei-00fzZ8; Fri, 02 Feb 2018 16:02:14 +0100 Subject: [PATCH 2/2] i2c-puv3: Improve a size determination in puv3_i2c_probe() From: SF Markus Elfring To: linux-i2c@vger.kernel.org, Guan Xuetao , Wolfram Sang Cc: LKML , kernel-janitors@vger.kernel.org References: <2fbd709c-2790-af6c-4fd4-0c8ec6b5b3d7@users.sourceforge.net> Message-ID: Date: Fri, 2 Feb 2018 16:02:12 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <2fbd709c-2790-af6c-4fd4-0c8ec6b5b3d7@users.sourceforge.net> Content-Language: en-US X-Provags-ID: V03:K0:Ua2G3WRqNVqnD+k/zqkJsE+0L/zoo4dGxKxZg9y6LuZVsGAUvPu Va6Kb3o4/yF5+Hd5AD30KXB4ifXHzFNqPJ/3o1w721PT2JtPGHMo23z4CfYjWWCn0sLQDZs DS3sNwc97JOeO8nzIP5Gu049o+ZF/nVZMbLyF5z5EJF06pksF2WdUQyNMSX7WZBFg9LSk2/ z0/31FzMY9nk5Keh8PD2g== X-UI-Out-Filterresults: notjunk:1; V01:K0:3nJjAmos3kE=:/NdtjFMYvbFMoIgtQaIg7u zFzeD8ks/0Ua/avMvrTeiAC0LRa0DWwoHa5YbT0a3VKw7jkYwNQ7duc/8H9L6Zx6ipvCWxLau ldojKkCnEwej4t+8Y/JStP37nf3EDXme0xD8x5Vvslrf5n5jbzj34fWtZFLAi8U2Ca7J+GyYw c4Hlfkio80EULkpITr57GiTyVLeYnbKEOc4lAFXX3z0RDZE1gAybcR1+IH+L3I+ZPkZTTwclb Z/bkiUBtnJ4LP/YqIKFn0XWtQ3tnaJI6PpcUfOV5nQ2rtG25wOqfGzPyXDx2UStmwF9C/xAcB qI7HCG315qEEUipudIF6v/8ylYpf3C7id8BpCKtU81puaZ4958XRis+nqwkUK69cE3NIoYnJw 1iMlX5J0TUFnMCcONNj67MuoFZbkds0wpwdQzlnUIhM2zMlRPZoTzK0zvWWN6MAJ+pkSIoGwE nea76gkZys38OWAUxz1cGXjbDsrsNfC6OGP0ZD3duE1xMM7qq4XmgmgcicCVYmXFILeBrwgUa /uyHtqvsi/8V1jRgH8n5NI/CVLz6DXsQxjhJP9LetzkqpKyJWgiwUq4Q+GO+uQGgljPUFp5zb KthvtI6a2W+LvrwZsZPmk/jfZFpstd8rsEIVz/xccuIdjQXgEVa/zCChNzrVB/FUbTBm/c2QI ZICMP3YA92gZhQtr/BMtzdTpo9jhGACI6wIo+FmJB6lB50QW4/hHparZ7AbUBOf1mPZ3d0K9J /p3AmKX+9nWnHlv+tYcnZbpVCNDg6sd7W3kK95t7WiaMC6bvhJefrVZktDoACtp1hvsAJp35O SAnSj9JKUGAT8DzeVfe5L9fAfyLwd3xqU7A4UhUDlAqi5kgumA= Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org From: Markus Elfring Date: Fri, 2 Feb 2018 15:43:21 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/i2c/busses/i2c-puv3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-puv3.c b/drivers/i2c/busses/i2c-puv3.c index 5bd001d01bef..929fd12b92de 100644 --- a/drivers/i2c/busses/i2c-puv3.c +++ b/drivers/i2c/busses/i2c-puv3.c @@ -196,7 +196,7 @@ static int puv3_i2c_probe(struct platform_device *pdev) if (!request_mem_region(mem->start, resource_size(mem), "puv3_i2c")) return -EBUSY; - adapter = kzalloc(sizeof(struct i2c_adapter), GFP_KERNEL); + adapter = kzalloc(sizeof(*adapter), GFP_KERNEL); if (adapter == NULL) { rc = -ENOMEM; goto fail_nomem;