RE: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME

2014-07-21 Thread Divneil Wadhawan
Hi Hans, This patch is all messed up and doesn't apply. Check your mailer settings: it clearly replaced hard tabs by a space. Can you repost? I tried to find out if I can change the mailer settings. Seems, that is the problem. I tried using mutt, but, seems the configuration is missing.

Re: [PATCH v2 06/29] [media] coda: Add encoder/decoder support for CODA960

2014-07-21 Thread Robert Schwebel
Hi Fabio, On Fri, Jul 11, 2014 at 02:33:18PM +0200, Robert Schwebel wrote: On Wed, Jul 02, 2014 at 09:16:42PM +0200, Robert Schwebel wrote: It would be really nice if the firmware was available in the linux-firmware repository. Do you think this would be possible? Best wishes, --

Re: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME

2014-07-21 Thread Ricardo Ribalda Delgado
Why dont you use git send-email ? On Mon, Jul 21, 2014 at 8:03 AM, Divneil Wadhawan divn...@outlook.com wrote: Hi Hans, This patch is all messed up and doesn't apply. Check your mailer settings: it clearly replaced hard tabs by a space. Can you repost? I tried to find out if I can

[PATCH for v3.17] v4l2-ioctl: set V4L2_CAP_EXT_PIX_FORMAT for device_caps

2014-07-21 Thread Hans Verkuil
V4L2_CAP_EXT_PIX_FORMAT is set for capabilities, but it needs to be set for device_caps as well: device_caps should report all caps relevant to the device node, and this is one of them. Signed-off-by: Hans Verkuil hans.verk...@cisco.com diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c

Re: [PATCH for v3.17] v4l2-ioctl: set V4L2_CAP_EXT_PIX_FORMAT for device_caps

2014-07-21 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Monday 21 July 2014 09:14:46 Hans Verkuil wrote: V4L2_CAP_EXT_PIX_FORMAT is set for capabilities, but it needs to be set for device_caps as well: device_caps should report all caps relevant to the device node, and this is one of them. Signed-off-by:

Re: [PATCH for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Sunday 20 July 2014 15:10:03 Hans Verkuil wrote: Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of the win format this overwrites a pointer value that

Re: [PATCH for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Hans Verkuil
On 07/21/2014 11:11 AM, Laurent Pinchart wrote: Hi Hans, Thank you for the patch. On Sunday 20 July 2014 15:10:03 Hans Verkuil wrote: Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of

[PATCHv2 for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Hans Verkuil
Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of the win format this overwrote a pointer value that is passed in from userspace. Just set it for V4L2_BUF_TYPE_VIDEO_CAPTURE and OUTPUT and not

Re: [PATCHv2 for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Monday 21 July 2014 11:31:46 Hans Verkuil wrote: Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of the win format this overwrote a pointer value that is

[PATCHv3 for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Hans Verkuil
Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of the win format this overwrote a pointer value that is passed in from userspace. Just set it for V4L2_BUF_TYPE_VIDEO_CAPTURE and OUTPUT only. Also

[PATCHv4 for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Hans Verkuil
Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of the win format this overwrote a pointer value that is passed in from userspace. Just set it for V4L2_BUF_TYPE_VIDEO_CAPTURE and OUTPUT only. Set

Re: [PATCHv4 for v3.17] v4l2-ioctl: don't set PRIV_MAGIC unconditionally in g_fmt()

2014-07-21 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Monday 21 July 2014 12:50:39 Hans Verkuil wrote: Regression fix: V4L2_PIX_FMT_PRIV_MAGIC should only be set for the VIDEO_CAPTURE and VIDEO_OUTPUT buffer types, and not for any others. In the case of the win format this overwrote a pointer value that is

Re: [PATCH/RFC v4 15/21] media: Add registration helpers for V4L2 flash

2014-07-21 Thread Sakari Ailus
Hi Jacek, Jacek Anaszewski wrote: This patch adds helper functions for registering/unregistering LED class flash devices as V4L2 subdevs. The functions should be called from the LED subsystem device driver. In case the support for V4L2 Flash sub-devices is disabled in the kernel config the

Re: Configurable Video Controller Driver

2014-07-21 Thread Sakari Ailus
Hi Julien, On Thu, Jul 10, 2014 at 04:19:06PM +0200, Julien BERAUD wrote: We are developing a driver for our video controller which has the particularity of being very reconfigurable. We have reached a point at which the complexity and variety of the applications we need to implement forces

Re: [PATCH] DocBook media typo

2014-07-21 Thread Sakari Ailus
On Sun, Jul 20, 2014 at 10:16:41PM +0200, Hans Verkuil wrote: V4L2_CID_BASE_LASTP1 should be V4L2_CID_LASTP1. This has probably been wrong since the earliest days of this documentation until I did a copy-and-paste and found out that V4L2_CID_BASE_LASTP1 doesn't actually exist :-)

[no subject]

2014-07-21 Thread Peter Meerwald
Hello, how can I query the supported pixel format(s) of a sensor connected via media-ctl and exposed via /dev/videoX there is VIDIOC_ENUM_FMT (which fails) and VIDIOC_SUBDEV_ENUM_MBUS_CODE (which works, but on a subdev, not on /dev/videoX) v4l2_subdev_video_ops has .enum_mbus_fmt (this is

Re: [PATCH] [media] exynos-gsc: Remove PM_RUNTIME dependency

2014-07-21 Thread Sylwester Nawrocki
On 21/07/14 06:54, Shaik Ameer Basha wrote: 1] Currently Gscaler clock is enabled only inside pm_runtime callbacks. If PM_RUNTIME is disabled, driver hangs. This patch removes the PM_RUNTIME dependency by keeping the clock enable/disable functions in m2m start/stop streaming

[RFC PATCH] Docbook/media: improve data_offset/bytesused documentation

2014-07-21 Thread Hans Verkuil
This patch explicitly documents the relationship between bytesused and data_offset. But looking in the kernel there isn't a single driver that actually sets data_offset. Do such beasts exist? It's also annoying that there is no such equivalent for the single planar API, so it is asymmetrical.

Re: [RFC PATCH] Docbook/media: improve data_offset/bytesused documentation

2014-07-21 Thread Nicolas Dufresne
Le lundi 21 juillet 2014 à 15:16 +0200, Hans Verkuil a écrit : + Note that data_offset is included in structfieldbytesused/structfield. + So the size of the image in the plane is + structfieldbytesused/structfield-structfielddata_offset/structfield at +

[PATCH 4/7] v4l2-ctrls: add RX RDS controls.

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com The radio-miropcm20 driver has firmware that decodes the RDS signals. So in that case the RDS data becomes available in the form of controls. Add support for these controls to the control framework, allowing the miro driver to use them. Signed-off-by:

[PATCH 3/7] si4713: add the missing RDS functionality.

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Not all the RDS features of the si4713 were supported. Add the missing bits to fully support the hardware capabilities. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Cc: Eduardo Valentin edubez...@gmail.com --- drivers/media/radio/si4713/si4713.c |

[PATCH 5/7] DocBook/media: document the new RDS RX controls

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Document the new RDS receiver controls. This will be used by the radio-miropcm20 driver. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- Documentation/DocBook/media/v4l/controls.xml | 51 1 file changed, 51

[PATCH 1/7] v4l2-ctrls: add new RDS TX controls

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com The si4713 supports several RDS features not yet implemented in the driver. This patch adds the missing RDS functionality to the list of RDS controls. The ALT_FREQS control is a compound control containing an array of up to 25 (the maximum according to

[PATCH 7/7] radio-miropcm20: add RDS support.

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Once upon a time the radio-miropcm20 driver had RDS support. However, after some internal kernel changes that support was removed. Now that we have a nice RDS API I have been working on adding back this support. It has been tested with the si4713 RDS

[PATCH 2/7] DocBook/media: document the new RDS TX controls

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Document the new RDS features that will be supported by the si4713 driver. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Cc: Eduardo Valentin edubez...@gmail.com --- Documentation/DocBook/media/v4l/controls.xml | 62 1

[PATCH 0/7] si4713/miropcm20: RDS enhancements

2014-07-21 Thread Hans Verkuil
This patch series adds a bunch of missing RDS TX controls and implements them in the si4713 driver. It also adds back RDS support to the miropcm20 driver. The Alternate Frequencies control is a u32 array since there can be up to 25 alternate frequencies. This was also the reason why I am only now

[PATCH 6/7] v4l2-ctrls: add support for setting string controls

2014-07-21 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Rather than always having to use a v4l2_ext_control struct to set a control value from within a driver, switch to just setting the new value. This is faster and it makes it possible to set more complex types such as a string control as is added by this

Re: [PATCH 0/7] si4713/miropcm20: RDS enhancements

2014-07-21 Thread Hans Verkuil
On 07/21/2014 03:45 PM, Hans Verkuil wrote: This patch series adds a bunch of missing RDS TX controls and implements them in the si4713 driver. It also adds back RDS support to the miropcm20 driver. The Alternate Frequencies control is a u32 array since there can be up to 25 alternate

[GIT PULL FOR v3.17] Various fixes and moving solo6x10/go7007 to mainline

2014-07-21 Thread Hans Verkuil
Hi Mauro, Here is a set of various fixes: two important ones in v4l2-ioctl and one in vb2. Also some DocBook fixes, a few VBI defines added and documented, a davinci bugfix (reported by the new gcc-4.9 compiler I'm now using for the daily build) and last but not least the move of the solo6x10 and

Re: [PATCH/RFC v4 16/21] leds: Add support for max77693 mfd flash cell

2014-07-21 Thread Sakari Ailus
Hi Jacek, Thanks for the patchset. Jacek Anaszewski wrote: This patch adds led-flash support to Maxim max77693 chipset. A device can be exposed to user space through LED subsystem sysfs interface or through V4L2 subdevice when the support for V4L2 Flash sub-devices is enabled. Device

Re: [GIT PULL FOR v3.16] mt9p031 fixes

2014-07-21 Thread Enric Balletbo Serra
Hi Laurent, 2014-07-10 0:29 GMT+02:00 Laurent Pinchart laurent.pinch...@ideasonboard.com: Hi Enric, On Wednesday 09 July 2014 17:56:59 Enric Balletbo Serra wrote: 2014-05-16 2:45 GMT+02:00 Laurent Pinchart wrote: Hi Mauro, The following changes since commit

Re: Fresco Logic FL2000

2014-07-21 Thread Laurent Pinchart
Hi Michael, On Tuesday 20 May 2014 18:32:08 Michael Durkin wrote: It was suggested to me to inquire here if anyone was working on drivers or support for the Fresco Logic FL2000 1d5c:2000 Could you please post the output of lsusb -v -d 1d5c:2000 (if possible running as root) ? -- Regards,

[PATCH 2/3] xc4000: add module meta-tag with the firmware names

2014-07-21 Thread Mauro Carvalho Chehab
This meta-tag is used by some distros to help them package the firmware and generate proper initrd images. So, add the firmware names there. Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com --- drivers/media/tuners/xc4000.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 0/3] Fix support for PCTV 340e

2014-07-21 Thread Mauro Carvalho Chehab
Support for PCTV340e was broken for a long time. The regression happened due to changeset 6fe1099c7aec. The above changeset was right, as it improves device tuning, but there's a bug at xc4000 get_frequency(): instead of returning the tuned frequency, it was returning the frequency minus an

[PATCH 1/3] xc4000: Update firmware name

2014-07-21 Thread Mauro Carvalho Chehab
The firmware name at: http://www.kernellabs.com/firmware/xc4000/ Is different from the one at the Kernel. Update it try first the new name, falling back to the previous one if the new name can't be found. Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com ---

[PATCH 3/3] xc4000: Fix get_frequency()

2014-07-21 Thread Mauro Carvalho Chehab
The programmed frequency on xc4000 is not the middle frequency, but the initial frequency on the bandwidth range. However, the DVB API works with the middle frequency. This works fine on set_frontend, as the device calculates the needed offset. However, at get_frequency(), the returned value is

Re: non-working UVC device 058f:5608

2014-07-21 Thread Laurent Pinchart
On Monday 09 June 2014 12:27:37 Johannes Berg wrote: On Mon, 2014-06-09 at 11:59 +0200, Johannes Berg wrote: Johannes, could you enable USB debugging in the linus/master kernel and provide a kernel log ? I'll try to get some logs (wasn't there tracing added to xhci too? will check)

[PATCH] xc5000: Fix get_frequency()

2014-07-21 Thread Mauro Carvalho Chehab
The programmed frequency on xc5000 is not the middle frequency, but the initial frequency on the bandwidth range. However, the DVB API works with the middle frequency. Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com --- drivers/media/tuners/xc5000.c | 22 -- 1 file

[GIT PULL for 3.16-rc7] media fixes for master

2014-07-21 Thread Mauro Carvalho Chehab
Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media v4l_for_linus For a series of driver fixes: - Fix DVB-S tuning with tda1071; - Fix tuner probe on af9035 when the device has a bad eeprom; - Some fixes for the new si2168/2157

[PATCH v2] mb86a20s: fix ISDB-T mode handling

2014-07-21 Thread Mauro Carvalho Chehab
The driver was reporting an incorrect mode, when mode 2 is selected. While testing it, noticed that neither mode 1 or guard interval 1/32 is supported by this device. Document it, and ensure that it will report _AUTO when it doesn't lock, in order to not report a wrong detection to userspace.

Re: [PATCH] mb86a20s: fix ISDB-T mode handling

2014-07-21 Thread Mauro Carvalho Chehab
Em Sat, 12 Jul 2014 21:52:46 +0300 Antti Palosaari cr...@iki.fi escreveu: Mauro, you have forgot some debug hacks to that patch. Thanks for noticing. Actually, it got merged by mistake with a hack I'm using to debug another driver. Just sent a version 2 of this patch. Regards, Mauro Antti

Re: [PATCH v3 06/32] [media] coda: Add encoder/decoder support for CODA960

2014-07-21 Thread Mauro Carvalho Chehab
Em Fri, 11 Jul 2014 11:36:17 +0200 Philipp Zabel p.za...@pengutronix.de escreveu: This patch adds support for the CODA960 VPU in Freescale i.MX6 SoCs. It enables h.264 and MPEG4 encoding and decoding support. Besides the usual register shifting, the CODA960 gains frame memory control and GDI

Re: [PATCH v3 18/32] [media] v4l2-mem2mem: export v4l2_m2m_try_schedule

2014-07-21 Thread Mauro Carvalho Chehab
Em Fri, 11 Jul 2014 11:36:29 +0200 Philipp Zabel p.za...@pengutronix.de escreveu: From: Michael Olbrich m.olbr...@pengutronix.de Some drivers might allow to decode remaining frames from an internal ringbuffer after a decoder stop command. Allow those to call v4l2_m2m_try_schedule

Re: [PATCH v3 06/32] [media] coda: Add encoder/decoder support for CODA960

2014-07-21 Thread Robert Schwebel
Hi Mauro, On Mon, Jul 21, 2014 at 04:01:28PM -0300, Mauro Carvalho Chehab wrote: This patch adds support for the CODA960 VPU in Freescale i.MX6 SoCs. It enables h.264 and MPEG4 encoding and decoding support. Besides the usual register shifting, the CODA960 gains frame memory control and

[PATCH] v4l: Add ARGB555X and XRGB555X pixel formats

2014-07-21 Thread Laurent Pinchart
The existing RGB555X pixel format is ill-defined in respect to its alpha bit and its meaning is driver dependent. Create new standard ARGB555X and XRGB555X variants with clearly defined meanings and make the existing variant deprecated. The new pixel formats 4CC values have been selected to match

[PATCH] v4l: Clarify RGB666 pixel format definition

2014-07-21 Thread Laurent Pinchart
The RGB666 pixel format doesn't include an alpha channel. Document it as such. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- .../DocBook/media/v4l/pixfmt-packed-rgb.xml | 20 ++-- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git

Re: [PATCH v2 03/23] v4l: Support extending the v4l2_pix_format structure

2014-07-21 Thread Laurent Pinchart
Hi Hans, On Thursday 17 July 2014 23:04:09 Hans Verkuil wrote: Hi Laurent, Something that just caught my eye: On 06/24/2014 01:54 AM, Laurent Pinchart wrote: The v4l2_pix_format structure has no reserved field. It is embedded in the v4l2_framebuffer structure which has no reserved

Re: [PATCH] v4l: Clarify RGB666 pixel format definition

2014-07-21 Thread Hans Verkuil
On 07/21/2014 10:39 PM, Laurent Pinchart wrote: The RGB666 pixel format doesn't include an alpha channel. Document it as such. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- .../DocBook/media/v4l/pixfmt-packed-rgb.xml | 20 ++-- 1 file

Re: [PATCH] v4l: Add ARGB555X and XRGB555X pixel formats

2014-07-21 Thread Hans Verkuil
On 07/21/2014 10:37 PM, Laurent Pinchart wrote: The existing RGB555X pixel format is ill-defined in respect to its alpha bit and its meaning is driver dependent. Create new standard ARGB555X and XRGB555X variants with clearly defined meanings and make the existing variant deprecated. The

Re: [PATCH v2 03/23] v4l: Support extending the v4l2_pix_format structure

2014-07-21 Thread Hans Verkuil
On 07/21/2014 10:56 PM, Laurent Pinchart wrote: Hi Hans, On Thursday 17 July 2014 23:04:09 Hans Verkuil wrote: Hi Laurent, Something that just caught my eye: On 06/24/2014 01:54 AM, Laurent Pinchart wrote: The v4l2_pix_format structure has no reserved field. It is embedded in the

Re: [PATCH] v4l: Clarify RGB666 pixel format definition

2014-07-21 Thread Laurent Pinchart
Hi Hans, On Monday 21 July 2014 23:43:16 Hans Verkuil wrote: On 07/21/2014 10:39 PM, Laurent Pinchart wrote: The RGB666 pixel format doesn't include an alpha channel. Document it as such. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com ---

[PATCH] v4l: Fix ARGB32 fourcc value in the documentation

2014-07-21 Thread Laurent Pinchart
The ARGB32 pixel format's fourcc value is defined to 'BA24' in the videodev2.h header, but documented as 'AX24'. Fix the documentation. Reported-by: Acked-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com ---

Re: [PATCH] v4l: Clarify RGB666 pixel format definition

2014-07-21 Thread Hans Verkuil
On 07/22/2014 12:30 AM, Laurent Pinchart wrote: Hi Hans, On Monday 21 July 2014 23:43:16 Hans Verkuil wrote: On 07/21/2014 10:39 PM, Laurent Pinchart wrote: The RGB666 pixel format doesn't include an alpha channel. Document it as such. Signed-off-by: Laurent Pinchart

Re: [PATCH] v4l: Fix ARGB32 fourcc value in the documentation

2014-07-21 Thread Hans Verkuil
On 07/22/2014 12:41 AM, Laurent Pinchart wrote: The ARGB32 pixel format's fourcc value is defined to 'BA24' in the videodev2.h header, but documented as 'AX24'. Fix the documentation. Reported-by: Acked-by: Hans Verkuil hans.verk...@cisco.com Reported-by or Acked-by? :-) Anyway: Acked-by:

Re: [PATCH] v4l: Fix ARGB32 fourcc value in the documentation

2014-07-21 Thread Laurent Pinchart
On Tuesday 22 July 2014 00:45:05 Hans Verkuil wrote: On 07/22/2014 12:41 AM, Laurent Pinchart wrote: The ARGB32 pixel format's fourcc value is defined to 'BA24' in the videodev2.h header, but documented as 'AX24'. Fix the documentation. Reported-by: Acked-by: Hans Verkuil

Re: [PATCH] v4l: Fix ARGB32 fourcc value in the documentation

2014-07-21 Thread Hans Verkuil
On 07/22/2014 12:46 AM, Laurent Pinchart wrote: On Tuesday 22 July 2014 00:45:05 Hans Verkuil wrote: On 07/22/2014 12:41 AM, Laurent Pinchart wrote: The ARGB32 pixel format's fourcc value is defined to 'BA24' in the videodev2.h header, but documented as 'AX24'. Fix the documentation.

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Mauro Carvalho Chehab
Em Fri, 18 Jul 2014 04:14:32 +0300 Antti Palosaari cr...@iki.fi escreveu: * AirSpy SDR driver * all SDR drivers moved out of staging * few new SDR stream formats regards Antti The following changes since commit 3445857b22eafb70a6ac258979e955b116bfd2c6: [media] hdpvr: fix two

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Mauro Carvalho Chehab
Em Mon, 21 Jul 2014 20:50:05 -0300 Mauro Carvalho Chehab m.che...@samsung.com escreveu: Em Fri, 18 Jul 2014 04:14:32 +0300 Antti Palosaari cr...@iki.fi escreveu: * AirSpy SDR driver * all SDR drivers moved out of staging * few new SDR stream formats regards Antti The

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Antti Palosaari
So what. Those were mostly WARNING only and all but long lines were some new checks added to checkpatch recently. chekcpatch gets all the time new and new checks, these were added after I have made that driver. I will surely clean those later when I do some new changes to driver and my

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Antti Palosaari
On 07/22/2014 03:05 AM, Mauro Carvalho Chehab wrote: total: 1 errors, 45 warnings, 1517 lines checked drivers/media/usb/msi2500/msi2500.c has style problems, please review. FYI, I applied the rest of this patch series, except for those patches: msi2500: move msi3101 out of staging

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Mauro Carvalho Chehab
Em Tue, 22 Jul 2014 03:08:19 +0300 Antti Palosaari cr...@iki.fi escreveu: So what. Those were mostly WARNING only and all but long lines were some new checks added to checkpatch recently. chekcpatch gets all the time new and new checks, these were added after I have made that driver. I

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Antti Palosaari
On 07/22/2014 03:51 AM, Mauro Carvalho Chehab wrote: Em Tue, 22 Jul 2014 03:08:19 +0300 Antti Palosaari cr...@iki.fi escreveu: So what. Those were mostly WARNING only and all but long lines were some new checks added to checkpatch recently. chekcpatch gets all the time new and new checks,

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Mauro Carvalho Chehab
Em Tue, 22 Jul 2014 04:05:05 +0300 Antti Palosaari cr...@iki.fi escreveu: On 07/22/2014 03:51 AM, Mauro Carvalho Chehab wrote: Em Tue, 22 Jul 2014 03:08:19 +0300 Antti Palosaari cr...@iki.fi escreveu: So what. Those were mostly WARNING only and all but long lines were some new checks

Bug 80871 - WARNING: CPU: 0 PID: 1498 at fs/sysfs/group.c:219 sysfs_remove_group+0x98/0xa0()

2014-07-21 Thread João M. S. Silva
Hi, Per request of Greg Kroah-Hartman, I'm sending this bug's reference: https://bugzilla.kernel.org/show_bug.cgi?id=80871#c1 Regards, -- João M. S. Silva -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More

cron job: media_tree daily build: ERRORS

2014-07-21 Thread Hans Verkuil
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 Jul 22 04:00:38 CEST 2014 git branch: test git hash: 1b303e1a58599e42f858805285f03ccb5a4e18d2 gcc

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Antti Palosaari
On 07/22/2014 05:09 AM, Mauro Carvalho Chehab wrote: Em Tue, 22 Jul 2014 04:05:05 +0300 Antti Palosaari cr...@iki.fi escreveu: On 07/22/2014 03:51 AM, Mauro Carvalho Chehab wrote: Em Tue, 22 Jul 2014 03:08:19 +0300 Antti Palosaari cr...@iki.fi escreveu: So what. Those were mostly WARNING

[PATCH] rtl2832_sdr: remove plain 64-bit divisions

2014-07-21 Thread Antti Palosaari
Commit 0ba2aeb6dab80920edd9cf5b93b1ea4d6913b8f3 (v4l2-ctrls: increase internal min/max/step/def to 64 bit) changes v4l2 controls to 64-bit. Driver it not working on 32-bit arch as it uses directly control 'step' which is changed to 64-bit. Reported-by: kbuild test robot fengguang...@intel.com Cc:

Re: [GIT PULL FOR v3.17] Various fixes and moving solo6x10/go7007 to mainline

2014-07-21 Thread Mauro Carvalho Chehab
Em Mon, 21 Jul 2014 16:08:41 +0200 Hans Verkuil hverk...@xs4all.nl escreveu: Hi Mauro, Here is a set of various fixes: two important ones in v4l2-ioctl and one in vb2. Also some DocBook fixes, a few VBI defines added and documented, a davinci bugfix (reported by the new gcc-4.9 compiler I'm

[GIT PULL FOR v3.17] Move go7007 and solo6x10 out of staging

2014-07-21 Thread Hans Verkuil
Rebased and dropped the offending files as requested. I plan on making the saa7134-go7007 patch work (which was why I kept it in the mainline driver), but I'll just add it back later. That's probably going to be for 3.18 anyway. Regards, Hans The following changes since commit

Re: [GIT PULL] SDR stuff

2014-07-21 Thread Mauro Carvalho Chehab
Em Tue, 22 Jul 2014 05:48:26 +0300 Antti Palosaari cr...@iki.fi escreveu: On 07/22/2014 05:09 AM, Mauro Carvalho Chehab wrote: Em Tue, 22 Jul 2014 04:05:05 +0300 Antti Palosaari cr...@iki.fi escreveu: On 07/22/2014 03:51 AM, Mauro Carvalho Chehab wrote: Em Tue, 22 Jul 2014 03:08:19

[PATCH] rtl2832_sdr: fix Kconfig dependencies

2014-07-21 Thread Antti Palosaari
MEDIA_SDR_SUPPORT and I2C_MUX are needed for rtl2832_sdr. Reported-by: kbuild test robot fengguang...@intel.com Signed-off-by: Antti Palosaari cr...@iki.fi --- drivers/media/dvb-frontends/Kconfig | 2 +- drivers/media/usb/dvb-usb-v2/Kconfig | 2 +- 2 files changed, 2 insertions(+), 2

Re: [PATCH] [media] exynos-gsc: Remove PM_RUNTIME dependency

2014-07-21 Thread Sachin Kamat
Hi Shaik, On Mon, Jul 21, 2014 at 10:24 AM, Shaik Ameer Basha shaik.am...@samsung.com wrote: 1] Currently Gscaler clock is enabled only inside pm_runtime callbacks. If PM_RUNTIME is disabled, driver hangs. This patch removes the PM_RUNTIME dependency by keeping the clock enable/disable

Different Devices identical hardware

2014-07-21 Thread Matthias Schwarzott
Hi, I want to add support for Hauppauge WinTV 930C-HD and PCTV QuatroStick 521e. The namess and USB-IDs are different, but the hardware is the same. Should there be in this case one card entry in cx231xx driver or two? Two would have the advantage that the correct name of the device could be

Re: Different Devices identical hardware

2014-07-21 Thread Mauro Carvalho Chehab
Em Tue, 22 Jul 2014 07:33:27 +0200 Matthias Schwarzott z...@gentoo.org escreveu: Hi, I want to add support for Hauppauge WinTV 930C-HD and PCTV QuatroStick 521e. The namess and USB-IDs are different, but the hardware is the same. Should there be in this case one card entry in cx231xx

[PATCH] msi2500: correct style issues

2014-07-21 Thread Antti Palosaari
Correct some style issues, mostly reported by checkpatch.pl. Signed-off-by: Antti Palosaari cr...@iki.fi --- drivers/media/usb/msi2500/msi2500.c | 70 +++-- 1 file changed, 51 insertions(+), 19 deletions(-) diff --git a/drivers/media/usb/msi2500/msi2500.c