From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f175.google.com (mail-pf1-f175.google.com [209.85.210.175]) (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 BE2A937F317 for ; Fri, 24 Jul 2026 10:08:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784887711; cv=none; b=O7ouLNc1xONu/emAkZViSZ++STIUamD2dgOnSNaCNon9sYVXDnd1w9Fj4kBiZSL++KTqvoujJAfNO/qn+7aksi5+APeV5ng8tC1UrnsEj/mHWJXQ63hBk6QwYOY8W607QiYmhsXkB51nv9KRFWynT8op7sRs3qX+gbtt9XXNah8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784887711; c=relaxed/simple; bh=1bmHWLYY47ZpNTpMtlAnInIGn8xlZDiBPGUnrf8wO54=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=H5fgWAPLD/c5GGpvxcxN9le7fdMhcH4lIRrgG1vdMXnKe4b2Uh+bdj8oFA863tfV2WXbPNJaHv+dYWdLiJYV0qZAJ8XyMSgKUu3YY9jKto6Af9/Q9ag0wdoIzhKFulfYRiVjqkObMzG1nuB0W/3e/f65FQfVxLzptJuusjbmc+k= 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=FRIDJCmb; arc=none smtp.client-ip=209.85.210.175 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="FRIDJCmb" Received: by mail-pf1-f175.google.com with SMTP id d2e1a72fcca58-848593533cbso136470b3a.3 for ; Fri, 24 Jul 2026 03:08:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784887710; x=1785492510; 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=GPKCvlhCvKQtHeiybNzrxgoDIHKyA0cPY2rVSugIzn4=; b=FRIDJCmbvC/zP5WB0qGoqhhWhmc4qTsp3hVHlVTDbgobODo6JzYtOLV0ETjfLoBK7V PG4QG+X4EsrOJBcedYzYL2BgmAj9hTLJ0K32U5NYGuUoOHPta0Ls/WGnThIqoyQ+xshL xYSXGyzONI7HPoF86HvWQPxX6g11fTqJm29ej6KY18K57n83EvcBvPeLKGqM7muVkOag 1lk0ekmHL6M8Bf+eZkOObjAFO86sDiIvHssmizH0Oefnb9oi6oNRER4ru4qRSuJ6jmcx sNYhGO1L1KpqP8WKLK5QI+aH6nhnLlKc9m1f/C5649lo4zPU1nXA++mH4OpDo2246hLc M2+w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784887710; x=1785492510; 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=GPKCvlhCvKQtHeiybNzrxgoDIHKyA0cPY2rVSugIzn4=; b=mYGckCsvTvIvutqhCSuZ+qYiL7Dk6mVIXJaXkS4ey0mLOFNr7Zl4auYoO7DtU5u6Rt JK66qP9NFmp4+Z9+wTmFbD/os4NyYOeDP4bxCTBHvuBxsWl9LXtzLn0Xpir4cUFvyzzw ss/m/BQxpnaRdKvHc5sN85BrjkO8EhkJN1Sy/X+c1XMyI2dYzbJW0ARnEJGANU+LIuzm c6sMz9o0AkTP1kWI9Xpdckv7BOLAN6P4qyM/gWhLwg5GqrSIEMMHmYB70ZfdDcwvxLaE OJzNxEt4sad/3xIxsovIElCrS+ZvZINrpBUlaSbiFypST0lAxGQ05ft2n1+rTBvDPz9B 9SmA== X-Forwarded-Encrypted: i=1; AHgh+Rqq2eDZJVHj6qwn+mKWnDA4BmcglDanWpBjfD1N61m1BEhn6jFHwCRADY3P2cXqA21oVn+YbEICVGVL3zU=@vger.kernel.org X-Gm-Message-State: AOJu0YwcuYg38oEkiMogredOeUiNeXEYYiVal6bxsIQuqgJvicKC4s5A C1Ywm0+if6E2IfNAMUl6gW3BCaz76z9UtfwUNaYJuYTNpiNPgKwbAAr3 X-Gm-Gg: AR+sD10rUaiIM+XoiDwWwgKg3HC+t4dh9eWvVSOz3CN2h7gzdexsMm7fytWFHT/tlBz fzzt68Tjq0KRZbODus9b71FPp7BS4vm2sZqKvK0InZhLY4D8ZM11zoL0wjn3V9Yj9riMOgTQ/HD yBNBOycFLQ1CuuJcgvmznx58MAOExTBcPJlABa+P2xqgCUhG4JNQrn1Z6q7xg2vffIggSgH+8vn CaKuqWMZZPhiGOtWBYqu8i+cHSdBCedijWhWK6GLwJk/HxQkW27VxwyXaWpTN01h1PtTknYjUU9 P7vokJSNrMFnLeDt83ghI3l3r49JkeX/gMwjgY1Xdlm8b/bHxxOB0SQPQ7EfJixrVAf3JL+w/4G IpUAuYWYulvZnHLVSmwmWhwxbEG8GSTA83555GKOF/lQB/EhCIUwfxcz4XIniL1OBah+/TkU2xm ZJH90wlrTaJPHmDG6RogezqcrsxaN47CKdzBGj3XPl1waGpan8RXXw59SSTPIQIY05RLt5Gwv69 UVPl5pde4M1eaBu/ktTcw== X-Received: by 2002:a05:6a00:1f0b:b0:84e:23a:14ba with SMTP id d2e1a72fcca58-84e2bba3b90mr7354559b3a.55.1784887710170; Fri, 24 Jul 2026 03:08:30 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84e175cdc21sm4405335b3a.47.2026.07.24.03.08.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 24 Jul 2026 03:08:29 -0700 (PDT) From: phucduc.bui@gmail.com To: Andrzej Hajda , Neil Armstrong Cc: Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Luca Ceresoli , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, bui duc phuc Subject: [PATCH 2/2] drm/bridge: adv7511: handle regcache_sync() failure on power on Date: Fri, 24 Jul 2026 17:08:03 +0700 Message-ID: <20260724100803.50169-2-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260724100803.50169-1-phucduc.bui@gmail.com> References: <20260724100803.50169-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 regcache_sync() failure in adv7511_power_on() is silently ignored, leaving the chip marked as powered on even though register synchronization may not have completed successfully. Log the error and attempt to power the chip back off on failure, to avoid the driver believing the chip is fully configured when it may not be. Signed-off-by: bui duc phuc --- drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c index d98a8f7afc7f..3e470366bce0 100644 --- a/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_drv.c @@ -367,12 +367,20 @@ static void __adv7511_power_on(struct adv7511 *adv7511) static void adv7511_power_on(struct adv7511 *adv7511) { + int ret; + __adv7511_power_on(adv7511); /* * Most of the registers are reset during power down or when HPD is low. */ - regcache_sync(adv7511->regmap); + ret = regcache_sync(adv7511->regmap); + if (ret) { + dev_err(&adv7511->i2c_main->dev, + "Failed to sync register cache: %d\n", ret); + __adv7511_power_off(adv7511); + return; + } if (adv7511->info->has_dsi) adv7533_dsi_power_on(adv7511); -- 2.43.0