This message is generated daily by a cron job that builds media_tree for
the kernels and architectures in the list below.
Results of the daily build of media_tree:
date: Tue Sep 11 04:00:15 CEST 2018
media-tree git hash:d842a7cf938b6e0f8a1aa9f1aec0476c9a599310
media_build
Hi Hugues,
On Monday, 10 September 2018 18:14:45 EEST Hugues FRUCHET wrote:
> On 09/07/2018 04:18 PM, Laurent Pinchart wrote:
> > On Thursday, 16 August 2018 18:07:54 EEST Hugues FRUCHET wrote:
> >> On 08/16/2018 12:10 PM, jacopo mondi wrote:
> >>> On Mon, Aug 13, 2018 at 12:19:46PM +0200, Hugues
On Mon, Sep 10, 2018 at 11:34 AM, Mauro Carvalho Chehab
wrote:
> Em Mon, 10 Sep 2018 09:18:05 -0700
> Kees Cook escreveu:
>
>> On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
>> wrote:
>> > The strncpy() function is being deprecated upstream. Replace
>> > it by the safer strscpy().
>>
>>
Hi Hugues,
(Hans, there's a question for you below)
On Monday, 10 September 2018 17:43:27 EEST Hugues FRUCHET wrote:
> On 09/10/2018 12:46 PM, Laurent Pinchart wrote:
> > On Monday, 10 September 2018 13:23:41 EEST Hugues FRUCHET wrote:
> >> On 09/06/2018 03:31 PM, Laurent Pinchart wrote:
> >>>
On Mon, 2018-09-10 at 13:16 -0400, Nicolas Dufresne wrote:
> Le lundi 10 septembre 2018 à 12:37 -0300, Ezequiel Garcia a écrit :
> > On Mon, 2018-09-10 at 17:00 +0200, Hans Verkuil wrote:
> > > From: Hans Verkuil
> > >
> > > state->info was NULL since I completely forgot to set state->info.
> >
The strcpy() function is being deprecated upstream. Replace
it by the safer strscpy().
Signed-off-by: Mauro Carvalho Chehab
--
v2: removed the changes at the imon driver. There, the is a debugfs
node with a store function using DEVICE_ATTR() passing a char * buf
without any sizing information.
Em Mon, 10 Sep 2018 16:48:47 -0300
Mauro Carvalho Chehab escreveu:
> Em Mon, 10 Sep 2018 09:16:35 -0700
> Kees Cook escreveu:
>
> > On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
> > wrote:
> > > The strcpy() function is being deprecated upstream. Replace
> > > it by the safer
Em Mon, 10 Sep 2018 09:16:35 -0700
Kees Cook escreveu:
> On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
> wrote:
> > The strcpy() function is being deprecated upstream. Replace
> > it by the safer strscpy().
>
> Did you verify that all the destination buffers here are arrays and
>
Em Mon, 10 Sep 2018 09:18:05 -0700
Kees Cook escreveu:
> On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
> wrote:
> > The strncpy() function is being deprecated upstream. Replace
> > it by the safer strscpy().
>
> This one I'm quite concerned about. This could lead to kernel memory
>
Le lundi 10 septembre 2018 à 12:37 -0300, Ezequiel Garcia a écrit :
> On Mon, 2018-09-10 at 17:00 +0200, Hans Verkuil wrote:
> > From: Hans Verkuil
> >
> > state->info was NULL since I completely forgot to set state->info.
> > Oops.
> >
> > Reported-by: Ezequiel Garcia
> > Signed-off-by: Hans
On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
wrote:
> The strncpy() function is being deprecated upstream. Replace
> it by the safer strscpy().
This one I'm quite concerned about. This could lead to kernel memory
exposures if any of the callers depend on strncpy()'s trailing
On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
wrote:
> The strcpy() function is being deprecated upstream. Replace
> it by the safer strscpy().
Did you verify that all the destination buffers here are arrays and
not pointers? For example:
struct thing {
char buffer[64];
char *ptr;
On Mon, Sep 10, 2018 at 5:19 AM, Mauro Carvalho Chehab
wrote:
> The implementation of strscpy() is more robust and safer.
>
> That's now the recommended way to copy NUL terminated strings.
This looks fine since I don't see anything using the strlcpy() return
value (the return value meaning
On 09/10/2018 05:44 PM, Ezequiel Garcia wrote:
> On Mon, 2018-09-10 at 17:23 +0200, Hans Verkuil wrote:
>> On 09/10/2018 05:21 PM, Ezequiel Garcia wrote:
>>> The vicodec doesn't use the Subdev API, so drop the dependency.
>>>
>>> Signed-off-by: Ezequiel Garcia
>>> ---
>>>
On Mon, 2018-09-10 at 17:23 +0200, Hans Verkuil wrote:
> On 09/10/2018 05:21 PM, Ezequiel Garcia wrote:
> > The vicodec doesn't use the Subdev API, so drop the dependency.
> >
> > Signed-off-by: Ezequiel Garcia
> > ---
> > drivers/media/platform/vicodec/Kconfig | 2 +-
> > 1 file changed, 1
On Mon, 2018-09-10 at 17:00 +0200, Hans Verkuil wrote:
> From: Hans Verkuil
>
> state->info was NULL since I completely forgot to set state->info.
> Oops.
>
> Reported-by: Ezequiel Garcia
> Signed-off-by: Hans Verkuil
For both patches:
Tested-by: Ezequiel Garcia
With these changes, now
On 09/10/2018 05:21 PM, Ezequiel Garcia wrote:
> The vicodec doesn't use the Subdev API, so drop the dependency.
>
> Signed-off-by: Ezequiel Garcia
> ---
> drivers/media/platform/vicodec/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
The vicodec does not use the aborting field. In fact, this driver
can't really cancel any work, since it performs all the work
in device_run().
Signed-off-by: Ezequiel Garcia
---
drivers/media/platform/vicodec/vicodec-core.c | 11 ---
1 file changed, 11 deletions(-)
diff --git
The vicodec doesn't use the Subdev API, so drop the dependency.
Signed-off-by: Ezequiel Garcia
---
drivers/media/platform/vicodec/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/vicodec/Kconfig
b/drivers/media/platform/vicodec/Kconfig
index
Hi Laurent, Steve,
On 09/07/2018 04:18 PM, Laurent Pinchart wrote:
> Hello Hugues,
>
> On Thursday, 16 August 2018 18:07:54 EEST Hugues FRUCHET wrote:
>> On 08/16/2018 12:10 PM, jacopo mondi wrote:
>>> On Mon, Aug 13, 2018 at 12:19:46PM +0200, Hugues Fruchet wrote:
>>>
Mode setting depends
From: Hans Verkuil
state->info was NULL since I completely forgot to set state->info.
Oops.
Reported-by: Ezequiel Garcia
Signed-off-by: Hans Verkuil
---
drivers/media/platform/vicodec/vicodec-core.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git
From: Hans Verkuil
These functions did not return an error if state->info was NULL
or an unsupported pixelformat was selected (should not happen,
but just to be on the safe side).
Signed-off-by: Hans Verkuil
---
drivers/media/platform/vicodec/codec-v4l2-fwht.c | 15 +++
Hi Laurent,
On 09/10/2018 12:46 PM, Laurent Pinchart wrote:
> Hi Hugues,
>
> On Monday, 10 September 2018 13:23:41 EEST Hugues FRUCHET wrote:
>> On 09/06/2018 03:31 PM, Laurent Pinchart wrote:
>>> On Monday, 13 August 2018 13:19:45 EEST Hugues Fruchet wrote:
>>>
When switching from auto to
Hi Hans,
Thanks for the review.
On Mon, 2018-09-10 at 14:42 +0200, Hans Verkuil wrote:
> On 09/06/2018 12:00 AM, Ezequiel Garcia wrote:
> > From: Shunqian Zheng
> >
> > Add V4L2_CID_JPEG_QUANTIZATION compound control to allow userspace
> > configure the JPEG quantization tables.
> >
> >
On 09/06/2018 12:00 AM, Ezequiel Garcia wrote:
> From: Shunqian Zheng
>
> Add V4L2_CID_JPEG_QUANTIZATION compound control to allow userspace
> configure the JPEG quantization tables.
>
> Signed-off-by: Shunqian Zheng
> Signed-off-by: Ezequiel Garcia
> ---
>
Pull request for v3 of this new driver
(https://www.spinics.net/lists/arm-kernel/msg674871.html).
Regards,
Hans
The following changes since commit d842a7cf938b6e0f8a1aa9f1aec0476c9a599310:
media: adv7842: enable reduced fps detection (2018-08-31 10:03:51 -0400)
are available in the
The strcpy() function is being deprecated upstream. Replace
it by the safer strscpy().
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/common/saa7146/saa7146_video.c | 2 +-
drivers/media/dvb-core/dvb_frontend.c | 2 +-
drivers/media/dvb-frontends/mt312.c | 9
There are hot discussions upstream about getting rid of strcpy(), strncpy()
and strlcpy() in favor of the safer strscpy().
While there are exceptions where strscpy() may not be the best option
(for example, when filling records with fixed size), we don't have those
situations right now on media.
The strncpy() function is being deprecated upstream. Replace
it by the safer strscpy().
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-frontends/as102_fe.c | 2 +-
drivers/media/dvb-frontends/dib7000p.c | 3 ++-
drivers/media/dvb-frontends/dib8000.c
ARE YOU IN NEED OF LOAN @3% INTEREST RATE FOR BUSINESS AND PRIVATE
PURPOSES? IF YES:
FILL AND RETURN
Name:===
Amount needed:===
Duration:==
country:===
Purpose:===
Mobile number
Hi Hugues,
On Monday, 10 September 2018 13:23:41 EEST Hugues FRUCHET wrote:
> On 09/06/2018 03:31 PM, Laurent Pinchart wrote:
> > On Monday, 13 August 2018 13:19:45 EEST Hugues Fruchet wrote:
> >
> >> When switching from auto to manual mode, V4L2 core is calling
> >> g_volatile_ctrl() in manual
ARE YOU IN NEED OF LOAN @3% INTEREST RATE FOR BUSINESS AND PRIVATE
PURPOSES? IF YES:
FILL AND RETURN
Name:===
Amount needed:===
Duration:==
country:===
Purpose:===
Mobile number
On Fri, Aug 31, 2018 at 11:07:23AM +0300, Sakari Ailus wrote:
> Hi Sean,
>
> On Fri, Jul 13, 2018 at 01:22:29PM +0100, Sean Young wrote:
> > The generic pwm-ir-tx driver should work for the Nokia n900.
> >
> > Compile tested only.
> >
> > Cc: Rob Herring
> > Cc: Ivaylo Dimitrov
> > Cc: Pali
Hi Laurent,
On 09/06/2018 03:31 PM, Laurent Pinchart wrote:
> Hi Hugues,
>
> Thank you for the patch.
>
> On Monday, 13 August 2018 13:19:45 EEST Hugues Fruchet wrote:
>> When switching from auto to manual mode, V4L2 core is calling
>> g_volatile_ctrl() in manual mode in order to get the manual
Hi Mauro,
This is the cedrus Allwinner decoder driver. It is for the request_api topic
branch, but it assumes that this pull request is applied first:
https://patchwork.linuxtv.org/patch/51889/
The last two patches could optionally be squashed with the main driver patch:
they fix COMPILE_TEST
35 matches
Mail list logo