Re: [PATCH] drm/usb: select USB_SUPPORT in Kconfig
Hi Dave, What is your opinion about this patch? On 15 August 2012 01:27, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Aug 14, 2012 at 05:12:22PM +0530, Sachin Kamat wrote: In general what you suggested seems to be the right thing to do. However in this particular case, making this depends on creates recursive dependencies and probably that is the reason Dave Airlie decided to make it select USB (?) Probably, yes. Trying to fix this resulted in a race through Kconfig entries throughout the kernel. Given that your patch is probably ok, though this should really be fixed properly. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- With warm regards, Sachin ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH] drm/usb: select USB_SUPPORT in Kconfig
Hi Dave, What is your opinion about this patch? On 15 August 2012 01:27, Sascha Hauer wrote: > On Tue, Aug 14, 2012 at 05:12:22PM +0530, Sachin Kamat wrote: >> >> In general what you suggested seems to be the right thing to do. >> However in this particular case, making this "depends on" creates >> recursive dependencies >> and probably that is the reason Dave Airlie decided to make it "select USB" >> (?) >> > > Probably, yes. Trying to fix this resulted in a race through Kconfig > entries throughout the kernel. Given that your patch is probably ok, > though this should really be fixed properly. > > Sascha > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- With warm regards, Sachin
[PATCH] drm/usb: select USB_SUPPORT in Kconfig
On Tue, Aug 14, 2012 at 05:12:22PM +0530, Sachin Kamat wrote: > > In general what you suggested seems to be the right thing to do. > However in this particular case, making this "depends on" creates > recursive dependencies > and probably that is the reason Dave Airlie decided to make it "select USB" > (?) > Probably, yes. Trying to fix this resulted in a race through Kconfig entries throughout the kernel. Given that your patch is probably ok, though this should really be fixed properly. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- |
[PATCH] drm/usb: select USB_SUPPORT in Kconfig
On 10 August 2012 19:35, Sascha Hauer wrote: > On Fri, Aug 10, 2012 at 11:00:25AM +0530, Sachin Kamat wrote: >> DRM_USB selects USB. However, USB depends on USB_SUPPORT and >> USB_ARCH_HAS_HCD. >> Thus, selecting USB_SUPPORT in Kconfig avoids the following warning >> (detected when DisplayLink was selected using exynos4_defconfig): >> >> warning: (MOUSE_APPLETOUCH && MOUSE_BCM5974 && MOUSE_SYNAPTICS_USB && >> JOYSTICK_XPAD && TABLET_USB_ACECAD && TABLET_USB_AIPTEK && >> TABLET_USB_HANWANG && TABLET_USB_KBTAB && TABLET_USB_WACOM && >> TOUCHSCREEN_USB_COMPOSITE && INPUT_ATI_REMOTE2 && INPUT_KEYSPAN_REMOTE && >> INPUT_POWERMATE && INPUT_YEALINK && INPUT_CM109 && RC_ATI_REMOTE && IR_IMON >> && IR_MCEUSB && IR_REDRAT3 && IR_STREAMZAP && IR_IGUANA && DRM_USB) selects >> USB which has unmet direct dependencies (USB_SUPPORT && USB_ARCH_HAS_HCD) >> >> Signed-off-by: Sachin Kamat >> --- >> drivers/gpu/drm/Kconfig |1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig >> index 23120c0..45536db 100644 >> --- a/drivers/gpu/drm/Kconfig >> +++ b/drivers/gpu/drm/Kconfig >> @@ -23,6 +23,7 @@ config DRM_USB >> tristate >> depends on DRM >> select USB >> + select USB_SUPPORT > > User visible options shouldn't be selected. So instead of selecting > all dependencies this should rather be: > > depends on USB In general what you suggested seems to be the right thing to do. However in this particular case, making this "depends on" creates recursive dependencies and probably that is the reason Dave Airlie decided to make it "select USB" (?) > > Sascha > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- With warm regards, Sachin
Re: [PATCH] drm/usb: select USB_SUPPORT in Kconfig
On Tue, Aug 14, 2012 at 05:12:22PM +0530, Sachin Kamat wrote: In general what you suggested seems to be the right thing to do. However in this particular case, making this depends on creates recursive dependencies and probably that is the reason Dave Airlie decided to make it select USB (?) Probably, yes. Trying to fix this resulted in a race through Kconfig entries throughout the kernel. Given that your patch is probably ok, though this should really be fixed properly. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH] drm/usb: select USB_SUPPORT in Kconfig
On Fri, Aug 10, 2012 at 11:00:25AM +0530, Sachin Kamat wrote: > DRM_USB selects USB. However, USB depends on USB_SUPPORT and USB_ARCH_HAS_HCD. > Thus, selecting USB_SUPPORT in Kconfig avoids the following warning > (detected when DisplayLink was selected using exynos4_defconfig): > > warning: (MOUSE_APPLETOUCH && MOUSE_BCM5974 && MOUSE_SYNAPTICS_USB && > JOYSTICK_XPAD && TABLET_USB_ACECAD && TABLET_USB_AIPTEK && TABLET_USB_HANWANG > && TABLET_USB_KBTAB && TABLET_USB_WACOM && TOUCHSCREEN_USB_COMPOSITE && > INPUT_ATI_REMOTE2 && INPUT_KEYSPAN_REMOTE && INPUT_POWERMATE && INPUT_YEALINK > && INPUT_CM109 && RC_ATI_REMOTE && IR_IMON && IR_MCEUSB && IR_REDRAT3 && > IR_STREAMZAP && IR_IGUANA && DRM_USB) selects USB which has unmet direct > dependencies (USB_SUPPORT && USB_ARCH_HAS_HCD) > > Signed-off-by: Sachin Kamat > --- > drivers/gpu/drm/Kconfig |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > index 23120c0..45536db 100644 > --- a/drivers/gpu/drm/Kconfig > +++ b/drivers/gpu/drm/Kconfig > @@ -23,6 +23,7 @@ config DRM_USB > tristate > depends on DRM > select USB > + select USB_SUPPORT User visible options shouldn't be selected. So instead of selecting all dependencies this should rather be: depends on USB Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- |
[PATCH] drm/usb: select USB_SUPPORT in Kconfig
DRM_USB selects USB. However, USB depends on USB_SUPPORT and USB_ARCH_HAS_HCD. Thus, selecting USB_SUPPORT in Kconfig avoids the following warning (detected when DisplayLink was selected using exynos4_defconfig): warning: (MOUSE_APPLETOUCH && MOUSE_BCM5974 && MOUSE_SYNAPTICS_USB && JOYSTICK_XPAD && TABLET_USB_ACECAD && TABLET_USB_AIPTEK && TABLET_USB_HANWANG && TABLET_USB_KBTAB && TABLET_USB_WACOM && TOUCHSCREEN_USB_COMPOSITE && INPUT_ATI_REMOTE2 && INPUT_KEYSPAN_REMOTE && INPUT_POWERMATE && INPUT_YEALINK && INPUT_CM109 && RC_ATI_REMOTE && IR_IMON && IR_MCEUSB && IR_REDRAT3 && IR_STREAMZAP && IR_IGUANA && DRM_USB) selects USB which has unmet direct dependencies (USB_SUPPORT && USB_ARCH_HAS_HCD) Signed-off-by: Sachin Kamat --- drivers/gpu/drm/Kconfig |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig index 23120c0..45536db 100644 --- a/drivers/gpu/drm/Kconfig +++ b/drivers/gpu/drm/Kconfig @@ -23,6 +23,7 @@ config DRM_USB tristate depends on DRM select USB + select USB_SUPPORT config DRM_KMS_HELPER tristate -- 1.7.4.1
[PATCH] drm/usb: select USB_SUPPORT in Kconfig
DRM_USB selects USB. However, USB depends on USB_SUPPORT and USB_ARCH_HAS_HCD. Thus, selecting USB_SUPPORT in Kconfig avoids the following warning (detected when DisplayLink was selected using exynos4_defconfig): warning: (MOUSE_APPLETOUCH MOUSE_BCM5974 MOUSE_SYNAPTICS_USB JOYSTICK_XPAD TABLET_USB_ACECAD TABLET_USB_AIPTEK TABLET_USB_HANWANG TABLET_USB_KBTAB TABLET_USB_WACOM TOUCHSCREEN_USB_COMPOSITE INPUT_ATI_REMOTE2 INPUT_KEYSPAN_REMOTE INPUT_POWERMATE INPUT_YEALINK INPUT_CM109 RC_ATI_REMOTE IR_IMON IR_MCEUSB IR_REDRAT3 IR_STREAMZAP IR_IGUANA DRM_USB) selects USB which has unmet direct dependencies (USB_SUPPORT USB_ARCH_HAS_HCD) Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/gpu/drm/Kconfig |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig index 23120c0..45536db 100644 --- a/drivers/gpu/drm/Kconfig +++ b/drivers/gpu/drm/Kconfig @@ -23,6 +23,7 @@ config DRM_USB tristate depends on DRM select USB + select USB_SUPPORT config DRM_KMS_HELPER tristate -- 1.7.4.1 ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel