From: Benoit Parrot
this patch adds support for omnivision's ov2659
sensor, the driver supports following features:
1: Asynchronous probing
2: DT support
3: Media controller support
Signed-off-by: Benoit Parrot
Signed-off-by: Lad, Prabhakar
---
Changes for v3:
a: Fixed review comments pointe
From: "Lad, Prabhakar"
Instead of matching the subdevs with their name, match
it with OF node/ i2c address and adapter number.
Signed-off-by: Lad, Prabhakar
---
drivers/media/platform/am437x/am437x-vpfe.c | 27 +++
drivers/media/platform/am437x/am437x-vpfe.h | 1 -
2 f
From: "Lad, Prabhakar"
return error in case devm_kzalloc() fails.
Signed-off-by: Lad, Prabhakar
---
drivers/media/platform/am437x/am437x-vpfe.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/media/platform/am437x/am437x-vpfe.c
b/drivers/media/platform/am437x/am437x-vpfe.c
i
From: "Lad, Prabhakar"
Lad, Prabhakar (2):
media: am437x-vpfe: match the OF node/i2c addr instead of name
media: am437x-vpfe: return error in case memory allocation failure
drivers/media/platform/am437x/am437x-vpfe.c | 33 +++--
drivers/media/platform/am437x/am437x-
From: "Lad, Prabhakar"
Embed video_device struct (video_dev) in vpfe_device and
Unregister path doesn't need to free the video_device
structure, hence, change the video_device.release callback
point to video_device_release_empty.
Signed-off-by: Lad, Prabhakar
---
drivers/media/platform/am437x/
On Thu, Mar 5, 2015 at 10:26 AM, Hans Verkuil wrote:
> On 03/05/15 10:44, Scott Jiang wrote:
>> Hi Hans,
[snip]
cd utils/v4l2-compliance
cat *.cpp >x.cpp
g++ -o v4l2-compliance x.cpp -I . -I ../../include/ -DNO_LIBV4L2
I've never used uclibc, so I don't know what the
On 03/07/2015 05:22 PM, Lad, Prabhakar wrote:
> On Thu, Mar 5, 2015 at 10:26 AM, Hans Verkuil wrote:
>> On 03/05/15 10:44, Scott Jiang wrote:
>>> Hi Hans,
> [snip]
>
> cd utils/v4l2-compliance
> cat *.cpp >x.cpp
> g++ -o v4l2-compliance x.cpp -I . -I ../../include/ -DNO_LIBV4L2
>>>
On 03/07/2015 08:52 PM, Florian Echtler wrote:
> On 06.03.2015 12:47, Hans Verkuil wrote:
>> On 03/06/2015 12:24 PM, Florian Echtler wrote:
>>> On 21.02.2015 11:22, Hans Verkuil wrote:
On 02/20/2015 10:46 PM, Florian Echtler wrote:
> On 16.02.2015 12:40, Hans Verkuil wrote:
>> I prefer
Hi folks,
I've had this patchset hanging around for a long, long time, and now it's
time to send it out to linux-media.
For OMAP 3, first there are a few patches for random and tiny bugfixes and
then preparation for DT support (including compile tested rework of cm-t35
board code).
The lane-pola
The syscon register isn't part of the ISP, use it through the syscom driver
regmap instead. The syscom block is considered to be from 343x on ISP
revision 2.0 whereas 15.0 is assumed to have 3630 syscon.
Signed-off-by: Sakari Ailus
---
arch/arm/boot/dts/omap3.dtsi|2 +-
arch/
The video port clock is l3_ick divided by vpclk_div. This clock must be high
enough for the external pixel rate. The video port requires two clock cycles
to process a pixel.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3isp/ispcsi2.c |8 +++-
include/media/omap3isp.h
Only the histogram sub-block driver uses the physical address. Do not store
it for other sub-blocks.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3isp/isp.c |3 ++-
drivers/media/platform/omap3isp/isp.h |6 +++---
drivers/media/platform/omap3isp/isphist.c |2 +-
Move the code which connects the external entity to an ISP entity into a
separate function. This disconnects it from parsing the platform data.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3isp/isp.c | 147 +
1 file changed, 74 insertions(+), 73 del
Only check for call platform data callback functions if there's platform
data. Also take care of a few other cases where the NULL pdata pointer could
have been accessed, and remove the check for NULL dev->platform_data
pointer.
Removing the check for NULL dev->platform_data isn't strictly needed b
The omap3isp MMIO register block is contiguous in the MMIO register space
apart from the fact that the ISP IOMMU register block is in the middle of
the area. Ioremap it at two occasions, and keep the rest of the layout of
the register space internal to the omap3isp driver.
Signed-off-by: Sakari Ai
The resources the ISP needs are slightly different on 3[45]xx and 3[67]xx.
Especially the phy-type property is different.
Signed-off-by: Sakari Ailus
---
arch/arm/boot/dts/omap34xx.dtsi | 15 +++
arch/arm/boot/dts/omap36xx.dtsi | 15 +++
2 files changed, 30 insertions
From: Laurent Pinchart
Signed-off-by: Laurent Pinchart
---
drivers/media/platform/omap3isp/isp.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/drivers/media/platform/omap3isp/isp.c
b/drivers/media/platform/omap3isp/isp.c
index a607f26..01356dd 100644
--- a/dr
Make omap3isp configuration data structures more suitable for consumption by
the DT by separating the I2C bus information of all the sub-devices in a
group and the ISP bus information from each other. The ISP bus information
is made a pointer instead of being directly embedded in the struct.
In th
If an uninitialised v4l2_subdev struct was passed to
media_entity_create_link(), one of the BUG_ON()'s in the function will be
hit since media_entity.num_pads will be zero. Avoid this by checking whether
the num_pads field is non-zero for the interface.
Signed-off-by: Sakari Ailus
---
drivers/me
Add lane-polarity property to endpoint nodes. This essentially tells that
the order of the differential signal wires is inverted.
Signed-off-by: Sakari Ailus
---
Documentation/devicetree/bindings/media/video-interfaces.txt |5 +
1 file changed, 5 insertions(+)
diff --git a/Documentation
Add lane_polarity field to struct v4l2_of_bus_mipi_csi2 and write the
contents of the lane polarity property to it. The field tells the polarity
of the physical lanes starting from the first one. Any unused lanes are
ignored, i.e. only the polarity of the used lanes is specified.
Signed-off-by: Sa
Add the ISP device to omap3 DT include file and add support to the driver to
use it.
Also obtain information on the external entities and the ISP configuration
related to them through the Device Tree in addition to the platform data.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3i
The mutex was not destroyed correctly if dma_coerce_mask_and_coherent()
failed for some reason.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3isp/isp.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/omap3isp/isp.c
b/drivers/media/plat
Signed-off-by: Sakari Ailus
---
.../devicetree/bindings/media/ti,omap3isp.txt | 64
MAINTAINERS|1 +
2 files changed, 65 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/ti,omap3isp.txt
diff --git a/
Rename VDD_CSIPHY1 as vdd-csiphy1 and VDD_CSIPHY2 as vdd-csiphy2.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3isp/isp.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/omap3isp/isp.c
b/drivers/media/platform/omap3isp/isp.c
index
Print a warning when the driver is used with platform data. Existing
platform data user should move to DT now.
Signed-off-by: Sakari Ailus
---
drivers/media/platform/omap3isp/isp.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/platform/omap3isp/isp.c
b/drivers/media/pla
Add support for the primary camera of the Nokia N950 and N9.
Signed-off-by: Sakari Ailus
---
arch/arm/boot/dts/omap3-n9.dts | 39 ++
arch/arm/boot/dts/omap3-n950-n9.dtsi |4
arch/arm/boot/dts/omap3-n950.dts | 39 +
Hi Prabhakar,
On Sat, Mar 07, 2015 at 03:21:40PM +, Lad Prabhakar wrote:
> + pixel-clock-frequency = <7000>;
As commented privately, could you use the link-frequencies property for this
purpose? On parallel bus if often equals the pixel clock, but works
Hi Mauro,
This pull request reverts the smiapp driver's u64 array DT property read
workaround, and uses of_property_read_u64_array() (second patch) which is
the correct API function for reading u64 arrays from DT.
Please pull.
The following changes since commit 3d945be05ac1e806af075e9315bc1b340
Hi Mauro,
Just a single patch, a trivial sparse fix to make a variable which is only
used in a single file, static.
Please pull.
The following changes since commit 3d945be05ac1e806af075e9315bc1b3409adae2b:
[media] mn88473: simplify bandwidth registers setting code (2015-03-03
13:09:12 -0300
On Sat, 2015-03-07 at 15:21 +, Lad Prabhakar wrote:
> --- /dev/null
> +++ b/drivers/media/i2c/ov2659.c
> @@ -0,0 +1,1495 @@
> +/*
> + * Omnivision OV2659 CMOS Image Sensor driver
> + *
> + * Copyright (C) 2015 Texas Instruments, Inc.
> + *
> + * Benoit Parrot
> + * Lad, Prabhakar
> + *
> + *
Hi Sakari,
On Sunday 08 March 2015 00:06:34 Sakari Ailus wrote:
> Hi Mauro,
>
> This pull request reverts the smiapp driver's u64 array DT property read
> workaround, and uses of_property_read_u64_array() (second patch) which is
> the correct API function for reading u64 arrays from DT.
>
> Plea
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:40:58 Sakari Ailus wrote:
> The mutex was not destroyed correctly if dma_coerce_mask_and_coherent()
> failed for some reason.
>
> Signed-off-by: Sakari Ailus
Acked-by: Laurent Pinchart
> ---
> drivers/media/platform/omap3isp/i
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:40:59 Sakari Ailus wrote:
> If an uninitialised v4l2_subdev struct was passed to
> media_entity_create_link(), one of the BUG_ON()'s in the function will be
> hit since media_entity.num_pads will be zero. Avoid this by checking whet
Hi Laurent,
On Sun, Mar 08, 2015 at 01:15:38AM +0200, Laurent Pinchart wrote:
> > Sakari Ailus (2):
> > Revert "[media] smiapp: Don't compile of_read_number() if CONFIG_OF
> > isn't defined"
> > smiapp: Use of_property_read_u64_array() to read a 64-bit number array
>
> Won't this caus
Hi Sakari,
On Sunday 08 March 2015 01:20:40 Sakari Ailus wrote:
> On Sun, Mar 08, 2015 at 01:15:38AM +0200, Laurent Pinchart wrote:
> > > Sakari Ailus (2):
> > > Revert "[media] smiapp: Don't compile of_read_number() if
> > > CONFIG_OF isn't defined"
> > > smiapp: Use of_property
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:00 Sakari Ailus wrote:
> Move the code which connects the external entity to an ISP entity into a
> separate function. This disconnects it from parsing the platform data.
>
> Signed-off-by: Sakari Ailus
> ---
> drivers/media/p
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:04 Sakari Ailus wrote:
> Rename VDD_CSIPHY1 as vdd-csiphy1 and VDD_CSIPHY2 as vdd-csiphy2.
>
> Signed-off-by: Sakari Ailus
Acked-by: Laurent Pinchart
> ---
> drivers/media/platform/omap3isp/isp.c |4 ++--
> 1 file chang
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:05 Sakari Ailus wrote:
> The video port clock is l3_ick divided by vpclk_div. This clock must be high
> enough for the external pixel rate. The video port requires two clock
> cycles to process a pixel.
>
> Signed-off-by: Sakari
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:06 Sakari Ailus wrote:
> Only the histogram sub-block driver uses the physical address. Do not store
> it for other sub-blocks.
>
> Signed-off-by: Sakari Ailus
Acked-by: Laurent Pinchart
> ---
> drivers/media/platform/omap3
Hi Mauro,
This pull request reverts the smiapp driver's u64 array DT property read
workaround, and uses of_property_read_u64_array() which is the correct API
function for reading u64 arrays from DT.
since v1:
Based on a discussion with Laurent, I merged the two patches. The commit
message of the
Liebe Begünstigte,
Sie wurden ausgewählt, um (€950.000,00 EURO) als Charity-Spenden / Hilfe der
Qatar Foundation erhalten. Kontaktieren Sie uns über E-Mail für weitere
Informationen;
Mit freundlichen Grüßen,
Ingenieur Saad Al Muhannadi.
Kontakt e-mail: saadalm...@gmail.com
Präsident der Qata
Hi Sakari,
Thank you for the patch.
(CC'ing linux-omap and Tony)
On Saturday 07 March 2015 23:41:07 Sakari Ailus wrote:
> The syscon register isn't part of the ISP, use it through the syscom driver
> regmap instead. The syscom block is considered to be from 343x on ISP
> revision 2.0 whereas 15.
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:15 Sakari Ailus wrote:
> Print a warning when the driver is used with platform data. Existing
> platform data user should move to DT now.
>
> Signed-off-by: Sakari Ailus
Acked-by: Laurent Pinchart
> ---
> drivers/media/plat
Hi Sakari,
Thank you for the patch.
(CC'ing linux-omap and Tony)
On Saturday 07 March 2015 23:41:08 Sakari Ailus wrote:
> The omap3isp MMIO register block is contiguous in the MMIO register space
> apart from the fact that the ISP IOMMU register block is in the middle of
> the area. Ioremap it a
Hi Laurent,
On Sun, Mar 08, 2015 at 01:34:17AM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> (CC'ing linux-omap and Tony)
Thanks.
> On Saturday 07 March 2015 23:41:07 Sakari Ailus wrote:
> > The syscon register isn't part of the ISP, use it through the syscom dri
Hi Sakari,
Thank you for the patch.
(CC'ing Sylwester)
On Saturday 07 March 2015 23:41:09 Sakari Ailus wrote:
> Add lane-polarity property to endpoint nodes. This essentially tells that
> the order of the differential signal wires is inverted.
>
> Signed-off-by: Sakari Ailus
> ---
> Documenta
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:10 Sakari Ailus wrote:
> Add lane_polarity field to struct v4l2_of_bus_mipi_csi2 and write the
> contents of the lane polarity property to it. The field tells the polarity
> of the physical lanes starting from the first one. Any
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:02 Sakari Ailus wrote:
> Only check for call platform data callback functions if there's platform
> data. Also take care of a few other cases where the NULL pdata pointer could
> have been accessed, and remove the check for NULL
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:13 Sakari Ailus wrote:
> The resources the ISP needs are slightly different on 3[45]xx and 3[67]xx.
> Especially the phy-type property is different.
>
> Signed-off-by: Sakari Ailus
> ---
> arch/arm/boot/dts/omap34xx.dtsi | 1
Hi Sakari,
Thank you for the patch.
On Saturday 07 March 2015 23:41:14 Sakari Ailus wrote:
> Add support for the primary camera of the Nokia N950 and N9.
>
> Signed-off-by: Sakari Ailus
> ---
> arch/arm/boot/dts/omap3-n9.dts | 39 +++
> arch/arm/boot/dts/oma
Hi Laurent,
On Sun, Mar 08, 2015 at 01:46:02AM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> (CC'ing Sylwester)
>
> On Saturday 07 March 2015 23:41:09 Sakari Ailus wrote:
> > Add lane-polarity property to endpoint nodes. This essentially tells that
> > the order o
Hi Laurent,
On Sun, Mar 08, 2015 at 01:56:13AM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> On Saturday 07 March 2015 23:41:14 Sakari Ailus wrote:
> > Add support for the primary camera of the Nokia N950 and N9.
> >
> > Signed-off-by: Sakari Ailus
> > ---
> > a
On Friday 06 March 2015 21:25:36 Guennadi Liakhovetski wrote:
> On Fri, 6 Mar 2015, Josh Wu wrote:
> > On 3/5/2015 6:41 PM, Laurent Pinchart wrote:
> >> On Thursday 05 March 2015 13:01:01 Josh Wu wrote:
> >>> The master clock should handled by sensor itself.
> >>
> >> I like that :-)
> >>
> >>> S
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: Sun Mar 8 04:00:18 CET 2015
git branch: test
git hash: 3d945be05ac1e806af075e9315bc1b3409adae2b
gcc versio
Make sure the format fields are reported consistently.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ioctl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c
b/drivers/media/v4l2-core/v4l2-ioctl.c
index b084072..84fb034
The debug attribute was renamed to dev_debug. Update the doc accordingly.
Signed-off-by: Hans Verkuil
---
Documentation/video4linux/v4l2-framework.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/video4linux/v4l2-framework.txt
b/Documentation/video4lin
From: Hans Verkuil
In contrast to all other colorspaces, the BT.2020 colorspace uses
limited range R'G'B' quantization as the default.
This was incorrected documented, so fix this.
Signed-off-by: Hans Verkuil
---
Documentation/DocBook/media/v4l/pixfmt.xml | 11 ++-
1 file changed, 6 i
From: Hans Verkuil
Two colorspace fixes:
- fix wrong xv601/709 formulas in the colorspace chapter
- BT.2020 RGB is limited range, not full range. Update where needed.
Regards,
Hans
Hans Verkuil (4):
DocBook media: fix xv601/709 formulas
DocBook media: BT.2020 RGB uses limited quan
From: Hans Verkuil
The denominator for the scaling and offsets is 256, not 255. Fix this.
Signed-off-by: Hans Verkuil
---
Documentation/DocBook/media/v4l/pixfmt.xml | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/DocBook/media/v4l/pixfmt.xml
b/Do
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
include/uapi/linux/videodev2.h | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
index fbdc360..15b21e4 100644
--- a/include/uapi/linux/videodev2.h
+++ b/i
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
drivers/media/platform/vivid/vivid-tpg.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/media/platform/vivid/vivid-tpg.c
b/drivers/media/platform/vivid/vivid-tpg.c
index 34493f4..acb73b6 100644
--- a/drivers/media/platform/viv
62 matches
Mail list logo