lkml.org 
[lkml]   [2016]   [Nov]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v4] drm: bridge: add DesignWare HDMI I2S audio support
Date

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 ?

\
 
 \ /
  Last update: 2016-11-09 01:48    [W:0.117 / U:0.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site