From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f51.google.com (mail-wm1-f51.google.com [209.85.128.51]) (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 A3BCB307AF4 for ; Fri, 3 Jul 2026 16:35:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783096510; cv=none; b=GqoQTrjuJAx/NNvtqpmzWIh9PhjdVT4w5ULfKPhcXgW0t/Y5DXmKEBJ9zrUoRVIvZJpk3CdeEHRzh0+1tMDiXdbvpP7aUjn+9JkXZx5Jz2RcuE8NSEMIrL8Ls+59OhJyFvQ2gVuI7xdlSNYGC0KRyiwLLUGzAXrQzM6tJ1SkIVA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783096510; c=relaxed/simple; bh=WdeqKTbBw0p9IN6WCIHfQW9SJlRN9D8ekRRYYDXPGYk=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=Hyob2H2VXdbwl8mL2HqEy7LLhh3bGL6A9MciQJChXER2/Ao+N6pJaB2wU2sJQPBmELxZRkCASusrD6vMB/38vxGYS3WNL2sOhfskMkVYoF8DzIWDt5fWXhs4VCVmZFa9Ilh6GsyNYq/mTXqNrewf5B0g9594R6leQKnVDj5ou98= 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=U0t6arJr; arc=none smtp.client-ip=209.85.128.51 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="U0t6arJr" Received: by mail-wm1-f51.google.com with SMTP id 5b1f17b1804b1-493c2c0b9a8so6078145e9.1 for ; Fri, 03 Jul 2026 09:35:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783096507; x=1783701307; 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:content-type; bh=ELIgzqdNSMVVWgszVoq642vcRbk3MvTzwUI4sM/DChA=; b=U0t6arJrWrNySAOsulmooDTgY5VjIUfVjjqwQsOXjXfAvMkaaFxeFE9zZTo54OGIxP VgzG1+XNNVydfLOQs6gIQjRv758XTsgIlUsHw1QrDxM/0mx/lQ6a/rqL8ICDbmv/LC05 q1X4IRwgLahxvKhgTHpRiGtMGKks4I950npo498a7A4SEkPUuuR/UpxxbAFX4cO5492k v8GgZKIUG7T1kn4B/My2R1UQmbEvzL7VVpAu1LWTlMvUCS8l1dfsXgDaCnTO1NsUglX0 cp92xE+jY3BSsJEDR91bRCbkeemadUQqBdQhP3oynQORvZ2lpmKQuXOMLkmMXQgLnWDU XObQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783096507; x=1783701307; 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:content-type; bh=ELIgzqdNSMVVWgszVoq642vcRbk3MvTzwUI4sM/DChA=; b=r+ksXwNHTKV4/iPxupGEHxe/C9vp0hic4Cg9IY8enM5XsifJIUB9QttpoK4aDfJhbi s5X3SSZUCg8+IdxRn7P58LCSBo+3vDpD/lyBUOsW8jrg0qwi6JGvz8b8LavdF1cAic12 AyP4gfJNlY1AlCdHCgH3s8U9M+/sjZoRMr7SM9NlkY/AOGLLGxo29zWtpdv24NJgU+bI UxXiRIlWgVwjmU0gR2JSbURDUblPHt6Ytnrr01XyozWaSWLQZCVBgrd7ZMB21MA3JCH2 7sdo10WRXXnEbMO8QYlu2JWPOAyeltIv3ROTcGFTbgI6/r75U2OVNYzVl30gqidhRw/b U6LA== X-Forwarded-Encrypted: i=1; AFNElJ8xYHLDY4FdZqIs5DPUkJNNMTep2x/61NJ052dnQJk6a6nuxFKTpWfYH7xprX0q01M4hiiLLA/3VxXXvHQ=@vger.kernel.org X-Gm-Message-State: AOJu0YwJkCmWm1C9k1OZ+PM2a2qlFLNuDEJrf4iqPvwKTC8jOO3GC0iK 1SUmvmQa8D0OGRhyQVWNreTIwek1TvAClqWnJMRfV6cmGvDo1qqTfc7F X-Gm-Gg: AfdE7cmQbUg73C4DPKafEreltLa3+yyIiwav06xtggHIOb8XQHvHtqpBTB6HcWWEqli dsMN2U94S717mi9mwfmjtARY+zRooQDGgIgfsqkcGTgSgUAhBFdbOR1MBYV+svRI2yZ3iaUAKZR nAoM7WQgl7VP8R2KIb1a/RMsAegGeyWhl8KZluilHe4TlCCiyMOmGAP+psvYP7pL1qG5B+RwpCq iVGVAjUSA/ddc3VSMV5G5jReKN/hOj3vkvPJ55Hug4kL8FeCLV/HfVKsOhmv141v/T7ghMBtozH GEHmJHGbr+QOwKzWCM0o2l1Pt3+iy6hAUHWJsZY8fzDb2Y/kfHCUQETFi/LD2M04Hj9MFKKayG9 VFBOSmyNgaxO0UffYRs5ULHUBK8i/XAJfliI8otibwe1ti0IINZNzIFoQg9q7As4qzz6PDmTCCB /AtPT3MRIS8NL+btEq5qCfDL1X68pTojGIcAaBX+2+wv+//48VAEHo9MU5TEfqfjpASIkoatEs X-Received: by 2002:a05:600c:314f:b0:493:cfd2:cd06 with SMTP id 5b1f17b1804b1-493d11cef20mr620455e9.6.1783096506606; Fri, 03 Jul 2026 09:35:06 -0700 (PDT) Received: from dohko.chello.ie (188-141-5-72.dynamic.upc.ie. [188.141.5.72]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-493cce040b4sm81986795e9.10.2026.07.03.09.35.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Jul 2026 09:35:06 -0700 (PDT) From: David Carlier To: dan.scally@ideasonboard.com, jacopo.mondi@ideasonboard.com, mchehab@kernel.org Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, David Carlier , stable@vger.kernel.org Subject: [PATCH v2] media: mali-c55: Fix clock leak on reset deassert failure Date: Fri, 3 Jul 2026 17:35:03 +0100 Message-ID: <20260703163503.715606-1-devnexen@gmail.com> X-Mailer: git-send-email 2.53.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 __mali_c55_power_on() enables the clocks before deasserting the resets, but bails out on a deassert failure without disabling them again. Both callers treat a failed power-on as already cleaned up, so the clocks are left enabled. Disable them on the error path. Fixes: d5f281f3dd29 ("media: mali-c55: Add Mali-C55 ISP driver") Cc: stable@vger.kernel.org Signed-off-by: David Carlier Reviewed-by: Daniel Scally Reviewed-by: Jacopo Mondi --- v2: - Add Cc: stable@vger.kernel.org (requested by Jacopo Mondi). - Collect Reviewed-by from Daniel Scally and Jacopo Mondi. drivers/media/platform/arm/mali-c55/mali-c55-core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-core.c b/drivers/media/platform/arm/mali-c55/mali-c55-core.c index ee4a42674..fb81141d1 100644 --- a/drivers/media/platform/arm/mali-c55/mali-c55-core.c +++ b/drivers/media/platform/arm/mali-c55/mali-c55-core.c @@ -699,6 +699,8 @@ static int __mali_c55_power_on(struct mali_c55 *mali_c55) mali_c55->resets); if (ret) { dev_err(mali_c55->dev, "failed to deassert resets\n"); + clk_bulk_disable_unprepare(ARRAY_SIZE(mali_c55->clks), + mali_c55->clks); return ret; } -- 2.53.0