lkml.org 
[lkml]   [2019]   [Aug]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v4 00/25] drm: Kirin driver cleanups to prep for Kirin960 support
    On Tue, Aug 20, 2019 at 1:01 PM Sam Ravnborg <sam@ravnborg.org> wrote:
    >
    > Hi John.
    >
    > On Mon, Aug 19, 2019 at 11:02:56PM +0000, John Stultz wrote:
    > > Sending this out again, to get it based on drm-misc-next.
    > >
    > > This patchset contains one fix (in the front, so its easier to
    > > eventually backport), and a series of changes from YiPing to
    > > refactor the kirin drm driver so that it can be used on both
    > > kirin620 based devices (like the original HiKey board) as well
    > > as kirin960 based devices (like the HiKey960 board).
    > >
    > > The full kirin960 drm support is still being refactored, but as
    > > this base kirin rework was getting to be substantial, I wanted
    > > to send out the first chunk, so that the review burden wasn't
    > > overwhelming.
    > >
    > > The full HiKey960 patch stack can be found here:
    > > https://git.linaro.org/people/john.stultz/android-dev.git/log/?h=dev/hikey960-mainline-WIP
    > >
    > > thanks
    > > -john
    > >
    > >
    > > New in v4:
    > > * Rebased to drm-misc-next, minor tweaks to merge changes
    > > * Dropped "drm: kirin: Get rid of drmP.h includes" as similar change
    > > was already in drm-misc next
    > > * Added acked-by tag from Xinliang
    >
    > There was some checkpatch noises in some of the patches - please verify
    > with "--strict".

    Ah. Apologies. I had not run with --strict.

    > And then the build failed like this:
    > LD [M] drivers/gpu/drm/hisilicon/kirin/kirin-drm.o
    > aarch64-linux-gnu-ld: drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.o: in function `init_module':
    > dw_drm_dsi.c:(.init.text+0x0): multiple definition of `init_module'; drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.o:kirin_drm_drv.c:(.init.text+0x0): first defined here
    > aarch64-linux-gnu-ld: drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.o: in function `cleanup_module':
    > dw_drm_dsi.c:(.exit.text+0x0): multiple definition of `cleanup_module'; drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.o:kirin_drm_drv.c:(.exit.text+0x0): first defined here
    > make[3]: *** [/home/sam/drm/linux.git/scripts/Makefile.build:464: drivers/gpu/drm/hisilicon/kirin/kirin-drm.o] Error 1
    > make[2]: *** [/home/sam/drm/linux.git/scripts/Makefile.build:490: drivers/gpu/drm/hisilicon/kirin] Error 2
    > make[1]: *** [/home/sam/drm/linux.git/Makefile:1776: drivers/gpu/drm/hisilicon/] Error 2
    > make[1]: Leaving directory '/home/sam/drm/linux.git/.build/arm64-allmodconfig'
    > make: *** [Makefile:179: sub-make] Error 2
    >
    > It was a simple allmodconfig build where I did:
    >
    > make drivers/gpu/drm/hisilicon/

    Yes, I've not used modules much with the board. I'll fix this up.

    > Please fix and resend. I did not look further.

    Apologies again, thanks so much for finding these issues! I'll get a
    new series to you shortly once the issues are resolved and I've
    validated things.

    thanks
    -john

    \
     
     \ /
      Last update: 2019-08-20 22:39    [W:5.717 / U:0.144 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site