From:   Guo Zhengkui <guozhengkui@vivo.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Alan Stern <stern@rowland.harvard.edu>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Li Jun <jun.li@nxp.com>, Sasha Levin <sashal@kernel.org>,
        Guo Zhengkui <guozhengkui@vivo.com>,
        Sergey Shtylyov <s.shtylyov@omp.ru>,
        Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
        Peter Chen <peter.chen@nxp.com>,
        Andrey Konovalov <andreyknvl@gmail.com>,
        linux-usb@vger.kernel.org (open list:USB SUBSYSTEM),
        linux-kernel@vger.kernel.org (open list)
Cc:     kernel@vivo.com
Subject: [PATCH] usb: core: hcd: change sizeof(vaddr) to sizeof(unsigned long)
Date:   Thu,  9 Dec 2021 14:23:05 +0800
Message-Id: <20211209062441.9856-1-guozhengkui@vivo.com>
In-Reply-To: <Ya/eUbdN1+ABFVWf@rowland.harvard.edu>
References: <Ya/eUbdN1+ABFVWf@rowland.harvard.edu>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?QLH20e243wS71HNaPuxC8jpceP9+ee9TIn55+uBu/KGFDVBV2Ja4fFwo29tS?=
 =?us-ascii?Q?3STOoBFS8lH0h9uuTEOXAbZwLysaazNIa8sjOKLHZ22I2yIJ4TdH7RZrK/Go?=
 =?us-ascii?Q?QJ3LsjzbLEGdpkrhCH+DAVwL9A7aZ+37yQVfiNHaT68XB3wq+byOCd/gCt1D?=
 =?us-ascii?Q?VAUQi0UXeIMJsONHQyumCOdbdmJNtMyVWR3otK10TZW6+dFfKbIjXSX12830?=
 =?us-ascii?Q?Pa85b0NAHDux/1uKQSEeQSbgBfJRAO5kOkUzvFaukng2L+u97LSf/S95jVXB?=
 =?us-ascii?Q?ckEanwIc58G8oCkluASjX3cUI9TY9z9OvyaL9qR3heJaodidEOU3iOGqOxEg?=
 =?us-ascii?Q?TppXUDGE/sXOmEBpJlPpUij/vEkiil52QtvgmoIgQNND8tJuPBkxKxNtJhou?=
 =?us-ascii?Q?LM6dOwMB1tz9h7AdHM5QkkpRHtxOz3BitA20GhPxrodqWaKkyDf7JiVEPXce?=
 =?us-ascii?Q?lU2ZUTrBJak3zSvP4QWJXGUpix0zaWBbdymnp1Ut7h7BWXs2kFhUzB20kOAa?=
 =?us-ascii?Q?WWTtDe/ZIRS2aV9uTNWjvzORzwXGOkk8M5wTdrZKSmJQJSO/8gkHjzbqD0ol?=
 =?us-ascii?Q?GLsolBZtdUlvRE2p7PHtE1zEyFIrcFjxDjyzF6PXEw8SJt/id+G/GW5W4vlX?=
 =?us-ascii?Q?GGIehjfMHMNY/iUuh47eEG07Pe8Gw7x6MjCl9yMhnhaZ+b6ktF3RQ+ACCuyy?=
 =?us-ascii?Q?MB/Jxsu5aGNE0PqF+nPsX+811l8twavXjyxQiQzbHmUlTea08bGfPqqivrQJ?=
 =?us-ascii?Q?eun0pcayfnQJhEwuGGNmsEuenPtjV/Dd2ViS2B5yvCTzr757NAgrOO/702M0?=
 =?us-ascii?Q?w5PY0EjWgE9vGCGEjROGqRw5cw1i2HstRImuIdwPFqnndt9udSU5OVK8Zn+X?=
 =?us-ascii?Q?B1zh/U9Sbysfsvuldgh/jnVzG05+Gb9U1WG2TMFdBfltLYFf8JhQAavmsGv6?=
 =?us-ascii?Q?qSE3ElQqMzBW0jqREygqpFLIS3+b31vQFU5Sdr9J0wojFnXzRXdL1WeyCBrl?=
 =?us-ascii?Q?P+bxfR+L69oWmjX5Z2N2T2fJ42Jw7GeBM6ERb1MXqPLupROJXAlIXqy5uDX4?=
 =?us-ascii?Q?EIrWonnGfQ6Qry4XH1NBRbvidF7mGBm2PZWS10PdRLkU8EFL+DMyGTNRO64z?=
 =?us-ascii?Q?hJ6tIEjhmj8FyjcQCtaoenupC+ZSdI+tWE+TpaPR11/ZwlxeBOagGYSWRphD?=
 =?us-ascii?Q?Ue6bbbzScGMrvmmIGLdefPRf6axDQPOxN1EN/EInon42NtO2Sid7CV99FzZ3?=
 =?us-ascii?Q?Sn6uQC9M4P8pCabo6uNwIwXuddld3s7pNL5QKFOPKo5e0TyV3FjTiVqde0We?=
 =?us-ascii?Q?K5HhdIfBpERNKdCKIJJgHN14C9wtI7fNHIarAZS0B0LLqzKtXqSvTLUO63iV?=
 =?us-ascii?Q?k9lG2Phkm5fjrtPPB9FIn8S4dk06O7IKkSCJBLDYQCmH3Df4At5hl8Kn3cVt?=
 =?us-ascii?Q?cXSPFI0lA2QfCkCILH9jV9CTCmvZg/aMVuaqap7+Pqft4rFzEQZ9tLgR5G1K?=
 =?us-ascii?Q?6rmGE5E9W/uITDEG2TJm8hQjypvy0ATGZas8q8d0/DL6749Wdt2ytGxC26K7?=
 =?us-ascii?Q?NgTyMx1v+IlHsdiej7soXFvtI8LOk8vVkHgz1hjxatTnsZPFLQwqEX+UGQ/M?=
 =?us-ascii?Q?KxyY5U4NxQNU+CPrmlYXWek=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 80e8bc9d-4838-4f34-7cf4-08d9badcc6f5
X-MS-Exchange-CrossTenant-AuthSource: HK2PR06MB3492.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Dec 2021 06:26:04.5840
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Y4053IMW5ieGDypoL9fM5s+gVp3iuqQxWi4n4m/apOEhwYWhyL3fUrbed/BIiWdVDldWezZoraBaGB6mxZ5oKw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HK0PR06MB2785
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

`vaddr` is a pointer to unsigned char. sizeof(vaddr) here intends
to get the size of a pointer. But readers may get confused. Change
sizeof(vaddr) to sizeof(unsigned long) makes more sense.

Suggested-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Guo Zhengkui <guozhengkui@vivo.com>
---
 drivers/usb/core/hcd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
index 4d326ee12c36..9ffc63ae65ac 100644
--- a/drivers/usb/core/hcd.c
+++ b/drivers/usb/core/hcd.c
@@ -1281,7 +1281,7 @@ static int hcd_alloc_coherent(struct usb_bus *bus,
 		return -EFAULT;
 	}
 
-	vaddr = hcd_buffer_alloc(bus, size + sizeof(vaddr),
+	vaddr = hcd_buffer_alloc(bus, size + sizeof(unsigned long),
 				 mem_flags, dma_handle);
 	if (!vaddr)
 		return -ENOMEM;
-- 
2.20.1