* [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" @ 2013-03-13 19:48 Paul Bolle 2013-06-13 10:31 ` Paul Bolle 2013-06-15 10:09 ` Thierry Reding 0 siblings, 2 replies; 6+ messages in thread From: Paul Bolle @ 2013-03-13 19:48 UTC (permalink / raw) To: David Airlie; +Cc: dri-devel, linux-kernel Signed-off-by: Paul Bolle <pebolle@tiscali.nl> --- Untested. Perhaps the first test that people with access to the relevant hardware might do, is to test _before applying this patch_ with FB_OMAP2 set. Perhaps this negative dependency isn't needed at all. Or is it obvious? drivers/gpu/drm/omapdrm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/omapdrm/Kconfig b/drivers/gpu/drm/omapdrm/Kconfig index 09f65dc..45875a0 100644 --- a/drivers/gpu/drm/omapdrm/Kconfig +++ b/drivers/gpu/drm/omapdrm/Kconfig @@ -1,7 +1,7 @@ config DRM_OMAP tristate "OMAP DRM" - depends on DRM && !CONFIG_FB_OMAP2 + depends on DRM && !FB_OMAP2 depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM depends on OMAP2_DSS select DRM_KMS_HELPER -- 1.7.11.7 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" 2013-03-13 19:48 [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" Paul Bolle @ 2013-06-13 10:31 ` Paul Bolle 2013-06-15 12:20 ` Rob Clark 2013-06-15 10:09 ` Thierry Reding 1 sibling, 1 reply; 6+ messages in thread From: Paul Bolle @ 2013-06-13 10:31 UTC (permalink / raw) To: David Airlie; +Cc: dri-devel, linux-kernel On Wed, 2013-03-13 at 20:48 +0100, Paul Bolle wrote: > Signed-off-by: Paul Bolle <pebolle@tiscali.nl> > --- > Untested. Perhaps the first test that people with access to the relevant > hardware might do, is to test _before applying this patch_ with FB_OMAP2 > set. Perhaps this negative dependency isn't needed at all. Or is it > obvious? > > drivers/gpu/drm/omapdrm/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) This patch was sent exactly three months ago, shortly after v3.9-rc2 was released. This obvious typo is still present in v3.10-rc5. I didn't received any feedback on this patch. Did anyone had a look at it? Is it perhaps queued somewhere? Paul Bolle > diff --git a/drivers/gpu/drm/omapdrm/Kconfig b/drivers/gpu/drm/omapdrm/Kconfig > index 09f65dc..45875a0 100644 > --- a/drivers/gpu/drm/omapdrm/Kconfig > +++ b/drivers/gpu/drm/omapdrm/Kconfig > @@ -1,7 +1,7 @@ > > config DRM_OMAP > tristate "OMAP DRM" > - depends on DRM && !CONFIG_FB_OMAP2 > + depends on DRM && !FB_OMAP2 > depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM > depends on OMAP2_DSS > select DRM_KMS_HELPER ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" 2013-06-13 10:31 ` Paul Bolle @ 2013-06-15 12:20 ` Rob Clark 2013-06-28 9:59 ` Tomi Valkeinen 0 siblings, 1 reply; 6+ messages in thread From: Rob Clark @ 2013-06-15 12:20 UTC (permalink / raw) To: Paul Bolle; +Cc: linux-kernel, dri-devel On Thu, Jun 13, 2013 at 6:31 AM, Paul Bolle <pebolle@tiscali.nl> wrote: > On Wed, 2013-03-13 at 20:48 +0100, Paul Bolle wrote: >> Signed-off-by: Paul Bolle <pebolle@tiscali.nl> >> --- >> Untested. Perhaps the first test that people with access to the relevant >> hardware might do, is to test _before applying this patch_ with FB_OMAP2 >> set. Perhaps this negative dependency isn't needed at all. Or is it >> obvious? >> >> drivers/gpu/drm/omapdrm/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > This patch was sent exactly three months ago, shortly after v3.9-rc2 was > released. This obvious typo is still present in v3.10-rc5. > > I didn't received any feedback on this patch. Did anyone had a look at > it? Is it perhaps queued somewhere? oh, sorry, at the time I was in the middle of moving, and wasn't watching dri-devel so much. Yeah, this should probably either be merged, or we should just drop the negative dependency. (I guess in theory you could build both omapfb and omapdrm as modules and just load one.) BR -R > > Paul Bolle > >> diff --git a/drivers/gpu/drm/omapdrm/Kconfig b/drivers/gpu/drm/omapdrm/Kconfig >> index 09f65dc..45875a0 100644 >> --- a/drivers/gpu/drm/omapdrm/Kconfig >> +++ b/drivers/gpu/drm/omapdrm/Kconfig >> @@ -1,7 +1,7 @@ >> >> config DRM_OMAP >> tristate "OMAP DRM" >> - depends on DRM && !CONFIG_FB_OMAP2 >> + depends on DRM && !FB_OMAP2 >> depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM >> depends on OMAP2_DSS >> select DRM_KMS_HELPER > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" 2013-06-15 12:20 ` Rob Clark @ 2013-06-28 9:59 ` Tomi Valkeinen 2013-06-28 10:42 ` Dave Airlie 0 siblings, 1 reply; 6+ messages in thread From: Tomi Valkeinen @ 2013-06-28 9:59 UTC (permalink / raw) To: Rob Clark, Paul Bolle; +Cc: David Airlie, linux-kernel, dri-devel [-- Attachment #1: Type: text/plain, Size: 1728 bytes --] On 15/06/13 15:20, Rob Clark wrote: > On Thu, Jun 13, 2013 at 6:31 AM, Paul Bolle <pebolle@tiscali.nl> wrote: >> On Wed, 2013-03-13 at 20:48 +0100, Paul Bolle wrote: >>> Signed-off-by: Paul Bolle <pebolle@tiscali.nl> >>> --- >>> Untested. Perhaps the first test that people with access to the relevant >>> hardware might do, is to test _before applying this patch_ with FB_OMAP2 >>> set. Perhaps this negative dependency isn't needed at all. Or is it >>> obvious? >>> >>> drivers/gpu/drm/omapdrm/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> This patch was sent exactly three months ago, shortly after v3.9-rc2 was >> released. This obvious typo is still present in v3.10-rc5. >> >> I didn't received any feedback on this patch. Did anyone had a look at >> it? Is it perhaps queued somewhere? > > oh, sorry, at the time I was in the middle of moving, and wasn't > watching dri-devel so much. > > Yeah, this should probably either be merged, or we should just drop > the negative dependency. (I guess in theory you could build both > omapfb and omapdrm as modules and just load one.) Hmm, fixing that creates a problem: drivers/video/Kconfig:42:error: recursive dependency detected! drivers/video/Kconfig:42: symbol FB is selected by DRM_KMS_HELPER drivers/gpu/drm/Kconfig:29: symbol DRM_KMS_HELPER is selected by DRM_OMAP drivers/gpu/drm/omapdrm/Kconfig:2: symbol DRM_OMAP depends on FB_OMAP2 drivers/video/omap2/omapfb/Kconfig:1: symbol FB_OMAP2 depends on FB And it makes selecting omapfb not possible... omapfb already has a !DRM_OMAP dependency. I think it's enough to have that one there, and remove the !FB_OMAP dependency from omapdrm. Tomi [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 901 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" 2013-06-28 9:59 ` Tomi Valkeinen @ 2013-06-28 10:42 ` Dave Airlie 0 siblings, 0 replies; 6+ messages in thread From: Dave Airlie @ 2013-06-28 10:42 UTC (permalink / raw) To: Tomi Valkeinen Cc: Rob Clark, Paul Bolle, David Airlie, linux-kernel, dri-devel On Fri, Jun 28, 2013 at 7:59 PM, Tomi Valkeinen <tomi.valkeinen@iki.fi> wrote: > On 15/06/13 15:20, Rob Clark wrote: >> On Thu, Jun 13, 2013 at 6:31 AM, Paul Bolle <pebolle@tiscali.nl> wrote: >>> On Wed, 2013-03-13 at 20:48 +0100, Paul Bolle wrote: >>>> Signed-off-by: Paul Bolle <pebolle@tiscali.nl> >>>> --- >>>> Untested. Perhaps the first test that people with access to the relevant >>>> hardware might do, is to test _before applying this patch_ with FB_OMAP2 >>>> set. Perhaps this negative dependency isn't needed at all. Or is it >>>> obvious? >>>> >>>> drivers/gpu/drm/omapdrm/Kconfig | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> This patch was sent exactly three months ago, shortly after v3.9-rc2 was >>> released. This obvious typo is still present in v3.10-rc5. >>> >>> I didn't received any feedback on this patch. Did anyone had a look at >>> it? Is it perhaps queued somewhere? >> >> oh, sorry, at the time I was in the middle of moving, and wasn't >> watching dri-devel so much. >> >> Yeah, this should probably either be merged, or we should just drop >> the negative dependency. (I guess in theory you could build both >> omapfb and omapdrm as modules and just load one.) > > Hmm, fixing that creates a problem: > > drivers/video/Kconfig:42:error: recursive dependency detected! > drivers/video/Kconfig:42: symbol FB is selected by DRM_KMS_HELPER > drivers/gpu/drm/Kconfig:29: symbol DRM_KMS_HELPER is selected by DRM_OMAP > drivers/gpu/drm/omapdrm/Kconfig:2: symbol DRM_OMAP depends on FB_OMAP2 > drivers/video/omap2/omapfb/Kconfig:1: symbol FB_OMAP2 depends on FB > > And it makes selecting omapfb not possible... > > omapfb already has a !DRM_OMAP dependency. I think it's enough to have > that one there, and remove the !FB_OMAP dependency from omapdrm. I just nuked it upstream in drm-next. commit 2644ee9614be67abe155f1073bb9e1b737bbca53 Author: Dave Airlie <airlied@redhat.com> Date: Fri Jun 28 12:08:10 2013 +1000 drm/omap: drop the !FB_OMAP2 dep This ends up causing circularity and really let people shoot themselves in the foot. Acked-by: Rob Clark <robdclark@gmail.com> Signed-off-by: Dave Airlie <airlied@redhat.com> Dave. > > Tomi > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" 2013-03-13 19:48 [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" Paul Bolle 2013-06-13 10:31 ` Paul Bolle @ 2013-06-15 10:09 ` Thierry Reding 1 sibling, 0 replies; 6+ messages in thread From: Thierry Reding @ 2013-06-15 10:09 UTC (permalink / raw) To: Paul Bolle; +Cc: David Airlie, linux-kernel, dri-devel [-- Attachment #1: Type: text/plain, Size: 1217 bytes --] On Wed, Mar 13, 2013 at 08:48:22PM +0100, Paul Bolle wrote: > Signed-off-by: Paul Bolle <pebolle@tiscali.nl> > --- > Untested. Perhaps the first test that people with access to the relevant > hardware might do, is to test _before applying this patch_ with FB_OMAP2 > set. Perhaps this negative dependency isn't needed at all. Or is it > obvious? I don't think you need access to hardware to test this. Checking that the dependencies work properly (using menuconfig for example) and build testing should be enough. > drivers/gpu/drm/omapdrm/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/omapdrm/Kconfig b/drivers/gpu/drm/omapdrm/Kconfig > index 09f65dc..45875a0 100644 > --- a/drivers/gpu/drm/omapdrm/Kconfig > +++ b/drivers/gpu/drm/omapdrm/Kconfig > @@ -1,7 +1,7 @@ > > config DRM_OMAP > tristate "OMAP DRM" > - depends on DRM && !CONFIG_FB_OMAP2 > + depends on DRM && !FB_OMAP2 > depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM > depends on OMAP2_DSS > select DRM_KMS_HELPER Regardless of whether the dependency is needed or not, the typo should be fixed, so: Reviewed-by: Thierry Reding <thierry.reding@gmail.com> [-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-06-28 10:42 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-03-13 19:48 [PATCH] drm/omap: change "!CONFIG_FB_OMAP2" to "!FB_OMAP2" Paul Bolle 2013-06-13 10:31 ` Paul Bolle 2013-06-15 12:20 ` Rob Clark 2013-06-28 9:59 ` Tomi Valkeinen 2013-06-28 10:42 ` Dave Airlie 2013-06-15 10:09 ` Thierry Reding
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).