Hi,
Am 28.03.21 um 07:58 schrieb Ryutaroh Matsumoto:
>> I think the root cause of this issue is that both vc4.ko and snd_bcm2835.ko
>> try to provide ALSA sinks to HDMI audio outputs from RPi.
>> Why do the two drivers provide the same functionality for the same device?
>> It seems nonsense.
>> Th
10205&id=96ae327678eceabf455b11a88ba14ad540d4b046
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/staging/vc04_services?h=next-20210205&id=88753cc19f087abe0d39644b844e67a59cfb5a3d
Could you please try?
Best regards
Stefan
>
> I can consistently repr
ixes: 4184da4f316a ("staging: vchiq: fix __user annotations")
>
> Signed-off-by: Phil Elwell
Tested-by: Stefan Wahren
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
s://github.com/0day-ci/linux/commits/Stefan-Wahren/staging-bcm2835-camera-Clean-up-driver/20200329-225704
>
>
> in testcase: boot
>
> on test machine: qemu-system-x86_64 -enable-kvm -cpu SandyBridge -smp 2 -m 8G
>
> caused below changes (please refer t
The function mmal_setup_components has to many indention levels. So move
the setup code for encode component in its own function.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 185 ++---
1 file changed, 91 insertions(+), 94 deletions
Except of patch 2 all these patches tries to clean up the bcm2835-camera
driver.
Stefan Wahren (9):
staging: bcm2835-camera: Drop PREVIEW_LAYER
staging: bcm2835-camera: Activate V4L2_EXPOSURE_METERING_MATRIX
handling
staging: bcm2835-camera: Make struct indentation consistent
staging
This simplifies set_framerate_params and avoids the multiple assignment
in one line by moving the fps_high handling out of the if statement.
Signed-off-by: Stefan Wahren
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 15 ++-
1 file changed, 6 insertions(+), 9
There are a lot of multiline statements which can be reduced.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 57 +-
.../vc04_services/bcm2835-camera/controls.c| 3 +-
2 files changed, 23 insertions(+), 37 deletions(-)
diff
The indentation of struct members wasn't consistent over the whole driver.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/bcm2835-camera.h | 60 +++---
.../vc04_services/bcm2835-camera/controls.c| 4 +-
.../vc04_services/bcm2835-camera/mmal-com
The function mmal_setup_components has to many indention levels. So move
the setup code for video component in its own function.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 88 --
1 file changed, 48 insertions(+), 40 deletions
We can reduce the indentation in the loop by using continue in case the
effect doesn't match.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/controls.c| 63 +++---
1 file changed, 31 insertions(+), 32 deletions(-)
diff --git a/drivers/st
This define is used only once. So drop the define and init the layer
directly.
Signed-off-by: Stefan Wahren
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 2 +-
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.h | 2 --
2 files changed, 1 insertion(+), 3
I don't see any reason to keep this TODO, so activate the
V4L2_EXPOSURE_METERING_MATRIX handling.
Signed-off-by: Stefan Wahren
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/drivers/st
We can reduce the indentation in mmal_setup_components further by
returning early in error case.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 60 +++---
1 file changed, 31 insertions(+), 29 deletions(-)
diff --git a/drivers/staging
The struct bm2835_mmal_v4l2_ctrl contains an ignore_errors flag which
was always set to false. So drop the unused flag.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/controls.c| 34 +-
1 file changed, 1 insertion(+), 33 deletions(-)
diff --git a
The initializer lists for the V4L2 controls are hard to read.
So improve this by using designators.
Signed-off-by: Stefan Wahren
---
.../vc04_services/bcm2835-camera/controls.c| 424 ++---
1 file changed, 277 insertions(+), 147 deletions(-)
diff --git a/drivers/staging
These 2 patches improves the V4L2 controls in the bcm2835 camera driver.
Stefan Wahren (2):
staging: bcm2835-camera: Drop unused ignore_errors flag
staging: bcm2835-camera: Use designators to init V4L2 controls
.../vc04_services/bcm2835-camera/controls.c| 458
far I've basically just
> diffed the output of 'cat /dev/vchiq', run the program above with
> a few different values, and run vchiq_test a few times.
i consider this as sufficient, but i'm not the VCHIQ expert.
Thanks
Stefan
>
> These were applied to the staging-ne
Am 20.11.19 um 14:38 schrieb Krzysztof Kozlowski:
> Adjust indentation from spaces to tab (+optional two spaces) as in
> coding style with command like:
> $ sed -e 's/^/\t/' -i */Kconfig
>
> Signed-off-by: Krzysztof Kozlowski
Hello Marcelo,
Am 11.11.19 um 18:14 schrieb Marcelo Diop-Gonzalez:
> Moving the DT node check to vchiq_platform_init()
> removes the need for these.
this comment does match to your changes. Also i'm missing a why this is
necessary.
>
> Signed-off-by: Marcelo Diop-Gonzalez
> ---
> .../interface
ree.
Please make sure you use the following one (or a mirror):
https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
and operate on the branch staging-next
Please fix this up and send a new version.
Regards
Stefan
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
tter start with the 2nd part of Joe's suggestion
first.
Stefan
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Hi Rohit,
On 11.09.19 15:51, Rohit Sarkar wrote:
> When the number of bytes to be printed exceeds the limit snprintf
> returns the number of bytes that would have been printed (if there was
> no truncation). This might cause issues, hence use scnprintf which
> returns the actual number of bytes pr
sion, as described in commit fc1d8e7cca2d
> ("mm: introduce put_user_page*(), placeholder versions").
>
> Acked-by: Greg Kroah-Hartman
>
> Cc: Eric Anholt
> Cc: Stefan Wahren
> Cc: Greg Kroah-Hartman
> Cc: Mihaela Muraru
> Cc: Suniel Mahesh
> Cc: Al V
Hi Yue,
Am 26.07.19 um 11:26 schrieb YueHaibing:
> Fix gcc used-but-set-variable warning:
just a nit. It is call "unused-but-set-variable"
Acked-by: Stefan Wahren
>
> drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c: In function
> vchiq_release_inter
From: Dave Stevenson
RGB3/BGR3 end up being 3 bytes per pixel, which meant that
the alignment code ended up trying to align using bitmasking
with a mask of 96.
That doesn't work, so switch to an arithmetic alignment for
those formats.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan W
From: Dave Stevenson
Fixes a v4l2-compliance failure
v4l2-test-buffers.cpp(415): g_field() == V4L2_FIELD_ANY
The driver only ever produces progresive frames, so field should
always be set to V4L2_FIELD_NONE.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
From: Dave Stevenson
Fixes up a checkpatch error "Avoid using bool structure members
because of possible alignment issues".
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2
d-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/vc04_services/bcm2835-c
used in the REQBUFS case where it provides
the buffer properties.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c| 16
1 file changed, 16 inser
From: Dave Stevenson
Most of the headers use ifndef FOO_H, whilst mmal-parameters.h
used ifndef __FOO_H.
Revise mmal-parameters.h to drop the underscores and make the
headers all consistent.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro
From: Dave Stevenson
Clear checkpatch complaints of "multiple assignments should be avoided"
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 5 +++
From: Dave Stevenson
Set the sequence number in vb2_v4l2_buffer mainly so the
latest v4l2-ctl reports the frame rate correctly.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera
From: Dave Stevenson
mmal-common.h and mmal-msg.h didn't have the normal
ifndef FOO / define FOO / endif protection to stop it being
included multiple times. Add it.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho C
From: Dave Stevenson
Fixes up all the checkpatch error "line over 80 characters" in
mmal-parameters.h
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services/bcm2835-camera/mmal-paramete
7;t handled and returned an error up the stack.
Handle the condition correctly by returning it to the component
if streaming, or returning with an error if stopping streaming.
Fixes: 938416707071 ("staging: bcm2835-camera: Remove V4L2/MMAL buffer
remapping")
Signed-off-by: Dave Stevenson
Sig
ed value")
Reported-by: Colin Ian King
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/dr
the appropriate enum.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services/bcm2835-camera/controls.c| 37 --
1 file changed, 13 insertions(+), 24 deletions(-)
diff --git a/drivers
From: Dave Stevenson
Remove a todo which has been done.
Remove a template line that was redundant.
Make a comment clearer as to the non-obvious meaning of a field.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers
From: Dave Stevenson
We have numerous lines over 80 chars, or oddly split. Many
of these are due to using long enum names such as
MMAL_COMPONENT_CAMERA.
Reduce the length of these enum names.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro
This is an attempt to help Dave Stevenson to get all the fixes and
improvements of the bcm2835-camera driver into mainline.
Mostly i only polished the commit logs for upstream.
The series based on the latest bugfix V2 of staging: bcm2835-camera: Restore
return behavior of ctrl_set_bitrate().
Cha
From: Dave Stevenson
Fix checkpatch "Alignment should match open parenthesis"
errors.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 9
.../vc0
From: Dave Stevenson
Fix several instances where it is easier to return
early on error conditions than handle it as an else
clause. As requested by Mauro.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services
.
Add a workqueue to schedule the call vchi_bulk_queue_receive
in an alternate context to avoid the lock up.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services/bcm2835-camera/mmal-vchiq.c | 101
From: Dave Stevenson
Fix checkpatch errors "Avoid multiple line dereference"
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 11 ---
1 fi
s Dec 2013, so the chances of having a firmware that still
has that issue is so close to zero that the workaround can be
removed.
Link: https://github.com/raspberrypi/linux/pull/2782/
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho C
From: Dave Stevenson
Fix checkpatch errors for missing blank lines after variable
or structure declarations.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.h
From: Dave Stevenson
Fix mismatched or missing brace issues flagged by checkpatch.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 3 ++-
drivers/staging
From: Dave Stevenson
Forward MMAL key frame flags to the V4L2 buffers.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 3 +++
1 file changed, 3 insertions
therefore there is no need for that check, and indeed it is wrong
as there is no need to submit all the buffers before starting streaming.
Fixes: 938416707071 ("staging: bcm2835-camera: Remove V4L2/MMAL buffer
remapping")
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Ac
From: Dave Stevenson
None of the listed author email addresses were valid.
Keep list of authors and the companies they represented.
Update my email address.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
drivers/staging
his is not valid whilst in an atomic context
due to the spinlock.
There is no need for this to be a spinlock as a standard mutex is
sufficient.
Fixes: 950fd867c635 ("staging: bcm2835-camera: Replace open-coded idr with a
struct idr.")
Signed-off-by: Dave Stevenson
Signed-off-by: Ste
From: Dave Stevenson
With the recent change to match MMAL and V4L2 buffers there
is a need to wait for all MMAL buffers to be returned during
stop_streaming.
Fixes: 938416707071 ("staging: bcm2835-camera: Remove V4L2/MMAL buffer
remapping")
Signed-off-by: Dave Stevenson
Signed-off-
From: Dave Stevenson
Fix checkpatch warnings over spaces around operators.
Many were around operations that can be replaced with the
BIT(x) macro, so replace with that where appropriate.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro
From: Dave Stevenson
Fix comment style violations in the header files.
Signed-off-by: Dave Stevenson
Signed-off-by: Stefan Wahren
Acked-by: Hans Verkuil
Acked-by: Mauro Carvalho Chehab
---
.../vc04_services/bcm2835-camera/mmal-msg-format.h | 95 ---
.../vc04_services/bcm2835
Hi Dave,
Am 28.06.19 um 19:29 schrieb Dave Stevenson:
> Hi Stefan
>
> Could you try running
> v4l2-ctl -v width=640,height=480,pixelformat=YU12
> before running v4l2-compliance? The default format is JPEG, and I just
> wonder if there is an issue lurking in the compression s
Hi Hans,
Am 28.06.19 um 10:06 schrieb Hans Verkuil:
> Hi Stefan,
>
> On 6/27/19 8:55 PM, Stefan Wahren wrote:
>> This is an attempt to help Dave Stevenson to get all the fixes and
>> improvements of the bcm2835-camera driver into mainline.
>>
>> Mostly i only poli
From: Dave Stevenson
Fixes up a checkpatch error "Avoid using bool structure members
because of possible alignment issues".
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.c | 12 ++--
drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.h |
From: Dave Stevenson
Fixes a v4l2-compliance failure
v4l2-test-buffers.cpp(415): g_field() == V4L2_FIELD_ANY
The driver only ever produces progresive frames, so field should
always be set to V4L2_FIELD_NONE.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm283
From: Dave Stevenson
RGB3/BGR3 end up being 3 bytes per pixel, which meant that
the alignment code ended up trying to align using bitmasking
with a mask of 96.
That doesn't work, so switch to an arithmetic alignment for
those formats.
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835
From: Dave Stevenson
The V4L2 control API was expanded to take 64 bit values in commit
0ba2aeb6dab (Apr 16 2014), but as this driver wasn't in the mainline
kernel at that point this was overlooked.
Update to use 64 bit values. This also fixes a couple of warnings
in 64 bit builds.
Signed-off-by
From: Dave Stevenson
Fixes a v4l2-compliance failure when passed a buffer that is
too small.
queue_setup wasn't handling the case where !(*nplanes), as
used from CREATE_BUFS and requiring the driver to sanity
check the provided buffer parameters. It was assuming that
it was always being used in t
From: Dave Stevenson
Most of the headers use ifndef FOO_H, whilst mmal-parameters.h
used ifndef __FOO_H.
Revise mmal-parameters.h to drop the underscores and make the
headers all consistent.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/mmal-parameters.h | 4 +
From: Dave Stevenson
Controls of type MMAL_CONTROL_TYPE_STD_MENU call v4l2_ctrl_new_std_menu
with a max value and a mask. The max value is one of the defined
values for the control, however in the config array there are several
entries where raw numbers have been used instead. Replace these
with
From: Dave Stevenson
With V4L2_CID_COLORFX_CBCR calling ctrl_set_colfx it was incorrectly
assigning the colour values to the enable field of dev->colourfx
instead of the u and v fields.
Correct the assignments.
Reported as a Coverity issue
Detected by CoverityScan CID#1419711 ("Unused value")
From: Dave Stevenson
Fixes up all the checkpatch error "line over 80 characters" in
mmal-parameters.h
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835-camera/mmal-parameters.h | 261 +
1 file changed, 159 insertions(+), 102 deletions(-)
diff --git a/drivers/stag
From: Dave Stevenson
Remove a todo which has been done.
Remove a template line that was redundant.
Make a comment clearer as to the non-obvious meaning of a field.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 11 +--
1 file changed, 1 inse
From: Dave Stevenson
mmal-common.h and mmal-msg.h didn't have the normal
ifndef FOO / define FOO / endif protection to stop it being
included multiple times. Add it.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/mmal-common.h | 3 +++
drivers/staging/vc04_servi
From: Dave Stevenson
Clear checkpatch complaints of "multiple assignments should be avoided"
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/vc04_services/bc
From: Dave Stevenson
The commit "staging: bcm2835-camera: Replace open-coded idr with a struct idr."
replaced an internal implementation of an idr with the standard functions
and a spinlock. idr_alloc(GFP_KERNEL) can sleep whilst calling kmem_cache_alloc
to allocate the new node, but this is not
This is an attempt to help Dave Stevenson to get all the fixes and
improvements of the bcm2835-camera driver into mainline.
Mostly i only polished the commit logs for upstream.
The series based on the latest bugfix V2 of staging: bcm2835-camera: Restore
return behavior of ctrl_set_bitrate().
Dav
From: Dave Stevenson
Fix comment style violations in the header files.
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835-camera/mmal-msg-format.h | 95 ---
.../vc04_services/bcm2835-camera/mmal-msg-port.h | 124 +--
.../vc04_services/bcm2835-camera/mmal
From: Dave Stevenson
Set the sequence number in vb2_v4l2_buffer mainly so the
latest v4l2-ctl reports the frame rate correctly.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 4
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.
From: Dave Stevenson
Fix mismatched or missing brace issues flagged by checkpatch.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 3 ++-
drivers/staging/vc04_services/bcm2835-camera/controls.c | 3 ++-
drivers/staging/vc04_services/bcm28
From: Dave Stevenson
We have numerous lines over 80 chars, or oddly split. Many
of these are due to using long enum names such as
MMAL_COMPONENT_CAMERA.
Reduce the length of these enum names.
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 156 +++
From: Dave Stevenson
Before commit "staging: bcm2835-camera: Remove V4L2/MMAL buffer remapping"
there was a need to ensure that there were sufficient buffers supplied from
the user to cover those being sent to the VPU (always 1).
Now the buffers are linked 1:1 between MMAL and V4L2,
therefore th
From: Dave Stevenson
When handling for V4L2_CID_MPEG_VIDEO_REPEAT_SEQ_HEADER was added
the firmware would reject the setting if H264 hadn't already been
selected. This was fixed in the firmware at that point, but to
enable backwards compatibility the returned error was ignored.
That was Dec 2013
From: Dave Stevenson
vchi_bulk_queue_receive will queue up to a default of 4
bulk receives on a connection before blocking.
If called from the VCHI service_callback thread, then
that thread is unable to service the VCHI_CALLBACK_BULK_RECEIVED
events that would enable the queue call to succeed.
A
From: Dave Stevenson
Forward MMAL key frame flags to the V4L2 buffers.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c
b/drivers
From: Dave Stevenson
None of the listed author email addresses were valid.
Keep list of authors and the companies they represented.
Update my email address.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 9 +
drivers/staging/vc04_serv
From: Dave Stevenson
The change to mapping V4L2 to MMAL buffers 1:1 didn't handle
the condition we get with raw pixel buffers (eg YUV and RGB)
direct from the camera's stills port. That sends the pixel buffer
and then an empty buffer with the EOS flag set. The EOS buffer
wasn't handled and return
From: Dave Stevenson
Fix several instances where it is easier to return
early on error conditions than handle it as an else
clause. As requested by Mauro.
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 109 +++--
1 file changed, 58 insert
From: Dave Stevenson
There is an awkward situation with H264 header bytes. Currently
they are returned with a PTS of 0 because they aren't associated
with a timestamped frame to encode. These are handled by either
returning the timestamp of the last buffer to have been received,
or in the case of
From: Dave Stevenson
With the recent change to match MMAL and V4L2 buffers there
is a need to wait for all MMAL buffers to be returned during
stop_streaming.
Fixes: 938416707071 ("staging: bcm2835-camera: Remove V4L2/MMAL buffer
remapping")
Signed-off-by: Dave Stevenson
---
.../vc04_services/
From: Dave Stevenson
Fix checkpatch warnings over spaces around operators.
Many were around operations that can be replaced with the
BIT(x) macro, so replace with that where appropriate.
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835-camera/controls.c| 32 +++--
From: Dave Stevenson
Fix checkpatch errors for missing blank lines after variable
or structure declarations.
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/vc04_services/bcm2835
From: Dave Stevenson
Fix checkpatch errors "Avoid multiple line dereference"
Signed-off-by: Dave Stevenson
---
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/vc04_services/bcm2835-cam
From: Dave Stevenson
Fix checkpatch "Alignment should match open parenthesis"
errors.
Signed-off-by: Dave Stevenson
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 9
.../vc04_services/bcm2835-camera/controls.c| 25 ++
.../vc04_services/bcm2835-ca
From: Dave Stevenson
H264 header come from VC with 0 timestamps, which means they get a
strange timestamp when processed with VC/kernel start times,
particularly if used with the inline header option.
Remember the last frame timestamp and use that if set, or otherwise
use the kernel start time.
explaining comment and a debug message
to verify that the bug has been fixed in firmware.
Fixes: 52c4dfcead49 ("Staging: vc04_services: Cleanup in ctrl_set_bitrate()")
Signed-off-by: Stefan Wahren
---
.../staging/vc04_services/bcm2835-camera/controls.c | 19 ---
Hi Dan,
hi Dave,
Am 25.06.19 um 09:55 schrieb Dan Carpenter:
> On Tue, Jun 25, 2019 at 12:13:15AM +0200, Stefan Wahren wrote:
>> The commit 52c4dfcead49 ("Staging: vc04_services: Cleanup in
>> ctrl_set_bitrate()") changed the return behavior of ctrl_set_bitrate().
: vc04_services: Cleanup in ctrl_set_bitrate()")
Signed-off-by: Stefan Wahren
---
drivers/staging/vc04_services/bcm2835-camera/controls.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/vc04_services/bcm2835-camera/controls.c
b/drivers/staging/vc04_ser
he JAILLET
there has been a fix for this, which isn't upstreamed yet. The preferred
solution is to replace the spin_lock with a mutex. Since i'm currently
working on this i would take care of this.
Sorry about this.
Stefan
___
d
Am 16.05.2019 um 23:33 schrieb Madhumitha Prabakaran:
Fix the warning issued by checkpatch
Prefer kernel type 'u32' over 'uint32_t'.
Along with that include a blank line after a declaration
to maintain Linux kernel coding style.
Signed-off-by: Madhumitha Prabakaran
Ack
ole camera driver
and name the patch like
staging: bcm2835-camera: prefer kernel types
Regards
Stefan
>
> Signed-off-by: Madhumitha Prabakaran
> ---
> drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> dif
27;s mainly used as an interface to V4L2 and ALSA devices.
>
> Fixes: ff5979ad8636 ("staging: vchiq_2835_arm: quit using custom
> down_interruptible()")
> Signed-off-by: Nicolas Saenz Julienne
This whole series is:
Acked-by: Stefan Wahren
_
Signed-off-by: Dan Carpenter
Good catch
Acked-by: Stefan Wahren
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
ed Value")
> Signed-off-by: Colin Ian King
Acked-by: Stefan Wahren
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
hiq_arm/vchiq_arm.c | 21 +++--
> .../interface/vchiq_arm/vchiq_core.c | 31 ---
> .../interface/vchiq_arm/vchiq_util.c | 6 ++--
> 4 files changed, 35 insertions(+), 25 deletions(-)
>
against which tree
s no errors, so I would just leave it out...
> I agree, I'll fix it.
i also agree with Dan, but this specific patch should revert the changes
of a772f116702e3f0afdd7e6acadc1b8fb3b20b9ff . So mentioned style issue
should be fixed in a separate patch.
Regards Stefan
>
> R
1 - 100 of 460 matches
Mail list logo
