Hi Tomasz,
On Wed, 2019-09-04 at 21:19 +0800, Jerry-ch Chen wrote:
> On Wed, 2019-09-04 at 21:12 +0800, Tomasz Figa wrote:
> > On Wed, Sep 4, 2019 at 6:26 PM Jerry-ch Chen
> > wrote:
> > >
> > > Hi Tomasz,
> > >
> > > On Wed, 2019-09-04 at 17:03 +0800, Tomasz Figa wrote:
> > > > On Wed, Sep 4, 2
On Thu, Sep 5, 2019 at 4:02 PM Jerry-ch Chen wrote:
>
> Hi Tomasz,
>
> On Wed, 2019-09-04 at 21:19 +0800, Jerry-ch Chen wrote:
> > On Wed, 2019-09-04 at 21:12 +0800, Tomasz Figa wrote:
> > > On Wed, Sep 4, 2019 at 6:26 PM Jerry-ch Chen
> > > wrote:
> > > >
> > > > Hi Tomasz,
> > > >
> > > > On W
On 9/4/19 11:40 AM, Mauro Carvalho Chehab wrote:
> You can use any of them, provided that it is opened as if it were a
> normal file, using the read() interface. For example, this should work:
>
> cat /dev/videoX | mplayer -cache 8000 -
>
> The thing is that most apps assume that a V4L2 de
From: Dongchun Zhu
Hello,
Add a v4l2 sub-device driver for Dongwoon's DW9768 lens voice coil.
This is a voice coil module using the i2c bus to control the focus position.
The DW9768 can control the position with 10 bits value and
consists of two 8 bit registers show as below.
register 0x04(DW97
From: Dongchun Zhu
This patch adds a V4L2 sub-device driver for DW9768 lens voice coil,
and provides control to set the desired focus.
The DW9768 is a 10 bit DAC with 100mA output current sink capability
from Dongwoon, designed for linear control of voice coil motor,
and controlled via I2C seria
From: Dongchun Zhu
This patch is to add the Devicetree binding documentation and
MAINTAINERS entry for dw9768 actuator.
Signed-off-by: Dongchun Zhu
---
Documentation/devicetree/bindings/media/i2c/dongwoon,dw9768.txt | 9 +
MAINTAINERS
Instead of hardcoding the UI Command key values, use the new
defines.
Signed-off-by: Hans Verkuil
---
drivers/media/cec/cec-adap.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/media/cec/cec-adap.c b/drivers/media/cec/cec-adap.c
index 5ef7daeb8cbd..284f9b
These defines were never added to this CEC header, likely due
to laziness on the part of the original author, i.e. me.
But it is useful to have them, so add them.
Signed-off-by: Hans Verkuil
---
include/uapi/linux/cec.h | 89
1 file changed, 89 insertion
This series adds defines for the CEC UI Command Operand. I suspect
the reason these weren't added before was laziness :-)
Anyway, this adds them and uses the new defines in cec-adap.c.
I decided not to use them in drivers/media/rc/keymaps/rc-cec.c
since I think it would actually make that keymap
Hi Tomasz,
On Thu, 2019-09-05 at 15:13 +0800, Tomasz Figa wrote:
> On Thu, Sep 5, 2019 at 4:02 PM Jerry-ch Chen
> wrote:
> >
> > Hi Tomasz,
> >
> > On Wed, 2019-09-04 at 21:19 +0800, Jerry-ch Chen wrote:
> > > On Wed, 2019-09-04 at 21:12 +0800, Tomasz Figa wrote:
> > > > On Wed, Sep 4, 2019 at 6
Hi Dongchun,
Thanks for the update. A few comments below.
On Thu, Sep 05, 2019 at 03:21:42PM +0800, dongchun@mediatek.com wrote:
> From: Dongchun Zhu
>
> This patch adds a V4L2 sub-device driver for DW9768 lens voice coil,
> and provides control to set the desired focus.
>
> The DW9768 is
On 9/4/19 11:04 AM, Hans Verkuil wrote:
> poll_wait uses &buf->done, but buf is NULL. Move the poll_wait to later
> in the function once buf is correctly set and only call it if it is
> non-NULL.
>
> Signed-off-by: Hans Verkuil
> Reported-by: Dan Carpenter
> Fixes: bb436cbeb918 ("media: videobuf
Many thanks to Dan Carpenter for finding this stupid regression.
All vb1 drivers (luckily not that many anymore) are broken since 5.1.
Regards,
Hans
The following changes since commit 20a438d53fd9d12a894161bc56cbeab7a9993c39:
media: sun4i: Make sun4i_csi_formats static (2019-08-30 14
From: buil...@linuxtv.org
Pull request: https://patchwork.linuxtv.org/patch/58578/
Build log: https://builder.linuxtv.org/job/patchwork/14773/
Build time: 00:03:04
Link:
https://lore.kernel.org/linux-media/55501135-128f-f715-4608-07de0b8c4...@xs4all.nl
gpg: Signature made Thu 05 Sep 2019 08:26:3
On Thu, Sep 5, 2019 at 5:17 PM Jerry-ch Chen wrote:
>
> Hi Tomasz,
>
> On Thu, 2019-09-05 at 15:13 +0800, Tomasz Figa wrote:
> > On Thu, Sep 5, 2019 at 4:02 PM Jerry-ch Chen
> > wrote:
> > >
> > > Hi Tomasz,
> > >
> > > On Wed, 2019-09-04 at 21:19 +0800, Jerry-ch Chen wrote:
> > > > On Wed, 2019
Hi Sakari,
On Mon, 2019-08-19 at 11:30 +0300, Sakari Ailus wrote:
> Hi Dongchun,
>
> Thanks for the update.
>
> On Mon, Aug 19, 2019 at 11:43:31AM +0800, dongchun@mediatek.com wrote:
> > From: Dongchun Zhu
> >
> > This patch adds a V4L2 sub-device driver for OV02A10 image sensor.
> > The O
On Thu, Sep 05, 2019 at 03:21:41PM +0800, dongchun@mediatek.com wrote:
> From: Dongchun Zhu
>
> This patch is to add the Devicetree binding documentation and
> MAINTAINERS entry for dw9768 actuator.
>
> Signed-off-by: Dongchun Zhu
> ---
> Documentation/devicetree/bindings/media/i2c/dongwoo
This flag tells the kernel whether the slice header contained the
num_ref_idx_l[01]_active_minus1 syntax elements, or whether the
num_ref_idx_l[01]_default_active_minus1 from PPS should be used
instead.
Signed-off-by: Philipp Zabel
---
Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 3 +++
in
If the slice header had the num_ref_idx_active_override flag set, we
should use the num_ref_idx_l[01]_active_minus1 fields instead of the
defaults from the PPS.
Signed-off-by: Philipp Zabel
---
drivers/staging/media/hantro/hantro_g1_h264_dec.c | 13 +++--
1 file changed, 11 insertions(+)
On Thu, Sep 05, 2019 at 11:21:34AM +0300, Sakari Ailus wrote:
> On Thu, Sep 05, 2019 at 03:21:42PM +0800, dongchun@mediatek.com wrote:
> > From: Dongchun Zhu
> > +static const struct i2c_device_id dw9768_id_table[] = {
> > + { DW9768_NAME, 0 },
> > + { },
>
> Could you drop the I²C ID ta
This adds a label to the error path to avoid calling
v4l2_m2m_ctx_init() and kfree(ctx) in two places each.
Signed-off-by: Philipp Zabel
---
drivers/staging/media/hantro/hantro_drv.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/media/hantro/ha
On Thu, Sep 05, 2019 at 03:21:42PM +0800, dongchun@mediatek.com wrote:
> From: Dongchun Zhu
>
> This patch adds a V4L2 sub-device driver for DW9768 lens voice coil,
> and provides control to set the desired focus.
>
> The DW9768 is a 10 bit DAC with 100mA output current sink capability
> fro
On Thu, Sep 5, 2019 at 7:15 PM Philipp Zabel wrote:
>
> This flag tells the kernel whether the slice header contained the
> num_ref_idx_l[01]_active_minus1 syntax elements, or whether the
> num_ref_idx_l[01]_default_active_minus1 from PPS should be used
> instead.
>
> Signed-off-by: Philipp Zabel
On Thu, Sep 5, 2019 at 7:15 PM Philipp Zabel wrote:
>
> If the slice header had the num_ref_idx_active_override flag set, we
> should use the num_ref_idx_l[01]_active_minus1 fields instead of the
> defaults from the PPS.
>
> Signed-off-by: Philipp Zabel
> ---
> drivers/staging/media/hantro/hantr
On Thu, 2019-09-05 at 19:31 +0900, Tomasz Figa wrote:
> On Thu, Sep 5, 2019 at 7:15 PM Philipp Zabel wrote:
> >
> > This flag tells the kernel whether the slice header contained the
> > num_ref_idx_l[01]_active_minus1 syntax elements, or whether the
> > num_ref_idx_l[01]_default_active_minus1 fro
On Thu, Sep 05, 2019 at 01:19:08PM +0300, Andy Shevchenko wrote:
> On Thu, Sep 05, 2019 at 11:21:34AM +0300, Sakari Ailus wrote:
> > On Thu, Sep 05, 2019 at 03:21:42PM +0800, dongchun@mediatek.com wrote:
> > > From: Dongchun Zhu
>
> > > +static const struct i2c_device_id dw9768_id_table[] = {
Hi Dongchun,
On Thu, Sep 05, 2019 at 05:41:05PM +0800, Dongchun Zhu wrote:
...
> > > + ret = regulator_bulk_enable(OV02A10_NUM_SUPPLIES, ov02a10->supplies);
> > > + if (ret < 0) {
> > > + dev_err(dev, "Failed to enable regulators\n");
> > > + goto disable_clk;
> > > + }
> > > + m
On Thu, Sep 05, 2019 at 01:14:06PM +0300, Andy Shevchenko wrote:
> On Thu, Sep 05, 2019 at 03:21:41PM +0800, dongchun@mediatek.com wrote:
> > From: Dongchun Zhu
> >
> > This patch is to add the Devicetree binding documentation and
> > MAINTAINERS entry for dw9768 actuator.
> >
> > Signed-off
On 9/5/19 12:40 PM, Sakari Ailus wrote:
> On Thu, Sep 05, 2019 at 01:19:08PM +0300, Andy Shevchenko wrote:
>> On Thu, Sep 05, 2019 at 11:21:34AM +0300, Sakari Ailus wrote:
>>> On Thu, Sep 05, 2019 at 03:21:42PM +0800, dongchun@mediatek.com wrote:
From: Dongchun Zhu
>>
+static const s
On Thu, 2019-09-05 at 19:34 +0900, Tomasz Figa wrote:
> On Thu, Sep 5, 2019 at 7:15 PM Philipp Zabel wrote:
> >
> > If the slice header had the num_ref_idx_active_override flag set, we
> > should use the num_ref_idx_l[01]_active_minus1 fields instead of the
> > defaults from the PPS.
> >
> > Sig
On Thu, Sep 05, 2019 at 01:48:30PM +0300, Sakari Ailus wrote:
> On Thu, Sep 05, 2019 at 01:14:06PM +0300, Andy Shevchenko wrote:
> > On Thu, Sep 05, 2019 at 03:21:41PM +0800, dongchun@mediatek.com wrote:
> > > From: Dongchun Zhu
> > >
> > > This patch is to add the Devicetree binding document
On Thu, Sep 05, 2019 at 12:57:34PM +0200, Javier Martinez Canillas wrote:
> On 9/5/19 12:40 PM, Sakari Ailus wrote:
> > On Thu, Sep 05, 2019 at 01:19:08PM +0300, Andy Shevchenko wrote:
> >> On Thu, Sep 05, 2019 at 11:21:34AM +0300, Sakari Ailus wrote:
> >>> On Thu, Sep 05, 2019 at 03:21:42PM +0800,
To explain why num_ref_idx_active_override_flag is not part of the API,
describe how the num_ref_idx_l[01]_active_minus1 fields and the
num_ref_idx_l[01]_default_active_minus1 fields are used, depending on
whether the decoder parses slice headers itself or not.
Signed-off-by: Philipp Zabel
---
D
On Thu, Sep 5, 2019 at 1:35 PM Andy Shevchenko
wrote:
>
> On Thu, Sep 05, 2019 at 01:48:30PM +0300, Sakari Ailus wrote:
> > On Thu, Sep 05, 2019 at 01:14:06PM +0300, Andy Shevchenko wrote:
> > > On Thu, Sep 05, 2019 at 03:21:41PM +0800, dongchun@mediatek.com wrote:
> > > > From: Dongchun Zhu
On Thu, Sep 05, 2019 at 01:49:28PM +0200, Javier Martinez Canillas wrote:
> On Thu, Sep 5, 2019 at 1:35 PM Andy Shevchenko
> wrote:
> >
> > On Thu, Sep 05, 2019 at 01:48:30PM +0300, Sakari Ailus wrote:
> > > On Thu, Sep 05, 2019 at 01:14:06PM +0300, Andy Shevchenko wrote:
> > > > On Thu, Sep 05, 2
On Thu, Sep 05, 2019 at 03:00:12PM +0300, Sakari Ailus wrote:
> And I see no reason to add a separate patch just for
> MAINTAINERS change.
It's up to maintainers of the subsystem.
--
With Best Regards,
Andy Shevchenko
Hi Jerry,
Thank you for the patch.
On Tue, Jul 09, 2019 at 04:41:11PM +0800, Jerry-ch Chen wrote:
> From: Jerry-ch Chen
>
> This patch adds KConfig for Mediatek Face Detection driver (FD).
> FD is embedded in Mediatek SoCs. It can provide hardware
> accelerated face detection function.
>
> Sig
Hello,
On Fri, Aug 09, 2019 at 05:07:21PM +0900, Tomasz Figa wrote:
> On Mon, Jul 29, 2019 at 8:58 PM Jerry-ch Chen wrote:
> > On Mon, 2019-07-29 at 17:57 +0800, Tomasz Figa wrote:
> >> On Mon, Jul 29, 2019 at 3:01 PM Jerry-ch Chen wrote:
> >>> On Tue, 2019-07-09 at 18:56 +0800, Enrico Weigelt, me
Since dec_ref_pic_marking_bit_size is not a syntax element
itself, explicitly state that this is the size in bits of
the dec_ref_pic_marking() syntax element contained in the
slice.
Signed-off-by: Philipp Zabel
---
Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 2 +-
1 file changed, 1 insert
Since pic_order_cnt_bit_size is not a syntax element itself, explicitly
state that it is the total size in bits of the pic_order_cnt_lsb,
delta_pic_order_cnt_bottom, delta_pic_order_cnt[0], and
delta_pic_order_cnt[1] syntax elements contained in the slice.
Signed-off-by: Philipp Zabel
---
Docume
On Thu, 2019-09-05 at 16:52 +0800, Tomasz Figa wrote:
> On Thu, Sep 5, 2019 at 5:17 PM Jerry-ch Chen
> wrote:
> >
> > Hi Tomasz,
> >
> > On Thu, 2019-09-05 at 15:13 +0800, Tomasz Figa wrote:
> > > On Thu, Sep 5, 2019 at 4:02 PM Jerry-ch Chen
> > > wrote:
> > > >
> > > > Hi Tomasz,
> > > >
> > >
Hi Laurent,
On Thu, 2019-09-05 at 20:30 +0800, Laurent Pinchart wrote:
> Hi Jerry,
>
> Thank you for the patch.
>
> On Tue, Jul 09, 2019 at 04:41:11PM +0800, Jerry-ch Chen wrote:
> > From: Jerry-ch Chen
> >
> > This patch adds KConfig for Mediatek Face Detection driver (FD).
> > FD is embedded
Le jeudi 05 septembre 2019 à 12:39 +0200, Philipp Zabel a écrit :
> On Thu, 2019-09-05 at 19:31 +0900, Tomasz Figa wrote:
> > On Thu, Sep 5, 2019 at 7:15 PM Philipp Zabel wrote:
> > > This flag tells the kernel whether the slice header contained the
> > > num_ref_idx_l[01]_active_minus1 syntax ele
Hi Tomasz,
Thanks very much for your careful review.
Reply to the comments see belows.
Any missing please let me know.
On Wed, 2019-08-21 at 19:30 +0900, Tomasz Figa wrote:
> Hi Dongchun,
>
> On Mon, Aug 19, 2019 at 11:43:31AM +0800, dongchun@mediatek.com wrote:
> > From: Dongchun Zhu
> >
The image size is doubled for NV16 and is calculated as bytesperline *
height * 2 to accommodate the split of UV data. When writing the offset
to hardware width is used instead of bytesperline, fix this.
Signed-off-by: Niklas Söderlund
---
drivers/media/platform/rcar-vin/rcar-dma.c | 4 ++--
1 f
Hi,
This series add support for sequential filed formats to rcar-vin. The
series is based on [1] and tested on both R-Car Gen2 and Gen3 boards
without regressions.
Patch 1/2 prepares for the new filed formats by reworking and renaming
an existing struct member while 2/2 adds support for the tw
To support SEQ_TB/BT not all buffers given to the hardware will be
equal, the driver needs to keep track of different buffer types. Move
the tracking of buffers given to hardware into a struct so additional
tracking fields can be associated with each buffer.
Signed-off-by: Niklas Söderlund
---
d
The hardware do not support capturing the field types V4L2_FIELD_SEQ_TB
and V4L2_FIELD_SEQ_BT. To capture in these formats the driver needs to
adjust the offset of the capture buffer and capture twice to each vb2
buffer.
Signed-off-by: Niklas Söderlund
---
drivers/media/platform/rcar-vin/rcar-dm
For devices on a HDMI ARC link, sinks can only support transmitter
devices and sources and only support receiver devices.
This patch checks to see if the DUT is an transmitter or receiver and
has the ARC flag set properly from the device.
This should apply to devices that support the HDMI CEC 1.4
The HDMI CEC specification defines deterministic operands for
User Control Pressed that mute or unmute system audio.
Those being:
- Mute Function for mute on
- Restore Volume Function for mute off
This is different from the operand Mute which is a toggle.
This patch adds tests for these User C
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: Fri Sep 6 05:00:12 CEST 2019
media-tree git hash:6f51fdfd8229d5358c2d6e272cf73478866e8ddc
media_build gi
Hi Mauro,
Here are a few more patches for 5.4. Please pull.
The following changes since commit 6f51fdfd8229d5358c2d6e272cf73478866e8ddc:
media: videobuf-core.c: poll_wait needs a non-NULL buf pointer (2019-09-05
06:26:57 -0300)
are available in the Git repository at:
git://linuxtv.org/s
From: buil...@linuxtv.org
Pull request: https://patchwork.linuxtv.org/patch/58618/
Build log: https://builder.linuxtv.org/job/patchwork/14906/
Build time: 00:03:54
Link:
https://lore.kernel.org/linux-media/20190906064546.gd1...@valkosipuli.retiisi.org.uk
gpg: Signature made Fri 06 Sep 2019 06:40
On 9/6/19 5:51 AM, Jiunn Chang wrote:
> For devices on a HDMI ARC link, sinks can only support transmitter
> devices and sources and only support receiver devices.
>
> This patch checks to see if the DUT is an transmitter or receiver and
> has the ARC flag set properly from the device.
>
> This s
54 matches
Mail list logo
