Hi Russel, Thanks for both fixes. Just one nitpick. On 06/12/2014 06:09 PM, Russell King wrote: > DRM_PANEL_LD9040 and DRM_PANEL_S6E8AA0 both explicitly depended on > DRM_PANEL && DRM. This is nonsense for two reasons: > > (a) DRM_PANEL already depends on DRM, so DRM_PANEL can not be enabled > without DRM first being enabled. Hence the && DRM is useless. This reason is invalid and do not describe the patch anymore. Regards Andrzej > > (b) These two configs are already beneath a menu which depends on > DRM_PANEL being enabled; these options will not be visible without > DRM_PANEL already being enabled. > > Let's just remove this unnecessary verbosity. > > Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/panel/Kconfig | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig > index 4ec874da5668..d83958d39f43 100644 > --- a/drivers/gpu/drm/panel/Kconfig > +++ b/drivers/gpu/drm/panel/Kconfig > @@ -5,7 +5,7 @@ config DRM_PANEL > Panel registration and lookup framework. > > menu "Display Panels" > - depends on DRM_PANEL > + depends on DRM && DRM_PANEL > > config DRM_PANEL_SIMPLE > tristate "support for simple panels" > @@ -18,14 +18,12 @@ config DRM_PANEL_SIMPLE > > config DRM_PANEL_LD9040 > tristate "LD9040 RGB/SPI panel" > - depends on DRM && DRM_PANEL > depends on OF > select SPI > select VIDEOMODE_HELPERS > > config DRM_PANEL_S6E8AA0 > tristate "S6E8AA0 DSI video mode panel" > - depends on DRM && DRM_PANEL > depends on OF > select DRM_MIPI_DSI > select VIDEOMODE_HELPERS > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
