Message ID | 20240229-anx7625-defer-log-no-dsi-host-v2-0-00506941049a@collabora.com |
---|---|
Headers |
Return-Path: <linux-kernel+bounces-87719-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:2097:b0:108:e6aa:91d0 with SMTP id gs23csp762002dyb; Thu, 29 Feb 2024 16:12:52 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCX3oJqO3j9ifZmT4U0DirFjQhxPHJ7atyUVI1uYl5Pi1/AddGwW9jpGQuaVGVWX2/R3HgayYj1E3sQCAwpNa+UAA/o5bw== X-Google-Smtp-Source: AGHT+IETAbJHSOspWbdIqd0CX0Eq0NJP8LrckM2sqtTrMiOKVTTYl5Z21feYNHThEw8DExZkD9Qi X-Received: by 2002:a05:6a20:6f8c:b0:19c:90fc:f0d3 with SMTP id gv12-20020a056a206f8c00b0019c90fcf0d3mr4901491pzb.46.1709251972193; Thu, 29 Feb 2024 16:12:52 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709251972; cv=pass; d=google.com; s=arc-20160816; b=n6kCr9b92ZM2cbN4bfNr9nnUnoF1G6jP7bVDrUOWwVWRA39jf1qKmyqnGWQY0e//Nj Y8wY7KwI1VmJLDqgWuqOpCcv2Q4URvkrQJUzU42KJUpZAHNLe3wISAbkJXu0voPtMY+O 1kcJyXDfO886YXXU7otxtnT/JsH4RdXsLXsVs1kUiZcSFcwKk8JGizkFB2QH3cJi+kEY noWZWrzdBlpeEWNIOcplkpSOcK24l0H8Nd6nx3fbW3uvVipOyA0c73ZdBJ4gLn3ZwKto 8y+Z4T0wR/lkUFd/Th3VT1xjUwwdAdgdz1eb/pytorq+BjPQLneqre1T4hQR4FOGXODs JhrQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:from :dkim-signature; bh=p4DvBvelzexXqNrweASumIOganguNqo7m/a9CN7DVN8=; fh=EPbsNJMLwlXYTg/MTuk9bwilsoK94uNp6NRacKCAfCI=; b=Z88m/x3aing8AcbF812bTEuupvtvhM+SO4cgEN8q7zPTYcy3D9ELjTuVlJKsVLz+B7 S2OyyVRegWoL01tXueMS0CY3mTrtVqbVGvXz+ZPhdES/t7utjxBKUqPvsoqqTM3vNhin +7y+ZrC8P/h7GuoxUVJYbE9K+Pedad18r/Pwu18n0Nw4KP+HwM8ekCg/DtR36EX9+ZwX d5J0Ac3dx06+R2duSi3LtdWczDhClw7pX32DQwFuKwRocY7u10olNbK5eYeXTJUTlEaR arjRLIPb0CHnz8PEnYRyOz/+2Mo8Fqudn3ilLx/j2byFewm4u3oC/NZitUt+2ToFcsgS xR+A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=D7Km4Ojv; arc=pass (i=1 spf=pass spfdomain=collabora.com dkim=pass dkdomain=collabora.com dmarc=pass fromdomain=collabora.com); spf=pass (google.com: domain of linux-kernel+bounces-87719-ouuuleilei=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-87719-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id d2-20020a63bd42000000b005dbec91be93si2344089pgp.595.2024.02.29.16.12.52 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Feb 2024 16:12:52 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-87719-ouuuleilei=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=D7Km4Ojv; arc=pass (i=1 spf=pass spfdomain=collabora.com dkim=pass dkdomain=collabora.com dmarc=pass fromdomain=collabora.com); spf=pass (google.com: domain of linux-kernel+bounces-87719-ouuuleilei=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-87719-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com 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 sv.mirrors.kernel.org (Postfix) with ESMTPS id 004A7282D34 for <ouuuleilei@gmail.com>; Fri, 1 Mar 2024 00:12:52 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 62BAB1C20; Fri, 1 Mar 2024 00:12:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="D7Km4Ojv" Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3CFE8193 for <linux-kernel@vger.kernel.org>; Fri, 1 Mar 2024 00:12:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=46.235.227.194 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709251945; cv=none; b=TbFYLB64S1pR/5sJ0PkdlBux6Q/Rw0UjdXyLsIbPXq2J5s8AS/L/9imVJL3mqdBNLbQsqjel11hAbxn1tQKygHlNlRf9oFDmiNGd+/j2B/lGg5adeOCN2W0A9llqcnphOBJZizfVy5rx3c+mTVSVPkiFndOLEgmcD++Uzn+WNiw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709251945; c=relaxed/simple; bh=ddNDXXuct86kt9fa1yOGg6EFSRsJaqUkVKYspL0iIzM=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=ttjzWlFM0kPUbfL09Df2ukv5nykhVXeY7evjLtcyAx30qCEKGdoL2NSiSGTce8Vhh5PgBAN7qagfZtCPnNL9MmjD/PemdXzaZc80A64bh9XxXWb9ch9n/s4xlyLcBN1JAQOSKp1EDl9epEVXDEQ6Fif66DYHLLBVrv3Q+bsowIs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=D7Km4Ojv; arc=none smtp.client-ip=46.235.227.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1709251935; bh=ddNDXXuct86kt9fa1yOGg6EFSRsJaqUkVKYspL0iIzM=; h=From:Subject:Date:To:Cc:From; b=D7Km4Ojv9tZIrYtyn+iT03V9mKDPM3/jf96hLHKZ2I7VHIJlrFZ2LNgA5CgKz+J0D 8JfN9NZDqnwudR/RkCWIAgWbCKmO8G2GKznUx3fXrtw8F1cDLviJmnMz5APg6BYUrv VGbqPYn4i3VgW/IWXZtltgImXg4QX90KUosiMqBMqg3qp0Zj1aQx0VoulGbmfmwSW1 N67HKyN6WTLX8imZP2NXsfDjmjXETOvgB/9Yz0Lts76bH5kq6nyiYOba0VfFStWSkL Xw/1FmRAFWmiV8FY9ZRAjCqXum9BhKGUivMyMovnjEdlAx54OfL+6IvykySSMEiKpj M815uNpg3QwaA== Received: from [192.168.1.205] (zone.collabora.co.uk [167.235.23.81]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: nfraprado) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 49B4A378000E; Fri, 1 Mar 2024 00:12:09 +0000 (UTC) From: =?utf-8?b?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= <nfraprado@collabora.com> Subject: [PATCH v2 0/9] drm: Switch from dev_err to dev_err_probe for missing DSI host error path Date: Thu, 29 Feb 2024 19:12:06 -0500 Message-Id: <20240229-anx7625-defer-log-no-dsi-host-v2-0-00506941049a@collabora.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-B4-Tracking: v=1; b=H4sIAFYd4WUC/43NQQ6CMBCF4auYrh1DBwR05T0Mi3aYQhPsmJYQD OHuVk7g8nuL/20qcfSc1P20qciLT15CBp5PikYTBgbfZysssCoQazBhbWq8Qs+OI0wyQBDok4d R0gxUuhuRJYtto3LjHdn59eg/u+zRp1ni57hb9G/9t7xo0IAVWm1cqdu2epBMk7ESzYXkpbp93 79ZrL/tzgAAAA== To: Andrzej Hajda <andrzej.hajda@intel.com>, Neil Armstrong <neil.armstrong@linaro.org>, Robert Foss <rfoss@kernel.org>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, Jonas Karlman <jonas@kwiboo.se>, Jernej Skrabec <jernej.skrabec@gmail.com>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, Maxime Ripard <mripard@kernel.org>, Thomas Zimmermann <tzimmermann@suse.de>, David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, owen <qwt9588@gmail.com>, Jagan Teki <jagan@amarulasolutions.com>, Marek Vasut <marex@denx.de>, Adrien Grassein <adrien.grassein@gmail.com>, Srinivas Kandagatla <srinivas.kandagatla@linaro.org>, Sam Ravnborg <sam@ravnborg.org>, Bjorn Andersson <andersson@kernel.org>, Vinod Koul <vkoul@kernel.org>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org>, Vinay Simha BN <simhavcs@gmail.com>, Christopher Vollo <chris@renewoutreach.org>, Jessica Zhang <quic_jesszhan@quicinc.com>, Marijn Suijten <marijn.suijten@somainline.org>, AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org> Cc: kernel@collabora.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, =?utf-8?b?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= <nfraprado@collabora.com> X-Mailer: b4 0.13.0 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1792280596160247561 X-GMAIL-MSGID: 1792280596160247561 |
Series |
drm: Switch from dev_err to dev_err_probe for missing DSI host error path
|
|
Message
Nícolas F. R. A. Prado
March 1, 2024, 12:12 a.m. UTC
This series changes every occurence of the following pattern: dsi_host = of_find_mipi_dsi_host_by_node(dsi); if (!dsi_host) { dev_err(dev, "failed to find dsi host\n"); return -EPROBE_DEFER; } into dsi_host = of_find_mipi_dsi_host_by_node(dsi); if (!dsi_host) return dev_err_probe(dev, -EPROBE_DEFER, "failed to find dsi host\n"); This registers the defer probe reason (so it can later be printed by the driver core or checked on demand through the devices_deferred file in debugfs) and prevents errors to be spammed in the kernel log every time the driver retries to probe, unnecessarily alerting userspace about something that is a normal part of the boot process. I have omitted a Fixes: tag in the last patch, for the truly-nt35597 panel, because it predates the dev_err_probe() helper. Changes in v2: - Added patches 2 onwards to fix all occurences of this pattern instead of just for the anx7625 driver - Link to v1: https://lore.kernel.org/r/20240226-anx7625-defer-log-no-dsi-host-v1-1-242b1af31884@collabora.com --- Nícolas F. R. A. Prado (9): drm/bridge: anx7625: Don't log an error when DSI host can't be found drm/bridge: icn6211: Don't log an error when DSI host can't be found drm/bridge: lt8912b: Don't log an error when DSI host can't be found drm/bridge: lt9611: Don't log an error when DSI host can't be found drm/bridge: lt9611uxc: Don't log an error when DSI host can't be found drm/bridge: tc358775: Don't log an error when DSI host can't be found drm/bridge: dpc3433: Don't log an error when DSI host can't be found drm/panel: novatek-nt35950: Don't log an error when DSI host can't be found drm/panel: truly-nt35597: Don't log an error when DSI host can't be found drivers/gpu/drm/bridge/analogix/anx7625.c | 6 ++---- drivers/gpu/drm/bridge/chipone-icn6211.c | 6 ++---- drivers/gpu/drm/bridge/lontium-lt8912b.c | 6 ++---- drivers/gpu/drm/bridge/lontium-lt9611.c | 6 ++---- drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 6 ++---- drivers/gpu/drm/bridge/tc358775.c | 6 ++---- drivers/gpu/drm/bridge/ti-dlpc3433.c | 17 +++++++++-------- drivers/gpu/drm/panel/panel-novatek-nt35950.c | 6 ++---- drivers/gpu/drm/panel/panel-truly-nt35597.c | 6 ++---- 9 files changed, 25 insertions(+), 40 deletions(-) --- base-commit: 2ae0a045e6814c8c1d676d6153c605a65746aa29 change-id: 20240226-anx7625-defer-log-no-dsi-host-c3f9ccbcb287 Best regards,
Comments
Hi Nícolas, On Thu, Feb 29, 2024 at 07:12:06PM -0500, Nícolas F. R. A. Prado wrote: > This series changes every occurence of the following pattern: > > dsi_host = of_find_mipi_dsi_host_by_node(dsi); > if (!dsi_host) { > dev_err(dev, "failed to find dsi host\n"); > return -EPROBE_DEFER; > } > > into > > dsi_host = of_find_mipi_dsi_host_by_node(dsi); > if (!dsi_host) > return dev_err_probe(dev, -EPROBE_DEFER, "failed to find dsi host\n"); > > This registers the defer probe reason (so it can later be printed by the > driver core or checked on demand through the devices_deferred file in > debugfs) and prevents errors to be spammed in the kernel log every time > the driver retries to probe, unnecessarily alerting userspace about > something that is a normal part of the boot process. The idea is good, but I have a small issue with patches 1/9 to 7/9. They all patch a function that is called by the probe function. Calling dev_err_probe() in such functions is error-prone. I had to manually check when reviewing the patches that those functions were indeed called at probe time, and not through other code paths, and I also had to check that no callers were using dev_err_probe() in the error handling path, as that would have overridden the error message. Would there be a way to move the dev_err_probe() to the top-level ? I understand it's not always possible or convenient, but if it was doable in at least some of the drivers, I think it would be better. I'll let you be the judge. > I have omitted a Fixes: tag in the last patch, for the truly-nt35597 > panel, because it predates the dev_err_probe() helper. > > Changes in v2: > - Added patches 2 onwards to fix all occurences of this pattern instead > of just for the anx7625 driver > - Link to v1: https://lore.kernel.org/r/20240226-anx7625-defer-log-no-dsi-host-v1-1-242b1af31884@collabora.com > > --- > Nícolas F. R. A. Prado (9): > drm/bridge: anx7625: Don't log an error when DSI host can't be found > drm/bridge: icn6211: Don't log an error when DSI host can't be found > drm/bridge: lt8912b: Don't log an error when DSI host can't be found > drm/bridge: lt9611: Don't log an error when DSI host can't be found > drm/bridge: lt9611uxc: Don't log an error when DSI host can't be found > drm/bridge: tc358775: Don't log an error when DSI host can't be found > drm/bridge: dpc3433: Don't log an error when DSI host can't be found > drm/panel: novatek-nt35950: Don't log an error when DSI host can't be found > drm/panel: truly-nt35597: Don't log an error when DSI host can't be found > > drivers/gpu/drm/bridge/analogix/anx7625.c | 6 ++---- > drivers/gpu/drm/bridge/chipone-icn6211.c | 6 ++---- > drivers/gpu/drm/bridge/lontium-lt8912b.c | 6 ++---- > drivers/gpu/drm/bridge/lontium-lt9611.c | 6 ++---- > drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 6 ++---- > drivers/gpu/drm/bridge/tc358775.c | 6 ++---- > drivers/gpu/drm/bridge/ti-dlpc3433.c | 17 +++++++++-------- > drivers/gpu/drm/panel/panel-novatek-nt35950.c | 6 ++---- > drivers/gpu/drm/panel/panel-truly-nt35597.c | 6 ++---- > 9 files changed, 25 insertions(+), 40 deletions(-) > --- > base-commit: 2ae0a045e6814c8c1d676d6153c605a65746aa29 > change-id: 20240226-anx7625-defer-log-no-dsi-host-c3f9ccbcb287
On Fri, Mar 01, 2024 at 08:34:31AM +0200, Laurent Pinchart wrote: > Hi Nícolas, > > On Thu, Feb 29, 2024 at 07:12:06PM -0500, Nícolas F. R. A. Prado wrote: > > This series changes every occurence of the following pattern: > > > > dsi_host = of_find_mipi_dsi_host_by_node(dsi); > > if (!dsi_host) { > > dev_err(dev, "failed to find dsi host\n"); > > return -EPROBE_DEFER; > > } > > > > into > > > > dsi_host = of_find_mipi_dsi_host_by_node(dsi); > > if (!dsi_host) > > return dev_err_probe(dev, -EPROBE_DEFER, "failed to find dsi host\n"); > > > > This registers the defer probe reason (so it can later be printed by the > > driver core or checked on demand through the devices_deferred file in > > debugfs) and prevents errors to be spammed in the kernel log every time > > the driver retries to probe, unnecessarily alerting userspace about > > something that is a normal part of the boot process. > > The idea is good, but I have a small issue with patches 1/9 to 7/9. They > all patch a function that is called by the probe function. Calling > dev_err_probe() in such functions is error-prone. I had to manually > check when reviewing the patches that those functions were indeed called > at probe time, and not through other code paths, and I also had to check > that no callers were using dev_err_probe() in the error handling path, > as that would have overridden the error message. > > Would there be a way to move the dev_err_probe() to the top-level ? I > understand it's not always possible or convenient, but if it was doable > in at least some of the drivers, I think it would be better. I'll let > you be the judge. Hey Laurent, thanks for the review. I get where you're coming from, as I checked those things myself while writing the patch. That said, I don't think moving dev_err_probe() to the top-level is a good move for a few reasons: * Keeping the log message as close to the source of the error makes it more specific, and consequently, more useful. * The original code already returned -EPROBE_DEFER, implying the function is expected to be called only from the probe function. With those points in mind, the only way I see to guarantee dev_err_probe(...,-EPROBE_DEFER...) would only be called by probe, and that the reason wouldn't be overriden, would be to move the entire code path of that function that calls into dev_err_probe() up into the probe function. But if we adopt this pattern consistently across the drivers in the tree, I think it would drastically worsen readability and cancel out the benefits. IMO the way forward with the API we have, is to make use of warnings and static checkers to catch cases where dev_err_probe() is overriding a defer probe reason, and where it's called outside of the probe function scope. So I'm inclined to leave the patches as they are, but am happy to discuss this further or other ideas. Thanks, Nícolas
On Fri, Mar 01, 2024 at 11:19:27AM -0500, Nícolas F. R. A. Prado wrote: > On Fri, Mar 01, 2024 at 08:34:31AM +0200, Laurent Pinchart wrote: > > Hi Nícolas, > > > > On Thu, Feb 29, 2024 at 07:12:06PM -0500, Nícolas F. R. A. Prado wrote: > > > This series changes every occurence of the following pattern: > > > > > > dsi_host = of_find_mipi_dsi_host_by_node(dsi); > > > if (!dsi_host) { > > > dev_err(dev, "failed to find dsi host\n"); > > > return -EPROBE_DEFER; > > > } > > > > > > into > > > > > > dsi_host = of_find_mipi_dsi_host_by_node(dsi); > > > if (!dsi_host) > > > return dev_err_probe(dev, -EPROBE_DEFER, "failed to find dsi host\n"); > > > > > > This registers the defer probe reason (so it can later be printed by the > > > driver core or checked on demand through the devices_deferred file in > > > debugfs) and prevents errors to be spammed in the kernel log every time > > > the driver retries to probe, unnecessarily alerting userspace about > > > something that is a normal part of the boot process. > > > > The idea is good, but I have a small issue with patches 1/9 to 7/9. They > > all patch a function that is called by the probe function. Calling > > dev_err_probe() in such functions is error-prone. I had to manually > > check when reviewing the patches that those functions were indeed called > > at probe time, and not through other code paths, and I also had to check > > that no callers were using dev_err_probe() in the error handling path, > > as that would have overridden the error message. > > > > Would there be a way to move the dev_err_probe() to the top-level ? I > > understand it's not always possible or convenient, but if it was doable > > in at least some of the drivers, I think it would be better. I'll let > > you be the judge. > > Hey Laurent, thanks for the review. > > I get where you're coming from, as I checked those things myself while writing > the patch. That said, I don't think moving dev_err_probe() to the top-level is a > good move for a few reasons: > * Keeping the log message as close to the source of the error makes it more > specific, and consequently, more useful. > * The original code already returned -EPROBE_DEFER, implying the function is > expected to be called only from the probe function. > > With those points in mind, the only way I see to guarantee > dev_err_probe(...,-EPROBE_DEFER...) would only be called by probe, and that the > reason wouldn't be overriden, would be to move the entire code path of that > function that calls into dev_err_probe() up into the probe function. But if we > adopt this pattern consistently across the drivers in the tree, I think it would > drastically worsen readability and cancel out the benefits. > > IMO the way forward with the API we have, is to make use of warnings and static > checkers to catch cases where dev_err_probe() is overriding a defer probe > reason, and where it's called outside of the probe function scope. > > So I'm inclined to leave the patches as they are, but am happy to discuss this > further or other ideas. Thanks for checking and having taken the time to explain your rationale. For the whole series, Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>