[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Z6I6tUa9W0jBbfqf@hovoldconsulting.com>
Date: Tue, 4 Feb 2025 17:05:09 +0100
From: Johan Hovold <johan@...nel.org>
To: Vikash Garodia <quic_vgarodia@...cinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
Abhinav Kumar <quic_abhinavk@...cinc.com>,
Krzysztof Kozlowski <krzk@...nel.org>,
Dikshita Agarwal <quic_dikshita@...cinc.com>, mchehab@...nel.org,
robh@...nel.org, krzk+dt@...nel.org, p.zabel@...gutronix.de,
hverkuil@...all.nl, sebastian.fricke@...labora.com,
bryan.odonoghue@...aro.org, neil.armstrong@...aro.org,
nicolas@...fresne.ca, u.kleine-koenig@...libre.com,
stefan.schmidt@...aro.org, lujianhua000@...il.com,
linux-arm-msm@...r.kernel.org, linux-media@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
krzysztof.kozlowski@...aro.org
Subject: Re: [RFC PATCH v10 1/2] media: iris: introduce helper module to
select video driver
On Tue, Feb 04, 2025 at 08:38:56PM +0530, Vikash Garodia wrote:
> On 2/4/2025 3:01 PM, Johan Hovold wrote:
> > And we're still waiting to hear the answers to all of Hans' questions. I
> > still haven't seen anyone explaining why any of this is needed. You
> > could have just continued letting Venus support 8250 so presumably there
> > is some benefit in using Iris instead. Which? And is that potential
> > benefit important enough to not just wait until Iris is up to par
> > feature wise?
> They are documented in this RFC [1] as comments, and would be added in v10 as we
> conclude the ongoing discussion to handle 2 drivers during the transition phase.
Ah, I forgot about the comment. Most of that should probably go in the
cover letter and commit messages of v10 instead (and with a more
condensed comment in the code), but it does indeed seem to provide some
answers.
> [1]
> https://patchwork.kernel.org/project/linux-media/patch/20250128080429.3911091-2-quic_dikshita@quicinc.com/
Johan
Powered by blists - more mailing lists