From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) (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 BCFFF38E8DF for ; Thu, 14 May 2026 11:06:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778756781; cv=none; b=YucE4vwm1hObssCflDnSgvkf1ytxrUAYMKMzUEn/tcWTWYMrWiTPDU02Prk+Nyt3TNBLxjfMb3YWcmCuG8N/AQj/lObvoCdAfgdlBeTiyKfSjMCtTi3kQBXDUR6FGQ+1MM7K7oT90Y9mS3eywZ3OuuHkxcb8a2jDt+8zRUJzvZ0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778756781; c=relaxed/simple; bh=47sog5D+xEQbGjajYDAtYcSG4JoXZ0powok8EU5fFOQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=lqGfX7aq+14lVMY/wXz3sBKFDJOkowWpNYYACybFAeZdXYtFOID2INTE9+KPjcVIsjXnxqZlMyH5qNjNUHm3vkVlWapl829x3DJMUSj4tt7vBlt+kYiquuvLGK1k1GMKBTK9JNwmlShdgjriKE0g535R4rW2P47rtwmxPST0ruk= 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=pJtW8UzZ; arc=none smtp.client-ip=209.85.214.176 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="pJtW8UzZ" Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-2bc85eda6b6so26404535ad.1 for ; Thu, 14 May 2026 04:06:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778756779; x=1779361579; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=umazs7xdzJh6gNqfVRcGHxjaKuxZgvm9MNq62f5KWtU=; b=pJtW8UzZgz5k0ZjtC/ENWGBiuQ17rpmscP0qT8I2/AysF7pOGrWlVggiEqA7q1zi3N D02Z+mDetFAHRz97v8PkHQYWsibQ+equM30NWp0yUYZvMNB3KeLSRE92V3XWJB4y5dq2 +0V/qMFQ7PIQCi7HNezEUDSkRnwN2VQSsNsXPo2pBTnT8Ya97yja8Lb/HE2prZLPJPP1 r/U4kfhrMEfGBR7NA6+Yflsg5sQEm5T0lxj88pPfGRPMeg0Nv9mOVnZ/cjk/QZHdHOc0 KXWRRMp5KSjDbx6gdCqlCFjoCBHSK8WMehXzjGPpD3SzurRC9h74sgwpxPw+dJRlnhyL 4w9g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778756779; x=1779361579; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=umazs7xdzJh6gNqfVRcGHxjaKuxZgvm9MNq62f5KWtU=; b=CH2fx4MBrvtIt+3zj+DgsfTlh8TVyFaR5su/UL967Fu/z1ipzCSLi/rdqJ+K/hgb9L lNPMsj7H6ny8iJwN2eP4FdRZmoIFoq4KAZtKXPEHa/QH4z+Pgr3i9o7AEH82YBbKSZpR xsbrqbZyCDMG3Kepx+NfFBVkEZCuUiFraC5A8DbTzBuPex1n1QWFEhJUWmJF5ZI1F68x ISNjqJtNQSlyN6xD4OJfkb3W6zquOWRhfX1qbbVdBUZW4Hq8zKG4L6rob2STl/XsM9YW hok2osHe9+fda2XiVwZp4edL3K/s9Sltdwtwh92nbXUFrIJNq7Zja9mVxjTFGYjYEh9M HjsQ== X-Forwarded-Encrypted: i=1; AFNElJ8rXQGlXEmS+uM0UMW+dAO3cULJEGzM7dJmnHAk24lrOhs5zlT0UyAGNLoRzwlK+7LlSMCfaWOOCBj4ZWY=@vger.kernel.org X-Gm-Message-State: AOJu0YwHLZxnttfpt3oBtqRx4jrqkfu92RHbYDQClxXJsaHFLd1Nf7zZ m6UP40e6lk0uelZAqvATytFk2iSxupfU0LUNEcUZPYQS9xWYtNn6dMq5 X-Gm-Gg: Acq92OEtkQOKAfDaAsxvC0OIV8Kmxmy3s1EjNvfB7lGB07SBumqKqr6t8Y+85bIrPON r5pkWQjlB0zMwe7mbOi4TTe2d+i1i/T2QhZXGkWoDwBtKHRKaDesIlEOJcxmPAB0CYs0X6gn5+T X8idFkZLeMJhVh60dQohaiYXJMchNvL8DScwjsHz60GK04FFua57BcYydV+wNT/zxBTuUDPiiEr G/36Rjk6slMTsU8lxTwDKnFrdwd/JhVCUqAWGObBpgLzWDKIPBSWNYW4TC49QZwu/+EPZHCFNF/ JZNlB4EoCq0RBfl6oPpuIKrpcqq8d2Bdyb3jYjBgNpW5U7XqVB/KdVgDdyt8DzKGjTfF3gOOvoz eIDEbGOIewJ/oEZcd6G4O5UNWI3B+Q1+8wf/i3eOIiToZOhYsxyPHFfTcxevucUzSmCO9nRRwag zu5bDgjNRG7v6x7RM/+tspDqMVsyxEby/fbq6rVEdJFrGBjl9+GudDwlOPkA== X-Received: by 2002:a17:902:d392:b0:2ba:7610:3f21 with SMTP id d9443c01a7336-2bd270fbe07mr63185415ad.5.1778756778730; Thu, 14 May 2026 04:06:18 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2bd5c0600b4sm23053735ad.28.2026.05.14.04.06.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 14 May 2026 04:06:18 -0700 (PDT) From: phucduc.bui@gmail.com To: Mark Brown Cc: Peter Ujfalusi , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , asahi@lists.linux.dev, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, bui duc phuc Subject: [PATCH] ASoC: ti: omap-mcbsp: Remove mixed goto/scoped cleanup handling Date: Thu, 14 May 2026 18:06:02 +0700 Message-ID: <20260514110602.30480-1-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 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 After converting to guard()/scoped_guard() helpers, omap_mcbsp_request() still mixes scoped cleanup with goto based error handling. Remove the remaining goto based cleanup paths for a more consistent cleanup flow. Suggested-by: Mark Brown Signed-off-by: bui duc phuc --- sound/soc/ti/omap-mcbsp.c | 46 +++++++++++++++++++-------------------- 1 file changed, 22 insertions(+), 24 deletions(-) diff --git a/sound/soc/ti/omap-mcbsp.c b/sound/soc/ti/omap-mcbsp.c index d82fef629867..26af616c33f5 100644 --- a/sound/soc/ti/omap-mcbsp.c +++ b/sound/soc/ti/omap-mcbsp.c @@ -307,7 +307,7 @@ static int omap_mcbsp_request(struct omap_mcbsp *mcbsp) reg_cache = NULL; } - if(mcbsp->pdata->ops && mcbsp->pdata->ops->request) + if (mcbsp->pdata->ops && mcbsp->pdata->ops->request) mcbsp->pdata->ops->request(mcbsp->id - 1); /* @@ -322,42 +322,40 @@ static int omap_mcbsp_request(struct omap_mcbsp *mcbsp) "McBSP", (void *)mcbsp); if (err != 0) { dev_err(mcbsp->dev, "Unable to request IRQ\n"); - goto err_clk_disable; } } else { err = request_irq(mcbsp->tx_irq, omap_mcbsp_tx_irq_handler, 0, "McBSP TX", (void *)mcbsp); if (err != 0) { dev_err(mcbsp->dev, "Unable to request TX IRQ\n"); - goto err_clk_disable; - } - - err = request_irq(mcbsp->rx_irq, omap_mcbsp_rx_irq_handler, 0, - "McBSP RX", (void *)mcbsp); - if (err != 0) { - dev_err(mcbsp->dev, "Unable to request RX IRQ\n"); - goto err_free_irq; + } else { + err = request_irq(mcbsp->rx_irq, omap_mcbsp_rx_irq_handler, 0, + "McBSP RX", (void *)mcbsp); + if (err != 0) { + dev_err(mcbsp->dev, "Unable to request RX IRQ\n"); + free_irq(mcbsp->tx_irq, (void *)mcbsp); + } } } - return 0; -err_free_irq: - free_irq(mcbsp->tx_irq, (void *)mcbsp); -err_clk_disable: - if(mcbsp->pdata->ops && mcbsp->pdata->ops->free) - mcbsp->pdata->ops->free(mcbsp->id - 1); + if (err != 0) { + if (mcbsp->pdata->ops && mcbsp->pdata->ops->free) + mcbsp->pdata->ops->free(mcbsp->id - 1); - /* Disable wakeup behavior */ - if (mcbsp->pdata->has_wakeup) - MCBSP_WRITE(mcbsp, WAKEUPEN, 0); + /* Disable wakeup behavior */ + if (mcbsp->pdata->has_wakeup) + MCBSP_WRITE(mcbsp, WAKEUPEN, 0); - scoped_guard(spinlock, &mcbsp->lock) { - reg_cache = mcbsp->reg_cache; - mcbsp->free = true; - mcbsp->reg_cache = NULL; + scoped_guard(spinlock, &mcbsp->lock) { + reg_cache = mcbsp->reg_cache; + mcbsp->free = true; + mcbsp->reg_cache = NULL; + } + + return err; } - return err; + return 0; } static void omap_mcbsp_free(struct omap_mcbsp *mcbsp) -- 2.43.0