On Thu, Sep 12, 2019 at 2:41 AM Qian Cai <cai@xxxxxx> wrote:
>
>
>
> > On Sep 11, 2019, at 1:37 PM, Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote:
> >
> > Hi,
> >
> > Le mer. 11 sept. 2019 à 19:35, Qian Cai <cai@xxxxxx> a écrit :
> > The commit c0e09200dc08 ("drm: reorganise drm tree to be more future
> > proof.") changed the behavior from only compiling drm/ if CONFIG_DRM=y
> > to always compiling drm/. This restores the behavior, so people don't
> > need to waste time compiling stuff they don't need.
> >
> > Fixes: c0e09200dc08 ("drm: reorganise drm tree to be more future proof.")
> >
> > You are missing your signed-off-by
>
> That is intentional because this is a RFC patch to gather the feedback as I am no
> expert in DRM and may miss something important.
>
> Once people are happy with it, I plan to send a formal patch with Signed-off-by.
>
CONFIG_DRM is tristate.
Your patch requires everything under drivers/gpu/drm/
modular when CONFIG_DRM=m.
At least, drivers/gpu/drm/drm_mipi_dsi.c
is built-in only.
I guess that is the reason why
they use 'obj-y += drm/'.
In other words, if you are able to
make all the drm code modular,
your patch is applicable.
--
Best Regards
Masahiro Yamada
