[PATCH v4] drm: bridge: add DesignWare HDMI I2S audio support

Kuninori Morimoto kuninori.morimoto.gx at renesas.com
Wed Nov 9 00:47:36 UTC 2016


Hi Russell

> > @@ -11,4 +11,11 @@ struct dw_hdmi_audio_data {
> >  	u8 *eld;
> >  };
> >  
> > +struct dw_hdmi_i2s_audio_data {
> > +	struct dw_hdmi *hdmi;
> > +
> > +	void (*write)(struct dw_hdmi *hdmi, u8 val, int offset);
> > +	u8 (*read)(struct dw_hdmi *hdmi, int offset);
> > +};
> 
> Another slight concern here is why we need this separate data for i2s -
> maybe adding the write/read ops to struct dw_hdmi_audio_data would be
> a better thing to do, which would then allow the AHB audio to drop
> the .base argument in the future.
> 
> I'm not that bothered about this though.  So...

Thank you for your feedback.
Maybe we can merge these in incremental patch if needed ?



More information about the dri-devel mailing list