linux-kernel - Re: [PATCH 1/3] drm: bridge: adv7511: fill i2s stream capabilities

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <g4beidwfuz7dtavflyovtkokdz5kcjgbmnw3q2r2ceycezxrwd@wlsorpagke3h>
Date: Fri, 20 Dec 2024 03:30:06 +0200
From: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To: Olivier MOYSAN <olivier.moysan@...s.st.com>
Cc: Andrzej Hajda <andrzej.hajda@...el.com>, 
	Neil Armstrong <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>, 
	Laurent Pinchart <Laurent.pinchart@...asonboard.com>, Jonas Karlman <jonas@...boo.se>, 
	Jernej Skrabec <jernej.skrabec@...il.com>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, 
	Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>, 
	David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>, dri-devel@...ts.freedesktop.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] drm: bridge: adv7511: fill i2s stream capabilities

On Thu, Dec 19, 2024 at 03:49:45PM +0100, Olivier MOYSAN wrote:
> Hi Dmitry,
> 
> On 12/11/24 23:52, Dmitry Baryshkov wrote:
> > On Tue, Dec 10, 2024 at 02:42:52PM +0100, Olivier Moysan wrote:
> > > Set no_i2s_capture flag in hdmi_codec_pdata structure to report
> > > that the ADV7511 HDMI bridge does not support i2s audio capture.
> > > 
> > > Signed-off-by: Olivier Moysan <olivier.moysan@...s.st.com>
> > > ---
> > >   drivers/gpu/drm/bridge/adv7511/adv7511_audio.c | 1 +
> > >   1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c b/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c
> > > index 61f4a38e7d2b..28ae81ca3651 100644
> > > --- a/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c
> > > +++ b/drivers/gpu/drm/bridge/adv7511/adv7511_audio.c
> > > @@ -234,6 +234,7 @@ static const struct hdmi_codec_pdata codec_data = {
> > >   	.ops = &adv7511_codec_ops,
> > >   	.max_i2s_channels = 2,
> > >   	.i2s = 1,
> > > +	.no_i2s_capture = 1,
> > 
> > Does it support spdif capture?
> > 
> 
> "no_i2s_spdif = 1" is relevant also for adv7511 bridge.
> I can add it in this serie.

I think you mean no_spdif_playback / no_spdif_capture. Yes, please.

> 
> > >   	.spdif = 1,
> > >   };
> > > -- 
> > > 2.25.1
> > > 
> > 
> 
> BRs
> Olivier

-- 
With best wishes
Dmitry