From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f169.google.com (mail-pf1-f169.google.com [209.85.210.169]) (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 E8E273ACF0C for ; Mon, 10 Aug 2026 09:31:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786354308; cv=none; b=Y8eBwSSTf05ZU2st49K4UWuxvWhEm2H7SSnhPsKByJ2O6IQfqxA4gGrRjbWIuqsdK7GgClA8sMbTHSQVtlTJVEwdO8adzGJzLeWpLSojVO0iTwg6TGP51zok/52ai4+LWzTymkONEJyln9gESEuQT6j5uPeKdAVo2CLAV7JduhE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786354308; c=relaxed/simple; bh=2M2xeAgdsVXI/pfTvlb2yIeWwS0e9oCjUL8hkbV4lvI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=EI/O5B/MrIL2TNepDPGu6faqn6vhF55ctkDtHLuwJRHvqpk+3EckJ2gSPv0S2IH32Ny2wD8l0qwtG4FqJGzdHNm5p4AxAPGR49ZArtnuMQl7AA8VyaJuoETT4o1rTA+1ySm+o2M0f0/Wq3FlcPn8h21CRh/AVZ2x/yX1lgh54mw= 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=MxzzOma8; arc=none smtp.client-ip=209.85.210.169 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="MxzzOma8" Received: by mail-pf1-f169.google.com with SMTP id d2e1a72fcca58-84a2dcede83so2108372b3a.3 for ; Mon, 10 Aug 2026 02:31:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786354306; x=1786959106; 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:content-type; bh=cgq4ngVWmvwH/7fZSBUdZHTs+zKxzmXCAvxr+GPkjcI=; b=MxzzOma8iIMiccF03ORdokrS0HqdSLg2ketlyV+zsgBQMiYYCECMMyocd+S3uSW0pG fgIVFHn03CJcceVFI4t8gnvU2+Jy8PLztaGwDyZzSOXeHofJcoNDfopLdmcJtw+NImQJ DzOXap5jRA5UlHZNY89hZ/XZIBBpMZSMqoi9q9zxb+q/7UM1jmr5rp77MgmnEV3Sfv4P qlJ9zXQgg8W3mc9bjziFlSb+eTZjzNjv/wElnkDD7Cd0kf6MfB5VDnzlheWpmR5vaJNH A1PRsAxBw/UukY+Oufb0lgJtvGPeYSpHdhVnuOpFmaHyoCCnyeI/GP7kaPW1kfDrTUfa XAPg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786354306; x=1786959106; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=cgq4ngVWmvwH/7fZSBUdZHTs+zKxzmXCAvxr+GPkjcI=; b=U3ckVd3GWWnUyw8XKEk/3lCyR2GICIOH2kbaI/wYXWyBfC++Q/AM8+AmBFQeMwmI31 D389kaaJCYv2MBvpqGZ1x1NjXkrIvMNXZ9wpx6LAUy2lijc6WTmCDYq5+Wv5z9VmJtsX O6e6DfERO+NEavJMPOYuw6AyuSxy5jiIDMA09RCSLUXW47lP2LcOpc91NF8q2B3vI7q+ leOciZTMx4+Uoj3BHndIkpeYQfxzxKPgjpa4tfV+FacD9flL5mB7zoIRoI3NSdJUPS+m ChMQ75n1RHLUBCBukeCmUgZe6dIdq4Cn6DMiDMdK6AtCsmgMCWOnVhB4RHLVHSeVA/t0 wPeg== X-Forwarded-Encrypted: i=1; AHgh+RpcDOjPaPpD7jVaINrTvoKd+3pqxYn0dR0DHlM+8iN/5LgwsJLYpgdBUqrV7i1Z4/l9h6fHh+Wyp7yfgU4=@vger.kernel.org X-Gm-Message-State: AOJu0YxX4Tf0zFTY79okwSmTcP2zbQSeAuaMvFdek97q/0a+wMwJ3mNh r25q5uqf1FTwB6Gc/wx9WxZ796tnTOodKupTOUrA6KK4R3rtrBB4Ne1y X-Gm-Gg: AR+sD11zKsVZe/J9JKVgnOq+6+FLC50J7pBrev7UApG3t6DXB+q6g9JiJgoA8ZABEXV VrrkwdM6CO/zFN9wRCXPq+Cd2E6+uYkcVUYmZ+Yqxu3mwA1AiJz+PlDaiEkjzYxaPizqJgz+Krb AWqhIyqAvOIi4eRQN3C2QMwhMEVKILs03iMmDXWyQGgr/OctTmW4gICUUJZrehCOFhENyTsg2Sz v7VAMpGvavlSbK9LmUab+6DmULTl+Sa5ZGvxCIyxbKy4Y0c7O88exmWqlCVbd7FMdWbfFY1Wlv0 l1eItufOcUFtPEZyBJPDUpBvHzHCIeYB8jq5f3tiZu8ToEbyz9bLQC/akmH69nVml88TcObGqK6 ymSJpDPgU/1HYVDOqa82Zb0LX6Wvz90SEPSINsHc4WbnF/mbrNf9WSzrByJzQ2GHVJxT36ao2Jl BgrmLsXIB+FuEmXbFABOkJORcDrhswXyYpOJzyvJAn3e+zw3frecP8pWq8tNG1RcDwbqgh9R1pv HD1DRIAcaDf606l5AqlCwAPA1t4/4w8OO3VC2xtWmK8e8zyiJEtGr8KpI8q3MYDdgX5Ye8a+f/0 CfJu X-Received: by 2002:a05:6a00:3c81:b0:848:50a5:a70b with SMTP id d2e1a72fcca58-84f5e068579mr22565587b3a.21.1786354306343; Mon, 10 Aug 2026 02:31:46 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84f5a3fe7d6sm3592046b3a.26.2026.08.10.02.31.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 Aug 2026 02:31:46 -0700 (PDT) From: phucduc.bui@gmail.com To: Peter Chen , Greg Kroah-Hartman , Frank Li , Sascha Hauer , Pengutronix Kernel Team , linux-usb@vger.kernel.org Cc: Fabio Estevam , imx@lists.linux.dev, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, bui duc phuc , Sashiko Bot Subject: [PATCH v3 2/2] usb: chipidea: ci_hdrc_imx: Fix error handling when creating IRQ name Date: Mon, 10 Aug 2026 16:31:12 +0700 Message-ID: <20260810093112.47807-2-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260810093112.47807-1-phucduc.bui@gmail.com> References: <20260810093112.47807-1-phucduc.bui@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: bui duc phuc devm_kasprintf() can fail when allocating the wakeup IRQ name. The error was logged with dev_err_probe(), but its return value was ignored and probe could continue with a successful return value. Store the error returned by dev_err_probe() in ret and jump to the error path to propagate the failure. Reviewed-by: Frank Li Acked-by: Peter Chen Reported-by: Sashiko Bot Link: https://lore.kernel.org/all/20260806065755.C9F361F000E9@smtp.kernel.org/ Signed-off-by: bui duc phuc --- Link v2: https://lore.kernel.org/all/20260807065619.30604-2-phucduc.bui@gmail.com/ Changes in v3: - Use the phy_shutdown label instead of err_clk as suggested by Sashiko's review. - Add Peter Chen's Acked-by tag. - Add Frank Li's Reviewed-by tag. drivers/usb/chipidea/ci_hdrc_imx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c b/drivers/usb/chipidea/ci_hdrc_imx.c index 9e529398c607..edbf9df9b8f2 100644 --- a/drivers/usb/chipidea/ci_hdrc_imx.c +++ b/drivers/usb/chipidea/ci_hdrc_imx.c @@ -532,8 +532,8 @@ static int ci_hdrc_imx_probe(struct platform_device *pdev) if (data->wakeup_irq > 0) { irq_name = devm_kasprintf(dev, GFP_KERNEL, "%s:wakeup", pdata.name); if (!irq_name) { - dev_err_probe(dev, -ENOMEM, "failed to create irq_name\n"); - goto err_clk; + ret = dev_err_probe(dev, -ENOMEM, "failed to create irq_name\n"); + goto phy_shutdown; } ret = devm_request_threaded_irq(dev, data->wakeup_irq, @@ -541,7 +541,7 @@ static int ci_hdrc_imx_probe(struct platform_device *pdev) IRQF_ONESHOT | IRQF_NO_AUTOEN, irq_name, data); if (ret) - goto err_clk; + goto phy_shutdown; } ret = imx_usbmisc_init(data->usbmisc_data); -- 2.43.0