From nobody Wed Dec 17 07:27:16 2025 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3137F2376F4; Mon, 7 Apr 2025 10:50:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744023012; cv=none; b=t96VODVm27UBJMzMl4Yj+M3zbDO/nGSUjkFHQ0fjX30SKEZKqye36JbIkApeW31impsQlrKbj9PG8GIOzN8+teo0EoO5MMDAEBZZHRlVu3JZcYFNjelnDv7CIERBF99AvtXuhqedzcOrGefYvdXuXItoR/79vO7E1/SFcjL9oFQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744023012; c=relaxed/simple; bh=JPr4IPusXu6E+MAPG2d2TogsW9Kq03h6kaAEvoKbtTM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=H+GfwTe/cv4i8a/6gQgKEiQa3Ldhr/cgZ5vKFIrbmRRgWbS33lYF1yOGP3nizfOujR/PQZE62sNhzQ7s+OjTq67KX+u0dK6trZJCuyPk7/rgLQHAhwKshrJMNDocgKYeEd1SW2/13ZACQT3Jld7Q3lc5BZrIEtzSwQ3/0s929og= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=GE+10KQU; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="GE+10KQU" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-43cf848528aso34567935e9.2; Mon, 07 Apr 2025 03:50:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1744023008; x=1744627808; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=pgyJSudWfh+2N/OtRxBUgvnd0zEKKZ9y7XXQkkvhRX8=; b=GE+10KQU3+D4zb3IXdUTBzgKWTUmJ6mRMXQW68/GlaCuNb9aZUTYmI1eJN8rIMJQvE qlQIPZkLvcuPTDvKnoNyr9ytbspkszeEYADGgh88rMvxZmvJmTXTdzJu6DAjZj8GzOQM rypgNry5Ln31hSrt1ffXri9r2FOB4dNs+zuscTA03eRqWlc38kn8nCbEMcbGwhTXNA/h a79CIHS4lx3LcrmwYUsGqkcCjbktSq2Zwj45uS3whGdDhg5LQsuIKgPgWBH7HR1ZWNJI SOwn/JIRBlJ/ticBgrl1cKgV1ZNgksRJYop5uqsyavIn/K3Kbq0DIkd5R0VMute1PcLF oQHQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744023008; x=1744627808; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=pgyJSudWfh+2N/OtRxBUgvnd0zEKKZ9y7XXQkkvhRX8=; b=RRTWQeuZOLJyREjH/wHqhs4gRNK3FcrnUmwUOjNvYSvu++bmzADJl3C5L7LUfB7EtE 7nAU91Yf3rUZTHVN35hjS8QUgNFbbRv+i41sfyncSH0J4g6Q0cxL64esMFHF9wjHEoeH J1kexq78zw/dIaNPe17wKgZhk7oAoN6Oiitxdm0UVXHJ51Ar2+W0pEi8ENipiOwjOD43 KPOWwUV9fuVzN9FW6LGU2RPI7rKRJHm1U0URg/DMtIXitEpYv3obDsNwo9GZuQ6tvEwG lHJpRhqVoxkaHQsT/macpQn0oSf4teOGRuBFjJO7+P4IzpVVRDYZwe8Jso05quoVSPYG ke8A== X-Forwarded-Encrypted: i=1; AJvYcCU2q1Ef6ShGV7EVnzG1XKTVmfURext+Urm/7JpQLVxKRU4lmVKZlwWKiZU1Zpkds4tdo7Dv9syvaL17+lQ=@vger.kernel.org, AJvYcCVLHiSoa3RBpJ80glKbOR1Szw6Vat0M39MB6d6AGS1wDtnxrVw3CVf9j662faF/nlVPV/SbGpmgFLxjLS/Xf6AId+E=@vger.kernel.org X-Gm-Message-State: AOJu0YwOgNb2n189n/NEdyhXFUyEdkKRuC+4P4g79znQethzjd+5Sxi7 gmhFvGiYlB+3D0zGPkRJdQHOHtiGpxZy1fGiX13YByOrCLzP+U+/ X-Gm-Gg: ASbGnctIu+ssnzNxugi1ioQRSydUVHEczuuR3LwrbhZX0Ds/UTrD/xCyJcttlY0K/tZ 8jOfWcCGB68CFCeTC3+CriHn1GB5B1dWq9b+KjhRWozYImfWRVTJ6zHIM8LjrREqp7II1BOir7r HbwvasOXND3s2jeIv7svLjh7tryjajWsel+IT/DRZYsiI4N5uqh159QjVUHO/xwkzhjCn8Ib/Uh AHo1kH3OkLrPM9axU6sQAAWvg8aGjavc2hcQnBowXAc8F2NtidutCV1SvVg4lvJlrNojUtUOQce qmWFuf7PEnVGq1lWN1vHsNuw71ov/9RQktZ5F4og0RPIlAZLBjkbU4t0k5+wYodK4h+hiw== X-Google-Smtp-Source: AGHT+IFtRQyH/n4F+U3bXsWJ7B32IMwaMOKnWA5MjHmQnzLYzKFFsBT3Bt/ietEKEmSnfVwLGUJOWw== X-Received: by 2002:a05:600c:5117:b0:43d:40b0:5b with SMTP id 5b1f17b1804b1-43ee076924bmr55522275e9.25.1744023008329; Mon, 07 Apr 2025 03:50:08 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:78b9:80c2:5373:1b49]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43ec3174cf0sm129975765e9.0.2025.04.07.03.50.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Apr 2025 03:50:07 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Yoshihiro Shimoda , Geert Uytterhoeven , Kuninori Morimoto Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Claudiu Beznea , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v2 1/3] usb: renesas_usbhs: Correct function references in comment Date: Mon, 7 Apr 2025 11:50:00 +0100 Message-ID: <20250407105002.107181-2-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250407105002.107181-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250407105002.107181-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Lad Prabhakar Update the comment to reference `usbhs_mod_probe()` instead of `usbhs_mod_init()`, and replace `dev_set_drvdata()` with `platform_set_drvdata()`, as these are the correct functions used in this context. Signed-off-by: Lad Prabhakar Acked-by: Kuninori Morimoto --- drivers/usb/renesas_usbhs/common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/renesas_usbhs/common.c b/drivers/usb/renesas_usbhs= /common.c index 4b35ef216125..03d4d40c90b3 100644 --- a/drivers/usb/renesas_usbhs/common.c +++ b/drivers/usb/renesas_usbhs/common.c @@ -698,7 +698,7 @@ static int usbhs_probe(struct platform_device *pdev) if (ret < 0) goto probe_end_fifo_exit; =20 - /* dev_set_drvdata should be called after usbhs_mod_init */ + /* platform_set_drvdata() should be called after usbhs_mod_probe() */ platform_set_drvdata(pdev, priv); =20 ret =3D reset_control_deassert(priv->rsts); --=20 2.49.0 From nobody Wed Dec 17 07:27:16 2025 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 56B1C238D2B; Mon, 7 Apr 2025 10:50:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744023014; cv=none; b=bM8YnsN3ovQipxmZM4f0vspNH5YNJ1DRhbXLUDLl0+o5W8mHAHf99jH4P8Q4PPzAIsk+YABP5jWDJjXba+9yZfAaVu66bzQF4iNW4n4VaN+GusKui6psKUn3Cr7QnMlkjKuEA10ZruVVqMnY3YMKShHvRqgkvxgdynblpeR91w0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744023014; c=relaxed/simple; bh=ATUIayw5g2MB/P4Cf28yTaKbzDcul5/51x1bj+KByn0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=lqqyLtchguS8OcDWhbKqETVDZxPlIkvbd5NBZRSu30H9ZTq/Ww+37o+ixz3XozAC8i7CCwGX8Gq1TsWOlHrbFIB5EQWEwgDuURcpkQo2ZWvz24+f/+tqFG/64jCkvHqk5/4E/4KEQy2NfJyv4XPD9nQB9mu7qVigicZK9IvCToA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=UEiWEpnU; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="UEiWEpnU" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-43cfb6e9031so38955555e9.0; Mon, 07 Apr 2025 03:50:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1744023009; x=1744627809; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=i42+usEMrEMKE2Zbz9YF9lzk6FXr0K6ureFV6wMnEjA=; b=UEiWEpnU9/C6goqOEo6ry0ZAjc2UQnR7oFOg2/f9cM+Sg5MlqqhO5scVCfUC3F8StC L2ZZnJ0BLCuH35Dz2aZGxPWxvk+VH/OD0u976GjfQJWVTQFawmXp/3PklG2dzbwNXnOW BZvLPD4AXxt+j55e2cuzmd0K18MsDYfG07I/DfomR+AqmwGzNIRr7ITH9LoL+cUFslz5 9tglTH/Aq4T+1HxiwTMf/mRJd5WGzFdNWR/rtqshlAmmKMI0OlBaBIXo9IumheQsJOiA ckd/yjaXYG7EsPUudD4MmIPQM73224OPvLSL0+liMRjXQ/az6dTQpZw0zJuaURKpjno7 Z+FQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744023009; x=1744627809; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=i42+usEMrEMKE2Zbz9YF9lzk6FXr0K6ureFV6wMnEjA=; b=sG9aad00j9lNO06dzrzZWMt5bQXsr99URG8SG1zsCVcpAsF/9gH4lZn2ag2s4aFRAB 1TPcyegq5LJLnvHc8zJ0mAoKO6vOCTyZNFBIsFATrMQHRjmRVRntVjQ66Bvbp7H8LOqy neIetCEHVKgkaTlZjHpEIjGVqiD2Qu8eQNtP5t1TdIU2hwuE0IXJZEq2Ge6dRZm/JGOR IjY6EPHCIK9MUIwBIOfjWpgeU/WJDmGlvJ0amreoOKeYeW/iJalqWO1q+NryT6gJzPZx jBNcgsoeJ5I1/VBpSloOqCj1jOcudLWi8sFUW8Z8RumvFaq/G5lNM5itfwldmFEGDYKK hr/Q== X-Forwarded-Encrypted: i=1; AJvYcCUAx782JikfewSIAo6K48lY48/WfmbI+TNoECwYm8ghVHzYJwtvkE3r1my7sg0guD0Ze8mmPmhSGyNzG1k=@vger.kernel.org, AJvYcCWUzOUJWlnhHWfy1wHo6/i3wBxIUBSsNrXWQVCthqllFp6eh/rAWoVT5O/1k+0rswcsErPU0CEHmVg2JzFQNmr2v/g=@vger.kernel.org X-Gm-Message-State: AOJu0Yxw30xiOKZOgDTPXrmaxL9VJkaGRWr3PWMFsUCgSTdT2Pb0t4Wx sUeFMhKby+dQcFYgMDj++Y52zpShhSRAjZmMgeZ9I2mpmFEbg4Pc X-Gm-Gg: ASbGncuBRwkwX+YGH1rzWO6AiO3fOICyCydKv7tkG0oK66ttSyoYeTy61CyYxOZ+ppV Ux2nADp0usM7d5fGjV27V2piGQfE7gA6n7uj4RtLyMxCOW7+U4X9jziGrKcMgfdFlyiIvSQl0CI ZzDMf7JBvAa/D9OwCkynoOulJTrJj76i7O/tRRXyYAPwS1MxMiCL8J4ntD6cyMf1y/biUu13izz MkPinFISpa33AnQSRbhQpAHTiUUt/YKbQMqGMA21OID9KCfAf3TL6HXLMocTLkkwYxvkakLRTaG x52BKQ9tHyK/TOJkU1Rn5xCPeh8/A4gn7cnyh9igcHs4C9JWy3PMZXUUFLLcCqLn/c0t0OBiqNK 0qmWQ X-Google-Smtp-Source: AGHT+IHQlntxN4Yte+Z25fvQMt+ZJ4riNzZQNUAzoCOp/q+xF35N8bB/U0sBBM35bJV4TW2DFangTw== X-Received: by 2002:a05:600c:c11:b0:43c:ea36:9840 with SMTP id 5b1f17b1804b1-43ecf9c7c07mr91540105e9.22.1744023009402; Mon, 07 Apr 2025 03:50:09 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:78b9:80c2:5373:1b49]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43ec3174cf0sm129975765e9.0.2025.04.07.03.50.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Apr 2025 03:50:08 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Yoshihiro Shimoda , Geert Uytterhoeven , Kuninori Morimoto Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Claudiu Beznea , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v2 2/3] usb: renesas_usbhs: Fix typo in comment Date: Mon, 7 Apr 2025 11:50:01 +0100 Message-ID: <20250407105002.107181-3-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250407105002.107181-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250407105002.107181-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Lad Prabhakar Fix a typo in the comment by correcting "deviece" to "device" for clarity and readability. Signed-off-by: Lad Prabhakar Acked-by: Kuninori Morimoto --- drivers/usb/renesas_usbhs/common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/renesas_usbhs/common.c b/drivers/usb/renesas_usbhs= /common.c index 03d4d40c90b3..703cf5d0cb41 100644 --- a/drivers/usb/renesas_usbhs/common.c +++ b/drivers/usb/renesas_usbhs/common.c @@ -710,7 +710,7 @@ static int usbhs_probe(struct platform_device *pdev) goto probe_fail_clks; =20 /* - * deviece reset here because + * device reset here because * USB device might be used in boot loader. */ usbhs_sys_clock_ctrl(priv, 0); --=20 2.49.0 From nobody Wed Dec 17 07:27:16 2025 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6FBA3238D31; Mon, 7 Apr 2025 10:50:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.42 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744023014; cv=none; b=itGiAz3icykfo52uVJBfC/PJmxFcSu+A13izbxXccSAQTSNbx4JjXu3ITbvyUJz/PoXJ/aXsWUAMOSDOixcrDXXgAJpHLuwNlDXTPhIdV16+R5Yuo1Nnu1msIinpAOI58hj0trKhdwLpBF02RZ8fado2mX6+4LqWD0gK38RzFMU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744023014; c=relaxed/simple; bh=M1ZKOgW1n57B2nbBeuZf06udUcxV8H5OpaY2xRZXsfk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=l9dszPuggqbBpdeJ6e8JF0M3rTqQEr61jGC/o0/iq5DDu6m7ySFtCZ6CsD1D67lQ0HJkIiCJw6rHw2lV8Q/780+MRTzwCuJDkaf7DTjwpnZ3MNDg30p04uE6WP1hj23jVftRTdCaYl+Ja3Dszed+cyBJIZ3wWKtiN5fcIwyiwl0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=DgpOFPgi; arc=none smtp.client-ip=209.85.128.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DgpOFPgi" Received: by mail-wm1-f42.google.com with SMTP id 5b1f17b1804b1-4394a823036so38321025e9.0; Mon, 07 Apr 2025 03:50:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1744023011; x=1744627811; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=x5PfyvK0EHksIxahc3vrkGpOF60charYPKbKSdBdtqg=; b=DgpOFPgifkzKRSgLlJ+YhQdMcwalA4odQ8tOluEH7gxyaZ8aB4OjLdegQJGrb3zUD8 AzqmlI0KHB99ei5oEB0Gm1/hZV6vOFo8XB4712E6UGCLQ5GEIbv35DdgkgVrkYEUxNKL HY0qKcUJ+Sb1pP7YQZ4kmEDLkyyTEt8F9ngvQhe+KBc6PIH830YJi6GaYiv4phF3j98e 9G8k5vDHB2J0ol6+BpKkYdZn1QMZTnIHPTWaS3PWUSqcowaAexReRX3mk6rVNjv3qc20 uABRAOoMr2mPf7aDv+23IY9KsSo07Kx4WDckHNDwSl8tSwLeHBHjfryAC08gXWrxFOK0 3s6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744023011; x=1744627811; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=x5PfyvK0EHksIxahc3vrkGpOF60charYPKbKSdBdtqg=; b=B77MCZTCmV/LES78gkkd/I7azMW5Bulr7fnrNQLyPsPXMdt+8wUdVBf63I90GwQolk V+ViuBKnGpyOoS6S1TG2ymIsB8/7EDakoCKWedv264vkHNm4LPFUvQOOfb3bfERl0nCF I/MIDZo1NE5CTnY1BDxwhpHHLVmLjDgGWRwiT/1ccFNmIFXk2aer1x1L0fOknMimz5ax FZ0XfrVM++T0tm3lHZpr9vMlToK1jCnbcQFx5L1E27VMsYLh3DxCJ67FIsuysUCu/OmN Rj6b5CbPcObBILGk6KLZ9kJu25qEbd+jA0ig9tUVKV88YGABoxiIzwDt7wiT6LRp5FMX iFcg== X-Forwarded-Encrypted: i=1; AJvYcCWH7/uOowwuykwxdAxqcDpQHz+JHEMNCaLk3/CsYcNtJwxnCspR4LrpQDbsZYhb7oflE2WAGS+jjIqb3IW9aDAOLFc=@vger.kernel.org, AJvYcCXs/lKVM+ViZNkCbFJmLvDV5TxpTdiiQIb2wbQVZ9nLK9Q/ubr872nvM9bPN5BFmQ3NHvc3Ryn19iZSct8=@vger.kernel.org X-Gm-Message-State: AOJu0YxM6DwSRouqhOCvX1+HQ42Oot16nP22V0DYweuaPMuzlPNKFdun fHtJ767sZjnHJzdoR0FVqgKF18rzolM+yqCXL/citGV005rdg1sj X-Gm-Gg: ASbGncvJbYsIeYyLRnUvbmc1QjdjphpLvV6kTZB+kYpum1uqrWOxMjba4725cmhPwPB xDS73OS2UAWFOczN9D702LJNNs10juYdMeCF3n7YWw0AMVckjYQ5YjsJK/AGDto/NaUKI0mT/ar CngTdGB/XNdqfNGyuvO98cWrhQWRjjFpKH5MGsjZLYixoQadGjgx7bAcA7rbrES6NmebW9cmzv9 y7V8bB8csjMkGi+1SegAhEUOsx4rcN/6b94plUXXxx8+uQCR5yd7tH9JYcDoc8MtvpHD6CdfaZ4 mDz165B0sxxBLFywkkrIoulMiL15qsIFJpcLTUaHgXGnR3GC1d9952ImhXwot9UZLD3gnQ== X-Google-Smtp-Source: AGHT+IGnt5id742CxRyj846Uuof7YiHdnbsgHm7Cy93tRKm15j6a6Hy36m/bf40XP4sXhp33LIkiLQ== X-Received: by 2002:a05:600c:34ca:b0:43d:49eb:9675 with SMTP id 5b1f17b1804b1-43ee0768b1cmr74357675e9.22.1744023010610; Mon, 07 Apr 2025 03:50:10 -0700 (PDT) Received: from iku.Home ([2a06:5906:61b:2d00:78b9:80c2:5373:1b49]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43ec3174cf0sm129975765e9.0.2025.04.07.03.50.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Apr 2025 03:50:09 -0700 (PDT) From: Prabhakar X-Google-Original-From: Prabhakar To: Greg Kroah-Hartman , Yoshihiro Shimoda , Geert Uytterhoeven , Kuninori Morimoto Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Prabhakar , Biju Das , Claudiu Beznea , Fabrizio Castro , Lad Prabhakar Subject: [PATCH v2 3/3] usb: renesas_usbhs: Reorder clock handling and power management in probe Date: Mon, 7 Apr 2025 11:50:02 +0100 Message-ID: <20250407105002.107181-4-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250407105002.107181-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20250407105002.107181-1-prabhakar.mahadev-lad.rj@bp.renesas.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Lad Prabhakar Reorder the initialization sequence in `usbhs_probe()` to enable runtime PM before accessing registers, preventing potential crashes due to uninitialized clocks. Currently, in the probe path, registers are accessed before enabling the clocks, leading to a synchronous external abort on the RZ/V2H SoC. The problematic call flow is as follows: usbhs_probe() usbhs_sys_clock_ctrl() usbhs_bset() usbhs_write() iowrite16() <-- Register access before enabling clocks Since `iowrite16()` is performed without ensuring the required clocks are enabled, this can lead to access errors. To fix this, enable PM runtime early in the probe function and ensure clocks are acquired before register access, preventing crashes like the following on RZ/V2H: [13.272640] Internal error: synchronous external abort: 0000000096000010 [#= 1] PREEMPT SMP [13.280814] Modules linked in: cec renesas_usbhs(+) drm_kms_helper fuse drm= backlight ipv6 [13.289088] CPU: 1 UID: 0 PID: 195 Comm: (udev-worker) Not tainted 6.14.0-r= c7+ #98 [13.296640] Hardware name: Renesas RZ/V2H EVK Board based on r9a09g057h44 (= DT) [13.303834] pstate: 60400005 (nZCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=3D-= -) [13.310770] pc : usbhs_bset+0x14/0x4c [renesas_usbhs] [13.315831] lr : usbhs_probe+0x2e4/0x5ac [renesas_usbhs] [13.321138] sp : ffff8000827e3850 [13.324438] x29: ffff8000827e3860 x28: 0000000000000000 x27: ffff8000827e3c= a0 [13.331554] x26: ffff8000827e3ba0 x25: ffff800081729668 x24: 00000000000000= 25 [13.338670] x23: ffff0000c0f08000 x22: 0000000000000000 x21: ffff0000c0f080= 10 [13.345783] x20: 0000000000000000 x19: ffff0000c3b52080 x18: 00000000ffffff= ff [13.352895] x17: 0000000000000000 x16: 0000000000000000 x15: ffff8000827e36= ce [13.360009] x14: 00000000000003d7 x13: 00000000000003d7 x12: 00000000000000= 00 [13.367122] x11: 0000000000000000 x10: 0000000000000aa0 x9 : ffff8000827e37= 50 [13.374235] x8 : ffff0000c1850b00 x7 : 0000000003826060 x6 : 00000000000000= 1c [13.381347] x5 : 000000030d5fcc00 x4 : ffff8000825c0000 x3 : 00000000000000= 00 [13.388459] x2 : 0000000000000400 x1 : 0000000000000000 x0 : ffff0000c3b520= 80 [13.395574] Call trace: [13.398013] usbhs_bset+0x14/0x4c [renesas_usbhs] (P) [13.403076] platform_probe+0x68/0xdc [13.406738] really_probe+0xbc/0x2c0 [13.410306] __driver_probe_device+0x78/0x120 [13.414653] driver_probe_device+0x3c/0x154 [13.418825] __driver_attach+0x90/0x1a0 [13.422647] bus_for_each_dev+0x7c/0xe0 [13.426470] driver_attach+0x24/0x30 [13.430032] bus_add_driver+0xe4/0x208 [13.433766] driver_register+0x68/0x130 [13.437587] __platform_driver_register+0x24/0x30 [13.442273] renesas_usbhs_driver_init+0x20/0x1000 [renesas_usbhs] [13.448450] do_one_initcall+0x60/0x1d4 [13.452276] do_init_module+0x54/0x1f8 [13.456014] load_module+0x1754/0x1c98 [13.459750] init_module_from_file+0x88/0xcc [13.464004] __arm64_sys_finit_module+0x1c4/0x328 [13.468689] invoke_syscall+0x48/0x104 [13.472426] el0_svc_common.constprop.0+0xc0/0xe0 [13.477113] do_el0_svc+0x1c/0x28 [13.480415] el0_svc+0x30/0xcc [13.483460] el0t_64_sync_handler+0x10c/0x138 [13.487800] el0t_64_sync+0x198/0x19c [13.491453] Code: 2a0103e1 12003c42 12003c63 8b010084 (79400084) [13.497522] ---[ end trace 0000000000000000 ]--- Fixes: f1407d5c66240 ("usb: renesas_usbhs: Add Renesas USBHS common code") Signed-off-by: Lad Prabhakar --- drivers/usb/renesas_usbhs/common.c | 50 +++++++++++++++++++++++------- 1 file changed, 38 insertions(+), 12 deletions(-) diff --git a/drivers/usb/renesas_usbhs/common.c b/drivers/usb/renesas_usbhs= /common.c index 703cf5d0cb41..f52418fe3fd4 100644 --- a/drivers/usb/renesas_usbhs/common.c +++ b/drivers/usb/renesas_usbhs/common.c @@ -685,10 +685,29 @@ static int usbhs_probe(struct platform_device *pdev) INIT_DELAYED_WORK(&priv->notify_hotplug_work, usbhsc_notify_hotplug); spin_lock_init(usbhs_priv_to_lock(priv)); =20 + /* + * Acquire clocks and enable power management (PM) early in the + * probe process, as the driver accesses registers during + * initialization. Ensure the device is active before proceeding. + */ + pm_runtime_enable(dev); + + ret =3D usbhsc_clk_get(dev, priv); + if (ret) + goto probe_pm_disable; + + ret =3D pm_runtime_resume_and_get(dev); + if (ret) + goto probe_clk_put; + + ret =3D usbhsc_clk_prepare_enable(priv); + if (ret) + goto probe_pm_put; + /* call pipe and module init */ ret =3D usbhs_pipe_probe(priv); if (ret < 0) - return ret; + goto probe_clk_dis_unprepare; =20 ret =3D usbhs_fifo_probe(priv); if (ret < 0) @@ -705,10 +724,6 @@ static int usbhs_probe(struct platform_device *pdev) if (ret) goto probe_fail_rst; =20 - ret =3D usbhsc_clk_get(dev, priv); - if (ret) - goto probe_fail_clks; - /* * device reset here because * USB device might be used in boot loader. @@ -721,7 +736,7 @@ static int usbhs_probe(struct platform_device *pdev) if (ret) { dev_warn(dev, "USB function not selected (GPIO)\n"); ret =3D -ENOTSUPP; - goto probe_end_mod_exit; + goto probe_assert_rest; } } =20 @@ -735,14 +750,19 @@ static int usbhs_probe(struct platform_device *pdev) ret =3D usbhs_platform_call(priv, hardware_init, pdev); if (ret < 0) { dev_err(dev, "platform init failed.\n"); - goto probe_end_mod_exit; + goto probe_assert_rest; } =20 /* reset phy for connection */ usbhs_platform_call(priv, phy_reset, pdev); =20 - /* power control */ - pm_runtime_enable(dev); + /* + * Disable the clocks that were enabled earlier in the probe path, + * and let the driver handle the clocks beyond this point. + */ + usbhsc_clk_disable_unprepare(priv); + pm_runtime_put(dev); + if (!usbhs_get_dparam(priv, runtime_pwctrl)) { usbhsc_power_ctrl(priv, 1); usbhs_mod_autonomy_mode(priv); @@ -759,9 +779,7 @@ static int usbhs_probe(struct platform_device *pdev) =20 return ret; =20 -probe_end_mod_exit: - usbhsc_clk_put(priv); -probe_fail_clks: +probe_assert_rest: reset_control_assert(priv->rsts); probe_fail_rst: usbhs_mod_remove(priv); @@ -769,6 +787,14 @@ static int usbhs_probe(struct platform_device *pdev) usbhs_fifo_remove(priv); probe_end_pipe_exit: usbhs_pipe_remove(priv); +probe_clk_dis_unprepare: + usbhsc_clk_disable_unprepare(priv); +probe_pm_put: + pm_runtime_put(dev); +probe_clk_put: + usbhsc_clk_put(priv); +probe_pm_disable: + pm_runtime_disable(dev); =20 dev_info(dev, "probe failed (%d)\n", ret); =20 --=20 2.49.0