On Wed, Apr 22, 2015 at 01:39:07AM +0200, Luis R. Rodriguez wrote:
> > Mike, do you think the time is right to just remove the iPath driver?
>
> With PAT now being default the driver effectively won't work
> with write-combining on modern kernels. Even if systems are old
> they likely had PAT supp
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: Wed Apr 22 04:00:22 CEST 2015
git branch: test
git hash: e183201b9e917daf2530b637b2f34f1d5afb934d
gcc versi
On Tue, Apr 21, 2015 at 04:57:32PM -0600, Jason Gunthorpe wrote:
> On Wed, Apr 22, 2015 at 12:46:01AM +0200, Luis R. Rodriguez wrote:
>
> > are talking about annotating the qib driver as "known to be broken without
> > PAT"
> > and since the ipath driver needs considerable work to be ported to
>
On Tue, Apr 21, 2015 at 06:51:26PM -0400, Andy Walls wrote:
> Sorry for the top post; mobile work email account.
>
> Luis,
>
> You do the changes to remove MTTR and point me to your dev repo and branch.
> Also point me to the new functions/primitives I'll need.
There is nothing new actually need
On Wed, Apr 22, 2015 at 12:46:01AM +0200, Luis R. Rodriguez wrote:
> are talking about annotating the qib driver as "known to be broken without
> PAT"
> and since the ipath driver needs considerable work to be ported to
> use PAT (the
This only seems to be true for one of the chips that driver s
On Wed, Apr 15, 2015 at 01:42:47PM -0700, Andy Lutomirski wrote:
> On Mon, Apr 13, 2015 at 10:49 AM, Luis R. Rodriguez wrote:
>
> > c) ivtv: the driver does not have the PCI space mapped out separately, and
> > in fact it actually does not do the math for the framebuffer, instead it
> > lets
> >
On Tue, Apr 21, 2015 at 3:08 PM, Luis R. Rodriguez wrote:
> On Tue, Apr 21, 2015 at 3:02 PM, Luis R. Rodriguez wrote:
>> Andy, can we live without MTRR support on this driver for future kernels?
>> This
>> would only leave ipath as the only offending driver.
>
> Sorry to be clear, can we live wi
On Tue, Apr 21, 2015 at 3:02 PM, Luis R. Rodriguez wrote:
> Andy, can we live without MTRR support on this driver for future kernels? This
> would only leave ipath as the only offending driver.
Sorry to be clear, can we live with removal of write-combining on this driver?
Luis
--
To unsubscribe
On Wed, Apr 15, 2015 at 09:07:37PM -0400, Andy Walls wrote:
> On Thu, 2015-04-16 at 01:58 +0200, Luis R. Rodriguez wrote:
> > Hey Andy, thanks for your review, adding Hyong-Youb Kim for review of the
> > full range ioremap_wc() idea below.
> >
> > On Wed, Apr 15, 2015 at 06:38:51PM -0400, Andy W
Hello Laurent
On Tue, Apr 21, 2015 at 7:44 PM, Laurent Pinchart
wrote:
> Hi Ricardo,
>
> Thank you for the patch, and sorry for the late review (so late that the patch
> has already been merged).
No worries.
>
> On Friday 20 March 2015 14:30:46 Ricardo Ribalda Delgado wrote:
>> Volatile control
Laurent -
On 04/21/15 12:58, Laurent Pinchart wrote:
Hi Tim,
On Wednesday 18 March 2015 10:25:34 Tim Nordell wrote:
I'll give that a shot and try add code into the adv7180 driver to turn on
and off its output signals. However, it seems like if the driver can avoid
a problem presented by exter
Hi Tim,
On Wednesday 18 March 2015 10:25:34 Tim Nordell wrote:
> On 03/18/15 10:19, Laurent Pinchart wrote:
> > On Tuesday 10 March 2015 14:24:53 Tim Nordell wrote:
> >> During testing there appeared to be a race condition where the IRQs
> >> for VD0 and VD1 could be triggered while enabling the C
Hi Ricardo,
Thank you for the patch, and sorry for the late review (so late that the patch
has already been merged).
On Friday 20 March 2015 14:30:46 Ricardo Ribalda Delgado wrote:
> Volatile controls should not generate CH_VALUE events.
What's the rationale for that ? I would actually expect t
On Wed, Apr 15, 2015 at 05:58:14PM -0700, Andy Lutomirski wrote:
> On Wed, Apr 15, 2015 at 4:59 PM, Andy Walls wrote:
> > On Wed, 2015-04-15 at 16:42 -0700, Andy Lutomirski wrote:
> >> On Wed, Apr 15, 2015 at 3:38 PM, Andy Walls
> >> wrote:
> >
> >> >
> >>
> >> IMO the right solution would be to
Hi Chris,
On Tuesday 21 April 2015 11:04:14 Chris Whittenburg wrote:
> On Fri, Apr 17, 2015 at 4:39 AM, Laurent Pinchart wrote:
> > On Thursday 16 April 2015 13:05:30 Chris Whittenburg wrote:
> >> On Tue, Apr 7, 2015 at 10:51 AM, Laurent Pinchart wrote:
> >>> Black level compensation is applied by
On Fri, Apr 17, 2015 at 4:39 AM, Laurent Pinchart
wrote:
> Hi Chris,
>
> On Thursday 16 April 2015 13:05:30 Chris Whittenburg wrote:
>> On Tue, Apr 7, 2015 at 10:51 AM, Laurent Pinchart wrote:
>> > Black level compensation is applied by the CCDC before writing raw frames
>> > to memory. If your ra
Hi Linus,
Please pull from:
git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media
tags/media/v4.1-2
For:
- A new frontend driver for new ATSC devices: lgdt3306a
- A new sensor driver: ov2659
- a new platform driver: xilinx
- The m88ts2022 tuner driver was merged at ts2020 d
Hi Hans,
On Tue, Apr 21, 2015 at 5:32 PM, Hans de Goede wrote:
>> diff --git a/drivers/media/usb/gspca/sn9c2028.h
>> b/drivers/media/usb/gspca/sn9c2028.h
>> index 8fd1d3e..6f20c0f 100644
>> --- a/drivers/media/usb/gspca/sn9c2028.h
>> +++ b/drivers/media/usb/gspca/sn9c2028.h
>> @@ -21,8 +21,17 @@
Hi Hans,
On Tue, Apr 21, 2015 at 5:21 PM, Hans de Goede wrote:
>> @@ -128,7 +129,7 @@ static int sn9c2028_long_command(struct gspca_dev
>> *gspca_dev, u8 *command)
>> status = -1;
>> for (i = 0; i < 256 && status < 2; i++)
>> status = sn9c2028_read1(gspca_dev);
>>
Hi,
On 19-04-15 20:52, Vasily Khoruzhick wrote:
Autogain algorithm is very simple, if average luminance is low - increase gain,
if it's high - decrease gain. Gain granularity is low enough for this algo to
stabilize quickly.
Signed-off-by: Vasily Khoruzhick
---
drivers/media/usb/gspca/sn9c20
Hi Vasily,
Thanks for the patches.
On 19-04-15 20:52, Vasily Khoruzhick wrote:
Signed-off-by: Vasily Khoruzhick
---
drivers/media/usb/gspca/sn9c2028.c | 120 -
1 file changed, 119 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/gspca/sn9c2028
Ensure standard format descriptions by filling it in in the v4l2 core.
Currently these descriptions are all over the place and every driver dreams
up its own description. That's not good.
Regards,
Hans
The following changes since commit e183201b9e917daf2530b637b2f34f1d5afb934d:
[media
From: Hans Verkuil
The format description is now filled in by the core, so we can
drop this in this skeleton driver.
Signed-off-by: Hans Verkuil
---
Documentation/video4linux/v4l2-pci-skeleton.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/Documentation/video4linux/v4l2-pci-skeleton.c
From: Hans Verkuil
The format description is now filled in by the core, so we can
drop this in this virtual m2m driver.
Signed-off-by: Hans Verkuil
---
drivers/media/platform/vim2m.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/media/platform/vim2m.c b/drivers/media/platform/
From: Hans Verkuil
The descriptions used in drivers for the formats returned with ENUM_FMT
are all over the place.
So instead allow the core to fill them in if the driver didn't. This
allows drivers to drop the description and flags.
Based on an earlier patch from Philipp Zabel:
http://comments
From: Hans Verkuil
The format description is now filled in by the core, so we can
drop this in this virtual driver.
Signed-off-by: Hans Verkuil
---
drivers/media/platform/vivid/vivid-core.h | 1 -
drivers/media/platform/vivid/vivid-vid-cap.c| 4 --
drivers/media/platform/vivid/vivi
From: Hans Verkuil
This patch series is identical to https://patchwork.linuxtv.org/patch/29080/
but it removes the description from the skeleton driver and the virtual
drivers. Since those are basically reference drivers it makes sense to update
those first.
I'll make a pull request of this as w
These patches replace most duplicate video ops by their pad ops counterparts.
The only remaining duplicate ops deal with cropping and selection, and I want
to be able to test that first. I have difficulty doing that, but I am
expecting hardware in the near future that should enable me to do this wo
Hi Dan,
Thank you for the patch.
On Tuesday 21 April 2015 12:31:10 Dan Carpenter wrote:
> My static checker warns that the name of the port can be 15 characters
> when you consider the NUL terminator and that's one more than the 14
> characters in name[]. Maybe it's an off-by-one?
>
> It's unlik
From: Hans Verkuil
In order to test the request API in applications we add request support to
vivid. The brightness, contrast, saturation and hue controls now can be used
in requests. Those were chosen because the test pattern generator supports
those controls and will adjust the TPG colors accor
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ctrls.c | 60 --
drivers/media/v4l2-core/v4l2-dev.c| 1 +
drivers/media/v4l2-core/v4l2-ioctl.c | 81 ++-
drivers/media/v4l2-core/v4l2-subdev.c | 63
From: Hans Verkuil
Add documentation about requests.
Signed-off-by: Hans Verkuil
---
Documentation/video4linux/v4l2-requests.txt | 233
1 file changed, 233 insertions(+)
create mode 100644 Documentation/video4linux/v4l2-requests.txt
diff --git a/Documentation/vid
From: Hans Verkuil
The vb2_qbuf_request() function will queue any buffers for the given request
that are in state PREPARED.
Useful when drivers have to implement the req_queue callback.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/videobuf2-core.c | 18 ++
include/m
From: Hans Verkuil
Add the V4L2_REQ_CMD_QUEUE command and the req_queue callback to struct
v4l2_device. Call it if set from v4l2-ioctl.c and v4l2-subdev.c. Make sure
in v4l2-ioctl.c to unlock any current lock first (and relock afterwards).
That way req_queue is called with the assurance that ther
From: Hans Verkuil
Experimental: I am still not certain whether this is desired or not.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ctrls.c | 28
drivers/media/v4l2-core/v4l2-ioctl.c | 9 -
drivers/media/v4l2-core/v4l2-subdev.c | 11
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ctrls.c | 42
include/media/v4l2-ctrls.h | 1 +
2 files changed, 43 insertions(+)
diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c
b/drivers/media/v4l2-core/v4l2-ct
From: Hans Verkuil
The v4l2_device_req_queue() function is a helper that can be used
as the req_queue callback in simple cases: it will walk over all
registered video_devices and call vb2_qbuf_request() for each video
device.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-device.
From: Hans Verkuil
In order to efficiently handle V4L2_REQ_CMD_QUEUE we need to know which
video_device structs are registered for the given v4l2_device struct.
So create a list of vdevs in v4l2_device and add/remove each video_device
there as it is registered/unregistered.
Signed-off-by: Hans
From: Hans Verkuil
Drivers need to be able to select a specific request. Add a new function that
can
be used to apply a given request.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ctrls.c | 112 +--
include/media/v4l2-ctrls.h | 3 +
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ctrls.c | 228 ++-
drivers/media/v4l2-core/v4l2-ioctl.c | 8 ++
include/media/v4l2-ctrls.h | 20 +++
3 files changed, 228 insertions(+), 28 deletions(-)
diff --git a/dri
From: Hans Verkuil
When queuing buffers allow for passing the request ID that
should be associated with this buffer. Split the u32 reserved2 field
into two u16 fields, one for request, one with the old reserved2 name.
Signed-off-by: Hans Verkuil
---
drivers/media/usb/cpia2/cpia2_v4l.c
From: Hans Verkuil
The driver has to set allow_requests explicitly in order to allow
queuing or preparing buffers for a specific request ID.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/videobuf2-core.c | 5 +
include/media/videobuf2-core.h | 2 ++
2 files changed, 7 i
From: Hans Verkuil
struct v4l2_query_ext_ctrl is extended with a new 'max_reqs' field to store
the maximum number of outstanding requests that contain this control.
Signed-off-by: Hans Verkuil
---
drivers/media/v4l2-core/v4l2-ioctl.c | 5 +++--
include/uapi/linux/videodev2.h | 4 +++-
2
From: Hans Verkuil
The ctrl_class is fairly pointless when used with drivers that use the control
framework: you can just fill in 0 and it will just work fine. There are still
some old unconverted drivers that do not support 0 and instead want the control
class there. The idea being that all cont
From: Hans Verkuil
This patch series adds support for the request API (formerly called the
configuration store API:
http://www.spinics.net/lists/linux-media/msg81024.html).
This second version takes into account all the feedback I received from
various people and from the discussions in Düsseld
My static checker warns that the name of the port can be 15 characters
when you consider the NUL terminator and that's one more than the 14
characters in name[]. Maybe it's an off-by-one?
It's unlikely that we hit the limit and even if we do the overflow will
only affect one of the two bytes of p
46 matches
Mail list logo