Re: [PATCH v7 6/7] drm: bridge/dw-hdmi: convert dw-hdmi to drm_bridge mode

[Andy Yan]

Hi ZubairLK:
On 2014å11æ11æ 22:36, Zubair Lutfullah Kakakhel wrote:
> Hi Andy,
>
> On 11/11/14 12:54, Andy Yan wrote:
> > From: Yakir Yang <ykk@xxxxxxxxxxxxxx>
> >
> > keep the connector & birdge in dw_hdmi.c, handle encoder in dw_hdmi-imx.c
> Is there a reason for this separation? Keeping encoder in platform files?
> If yes, then the commit message should explain that.
   in the encoder, most operation are platform specific, such as
  mux to select which crtc is used,  set panel format etc, these operation
  complete by platform specific interface
> > Signed-off-by: Andy Yan <andy.yan@xxxxxxxxxxxxxx>
> > Signed-off-by: Yakir Yang <ykk@xxxxxxxxxxxxxx>
> >
> > ---
> >
> > Changes in v7: None
> > Changes in v6:
> > - move some modification from patch#5
> >
> > Changes in v5: None
> > Changes in v4: None
> > Changes in v3: None
> > Changes in v2: None
> >
> >   drivers/gpu/drm/bridge/dw_hdmi.c      | 228 +++++++++++++++-------------------
> >   drivers/staging/imx-drm/dw_hdmi-imx.c | 145 ++++++++++++++-------
> >   include/drm/bridge/dw_hdmi.h          |  13 +-
> >   3 files changed, 199 insertions(+), 187 deletions(-)
>
> Regards
> ZubairLK
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-rockchip


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/