From: Peter Marshall <pm@petermarshall.ca>
To: Krzysztof Kozlowski <krzk@kernel.org>
Cc: Sakari Ailus <sakari.ailus@linux.intel.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Benjamin Mugnier <benjamin.mugnier@foss.st.com>,
Sylvain Petinot <sylvain.petinot@foss.st.com>,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-media@vger.kernel.org, platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH 5/7] media: i2c: st-vd55g: Add generic driver and VD55G0 support
Date: Fri, 11 Sep 2026 23:16:40 +0000 [thread overview]
Message-ID: <35a47b6a-e3db-442d-916c-e8d279df3605@petermarshall.ca> (raw)
In-Reply-To: <20260903-gainful-mellow-mantis-f436cf@quoll>
On 9/3/26 1:41 PM, Krzysztof Kozlowski wrote:
> Isn't vd55g_parse_dt() printing errors already? How many times same error
> should be printed?
The original driver already prints duplicate errors in many places, even
before my refactoring. The next version of this series will clean that
up. Many error messages can be replaced with a dev_dbg call, and return
codes can be reported from the probe and PM handlers instead of helper
functions.
Peter Marshall
next prev parent reply other threads:[~2026-09-11 23:23 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-02 20:45 [PATCH 0/7] media: i2c: st-vd55g1: Genericize driver and add " Peter Marshall
2026-09-02 20:45 ` [PATCH 1/7] media: i2c: vd55g: move patch array to a separate firmware header Peter Marshall
2026-09-03 13:37 ` Krzysztof Kozlowski
2026-09-02 20:45 ` [PATCH 2/7] dt-bindings: media: i2c: st,vd55g: rename and add VD55G0 variant Peter Marshall
2026-09-03 13:36 ` Krzysztof Kozlowski
2026-09-02 20:45 ` [PATCH 3/7] media: ipu-bridge: Add VD55G0 to the list of supported sensors Peter Marshall
2026-09-02 20:45 ` [PATCH 4/7] platform/x86: int3472: Add VD55G0 supply GPIO mapping Peter Marshall
2026-09-02 20:45 ` [PATCH 5/7] media: i2c: st-vd55g: Add generic driver and VD55G0 support Peter Marshall
2026-09-03 13:41 ` Krzysztof Kozlowski
2026-09-11 23:16 ` Peter Marshall [this message]
2026-09-02 20:45 ` [PATCH 6/7] media: i2c: st-vd55g: Remove legacy vd55g1 remnants Peter Marshall
2026-09-03 13:43 ` Krzysztof Kozlowski
2026-09-02 20:45 ` [PATCH 7/7] MAINTAINERS: Update ST VD55G camera driver entry Peter Marshall
2026-09-03 13:43 ` Krzysztof Kozlowski
2026-09-04 12:06 ` [PATCH 0/7] media: i2c: st-vd55g1: Genericize driver and add VD55G0 support Benjamin Mugnier
[not found] ` <DL6TUYCM6AE9.3SFNKPI5T7CVS@petermarshall.ca>
2026-09-08 8:48 ` Benjamin Mugnier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=35a47b6a-e3db-442d-916c-e8d279df3605@petermarshall.ca \
--to=pm@petermarshall.ca \
--cc=benjamin.mugnier@foss.st.com \
--cc=devicetree@vger.kernel.org \
--cc=krzk@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=sylvain.petinot@foss.st.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®