Received: by 2002:a05:7412:8d1c:b0:fa:4c10:6cad with SMTP id bj28csp631228rdb; Wed, 17 Jan 2024 12:18:12 -0800 (PST) X-Google-Smtp-Source: AGHT+IEP92Gbl3/rSMr5PoydSWqR9iYStwaSe7HyOEeBtaM5zx7vm8f0BvIV5NWFAvTcc6yE5LXh X-Received: by 2002:a05:6a00:6ca3:b0:6d9:e2b7:4e2 with SMTP id jc35-20020a056a006ca300b006d9e2b704e2mr2097004pfb.11.1705522691717; Wed, 17 Jan 2024 12:18:11 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1705522691; cv=pass; d=google.com; s=arc-20160816; b=JL8vVZSwIiDJNMJvjDOLvkTctLfndpapN8DvfGh8KZbIfZ1pmohiejHGOWmb8emCrZ OrserNsYMB6IYpj4VQNuicI2Jxc2nxtLX9pN7SL5hNAjH7FaLsFmheVGj1FOzNVnL3Vn DQ3MToinXJ/MrhtkLcSVhOHfzuQjTWzRGQzRzn7zRZ8DM6CGDlqB1LuVU2V4qklmXYfz pNZbYXCPCplcO5SLBt+kuhX3wcRyqzmXxi1PDJZqwPLAB5sMMpg2IdXjdZY7tWRMsPc9 4ph+Yfz11fl3XoQZpcyctz0M5a/6lmuhai3hwduMroK/7Bbsz2664JFa4c9ks6DYEejA GRNg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:dkim-signature; bh=9EN9U2M6WcTO4kvqAssI2USJeGX9gxivbYmkNOhif20=; fh=w74rxpuMPkrQrVK4leDz9Wuj1gIWDKziT6acPTbR8XY=; b=TxcSwLS/rv3OaGbasvTrGCnw5Hqw3BJZz9zqdJG2p+ZHHofNPWvwVAlydlC+xcQYLn Dh1xp9iiSxr7nhRGc1F5huJRuzNPDE7GT4Y+WeIjbHvPawCzMsFmjx+StJCsuwr0gEUO d6QsByTwudjQZJ3Jl+xlaF8vLQ9yUIOfvYRyHxm+vEPol9qTSK8KC0qGPCIJd02nuW/4 p4vvMQJUrXeuVyjmj3ulHXEk5beSf9qG5HPFDtYC+eWHpKOLlCmeyzxi23s4q3Ctrqbw ppPgcaPlRk7t9uWMcyvLapBIh+oaE7m1TyfpLrhB5J1CkvM3ueVjCpJshqKgdXSx1WqO DxWQ== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=St78sYYf; arc=pass (i=1 spf=pass spfdomain=chromium.org dkim=pass dkdomain=chromium.org dmarc=pass fromdomain=chromium.org); spf=pass (google.com: domain of linux-kernel+bounces-29407-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-29407-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [2604:1380:40f1:3f00::1]) by mx.google.com with ESMTPS id dw7-20020a056a00368700b006d9e350111esi2444260pfb.30.2024.01.17.12.18.11 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 17 Jan 2024 12:18:11 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-29407-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=St78sYYf; arc=pass (i=1 spf=pass spfdomain=chromium.org dkim=pass dkdomain=chromium.org dmarc=pass fromdomain=chromium.org); spf=pass (google.com: domain of linux-kernel+bounces-29407-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-29407-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id A60A0B24D0A for ; Wed, 17 Jan 2024 20:18:06 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E62E924B2F; Wed, 17 Jan 2024 20:17:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="St78sYYf" Received: from mail-ej1-f41.google.com (mail-ej1-f41.google.com [209.85.218.41]) (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 93CBE24B25 for ; Wed, 17 Jan 2024 20:17:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705522678; cv=none; b=IKMwvd5TIJ0esDU2OHCnkErEgOGzyXrL8BpiAah8slisK7+qk8Fc23geXX5Cyq4DNoXDSPIUZx/E/jcc3EbWZ6omO42mbOYuL1VXpRyJXu97nr3sCBFpgVMYomYnpfY2w8oXI2B+HXA4dWX+7247wZFlJaucPNzgc0aS4NuJ0pc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705522678; c=relaxed/simple; bh=9EN9U2M6WcTO4kvqAssI2USJeGX9gxivbYmkNOhif20=; h=Received:DKIM-Signature:X-Google-DKIM-Signature: X-Gm-Message-State:X-Google-Smtp-Source:X-Received:Received: Received:X-Received:MIME-Version:References:In-Reply-To:From:Date: X-Gmail-Original-Message-ID:Message-ID:Subject:To:Cc:Content-Type: Content-Transfer-Encoding; b=WWA5xEUe2x+NosCwMByOsdXPMhIBxaXd6ycYKKkJCM5JSK1GPN54KzqBEBuWUbnOmyrVWs40jwWZPM+PLeem9XEjpti3izdAwHQMGUfSe4mqm31jXC7iGsswafTrylK+0vG0iXSHIWto1oWVS0gzBwA4KLo47eW67CDOUqmXKQc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=St78sYYf; arc=none smtp.client-ip=209.85.218.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Received: by mail-ej1-f41.google.com with SMTP id a640c23a62f3a-a28a997f3dfso864075366b.0 for ; Wed, 17 Jan 2024 12:17:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1705522674; x=1706127474; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=9EN9U2M6WcTO4kvqAssI2USJeGX9gxivbYmkNOhif20=; b=St78sYYf3ILU3GQR9LqxMbrE0QhHcjeCHivVbu0oX7sWqNidUXdE09uUHvi5d99qR4 pDL40J0lPiMADN4f1u03r0nIss6ePtvf0l1H9fc9pGmU3+4WxB+OVezYnxcQtMsx/LeG 76ji4WGMr8SjOAlvt9iBdP41swiCpBbm4W7zI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1705522674; x=1706127474; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=9EN9U2M6WcTO4kvqAssI2USJeGX9gxivbYmkNOhif20=; b=dz0umTT5A8R74xmOvFjpfzr/8S2GvdGCLy+tsmwNEwETL+IOvplPngWkKeRrzOVeYG bpPtNVxsmsB9Ljo1UrvSBQXKy2nnfu7oTUUywGtYIZpriYM/7WqfrkBZpifoqxhK7x6+ TG2uKccuUlHB2r/B/CSgLplV4QkAnOdKU+B4wMj+JMu7lRp6XlFRlAAMQgbzsRrZS2Gy fJZX0x3Ag8P1e1cQKwxnF9h+2rjwv9JFjV5qDIYXpkxwF/jd+8I+D9Wl0FinWy1yF8q7 PR/kIG23wmUjCGeq/Q6h8huTWwzVEyMLJvuR9VQZb8SctEZ516vU2PjBo5B3eSDLVnqk dCPg== X-Gm-Message-State: AOJu0YzT6G4EagjuXUSL+J80kQOUNv/7VerdtioH5S0CWjyEHQ9m6N4A R8m/HK9hHCWBHcrJVsKQ8j23Lbn6qx2+CrMDgVgt/fJ/ddOZhXM= X-Received: by 2002:a17:906:fd55:b0:a28:e3e9:bc8f with SMTP id wi21-20020a170906fd5500b00a28e3e9bc8fmr4074896ejb.32.1705522674547; Wed, 17 Jan 2024 12:17:54 -0800 (PST) Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com. [209.85.128.49]) by smtp.gmail.com with ESMTPSA id l9-20020a1709066b8900b00a2c34d90a48sm8152309ejr.138.2024.01.17.12.17.54 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 17 Jan 2024 12:17:54 -0800 (PST) Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-40e43e55b87so70615e9.0 for ; Wed, 17 Jan 2024 12:17:54 -0800 (PST) X-Received: by 2002:a05:600c:1c8d:b0:40e:5274:e2ba with SMTP id k13-20020a05600c1c8d00b0040e5274e2bamr205912wms.4.1705522191757; Wed, 17 Jan 2024 12:09:51 -0800 (PST) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240117103502.1.Ib726a0184913925efc7e99c4d4fc801982e1bc24@changeid> In-Reply-To: From: Doug Anderson Date: Wed, 17 Jan 2024 12:09:37 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] drm/bridge: parade-ps8640: Make sure we drop the AUX mutex in the error case To: Hsin-Yi Wang Cc: dri-devel@lists.freedesktop.org, Andrzej Hajda , Daniel Vetter , David Airlie , Jernej Skrabec , Jonas Karlman , Laurent Pinchart , Maarten Lankhorst , Maxime Ripard , Neil Armstrong , Pin-yen Lin , Robert Foss , Thomas Zimmermann , linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi, On Wed, Jan 17, 2024 at 11:39=E2=80=AFAM Hsin-Yi Wang = wrote: > > On Wed, Jan 17, 2024 at 10:35=E2=80=AFAM Douglas Anderson wrote: > > > > After commit 26db46bc9c67 ("drm/bridge: parade-ps8640: Ensure bridge > > is suspended in .post_disable()"), if we hit the error case in > > ps8640_aux_transfer() then we return without dropping the mutex. Fix > > this oversight. > > > > Fixes: 26db46bc9c67 ("drm/bridge: parade-ps8640: Ensure bridge is suspe= nded in .post_disable()") > > Signed-off-by: Douglas Anderson > > --- > > Sorry for missing this in my review! :( Given that this is really > > simple and I'd rather the buggy commit not be there for long, if I can > > get a quick Reviewed-by tag on this patch I'll land it without the > > typical stewing period. > > > > Reviewed-by: Hsin-Yi Wang Thanks! I've pushed this to avoid the breakage. If any additional follow up comes up I'm happy to post additional patches. a20f1b02bafc drm/bridge: parade-ps8640: Make sure we drop the AUX mutex in the error case -Doug