lkml.org 
[lkml]   [2018]   [Jul]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 12/12] drm/rockchip: add support for CDNS MHDP IP controller.
    Date
    Hi Damien,

    it's very cool to see collaboration from vendors on this.


    Am Dienstag, 3. Juli 2018, 12:02:23 CEST schrieb Damian Kos:
    >

    It would be really nice to explain a bit about the added controller support
    in the commit message, so that people reviewing the patch can get a
    feeling for it.

    > Signed-off-by: Damian Kos <dkos@cadence.com>
    > ---
    > drivers/gpu/drm/rockchip/cdn-dp-core.c | 953
    > +++++++++++++++++++++++++++++++- drivers/gpu/drm/rockchip/cdn-dp-core.h |
    > 25 +
    > drivers/gpu/drm/rockchip/cdn-dp-reg.c | 2 +-
    > drivers/gpu/drm/rockchip/cdn-dp-reg.h | 4 +

    From the changes below, it looks that this seems to add support for a
    bridge chip based on that IP block. So it seems like the bridge+glue driver
    model would be a perfect fit for this, instead of stapling this onto the
    Rockchip-specific driver.

    So essentially, you could take the Rockchip cdn-dp driver, move the common
    parts to drivers/gpu/drm/bridge and then create separate glue drivers for
    both Rockchip and your external bridge IP block.

    This would prevent code duplication and also allow your bridge driver to
    be compiled without the Rockchip drm being present :-) .
    And also pave the way for future socs using your DP ip block.


    Nowadays we have quite a number of examples you could take as
    inspiration for this:
    - bridge/analogix/* (shared between Exynos and Rockchip right now)
    - bridge/synopsys/dw-hdmi* (shared between a quite big number of users)
    - bridge/synopsys/dw-mipi-dsi.c (shared between Rockchip [pending] and stm)


    Thanks
    Heiko



    \
     
     \ /
      Last update: 2018-07-03 13:04    [W:3.261 / U:0.192 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site