On 22/01/18 13:00, Lars-Peter Clausen wrote:
> On 01/22/2018 01:50 PM, Kieran Bingham wrote:
>> The ADV7511 has four 256-byte maps that can be accessed via the main I²C
>> ports. Each map has it own I²C address and acts as a standard slave
>> device on the I²C bus.
>>
>> Allow a device tree node to
On Mon, 2018-01-22 at 13:15 +0100, Hans Verkuil wrote:
> On 08/12/17 10:08, Smitha T Murthy wrote:
> > Added V4l2 controls for HEVC encoder
> >
> > Signed-off-by: Smitha T Murthy
> > ---
> > Documentation/media/uapi/v4l/extended-controls.rst | 395
> > +
> > 1 file changed,
On Mon, 2018-01-22 at 13:18 +0100, Hans Verkuil wrote:
> Hi Smitha,
>
> Thank you for this v6 series!
>
> You can add my:
>
> Acked-by: Hans Verkuil
>
> to patches 1-9 and 11. See my review for patches 10 and 12. The comments
> are minor, so I hope I can Ack v7 once it's posted and this can be
On Mon, 2018-01-22 at 12:08 +0100, Hans Verkuil wrote:
> On 08/12/17 10:08, Smitha T Murthy wrote:
> > Add v4l2 controls for HEVC encoder
> >
> > Signed-off-by: Smitha T Murthy
> > Reviewed-by: Andrzej Hajda
> > ---
> > drivers/media/v4l2-core/v4l2-ctrls.c | 118
> > +++
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 Jan 23 05:00:16 CET 2018
media-tree git hash:e3ee691dbf24096ea51b3200946b11d68ce75361
media_build git
Add a V4L2 sub-device driver for the Sony IMX258 image sensor.
This is a camera sensor using the I2C bus for control and the
CSI-2 bus for data.
Signed-off-by: Andy Yeh
---
- v2->v3
-- Update the streaming function to remove SW_STANDBY in the beginning.
-- Adjust the delay time from 1ms to 12ms b
Fix a few sparse warnings related to conversion between CPU and big
endian. Also simplify the code in the process.
Signed-off-by: Sakari Ailus
---
since v2:
- Count loop downwards, not up.
drivers/media/i2c/imx258.c | 23 +--
1 file changed, 9 insertions(+), 14 deletions(-)
Hi Andy,
On Mon, Jan 22, 2018 at 03:19:00PM +, Yeh, Andy wrote:
> Hi Sakari,
>
> I made a minor fix. I2C write function works after the change. Please kindly
> review soon then I would submit v5.
>
> *buf++ = reg >> 8;
> *buf++ = reg & 0xff;
>
> - for (i = len - 1; i >= 0
On 01/22/2018 11:37 AM, Dan Carpenter wrote:
> The while loop is a post op, "while (i-- >= 0)" so the last iteration
> will read camif_mbus_formats[-1] and then the loop will exit with "i"
> set to -2 and so we do: "mf->code = camif_mbus_formats[-2];".
>
> I've changed it to a pre-op, I've added a
Benötigen Sie Privat- oder Geschäftskredite ohne Stress und schnelle
Zustimmung? Wenn ja, kontaktieren Sie uns bitte alexgr...@gmail.com
From: Daniel Scheller
The first two missing FECs in the modcod2fec fe_code_rate table were 1/4
and 1/3. Add them as they're now defined by the API.
Signed-off-by: Daniel Scheller
---
drivers/media/dvb-frontends/stv0910.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
From: Daniel Scheller
Report the active delivery system based on the receive_mode state of the
demodulator.
Suggested-by: Richard Scobie
Signed-off-by: Daniel Scheller
---
drivers/media/dvb-frontends/stv0910.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/dvb-frontends/s
From: Daniel Scheller
Report the currently used S2 rolloff factor in get_frontend(). For
cosmetic reasons, also change all feroll_off occurences to fe_rolloff.
Signed-off-by: Daniel Scheller
---
drivers/media/dvb-frontends/stv0910.c | 12
1 file changed, 8 insertions(+), 4 deleti
From: Daniel Scheller
Add 15%, 10% and 5% DVB-S2X rolloff factors. Also fix roloff typos.
Signed-off-by: Daniel Scheller
---
Documentation/media/frontend.h.rst.exceptions | 3 +++
drivers/media/dvb-core/dvb_frontend.c | 9 +
include/uapi/linux/dvb/frontend.h | 16
From: Daniel Scheller
Add 1/4 and 1/3 FEC ratios, 64/128/256-APSK S2X modulations and 64K
transmission mode. Update relevant doc items aswell.
Signed-off-by: Daniel Scheller
---
Documentation/media/frontend.h.rst.exceptions | 6 ++
include/uapi/linux/dvb/frontend.h | 13 ++
From: Daniel Scheller
dddvb brings a few additional FEC rates (1/4 and 1/3), 64/128/256APSK
modulations and more rolloff factors (DVB-S2X), and 64K transmission mode
(DVB-T2). These rather trivial patches bring them to mainline, and puts
these missing bits into the stv0910's get_frontend() callba
From: Alan Chiang
DW9807 is a 10 bit DAC from Dongwoon, designed for linear
control of voice coil motor.
This driver creates a V4L2 subdevice and
provides control to set the desired focus.
Signed-off-by: Andy Yeh
Signed-off-by: Sakari Ailus
---
v2: changed author
.../bindings/media/i2c/dong
DW9807 is a 10 bit DAC from Dongwoon, designed for linear
control of voice coil motor.
This driver creates a V4L2 subdevice and
provides control to set the desired focus.
Signed-off-by: Andy Yeh
Signed-off-by: Sakari Ailus
---
.../bindings/media/i2c/dongwoon,dw9807.txt | 9 +
MAINTAI
Hi Sakari,
Thanks for reviewing,
On 01/22/2018 01:52 PM, Sakari Ailus wrote:
> Hi Hugues,
>
> On Mon, Jan 22, 2018 at 11:46:36AM +0100, Hugues Fruchet wrote:
>> Add YUV422 encoded JPEG support.
>>
>> Signed-off-by: Hugues Fruchet
>> ---
>> drivers/media/i2c/ov5640.c | 82
>> ++
The IPUv3 code has 8-bit grayscale capture support.
Enable imx-media to use it.
Signed-off-by: Philipp Zabel
---
This patch depends on https://patchwork.kernel.org/patch/10178777/
to work, otherwise STREAMON will fail with -EINVAL.
---
drivers/staging/media/imx/imx-media-csi.c | 1 +
drivers/s
Add a V4L2 sub-device driver for the Sony IMX258 image sensor.
This is a camera sensor using the I2C bus for control and the
CSI-2 bus for data.
Signed-off-by: Andy Yeh
Signed-off-by: Sakari Ailus
---
v3:
-- Update the streaming function to remove SW_STANDBY in the beginning.
-- Adjust the dela
Hi Sakari,
I made a minor fix. I2C write function works after the change. Please kindly
review soon then I would submit v5.
*buf++ = reg >> 8;
*buf++ = reg & 0xff;
- for (i = len - 1; i >= 0; i++)
+ for (i = len - 1; i >= 0; i--)
if (i2c_master_send(client
Hi Hans,
On Mon, Jan 22, 2018 at 03:48:17PM +0100, Hans Verkuil wrote:
> On 22/01/18 15:45, jacopo mondi wrote:
> > Hi Hans,
> >
> > On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
> >> From: Hans Verkuil
> >>
> >>
> >
> > [snip]
> >
> >> diff --git a/drivers/media/platform/atmel/at
Hello Daniel,
On 20.01.2018 16:02, Daniel Glöckner wrote:
> The VM-009 has 10 data lines. Do you use a board designed by Phytec?
> If not, did you connect the lower or the upper 8 data lines to the i.MX6?
> Using the upper 8 data lines is correct.
its a VM-009 but not a Phytec baseboard and lucki
On 22/01/18 15:45, jacopo mondi wrote:
> Hi Hans,
>
> On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
>> From: Hans Verkuil
>>
>>
>
> [snip]
>
>> diff --git a/drivers/media/platform/atmel/atmel-isc.c
>> b/drivers/media/platform/atmel/atmel-isc.c
>> index 34676409ca08..92d695b29fa
Hi Hans,
On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
> From: Hans Verkuil
>
>
[snip]
> diff --git a/drivers/media/platform/atmel/atmel-isc.c
> b/drivers/media/platform/atmel/atmel-isc.c
> index 34676409ca08..92d695b29fa9 100644
> --- a/drivers/media/platform/atmel/atmel-isc.c
Hi Hans,
On 18.01.2018 16:13, Hans Verkuil wrote:
Only apply the change to v4l2_dev.c, ignore the changes to
v4l2_event. I think it is sufficient to just apply that bit.
I tried both variants, with changes to v4l2_event and without. Both did
not work out of the box. I think the problem is in
On 01/22/2018 01:50 PM, Kieran Bingham wrote:
> The ADV7511 has four 256-byte maps that can be accessed via the main I²C
> ports. Each map has it own I²C address and acts as a standard slave
> device on the I²C bus.
>
> Allow a device tree node to override the default addresses so that
> address c
Hi Hugues,
On Mon, Jan 22, 2018 at 11:46:36AM +0100, Hugues Fruchet wrote:
> Add YUV422 encoded JPEG support.
>
> Signed-off-by: Hugues Fruchet
> ---
> drivers/media/i2c/ov5640.c | 82
> --
> 1 file changed, 80 insertions(+), 2 deletions(-)
>
> diff
The ADV7511 has four 256-byte maps that can be accessed via the main I²C
ports. Each map has it own I²C address and acts as a standard slave
device on the I²C bus.
Allow a device tree node to override the default addresses so that
address conflicts with other devices on the same bus may be resolve
From: Jean-Michel Hautbois
The ADV7604 has thirteen 256-byte maps that can be accessed via the main
I²C ports. Each map has it own I²C address and acts as a standard slave
device on the I²C bus.
Allow a device tree node to override the default addresses so that
address conflicts with other devic
Back in 2014, Jean-Michel provided patches [0] to implement a means of
describing software defined I2C addresses for devices through the DT nodes.
The patch to implement the function "i2c_new_secondary_device()" was integrated,
but the corresponding driver update didn't get applied.
This short se
Hi Hans,
On Mon, Jan 22, 2018 at 01:31:16PM +0100, Hans Verkuil wrote:
> From: Hans Verkuil
>
> There are currently two subdev ops variants to get/set the frame interval:
> g/s_parm and g/s_frame_interval.
>
> This patch series replaces all g/s_parm calls by g/s_frame_interval.
>
> The first p
From: Gustavo Padovan
The link was showing both VIDIOC_QBUF, VIDIOC_DQBUF while it should show
only VIDIOC_QBUF in this case.
Signed-off-by: Gustavo Padovan
---
Documentation/media/uapi/v4l/buffer.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/media/uapi/
From: Sakari Ailus
The s_parm callback in mt9m114 driver did nothing, remove it.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-m
From: Hans Verkuil
There are currently two subdev ops variants to get/set the frame interval:
g/s_parm and g/s_frame_interval.
This patch series replaces all g/s_parm calls by g/s_frame_interval.
The first patch adds helper functions that can be used by bridge drivers.
Only em28xx can't use it
From: Hans Verkuil
Create helpers to handle VIDIOC_G/S_PARM by querying the
g/s_frame_interval v4l2_subdev ops.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-common.c | 48 +++
include/media/v4l2-common.h | 26 +++
2 file
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
include/media/v4l2-subdev.h | 6 --
1 file changed, 6 deletions(-)
diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
index 980a86c08fce..457917e9237f 100644
--- a/include/media/v4l2-subdev.h
+++ b/include/media/v4l2-su
From: Hans Verkuil
The specification mentions that type can be V4L2_BUF_TYPE_VIDEO_CAPTURE,
but the v4l2 core implementation also allows the _MPLANE variant.
Document this.
Signed-off-by: Hans Verkuil
---
Documentation/media/uapi/v4l/vidioc-g-parm.rst | 7 ---
1 file changed, 4 insertions
From: Sakari Ailus
These drivers already support g_frame_interval. Therefore just dropping
g_parm is enough.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/atomisp-gc0310.c | 27 --
drivers/staging/media/atomisp/i2c/atomisp-gc
From: Hans Verkuil
Convert all g/s_parm calls to g/s_frame_interval. This allows us
to remove the g/s_parm ops since those are a duplicate of
g/s_frame_interval.
Signed-off-by: Hans Verkuil
---
drivers/media/i2c/mt9v011.c | 31 +++-
drivers/media/i2c/ov6650.
From: Sakari Ailus
The s_parm and g_parm did nothing. Remove them.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
.../staging/media/atomisp/pci/atomisp2/atomisp_file.c| 16
drivers/staging/media/atomisp/pci/atomisp2/atomisp_tpg.c | 14 --
2 files
From: Sakari Ailus
Remove sensor driver's interface that made use of use case specific
knowledge of platform capabilities.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/atomisp-gc0310.c | 26 -
drivers/staging/media/atomisp/i2c/atomisp-g
From: Sakari Ailus
Disable configurations for non-preview modes until configuration selection
is improved.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/gc2235.h| 2 ++
drivers/staging/media/atomisp/i2c/ov2722.h| 2 ++
drivers/st
On 22/01/18 11:59, Sakari Ailus wrote:
> Hi Hans,
>
> On Mon, Jan 22, 2018 at 11:33:28AM +0100, Hans Verkuil wrote:
>> On 22/01/18 11:26, Sakari Ailus wrote:
>>> Hi Hans,
>>>
>>> On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
From: Hans Verkuil
Convert all g/s_parm c
Hi Smitha,
Thank you for this v6 series!
You can add my:
Acked-by: Hans Verkuil
to patches 1-9 and 11. See my review for patches 10 and 12. The comments
are minor, so I hope I can Ack v7 once it's posted and this can be merged
for 4.17.
Regards,
Hans
On 08/12/17 10:08, Smitha T Murt
On 08/12/17 10:08, Smitha T Murthy wrote:
> Added V4l2 controls for HEVC encoder
>
> Signed-off-by: Smitha T Murthy
> ---
> Documentation/media/uapi/v4l/extended-controls.rst | 395
> +
> 1 file changed, 395 insertions(+)
>
> diff --git a/Documentation/media/uapi/v4l/extend
On 08/12/17 10:08, Smitha T Murthy wrote:
> Add v4l2 controls for HEVC encoder
>
> Signed-off-by: Smitha T Murthy
> Reviewed-by: Andrzej Hajda
> ---
> drivers/media/v4l2-core/v4l2-ctrls.c | 118
> +++
> include/uapi/linux/v4l2-controls.h | 92
Hi Hans,
On Mon, Jan 22, 2018 at 11:33:28AM +0100, Hans Verkuil wrote:
> On 22/01/18 11:26, Sakari Ailus wrote:
> > Hi Hans,
> >
> > On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
> >> From: Hans Verkuil
> >>
> >> Convert all g/s_parm calls to g/s_frame_interval. This allows us
>
Add YUV422 encoded JPEG support.
Signed-off-by: Hugues Fruchet
---
drivers/media/i2c/ov5640.c | 82 --
1 file changed, 80 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c
index e2dd352..db9aeeb 100644
On 22/01/18 11:26, Sakari Ailus wrote:
> Hi Hans,
>
> On Mon, Jan 22, 2018 at 11:18:57AM +0100, Hans Verkuil wrote:
>> From: Hans Verkuil
>>
>> The specification mentions that type can be V4L2_BUF_TYPE_VIDEO_CAPTURE,
>> but the v4l2 core implementation also allows the _MPLANE variant.
>>
>> Docum
The while loop is a post op, "while (i-- >= 0)" so the last iteration
will read camif_mbus_formats[-1] and then the loop will exit with "i"
set to -2 and so we do: "mf->code = camif_mbus_formats[-2];".
I've changed it to a pre-op, I've added a check to ensure we found the
right format and I've rem
On 22/01/18 11:28, Sakari Ailus wrote:
> On Mon, Jan 22, 2018 at 11:18:49AM +0100, Hans Verkuil wrote:
>> From: Hans Verkuil
>>
>> Create helpers to handle VIDIOC_G/S_PARM by querying the
>> g/s_frame_interval v4l2_subdev ops.
>>
>> Signed-off-by: Hans Verkuil
>> ---
>> drivers/media/v4l2-core/v
On 22/01/18 11:26, Sakari Ailus wrote:
> Hi Hans,
>
> On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
>> From: Hans Verkuil
>>
>> Convert all g/s_parm calls to g/s_frame_interval. This allows us
>> to remove the g/s_parm ops since those are a duplicate of
>> g/s_frame_interval.
>>
>
On Mon, Jan 22, 2018 at 11:18:49AM +0100, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Create helpers to handle VIDIOC_G/S_PARM by querying the
> g/s_frame_interval v4l2_subdev ops.
>
> Signed-off-by: Hans Verkuil
> ---
> drivers/media/v4l2-core/v4l2-common.c | 49
> ++
Hi Hans,
On Mon, Jan 22, 2018 at 11:18:50AM +0100, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Convert all g/s_parm calls to g/s_frame_interval. This allows us
> to remove the g/s_parm ops since those are a duplicate of
> g/s_frame_interval.
>
> Signed-off-by: Hans Verkuil
> ---
> drivers/me
Hi Hans,
On Mon, Jan 22, 2018 at 11:18:57AM +0100, Hans Verkuil wrote:
> From: Hans Verkuil
>
> The specification mentions that type can be V4L2_BUF_TYPE_VIDEO_CAPTURE,
> but the v4l2 core implementation also allows the _MPLANE variant.
>
> Document this.
>
> Signed-off-by: Hans Verkuil
> ---
From: Hans Verkuil
The specification mentions that type can be V4L2_BUF_TYPE_VIDEO_CAPTURE,
but the v4l2 core implementation also allows the _MPLANE variant.
Document this.
Signed-off-by: Hans Verkuil
---
Documentation/media/uapi/v4l/vidioc-g-parm.rst | 6 +++---
1 file changed, 3 insertions(
From: Sakari Ailus
These drivers already support g_frame_interval. Therefore just dropping
g_parm is enough.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/atomisp-gc0310.c | 27 --
drivers/staging/media/atomisp/i2c/atomisp-gc
From: Sakari Ailus
The s_parm callback in mt9m114 driver did nothing, remove it.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-m
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
include/media/v4l2-subdev.h | 6 --
1 file changed, 6 deletions(-)
diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
index 980a86c08fce..457917e9237f 100644
--- a/include/media/v4l2-subdev.h
+++ b/include/media/v4l2-su
From: Sakari Ailus
The s_parm and g_parm did nothing. Remove them.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
.../staging/media/atomisp/pci/atomisp2/atomisp_file.c| 16
drivers/staging/media/atomisp/pci/atomisp2/atomisp_tpg.c | 14 --
2 files
From: Sakari Ailus
Remove sensor driver's interface that made use of use case specific
knowledge of platform capabilities.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/atomisp-gc0310.c | 26 -
drivers/staging/media/atomisp/i2c/atomisp-g
From: Hans Verkuil
Convert all g/s_parm calls to g/s_frame_interval. This allows us
to remove the g/s_parm ops since those are a duplicate of
g/s_frame_interval.
Signed-off-by: Hans Verkuil
---
drivers/media/i2c/mt9v011.c | 29 +--
drivers/media/i2c/ov6650.c
From: Hans Verkuil
Create helpers to handle VIDIOC_G/S_PARM by querying the
g/s_frame_interval v4l2_subdev ops.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-common.c | 49 +++
include/media/v4l2-common.h | 26 +++
2 file
From: Sakari Ailus
Disable configurations for non-preview modes until configuration selection
is improved.
Signed-off-by: Sakari Ailus
Signed-off-by: Hans Verkuil
---
drivers/staging/media/atomisp/i2c/gc2235.h| 2 ++
drivers/staging/media/atomisp/i2c/ov2722.h| 2 ++
drivers/st
From: Hans Verkuil
There are currently two subdev ops variants to get/set the frame interval:
g/s_parm and g/s_frame_interval.
This patch series replaces all g/s_parm calls by g/s_frame_interval.
The first patch adds helper functions that can be used by bridge drivers.
Only em28xx can't use it
Time to time, stream is failing with a strange positive error.
Error code is returned erroneously by ov5640_set_ctrl_exposure()
due to ov5640_get_vts() return value wrongly treated as error.
Fix this by forcing ret to 0 after ov5640_get_vts() success call,
in order that ret is set to success for re
Time to time, stream on is failing with a strange positive error.
Error code is returned erroneously by ov5640_set_ctrl_exposure()
due to ov5640_get_vts() return value wrongly treated as error.
Fix this by forcing ret to 0 after ov5640_get_vts() success call,
in order that ret is set to success for
On 21/01/18 18:49, jacopo mondi wrote:
> Hi Hans,
>
> On Fri, Jan 19, 2018 at 12:26:09PM +0100, Hans Verkuil wrote:
>> Hi Jacopo,
>>
>> On 01/16/18 22:44, Jacopo Mondi wrote:
>>> Hello,
>>>new version of CEU after Hans' review.
>>>
>>> Added his Acked-by to most patches and closed review comme
On 21/01/18 18:29, jacopo mondi wrote:
> Hi Hans,
>
> On Sun, Jan 21, 2018 at 11:23:12AM +0100, Hans Verkuil wrote:
>> On 21/01/18 11:21, Hans Verkuil wrote:
>>> On 21/01/18 10:53, jacopo mondi wrote:
Hi Hans,
On Fri, Jan 19, 2018 at 12:20:19PM +0100, Hans Verkuil wrote:
> stati
Hi Hans,
On Mon, Jan 22, 2018 at 10:19:13AM +0100, Hans Verkuil wrote:
> On 21/01/18 23:48, Sakari Ailus wrote:
> > Hi Hans,
> >
> > On Sun, Jan 21, 2018 at 11:46:46AM +0100, Hans Verkuil wrote:
> >> Hi Sakari,
> >>
> >> I looked a bit closer at how atomisp uses g/s_parm. They abuse the
> >> cap
On 21/01/18 23:48, Sakari Ailus wrote:
> Hi Hans,
>
> On Sun, Jan 21, 2018 at 11:46:46AM +0100, Hans Verkuil wrote:
>> Hi Sakari,
>>
>> I looked a bit closer at how atomisp uses g/s_parm. They abuse the
>> capturemode field
>> to select video/preview/still modes on the sensor, which actually chan
On 22/01/18 01:52, Laurent Pinchart wrote:
> Hi Hans,
>
> On Friday, 19 January 2018 14:25:39 EET Hans Verkuil wrote:
>> On 01/19/18 13:20, Laurent Pinchart wrote:
>>> On Friday, 19 January 2018 13:20:19 EET Hans Verkuil wrote:
On 01/16/18 22:44, Jacopo Mondi wrote:
> Add driver for Renes
On Mon, Jan 22, 2018 at 10:37:43AM +0200, Sakari Ailus wrote:
> Hi Andy,
>
> On Mon, Jan 22, 2018 at 08:03:23AM +, Yeh, Andy wrote:
> > Hi Sakari, Tomasz,
> >
> > As below discussion that other drivers are with this pattern, I would
> > prefer to defer to address the concern in later discuss
Hi Andy,
On Mon, Jan 22, 2018 at 08:03:23AM +, Yeh, Andy wrote:
> Hi Sakari, Tomasz,
>
> As below discussion that other drivers are with this pattern, I would prefer
> to defer to address the concern in later discussion with you and owners of
> other sensors.
>
> Thanks a lot.
I thought o
On Mon, Jan 22, 2018 at 08:03:23AM +, Yeh, Andy wrote:
> Hi Sakari, Tomasz,
>
> As below discussion that other drivers are with this pattern, I would
> prefer to defer to address the concern in later discussion with you and
> owners of other sensors.
>
> Thanks a lot.
Works for me.
--
Saka
Fix a few sparse warnings related to conversion between CPU and big
endian. Also simplify the code in the process.
Signed-off-by: Sakari Ailus
---
Hi Andy,
I think I figured out the problem. Could you test this?
Thanks.
since v1:
- Fix pointer passed to i2c_master_send. This is the entire buf
Hi Sakari,
I verified the patch multiple times but the code cannot work.
Regards, Andy
-Original Message-
From: Sakari Ailus [mailto:sakari.ai...@linux.intel.com]
Sent: Friday, January 19, 2018 5:23 PM
To: linux-media@vger.kernel.org
Cc: Yeh, Andy
Subject: [PATCH 1/1] imx258: Fix spa
Hi Sakari, Tomasz,
As below discussion that other drivers are with this pattern, I would prefer to
defer to address the concern in later discussion with you and owners of other
sensors.
Thanks a lot.
Regards, Andy
-Original Message-
From: Sakari Ailus [mailto:sakari.ai...@linux.intel.
80 matches
Mail list logo
