Re: [PATCH v3 0/1] drm/exynos: drop use of drmP.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 21 Jun 2019 at 20:49, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote:
>
> Build tested using allyesconfig, allmodconfig for various
> architectures.

Hi,

Thanks for fixes. Just for the record, allyesconfig/allmodconfig are
not a proper way for build test specific drivers. They are nice but
since all dependencies are in, they might miss a lot. You should build
mentioned platform (exynos_defconfig on ARMv7, defconfig on ARMv8) and
multi configs (multi_v7 on ARMv7). Such rule is also valid for other
drivers, not specific to Exynos.

Best regards,
Krzysztof

>
> v2:
> - add missing people to recipient list of mail
> - no change in actual patch
>
> v3:
> - fix build breakage (Inki Dae)
>
> The testing I had done with allyesconfig, allmodconfig
> did not trigger a configuration where exynos_drm_g2d.c
> was built.  So a build error had slipped through.
>
> Verified that all 23 .c files are built now.
>
>         Sam
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux