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

Krzysztof Kozlowski krzk at kernel.org
Fri Jun 21 19:33:22 UTC 2019


On Fri, 21 Jun 2019 at 20:49, Sam Ravnborg <sam at ravnborg.org> 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


More information about the dri-devel mailing list