* [PATCH 0/4] staging: Fix build of VCHIQ driver.
@ 2016-10-03 18:52 ` Eric Anholt
0 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley, Eric Anholt
This is a series of fixes for the vchiq driver that Greg recently
added to his staging-testing branch.
A full tree with nasty patches to add the other drivers necessary to
do "vcdbg log msg" is available at:
https://github.com/anholt/linux/tree/vchiq
Eric Anholt (4):
staging/vchi: Convert to current get_user_pages() arguments.
staging/vchi: Update for rename of page_cache_release() to put_page().
ARM: bcm2835: Add #define for VCHIQ property message.
drivers/vchi: Remove dependency on CONFIG_BROKEN.
drivers/staging/vc04_services/Kconfig | 2 +-
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 6 +++---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 5 ++---
include/soc/bcm2835/raspberrypi-firmware.h | 2 ++
4 files changed, 8 insertions(+), 7 deletions(-)
--
2.9.3
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 1/4] staging/vchi: Convert to current get_user_pages() arguments.
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-03 18:52 ` Eric Anholt
-1 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 2 +-
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
index c29040fdf9a7..18f268ee6d4b 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
@@ -420,7 +420,7 @@ create_pagelist(char __user *buf, size_t count, unsigned short type,
*need_release = 0; /* do not try and release vmalloc pages */
} else {
down_read(&task->mm->mmap_sem);
- actual_pages = get_user_pages(task, task->mm,
+ actual_pages = get_user_pages(
(unsigned long)buf & ~(PAGE_SIZE - 1),
num_pages,
(type == PAGELIST_READ) /*Write */ ,
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index e11c0e07471b..56c2c48f6801 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -1473,8 +1473,7 @@ dump_phys_mem(void *virt_addr, uint32_t num_bytes)
}
down_read(¤t->mm->mmap_sem);
- rc = get_user_pages(current, /* task */
- current->mm, /* mm */
+ rc = get_user_pages(
(unsigned long)virt_addr, /* start */
num_pages, /* len */
0, /* write */
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 1/4] staging/vchi: Convert to current get_user_pages() arguments.
@ 2016-10-03 18:52 ` Eric Anholt
0 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley, Eric Anholt
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 2 +-
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
index c29040fdf9a7..18f268ee6d4b 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
@@ -420,7 +420,7 @@ create_pagelist(char __user *buf, size_t count, unsigned short type,
*need_release = 0; /* do not try and release vmalloc pages */
} else {
down_read(&task->mm->mmap_sem);
- actual_pages = get_user_pages(task, task->mm,
+ actual_pages = get_user_pages(
(unsigned long)buf & ~(PAGE_SIZE - 1),
num_pages,
(type == PAGELIST_READ) /*Write */ ,
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index e11c0e07471b..56c2c48f6801 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -1473,8 +1473,7 @@ dump_phys_mem(void *virt_addr, uint32_t num_bytes)
}
down_read(¤t->mm->mmap_sem);
- rc = get_user_pages(current, /* task */
- current->mm, /* mm */
+ rc = get_user_pages(
(unsigned long)virt_addr, /* start */
num_pages, /* len */
0, /* write */
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 1/4] staging/vchi: Convert to current get_user_pages() arguments.
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-07 14:44 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 26+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-07 14:44 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Oct 03, 2016 at 11:52:06AM -0700, Eric Anholt wrote:
> Signed-off-by: Eric Anholt <eric@anholt.net>
In the future, it's good to put at least some changelog text in here. I
normally do not accept patches without it. But I'll take these two.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 1/4] staging/vchi: Convert to current get_user_pages() arguments.
@ 2016-10-07 14:44 ` Greg Kroah-Hartman
0 siblings, 0 replies; 26+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-07 14:44 UTC (permalink / raw)
To: Eric Anholt
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley
On Mon, Oct 03, 2016 at 11:52:06AM -0700, Eric Anholt wrote:
> Signed-off-by: Eric Anholt <eric@anholt.net>
In the future, it's good to put at least some changelog text in here. I
normally do not accept patches without it. But I'll take these two.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 1/4] staging/vchi: Convert to current get_user_pages() arguments.
2016-10-07 14:44 ` Greg Kroah-Hartman
@ 2016-10-07 17:36 ` Eric Anholt
-1 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-07 17:36 UTC (permalink / raw)
To: linux-arm-kernel
Greg Kroah-Hartman <gregkh@linuxfoundation.org> writes:
> On Mon, Oct 03, 2016 at 11:52:06AM -0700, Eric Anholt wrote:
>> Signed-off-by: Eric Anholt <eric@anholt.net>
>
> In the future, it's good to put at least some changelog text in here. I
> normally do not accept patches without it. But I'll take these two.
I'm generally verbose in commit messages, but these seemed like they
didn't have useful context to be added.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 800 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161007/5a2699fc/attachment.sig>
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 1/4] staging/vchi: Convert to current get_user_pages() arguments.
@ 2016-10-07 17:36 ` Eric Anholt
0 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-07 17:36 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley
[-- Attachment #1: Type: text/plain, Size: 426 bytes --]
Greg Kroah-Hartman <gregkh@linuxfoundation.org> writes:
> On Mon, Oct 03, 2016 at 11:52:06AM -0700, Eric Anholt wrote:
>> Signed-off-by: Eric Anholt <eric@anholt.net>
>
> In the future, it's good to put at least some changelog text in here. I
> normally do not accept patches without it. But I'll take these two.
I'm generally verbose in commit messages, but these seemed like they
didn't have useful context to be added.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 800 bytes --]
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 2/4] staging/vchi: Update for rename of page_cache_release() to put_page().
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-03 18:52 ` Eric Anholt
-1 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 4 ++--
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
index 18f268ee6d4b..4cb5bff23728 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
@@ -439,7 +439,7 @@ create_pagelist(char __user *buf, size_t count, unsigned short type,
while (actual_pages > 0)
{
actual_pages--;
- page_cache_release(pages[actual_pages]);
+ put_page(pages[actual_pages]);
}
kfree(pagelist);
if (actual_pages == 0)
@@ -578,7 +578,7 @@ free_pagelist(PAGELIST_T *pagelist, int actual)
offset = 0;
set_page_dirty(pg);
}
- page_cache_release(pg);
+ put_page(pg);
}
}
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index 56c2c48f6801..47df1af2219d 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -1512,7 +1512,7 @@ dump_phys_mem(void *virt_addr, uint32_t num_bytes)
kunmap(page);
for (page_idx = 0; page_idx < num_pages; page_idx++)
- page_cache_release(pages[page_idx]);
+ put_page(pages[page_idx]);
kfree(pages);
}
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 2/4] staging/vchi: Update for rename of page_cache_release() to put_page().
@ 2016-10-03 18:52 ` Eric Anholt
0 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley, Eric Anholt
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 4 ++--
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
index 18f268ee6d4b..4cb5bff23728 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
@@ -439,7 +439,7 @@ create_pagelist(char __user *buf, size_t count, unsigned short type,
while (actual_pages > 0)
{
actual_pages--;
- page_cache_release(pages[actual_pages]);
+ put_page(pages[actual_pages]);
}
kfree(pagelist);
if (actual_pages == 0)
@@ -578,7 +578,7 @@ free_pagelist(PAGELIST_T *pagelist, int actual)
offset = 0;
set_page_dirty(pg);
}
- page_cache_release(pg);
+ put_page(pg);
}
}
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index 56c2c48f6801..47df1af2219d 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -1512,7 +1512,7 @@ dump_phys_mem(void *virt_addr, uint32_t num_bytes)
kunmap(page);
for (page_idx = 0; page_idx < num_pages; page_idx++)
- page_cache_release(pages[page_idx]);
+ put_page(pages[page_idx]);
kfree(pages);
}
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH 3/4] ARM: bcm2835: Add #define for VCHIQ property message.
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-03 18:52 ` Eric Anholt
-1 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: linux-arm-kernel
This comes from the downstream tree and is needed for the new VCHIQ
driver in staging.
Signed-off-by: Eric Anholt <eric@anholt.net>
---
include/soc/bcm2835/raspberrypi-firmware.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/soc/bcm2835/raspberrypi-firmware.h b/include/soc/bcm2835/raspberrypi-firmware.h
index 3fb357193f09..a06baffdf580 100644
--- a/include/soc/bcm2835/raspberrypi-firmware.h
+++ b/include/soc/bcm2835/raspberrypi-firmware.h
@@ -109,6 +109,8 @@ enum rpi_firmware_property_tag {
RPI_FIRMWARE_FRAMEBUFFER_SET_OVERSCAN = 0x0004800a,
RPI_FIRMWARE_FRAMEBUFFER_SET_PALETTE = 0x0004800b,
+ RPI_FIRMWARE_VCHIQ_INIT = 0x00048010,
+
RPI_FIRMWARE_GET_COMMAND_LINE = 0x00050001,
RPI_FIRMWARE_GET_DMA_CHANNELS = 0x00060001,
};
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 3/4] ARM: bcm2835: Add #define for VCHIQ property message.
@ 2016-10-03 18:52 ` Eric Anholt
0 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley, Eric Anholt
This comes from the downstream tree and is needed for the new VCHIQ
driver in staging.
Signed-off-by: Eric Anholt <eric@anholt.net>
---
include/soc/bcm2835/raspberrypi-firmware.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/soc/bcm2835/raspberrypi-firmware.h b/include/soc/bcm2835/raspberrypi-firmware.h
index 3fb357193f09..a06baffdf580 100644
--- a/include/soc/bcm2835/raspberrypi-firmware.h
+++ b/include/soc/bcm2835/raspberrypi-firmware.h
@@ -109,6 +109,8 @@ enum rpi_firmware_property_tag {
RPI_FIRMWARE_FRAMEBUFFER_SET_OVERSCAN = 0x0004800a,
RPI_FIRMWARE_FRAMEBUFFER_SET_PALETTE = 0x0004800b,
+ RPI_FIRMWARE_VCHIQ_INIT = 0x00048010,
+
RPI_FIRMWARE_GET_COMMAND_LINE = 0x00050001,
RPI_FIRMWARE_GET_DMA_CHANNELS = 0x00060001,
};
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH 3/4] ARM: bcm2835: Add #define for VCHIQ property message.
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-15 7:53 ` Stefan Wahren
-1 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 7:53 UTC (permalink / raw)
To: linux-arm-kernel
Hi Greg,
> Eric Anholt <eric@anholt.net> hat am 3. Oktober 2016 um 20:52 geschrieben:
>
>
> This comes from the downstream tree and is needed for the new VCHIQ
> driver in staging.
maybe it's not clear from the commit message, but RPI_FIRMWARE_VCHIQ_INIT is
already used in
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
So this patch fixes also a build issue.
>
> Signed-off-by: Eric Anholt <eric@anholt.net>
> ---
> include/soc/bcm2835/raspberrypi-firmware.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/include/soc/bcm2835/raspberrypi-firmware.h
> b/include/soc/bcm2835/raspberrypi-firmware.h
> index 3fb357193f09..a06baffdf580 100644
> --- a/include/soc/bcm2835/raspberrypi-firmware.h
> +++ b/include/soc/bcm2835/raspberrypi-firmware.h
> @@ -109,6 +109,8 @@ enum rpi_firmware_property_tag {
> RPI_FIRMWARE_FRAMEBUFFER_SET_OVERSCAN = 0x0004800a,
> RPI_FIRMWARE_FRAMEBUFFER_SET_PALETTE = 0x0004800b,
>
> + RPI_FIRMWARE_VCHIQ_INIT = 0x00048010,
> +
> RPI_FIRMWARE_GET_COMMAND_LINE = 0x00050001,
> RPI_FIRMWARE_GET_DMA_CHANNELS = 0x00060001,
> };
> --
> 2.9.3
>
>
> _______________________________________________
> linux-rpi-kernel mailing list
> linux-rpi-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rpi-kernel
^ permalink raw reply [flat|nested] 26+ messages in thread* Re: [PATCH 3/4] ARM: bcm2835: Add #define for VCHIQ property message.
@ 2016-10-15 7:53 ` Stefan Wahren
0 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 7:53 UTC (permalink / raw)
To: Eric Anholt, Greg Kroah-Hartman
Cc: linux-kernel, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel
Hi Greg,
> Eric Anholt <eric@anholt.net> hat am 3. Oktober 2016 um 20:52 geschrieben:
>
>
> This comes from the downstream tree and is needed for the new VCHIQ
> driver in staging.
maybe it's not clear from the commit message, but RPI_FIRMWARE_VCHIQ_INIT is
already used in
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
So this patch fixes also a build issue.
>
> Signed-off-by: Eric Anholt <eric@anholt.net>
> ---
> include/soc/bcm2835/raspberrypi-firmware.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/include/soc/bcm2835/raspberrypi-firmware.h
> b/include/soc/bcm2835/raspberrypi-firmware.h
> index 3fb357193f09..a06baffdf580 100644
> --- a/include/soc/bcm2835/raspberrypi-firmware.h
> +++ b/include/soc/bcm2835/raspberrypi-firmware.h
> @@ -109,6 +109,8 @@ enum rpi_firmware_property_tag {
> RPI_FIRMWARE_FRAMEBUFFER_SET_OVERSCAN = 0x0004800a,
> RPI_FIRMWARE_FRAMEBUFFER_SET_PALETTE = 0x0004800b,
>
> + RPI_FIRMWARE_VCHIQ_INIT = 0x00048010,
> +
> RPI_FIRMWARE_GET_COMMAND_LINE = 0x00050001,
> RPI_FIRMWARE_GET_DMA_CHANNELS = 0x00060001,
> };
> --
> 2.9.3
>
>
> _______________________________________________
> linux-rpi-kernel mailing list
> linux-rpi-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rpi-kernel
^ permalink raw reply [flat|nested] 26+ messages in thread* [PATCH 3/4] ARM: bcm2835: Add #define for VCHIQ property message.
2016-10-15 7:53 ` Stefan Wahren
@ 2016-10-15 7:56 ` Stefan Wahren
-1 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 7:56 UTC (permalink / raw)
To: linux-arm-kernel
> Stefan Wahren <stefan.wahren@i2se.com> hat am 15. Oktober 2016 um 09:53
> geschrieben:
>
>
> Hi Greg,
>
> > Eric Anholt <eric@anholt.net> hat am 3. Oktober 2016 um 20:52 geschrieben:
> >
> >
> > This comes from the downstream tree and is needed for the new VCHIQ
> > driver in staging.
>
> maybe it's not clear from the commit message, but RPI_FIRMWARE_VCHIQ_INIT is
> already used in
> drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
>
> So this patch fixes also a build issue.
>
Sorry about the noise. You already applied it.
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 3/4] ARM: bcm2835: Add #define for VCHIQ property message.
@ 2016-10-15 7:56 ` Stefan Wahren
0 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 7:56 UTC (permalink / raw)
To: Eric Anholt, Greg Kroah-Hartman
Cc: linux-kernel, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel
> Stefan Wahren <stefan.wahren@i2se.com> hat am 15. Oktober 2016 um 09:53
> geschrieben:
>
>
> Hi Greg,
>
> > Eric Anholt <eric@anholt.net> hat am 3. Oktober 2016 um 20:52 geschrieben:
> >
> >
> > This comes from the downstream tree and is needed for the new VCHIQ
> > driver in staging.
>
> maybe it's not clear from the commit message, but RPI_FIRMWARE_VCHIQ_INIT is
> already used in
> drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c
>
> So this patch fixes also a build issue.
>
Sorry about the noise. You already applied it.
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 4/4] drivers/vchi: Remove dependency on CONFIG_BROKEN.
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-03 18:52 ` Eric Anholt
-1 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: linux-arm-kernel
The driver builds now.
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/staging/vc04_services/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vc04_services/Kconfig b/drivers/staging/vc04_services/Kconfig
index 9676fb29075a..db8e1beb89f9 100644
--- a/drivers/staging/vc04_services/Kconfig
+++ b/drivers/staging/vc04_services/Kconfig
@@ -1,6 +1,6 @@
config BCM2708_VCHIQ
tristate "Videocore VCHIQ"
- depends on RASPBERRYPI_FIRMWARE && BROKEN
+ depends on RASPBERRYPI_FIRMWARE
default y
help
Kernel to VideoCore communication interface for the
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH 4/4] drivers/vchi: Remove dependency on CONFIG_BROKEN.
@ 2016-10-03 18:52 ` Eric Anholt
0 siblings, 0 replies; 26+ messages in thread
From: Eric Anholt @ 2016-10-03 18:52 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley, Eric Anholt
The driver builds now.
Signed-off-by: Eric Anholt <eric@anholt.net>
---
drivers/staging/vc04_services/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vc04_services/Kconfig b/drivers/staging/vc04_services/Kconfig
index 9676fb29075a..db8e1beb89f9 100644
--- a/drivers/staging/vc04_services/Kconfig
+++ b/drivers/staging/vc04_services/Kconfig
@@ -1,6 +1,6 @@
config BCM2708_VCHIQ
tristate "Videocore VCHIQ"
- depends on RASPBERRYPI_FIRMWARE && BROKEN
+ depends on RASPBERRYPI_FIRMWARE
default y
help
Kernel to VideoCore communication interface for the
--
2.9.3
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
2016-10-03 18:52 ` Eric Anholt
@ 2016-10-13 7:05 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 26+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-13 7:05 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> The driver builds now.
>
> Signed-off-by: Eric Anholt <eric@anholt.net>
> ---
> drivers/staging/vc04_services/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/vc04_services/Kconfig b/drivers/staging/vc04_services/Kconfig
> index 9676fb29075a..db8e1beb89f9 100644
> --- a/drivers/staging/vc04_services/Kconfig
> +++ b/drivers/staging/vc04_services/Kconfig
> @@ -1,6 +1,6 @@
> config BCM2708_VCHIQ
> tristate "Videocore VCHIQ"
> - depends on RASPBERRYPI_FIRMWARE && BROKEN
> + depends on RASPBERRYPI_FIRMWARE
> default y
> help
> Kernel to VideoCore communication interface for the
I've dropped this patch from my branch as there are build errors on
arm64 systems still, and we don't want regressions like that.
I've forwarded you the error messages, and I'll be glad to add this
patch back once these issues are fixed.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
@ 2016-10-13 7:05 ` Greg Kroah-Hartman
0 siblings, 0 replies; 26+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-13 7:05 UTC (permalink / raw)
To: Eric Anholt
Cc: linux-rpi-kernel, linux-arm-kernel, linux-kernel, Stephen Warren,
Lee Jones, bcm-kernel-feedback-list, Dom Cobley
On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> The driver builds now.
>
> Signed-off-by: Eric Anholt <eric@anholt.net>
> ---
> drivers/staging/vc04_services/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/vc04_services/Kconfig b/drivers/staging/vc04_services/Kconfig
> index 9676fb29075a..db8e1beb89f9 100644
> --- a/drivers/staging/vc04_services/Kconfig
> +++ b/drivers/staging/vc04_services/Kconfig
> @@ -1,6 +1,6 @@
> config BCM2708_VCHIQ
> tristate "Videocore VCHIQ"
> - depends on RASPBERRYPI_FIRMWARE && BROKEN
> + depends on RASPBERRYPI_FIRMWARE
> default y
> help
> Kernel to VideoCore communication interface for the
I've dropped this patch from my branch as there are build errors on
arm64 systems still, and we don't want regressions like that.
I've forwarded you the error messages, and I'll be glad to add this
patch back once these issues are fixed.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
2016-10-13 7:05 ` Greg Kroah-Hartman
@ 2016-10-15 8:35 ` Stefan Wahren
-1 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 8:35 UTC (permalink / raw)
To: linux-arm-kernel
> Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 13. Oktober 2016 um
> 09:05 geschrieben:
>
>
> On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> > The driver builds now.
> >
> > Signed-off-by: Eric Anholt <eric@anholt.net>
> > ---
> > drivers/staging/vc04_services/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/staging/vc04_services/Kconfig
> > b/drivers/staging/vc04_services/Kconfig
> > index 9676fb29075a..db8e1beb89f9 100644
> > --- a/drivers/staging/vc04_services/Kconfig
> > +++ b/drivers/staging/vc04_services/Kconfig
> > @@ -1,6 +1,6 @@
> > config BCM2708_VCHIQ
> > tristate "Videocore VCHIQ"
> > - depends on RASPBERRYPI_FIRMWARE && BROKEN
> > + depends on RASPBERRYPI_FIRMWARE
> > default y
> > help
> > Kernel to VideoCore communication interface for the
>
> I've dropped this patch from my branch as there are build errors on
> arm64 systems still, and we don't want regressions like that.
>
> I've forwarded you the error messages, and I'll be glad to add this
> patch back once these issues are fixed.
I ask the author of this downstream pull request [1] to send the VHCIQ part as
indiviual patches.
He is interested to submit them upstream.
[1] - https://github.com/raspberrypi/linux/pull/1611
>
> thanks,
>
> greg k-h
>
> _______________________________________________
> linux-rpi-kernel mailing list
> linux-rpi-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rpi-kernel
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
@ 2016-10-15 8:35 ` Stefan Wahren
0 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 8:35 UTC (permalink / raw)
To: Eric Anholt, Greg Kroah-Hartman
Cc: linux-kernel, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel
> Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 13. Oktober 2016 um
> 09:05 geschrieben:
>
>
> On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> > The driver builds now.
> >
> > Signed-off-by: Eric Anholt <eric@anholt.net>
> > ---
> > drivers/staging/vc04_services/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/staging/vc04_services/Kconfig
> > b/drivers/staging/vc04_services/Kconfig
> > index 9676fb29075a..db8e1beb89f9 100644
> > --- a/drivers/staging/vc04_services/Kconfig
> > +++ b/drivers/staging/vc04_services/Kconfig
> > @@ -1,6 +1,6 @@
> > config BCM2708_VCHIQ
> > tristate "Videocore VCHIQ"
> > - depends on RASPBERRYPI_FIRMWARE && BROKEN
> > + depends on RASPBERRYPI_FIRMWARE
> > default y
> > help
> > Kernel to VideoCore communication interface for the
>
> I've dropped this patch from my branch as there are build errors on
> arm64 systems still, and we don't want regressions like that.
>
> I've forwarded you the error messages, and I'll be glad to add this
> patch back once these issues are fixed.
I ask the author of this downstream pull request [1] to send the VHCIQ part as
indiviual patches.
He is interested to submit them upstream.
[1] - https://github.com/raspberrypi/linux/pull/1611
>
> thanks,
>
> greg k-h
>
> _______________________________________________
> linux-rpi-kernel mailing list
> linux-rpi-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rpi-kernel
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
2016-10-15 8:35 ` Stefan Wahren
@ 2016-10-15 11:27 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 26+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-15 11:27 UTC (permalink / raw)
To: linux-arm-kernel
On Sat, Oct 15, 2016 at 10:35:25AM +0200, Stefan Wahren wrote:
>
> > Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 13. Oktober 2016 um
> > 09:05 geschrieben:
> >
> >
> > On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> > > The driver builds now.
> > >
> > > Signed-off-by: Eric Anholt <eric@anholt.net>
> > > ---
> > > drivers/staging/vc04_services/Kconfig | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/staging/vc04_services/Kconfig
> > > b/drivers/staging/vc04_services/Kconfig
> > > index 9676fb29075a..db8e1beb89f9 100644
> > > --- a/drivers/staging/vc04_services/Kconfig
> > > +++ b/drivers/staging/vc04_services/Kconfig
> > > @@ -1,6 +1,6 @@
> > > config BCM2708_VCHIQ
> > > tristate "Videocore VCHIQ"
> > > - depends on RASPBERRYPI_FIRMWARE && BROKEN
> > > + depends on RASPBERRYPI_FIRMWARE
> > > default y
> > > help
> > > Kernel to VideoCore communication interface for the
> >
> > I've dropped this patch from my branch as there are build errors on
> > arm64 systems still, and we don't want regressions like that.
> >
> > I've forwarded you the error messages, and I'll be glad to add this
> > patch back once these issues are fixed.
>
> I ask the author of this downstream pull request [1] to send the VHCIQ part as
> indiviual patches.
> He is interested to submit them upstream.
>
> [1] - https://github.com/raspberrypi/linux/pull/1611
Ok, but what does that mean for us here? The code should be all now
merged into our "upstream" trees, it just needs to be cleaned up
properly.
confused,
greg k-h
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
@ 2016-10-15 11:27 ` Greg Kroah-Hartman
0 siblings, 0 replies; 26+ messages in thread
From: Greg Kroah-Hartman @ 2016-10-15 11:27 UTC (permalink / raw)
To: Stefan Wahren
Cc: Eric Anholt, linux-kernel, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel
On Sat, Oct 15, 2016 at 10:35:25AM +0200, Stefan Wahren wrote:
>
> > Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 13. Oktober 2016 um
> > 09:05 geschrieben:
> >
> >
> > On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> > > The driver builds now.
> > >
> > > Signed-off-by: Eric Anholt <eric@anholt.net>
> > > ---
> > > drivers/staging/vc04_services/Kconfig | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/staging/vc04_services/Kconfig
> > > b/drivers/staging/vc04_services/Kconfig
> > > index 9676fb29075a..db8e1beb89f9 100644
> > > --- a/drivers/staging/vc04_services/Kconfig
> > > +++ b/drivers/staging/vc04_services/Kconfig
> > > @@ -1,6 +1,6 @@
> > > config BCM2708_VCHIQ
> > > tristate "Videocore VCHIQ"
> > > - depends on RASPBERRYPI_FIRMWARE && BROKEN
> > > + depends on RASPBERRYPI_FIRMWARE
> > > default y
> > > help
> > > Kernel to VideoCore communication interface for the
> >
> > I've dropped this patch from my branch as there are build errors on
> > arm64 systems still, and we don't want regressions like that.
> >
> > I've forwarded you the error messages, and I'll be glad to add this
> > patch back once these issues are fixed.
>
> I ask the author of this downstream pull request [1] to send the VHCIQ part as
> indiviual patches.
> He is interested to submit them upstream.
>
> [1] - https://github.com/raspberrypi/linux/pull/1611
Ok, but what does that mean for us here? The code should be all now
merged into our "upstream" trees, it just needs to be cleaned up
properly.
confused,
greg k-h
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
2016-10-15 11:27 ` Greg Kroah-Hartman
@ 2016-10-15 12:03 ` Stefan Wahren
-1 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 12:03 UTC (permalink / raw)
To: linux-arm-kernel
Hi Greg,
> Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 15. Oktober 2016 um
> 13:27 geschrieben:
>
>
> On Sat, Oct 15, 2016 at 10:35:25AM +0200, Stefan Wahren wrote:
> >
> > > Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 13. Oktober 2016 um
> > > 09:05 geschrieben:
> > >
> > >
> > > On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> > > > The driver builds now.
> > > >
> > > > Signed-off-by: Eric Anholt <eric@anholt.net>
> > > > ---
> > > > drivers/staging/vc04_services/Kconfig | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/drivers/staging/vc04_services/Kconfig
> > > > b/drivers/staging/vc04_services/Kconfig
> > > > index 9676fb29075a..db8e1beb89f9 100644
> > > > --- a/drivers/staging/vc04_services/Kconfig
> > > > +++ b/drivers/staging/vc04_services/Kconfig
> > > > @@ -1,6 +1,6 @@
> > > > config BCM2708_VCHIQ
> > > > tristate "Videocore VCHIQ"
> > > > - depends on RASPBERRYPI_FIRMWARE && BROKEN
> > > > + depends on RASPBERRYPI_FIRMWARE
> > > > default y
> > > > help
> > > > Kernel to VideoCore communication interface for the
> > >
> > > I've dropped this patch from my branch as there are build errors on
> > > arm64 systems still, and we don't want regressions like that.
> > >
> > > I've forwarded you the error messages, and I'll be glad to add this
> > > patch back once these issues are fixed.
> >
> > I ask the author of this downstream pull request [1] to send the VHCIQ part
> > as
> > indiviual patches.
> > He is interested to submit them upstream.
> >
> > [1] - https://github.com/raspberrypi/linux/pull/1611
>
> Ok, but what does that mean for us here? The code should be all now
> merged into our "upstream" trees, it just needs to be cleaned up
> properly.
>
> confused,
>
> greg k-h
sorry for the confusion. I mean that he wants to send email patches based on
your staging tree. So we avoid patching the kernel twice.
The link was only for illustration that he already fixed most of the arm64 build
issues.
Stefan
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH 4/4] staging/vchi: Remove dependency on CONFIG_BROKEN.
@ 2016-10-15 12:03 ` Stefan Wahren
0 siblings, 0 replies; 26+ messages in thread
From: Stefan Wahren @ 2016-10-15 12:03 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Eric Anholt, linux-kernel, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel
Hi Greg,
> Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 15. Oktober 2016 um
> 13:27 geschrieben:
>
>
> On Sat, Oct 15, 2016 at 10:35:25AM +0200, Stefan Wahren wrote:
> >
> > > Greg Kroah-Hartman <gregkh@linuxfoundation.org> hat am 13. Oktober 2016 um
> > > 09:05 geschrieben:
> > >
> > >
> > > On Mon, Oct 03, 2016 at 11:52:09AM -0700, Eric Anholt wrote:
> > > > The driver builds now.
> > > >
> > > > Signed-off-by: Eric Anholt <eric@anholt.net>
> > > > ---
> > > > drivers/staging/vc04_services/Kconfig | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/drivers/staging/vc04_services/Kconfig
> > > > b/drivers/staging/vc04_services/Kconfig
> > > > index 9676fb29075a..db8e1beb89f9 100644
> > > > --- a/drivers/staging/vc04_services/Kconfig
> > > > +++ b/drivers/staging/vc04_services/Kconfig
> > > > @@ -1,6 +1,6 @@
> > > > config BCM2708_VCHIQ
> > > > tristate "Videocore VCHIQ"
> > > > - depends on RASPBERRYPI_FIRMWARE && BROKEN
> > > > + depends on RASPBERRYPI_FIRMWARE
> > > > default y
> > > > help
> > > > Kernel to VideoCore communication interface for the
> > >
> > > I've dropped this patch from my branch as there are build errors on
> > > arm64 systems still, and we don't want regressions like that.
> > >
> > > I've forwarded you the error messages, and I'll be glad to add this
> > > patch back once these issues are fixed.
> >
> > I ask the author of this downstream pull request [1] to send the VHCIQ part
> > as
> > indiviual patches.
> > He is interested to submit them upstream.
> >
> > [1] - https://github.com/raspberrypi/linux/pull/1611
>
> Ok, but what does that mean for us here? The code should be all now
> merged into our "upstream" trees, it just needs to be cleaned up
> properly.
>
> confused,
>
> greg k-h
sorry for the confusion. I mean that he wants to send email patches based on
your staging tree. So we avoid patching the kernel twice.
The link was only for illustration that he already fixed most of the arm64 build
issues.
Stefan
^ permalink raw reply [flat|nested] 26+ messages in thread