Re: [PATCH] drm/xen: adjust Kconfig
On 2/24/21 6:17 PM, Daniel Vetter wrote: > On Wed, Feb 24, 2021 at 8:55 AM Oleksandr Andrushchenko > wrote: >> Hello, Jan! >> >> On 2/23/21 6:41 PM, Jan Beulich wrote: >>> By having selected DRM_XEN, I was assuming I would build the frontend >>> driver. As it turns out this is a dummy option, and I have really not >>> been building this (because I had DRM disabled). Make it a promptless >>> one, moving the "depends on" to the other, real option, and "select"ing >>> the dummy one. >>> >>> Signed-off-by: Jan Beulich >> Reviewed-by: Oleksandr Andrushchenko > Since you're maintainer/committer, I'm assuming you'll also merge > this? Always confusing when there's an r-b but nothing about whether > the patch will get merged or not. Applied to drm-misc-next, thank you > -Daniel > >>> --- a/drivers/gpu/drm/xen/Kconfig >>> +++ b/drivers/gpu/drm/xen/Kconfig >>> @@ -1,15 +1,11 @@ >>># SPDX-License-Identifier: GPL-2.0-only >>>config DRM_XEN >>> - bool "DRM Support for Xen guest OS" >>> - depends on XEN >>> - help >>> - Choose this option if you want to enable DRM support >>> - for Xen. >>> + bool >>> >>>config DRM_XEN_FRONTEND >>>tristate "Para-virtualized frontend driver for Xen guest OS" >>> - depends on DRM_XEN >>> - depends on DRM >>> + depends on XEN && DRM >>> + select DRM_XEN >>>select DRM_KMS_HELPER >>>select VIDEOMODE_HELPERS >>>select XEN_XENBUS_FRONTEND >> ___ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://urldefense.com/v3/__https://lists.freedesktop.org/mailman/listinfo/dri-devel__;!!GF_29dbcQIUBPA!l8ITnylvjn3TuANLahtryeLJhjH20i-9Oi_1tYkdA59vXUpBHtKnlQ-Kg-zURW_ruUkKKaUAZQ$ >> [lists[.]freedesktop[.]org] > > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH] drm/xen: adjust Kconfig
On Wed, Feb 24, 2021 at 8:55 AM Oleksandr Andrushchenko wrote: > > Hello, Jan! > > On 2/23/21 6:41 PM, Jan Beulich wrote: > > By having selected DRM_XEN, I was assuming I would build the frontend > > driver. As it turns out this is a dummy option, and I have really not > > been building this (because I had DRM disabled). Make it a promptless > > one, moving the "depends on" to the other, real option, and "select"ing > > the dummy one. > > > > Signed-off-by: Jan Beulich > Reviewed-by: Oleksandr Andrushchenko Since you're maintainer/committer, I'm assuming you'll also merge this? Always confusing when there's an r-b but nothing about whether the patch will get merged or not. -Daniel > > --- a/drivers/gpu/drm/xen/Kconfig > > +++ b/drivers/gpu/drm/xen/Kconfig > > @@ -1,15 +1,11 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > config DRM_XEN > > - bool "DRM Support for Xen guest OS" > > - depends on XEN > > - help > > - Choose this option if you want to enable DRM support > > - for Xen. > > + bool > > > > config DRM_XEN_FRONTEND > > tristate "Para-virtualized frontend driver for Xen guest OS" > > - depends on DRM_XEN > > - depends on DRM > > + depends on XEN && DRM > > + select DRM_XEN > > select DRM_KMS_HELPER > > select VIDEOMODE_HELPERS > > select XEN_XENBUS_FRONTEND > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH] drm/xen: adjust Kconfig
Hello, Jan! On 2/23/21 6:41 PM, Jan Beulich wrote: > By having selected DRM_XEN, I was assuming I would build the frontend > driver. As it turns out this is a dummy option, and I have really not > been building this (because I had DRM disabled). Make it a promptless > one, moving the "depends on" to the other, real option, and "select"ing > the dummy one. > > Signed-off-by: Jan Beulich Reviewed-by: Oleksandr Andrushchenko > --- a/drivers/gpu/drm/xen/Kconfig > +++ b/drivers/gpu/drm/xen/Kconfig > @@ -1,15 +1,11 @@ > # SPDX-License-Identifier: GPL-2.0-only > config DRM_XEN > - bool "DRM Support for Xen guest OS" > - depends on XEN > - help > - Choose this option if you want to enable DRM support > - for Xen. > + bool > > config DRM_XEN_FRONTEND > tristate "Para-virtualized frontend driver for Xen guest OS" > - depends on DRM_XEN > - depends on DRM > + depends on XEN && DRM > + select DRM_XEN > select DRM_KMS_HELPER > select VIDEOMODE_HELPERS > select XEN_XENBUS_FRONTEND ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH] drm/xen: adjust Kconfig
By having selected DRM_XEN, I was assuming I would build the frontend driver. As it turns out this is a dummy option, and I have really not been building this (because I had DRM disabled). Make it a promptless one, moving the "depends on" to the other, real option, and "select"ing the dummy one. Signed-off-by: Jan Beulich --- a/drivers/gpu/drm/xen/Kconfig +++ b/drivers/gpu/drm/xen/Kconfig @@ -1,15 +1,11 @@ # SPDX-License-Identifier: GPL-2.0-only config DRM_XEN - bool "DRM Support for Xen guest OS" - depends on XEN - help - Choose this option if you want to enable DRM support - for Xen. + bool config DRM_XEN_FRONTEND tristate "Para-virtualized frontend driver for Xen guest OS" - depends on DRM_XEN - depends on DRM + depends on XEN && DRM + select DRM_XEN select DRM_KMS_HELPER select VIDEOMODE_HELPERS select XEN_XENBUS_FRONTEND ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel