Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1638333 > unrolled thread
| Started by | Eric Anholt <eric@anholt.net> |
|---|---|
| First post | 2017-05-09 20:20 +0200 |
| Last post | 2017-05-10 01:30 +0200 |
| Articles | 5 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. Eric Anholt <eric@anholt.net> - 2017-05-09 20:20 +0200
Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. Scott Branden <scott.branden@broadcom.com> - 2017-05-09 20:30 +0200
Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. Florian Fainelli <f.fainelli@gmail.com> - 2017-05-10 01:10 +0200
Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. Eric Anholt <eric@anholt.net> - 2017-05-10 01:20 +0200
Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. Florian Fainelli <f.fainelli@gmail.com> - 2017-05-10 01:30 +0200
| From | Eric Anholt <eric@anholt.net> |
|---|---|
| Date | 2017-05-09 20:20 +0200 |
| Subject | [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. |
| Message-ID | <tFhTP-5nA-17@gated-at.bofh.it> |
With the Cygnus port, we needed to add at least "|| ARCH_BCM_CYGNUS"
to let the module get built on a cygnus-only kernel. However, I
anticipate having a port for Kona soon, so just present the module on
all of BCM.
v2: Keep allowing selection with ARCH_BCM2835, since ARCH_BCM doesn't
exist on arm64.
Signed-off-by: Eric Anholt <eric@anholt.net>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> (v1)
---
drivers/gpu/drm/vc4/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/vc4/Kconfig b/drivers/gpu/drm/vc4/Kconfig
index 973b4203c0b2..b16aefe4a8d3 100644
--- a/drivers/gpu/drm/vc4/Kconfig
+++ b/drivers/gpu/drm/vc4/Kconfig
@@ -1,6 +1,6 @@
config DRM_VC4
tristate "Broadcom VC4 Graphics"
- depends on ARCH_BCM2835 || COMPILE_TEST
+ depends on ARCH_BCM || ARCH_BCM2835 || COMPILE_TEST
depends on DRM
depends on SND && SND_SOC
depends on COMMON_CLK
--
2.11.0
[toc] | [next] | [standalone]
| From | Scott Branden <scott.branden@broadcom.com> |
|---|---|
| Date | 2017-05-09 20:30 +0200 |
| Subject | Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. |
| Message-ID | <tFi3v-5s6-11@gated-at.bofh.it> |
| In reply to | #1638333 |
Looks good for Cygnus. On 17-05-09 11:15 AM, Eric Anholt wrote: > With the Cygnus port, we needed to add at least "|| ARCH_BCM_CYGNUS" > to let the module get built on a cygnus-only kernel. However, I > anticipate having a port for Kona soon, so just present the module on > all of BCM. > > v2: Keep allowing selection with ARCH_BCM2835, since ARCH_BCM doesn't > exist on arm64. > > Signed-off-by: Eric Anholt <eric@anholt.net> > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> (v1) Acked-by: Scott Branden <scott.branden@broadcom.com> > --- > drivers/gpu/drm/vc4/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/vc4/Kconfig b/drivers/gpu/drm/vc4/Kconfig > index 973b4203c0b2..b16aefe4a8d3 100644 > --- a/drivers/gpu/drm/vc4/Kconfig > +++ b/drivers/gpu/drm/vc4/Kconfig > @@ -1,6 +1,6 @@ > config DRM_VC4 > tristate "Broadcom VC4 Graphics" > - depends on ARCH_BCM2835 || COMPILE_TEST > + depends on ARCH_BCM || ARCH_BCM2835 || COMPILE_TEST > depends on DRM > depends on SND && SND_SOC > depends on COMMON_CLK >
[toc] | [prev] | [next] | [standalone]
| From | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| Date | 2017-05-10 01:10 +0200 |
| Subject | Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. |
| Message-ID | <tFmqt-8sO-1@gated-at.bofh.it> |
| In reply to | #1638333 |
On 05/09/2017 11:15 AM, Eric Anholt wrote: > With the Cygnus port, we needed to add at least "|| ARCH_BCM_CYGNUS" > to let the module get built on a cygnus-only kernel. However, I > anticipate having a port for Kona soon, so just present the module on > all of BCM. > > v2: Keep allowing selection with ARCH_BCM2835, since ARCH_BCM doesn't > exist on arm64. Nit: the patch changelog usually goes after the "---" line so it gets stripped with git am. Not necessary to resubmit just because of that. > > Signed-off-by: Eric Anholt <eric@anholt.net> > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> (v1) Acked-by: Florian Fainelli <f.fainelli@gmail.com> > --- > drivers/gpu/drm/vc4/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/vc4/Kconfig b/drivers/gpu/drm/vc4/Kconfig > index 973b4203c0b2..b16aefe4a8d3 100644 > --- a/drivers/gpu/drm/vc4/Kconfig > +++ b/drivers/gpu/drm/vc4/Kconfig > @@ -1,6 +1,6 @@ > config DRM_VC4 > tristate "Broadcom VC4 Graphics" > - depends on ARCH_BCM2835 || COMPILE_TEST > + depends on ARCH_BCM || ARCH_BCM2835 || COMPILE_TEST > depends on DRM > depends on SND && SND_SOC > depends on COMMON_CLK > -- Florian
[toc] | [prev] | [next] | [standalone]
| From | Eric Anholt <eric@anholt.net> |
|---|---|
| Date | 2017-05-10 01:20 +0200 |
| Message-ID | <tFmA9-5P-3@gated-at.bofh.it> |
| In reply to | #1638435 |
[Multipart message — attachments visible in raw view] — view raw
Florian Fainelli <f.fainelli@gmail.com> writes: > On 05/09/2017 11:15 AM, Eric Anholt wrote: >> With the Cygnus port, we needed to add at least "|| ARCH_BCM_CYGNUS" >> to let the module get built on a cygnus-only kernel. However, I >> anticipate having a port for Kona soon, so just present the module on >> all of BCM. >> >> v2: Keep allowing selection with ARCH_BCM2835, since ARCH_BCM doesn't >> exist on arm64. > > Nit: the patch changelog usually goes after the "---" line so it gets > stripped with git am. Not necessary to resubmit just because of that. Behavior on that front differs between subsystems. DRM is one where the changelog is generally retained.
[toc] | [prev] | [next] | [standalone]
| From | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| Date | 2017-05-10 01:30 +0200 |
| Subject | Re: [PATCH v2] drm/vc4: Enable selection in Kconfig on any 32-bit BCM platform. |
| Message-ID | <tFmJQ-9n-1@gated-at.bofh.it> |
| In reply to | #1638444 |
On 05/09/2017 04:16 PM, Eric Anholt wrote: > Florian Fainelli <f.fainelli@gmail.com> writes: > >> On 05/09/2017 11:15 AM, Eric Anholt wrote: >>> With the Cygnus port, we needed to add at least "|| ARCH_BCM_CYGNUS" >>> to let the module get built on a cygnus-only kernel. However, I >>> anticipate having a port for Kona soon, so just present the module on >>> all of BCM. >>> >>> v2: Keep allowing selection with ARCH_BCM2835, since ARCH_BCM doesn't >>> exist on arm64. >> >> Nit: the patch changelog usually goes after the "---" line so it gets >> stripped with git am. Not necessary to resubmit just because of that. > > Behavior on that front differs between subsystems. DRM is one where the > changelog is generally retained. Once the patch lands in git, it's sort of interesting to know its history and the context surrounding this acceptance, but there is already so much context being lost already (like where are all other patches from the same patch series for instance?) that I wonder if we should not add more to it (like links to past iterations and so on). Thanks for explaining how DRM works in that regard, though. -- Florian
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web