All of lore.kernel.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] drm/panel: make DRM_PANEL_LD9040 depend on SPI
Date: Thu, 12 Jun 2014 18:01:53 +0200	[thread overview]
Message-ID: <4631432.m2S6KTlMps@wuerfel> (raw)
In-Reply-To: <E1Wv7Ge-0004SN-Sk@rmk-PC.arm.linux.org.uk>

On Thursday 12 June 2014 16:50:20 Russell King wrote:
> Rather than DRM_PANEL_LD9040 selecting SPI, which then results in an
> increase in the probability of Kconf reporting circular dependencies
> (we're one "select" away from that right now), make this depend on
> SPI instead.  This is akin to having some driver select DRM.
> 
> Having some drivers depend on a subsystem, and other drivers select a
> subsystem is a recipe for circular dependencies, and there's really no
> need for it.
> 
> The potential circular dependency (which can be caused today by the
> addition of selecting DRM_PANEL from DRM_IMX_LDB) is:
> 
>   symbol DMADEVICES is selected by SAMSUNG_DMADEV
>   symbol SAMSUNG_DMADEV is selected by S3C64XX_PL080
>   symbol S3C64XX_PL080 is selected by SPI_S3C64XX
>   symbol SPI_S3C64XX depends on SPI
>   symbol SPI is selected by DRM_PANEL_LD9040
>   symbol DRM_PANEL_LD9040 depends on DRM_PANEL
>   symbol DRM_PANEL is selected by DRM_IMX_LDB
>   symbol DRM_IMX_LDB depends on MFD_SYSCON
>   symbol MFD_SYSCON is selected by POWER_RESET_KEYSTONE
>   symbol POWER_RESET_KEYSTONE depends on POWER_SUPPLY
>   symbol POWER_SUPPLY is selected by HID_SONY
>   symbol HID_SONY depends on NEW_LEDS
>   symbol NEW_LEDS is selected by BACKLIGHT_ADP8860
>   symbol BACKLIGHT_ADP8860 depends on BACKLIGHT_CLASS_DEVICE
>   symbol BACKLIGHT_CLASS_DEVICE is selected by FB_MX3
>   symbol FB_MX3 depends on MX3_IPU
>   symbol MX3_IPU depends on DMADEVICES
> 
> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
> 

Acked-by: Arnd Bergmann <arnd@arndb.de>

WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Russell King <rmk+kernel@arm.linux.org.uk>
Cc: linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 2/2] drm/panel: make DRM_PANEL_LD9040 depend on SPI
Date: Thu, 12 Jun 2014 18:01:53 +0200	[thread overview]
Message-ID: <4631432.m2S6KTlMps@wuerfel> (raw)
In-Reply-To: <E1Wv7Ge-0004SN-Sk@rmk-PC.arm.linux.org.uk>

On Thursday 12 June 2014 16:50:20 Russell King wrote:
> Rather than DRM_PANEL_LD9040 selecting SPI, which then results in an
> increase in the probability of Kconf reporting circular dependencies
> (we're one "select" away from that right now), make this depend on
> SPI instead.  This is akin to having some driver select DRM.
> 
> Having some drivers depend on a subsystem, and other drivers select a
> subsystem is a recipe for circular dependencies, and there's really no
> need for it.
> 
> The potential circular dependency (which can be caused today by the
> addition of selecting DRM_PANEL from DRM_IMX_LDB) is:
> 
>   symbol DMADEVICES is selected by SAMSUNG_DMADEV
>   symbol SAMSUNG_DMADEV is selected by S3C64XX_PL080
>   symbol S3C64XX_PL080 is selected by SPI_S3C64XX
>   symbol SPI_S3C64XX depends on SPI
>   symbol SPI is selected by DRM_PANEL_LD9040
>   symbol DRM_PANEL_LD9040 depends on DRM_PANEL
>   symbol DRM_PANEL is selected by DRM_IMX_LDB
>   symbol DRM_IMX_LDB depends on MFD_SYSCON
>   symbol MFD_SYSCON is selected by POWER_RESET_KEYSTONE
>   symbol POWER_RESET_KEYSTONE depends on POWER_SUPPLY
>   symbol POWER_SUPPLY is selected by HID_SONY
>   symbol HID_SONY depends on NEW_LEDS
>   symbol NEW_LEDS is selected by BACKLIGHT_ADP8860
>   symbol BACKLIGHT_ADP8860 depends on BACKLIGHT_CLASS_DEVICE
>   symbol BACKLIGHT_CLASS_DEVICE is selected by FB_MX3
>   symbol FB_MX3 depends on MX3_IPU
>   symbol MX3_IPU depends on DMADEVICES
> 
> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
> 

Acked-by: Arnd Bergmann <arnd@arndb.de>

  reply	other threads:[~2014-06-12 16:01 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-12 15:50 [PATCH 2/2] drm/panel: make DRM_PANEL_LD9040 depend on SPI Russell King
2014-06-12 15:50 ` Russell King
2014-06-12 16:01 ` Arnd Bergmann [this message]
2014-06-12 16:01   ` Arnd Bergmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4631432.m2S6KTlMps@wuerfel \
    --to=arnd@arndb.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.