On Wed, Jan 29, 2025 at 01:45:56AM +0200, Dmitry Baryshkov wrote:
> On Tue, Jan 28, 2025 at 05:11:16PM +0100, Maxime Ripard wrote:
> > On Sun, Jan 26, 2025 at 03:29:14PM +0200, Dmitry Baryshkov wrote:
> > > diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h
> > > index b55e80a57758e8b652eac0cd01cb245a04e221f5..d16af5fe90cb48f6671e798d9dee61a359c9233f 100644
> > > --- a/include/drm/drm_bridge.h
> > > +++ b/include/drm/drm_bridge.h
> > > @@ -920,6 +920,11 @@ struct drm_bridge {
> > > */
> > > int hdmi_audio_max_i2s_playback_channels;
> > >
> > > + /**
> > > + * @hdmi_audio_i2s_formats: supported I2S formats, optional
> > > + */
> > > + u64 hdmi_audio_i2s_formats;
> > > +
> >
> > We should document what the default value is if it's optional.
>
> "The default is to allow all formats supported by the corresponding
> I2S bus driver." Does that sound fine from your POV?
Yep, sounds good.
Maxime
Attachment:
signature.asc
Description: PGP signature
