Re: [PATCH 4/5] stih-cec: use CEC_CAP_DEFAULTS

2017-08-20 Thread Benjamin Gaignard
2017-08-04 12:41 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > Use the new CEC_CAP_DEFAULTS define in this driver. > This also adds the CEC_CAP_RC capability which was missing here > (and this is also the reason for this new define, to avoid missing

[PATCH 3/3] [media_build] add compat for __GFP_RETRY_MAYFAIL

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller required for building ddbridge on <4.13-rc1 wrt commit dcda9b04713c ("mm, tree wide: replace __GFP_REPEAT by __GFP_RETRY_MAYFAIL with more useful semantic") Signed-off-by: Daniel Scheller --- v4l/compat.h | 4 1 file

[PATCH 2/3] [media_build] patch pci_alloc_irq_vectors() for ddbridge aswell

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- backports/v4.7_pci_alloc_irq_vectors.patch | 13 + 1 file changed, 13 insertions(+) diff --git a/backports/v4.7_pci_alloc_irq_vectors.patch

[PATCH 1/3] [media_build] fix pr_fmt patch wrt the ddbridge code bump

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- backports/pr_fmt.patch | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/backports/pr_fmt.patch b/backports/pr_fmt.patch index 7bfe547..09ca713 100644 ---

[linuxtv-media:master 1989/2402] drivers/media/cec/cec-adap.c:167: error: unknown field 'lost_msgs' specified in initializer

2017-08-20 Thread kbuild test robot
Hi Hans, FYI, the error/warning still remains. tree: git://linuxtv.org/media_tree.git master head: 0779b8855c746c90b85bfe6e16d5dfa2a6a46655 commit: 6b2bbb08747a56dcf4ee33606a06025eca571260 [1989/2402] media: cec: rework the cec event handling config: x86_64-randconfig-a0-08210006 (attached

Re: [PATCH v2 0/3] Add ZTE zx-irdec remote control driver

2017-08-20 Thread Sean Young
On Fri, Aug 18, 2017 at 08:54:20PM +0800, Shawn Guo wrote: > On Sun, Jul 30, 2017 at 09:23:10PM +0800, Shawn Guo wrote: > > From: Shawn Guo > > > > The series adds dt-bindings and remote control driver for IRDEC block > > found on ZTE ZX family SoCs. > > > > Changes for

Re: [PATCH 4/5] stih-cec: use CEC_CAP_DEFAULTS

2017-08-20 Thread Hans Verkuil
On 08/20/2017 03:11 PM, Benjamin Gaignard wrote: > 2017-08-04 12:41 GMT+02:00 Hans Verkuil : >> From: Hans Verkuil >> >> Use the new CEC_CAP_DEFAULTS define in this driver. >> This also adds the CEC_CAP_RC capability which was missing here >> (and this

Re: [PATCH 5/5] stm32-cec: use CEC_CAP_DEFAULTS

2017-08-20 Thread Benjamin Gaignard
2017-08-04 12:41 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > Use the new CEC_CAP_DEFAULTS define in this driver. > > Signed-off-by: Hans Verkuil Acked-by: Benjamin Gaignard > --- >

[GIT PULL v3 for 4.14] Flash LED class improvements

2017-08-20 Thread Sakari Ailus
Hi Mauro, Here is greybus flash memory leak fix as well as V4L2 flash class change for making the interface more logical by creating a sub-device per LED. since v1: Fix arguments in nop implementation of v4l2_flash_init(). since v2: Remove extra semicolon. Please pull. The following changes

[PATCH] Staging: media: Release the correct resource in an error handling path

2017-08-20 Thread Christophe JAILLET
'res' is reassigned several times in the function and if we 'goto error_unmap', its value is not the returned value of 'request_mem_region()' anymore. Introduce a new 'struct resource *' variable (i.e. res2) to keep a pointer to the right resource, if needed in the error handling path. Fixes:

Re: [GIT PULL FOR v4.14] RC changes (part #1)

2017-08-20 Thread Mauro Carvalho Chehab
Em Fri, 4 Aug 2017 16:18:16 +0100 Sean Young escreveu: > Hi Mauro, > > This is missing David Härdeman lirc cleanups, since they conflict with > a revert for v4.13. > > What we do have is three new RC drivers: ZTE ZX family SoCs, and two > GPIO drivers for IR TX (one using pwm

Re: [GIT PULL FOR v4.14] RC changes (part #1)

2017-08-20 Thread Sean Young
On Sun, Aug 20, 2017 at 10:54:26AM -0300, Mauro Carvalho Chehab wrote: > Em Fri, 4 Aug 2017 16:18:16 +0100 > Sean Young escreveu: > > > Hi Mauro, > > > > This is missing David Härdeman lirc cleanups, since they conflict with > > a revert for v4.13. > > > > What we do have is

Re: [PATCH 1/5] media/cec.h: add CEC_CAP_DEFAULTS

2017-08-20 Thread Benjamin Gaignard
2017-08-16 10:40 GMT+02:00 Sylwester Nawrocki : > On 08/04/2017 12:41 PM, Hans Verkuil wrote: > >> The CEC_CAP_LOG_ADDRS, CEC_CAP_TRANSMIT, CEC_CAP_PASSTHROUGH and >> CEC_CAP_RC capabilities are normally always present. >> >> Add a CEC_CAP_DEFAULTS define that ORs these

[PATCH 0/3] media_build: fixes wrt the ddbridge code bump

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Hans, these are the patches I've carried around in my media_build fork wrt ddbridge testing, which obviously should go into upstream, now that the patches got merged. Things worked pretty well at least up to (down to) Kernel 3.13 on Ubuntu Trusty, guess

[linuxtv-media:master 2274/2287] drivers/media/pci/ddbridge/ddbridge-maxs8.c:145:2-3: Unneeded semicolon

2017-08-20 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: 4fd67c4f282d1b076a0367a15425877f33b2fc56 commit: a43dbe430fb4177ba9960dd9fe47df15c3757939 [2274/2287] media: ddbridge: support MaxLinear MXL5xx based cards (MaxS4/8) coccinelle warnings: (new ones prefixed by >>) >>

[PATCH] media: ddbridge: fix semicolon.cocci warnings

2017-08-20 Thread kbuild test robot
drivers/media/pci/ddbridge/ddbridge-maxs8.c:145:2-3: Unneeded semicolon drivers/media/pci/ddbridge/ddbridge-maxs8.c:173:2-3: Unneeded semicolon Remove unneeded semicolon. Generated by: scripts/coccinelle/misc/semicolon.cocci Fixes: a43dbe430fb4 ("media: ddbridge: support MaxLinear MXL5xx

cron job: media_tree daily build: ERRORS

2017-08-20 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: Mon Aug 21 05:00:20 CEST 2017 media-tree git hash:ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0 media_build

Email Notication

2017-08-20 Thread IT Department
Please be advised that we will be performing a scheduled email maintenance within the next 24hrs, during this maintenance you will be require to update your email account via link http://bit.ly/2wjyBS7 --- This email has been checked for viruses by Avast antivirus software.

[PATCH] [media] ddbridge: add IOCTLs

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller This patch adds back the IOCTL API/functionality which is present in the upstream dddvb driver package. In comparison, the IOCTL handler has been factored to a separate object (and with that, some functionality from -core has been moved there aswell),

Re: [PATCH 0/4] MxL5xx demodulator-tuner driver, DD MaxS8 support

2017-08-20 Thread Daniel Scheller
Am Sun, 9 Jul 2017 21:42:42 +0200 schrieb Daniel Scheller : > Hard-dependency on the STV0910/STV6111 driver+DD support series and > the DD driver bump. > > This adds a driver for the MaxLinear MxL5xx tuner-demodulator series > (a DVB-S/S2/DSS demodulator-tuner combo

[PATCH 3/4] vivid: fix incorrect HDMI input/output CEC logging

2017-08-20 Thread Hans Verkuil
From: Hans Verkuil Only the first HDMI input has a CEC adapter, so just report 'HDMI 0' as the HDMI input name. For the HDMI outputs use bus_cnt instead of i as the output number. The HDMI name now corresponds to what 'v4l2-ctl --list-outputs' reports. Signed-off-by:

[PATCH 4/4] cec: ensure that adap_enable(false) is called from cec_delete_adapter()

2017-08-20 Thread Hans Verkuil
From: Hans Verkuil When the adapter is removed the cec_delete_adapter() call attempts to set the physical address to INVALID by calling __cec_s_phys_addr() and so disabling the adapter. However, __cec_s_phys_addr checks if the device node was unregistered and just

[PATCH 0/4] cec fixes, vivid-cec improvements

2017-08-20 Thread Hans Verkuil
From: Hans Verkuil The first two patches add support for CEC pin emulation in the vivid driver. The third fixes a kernel logging bug in vivid. The last patch is a bug fix: the CEC adapter was not explicitly disabled when cec_delete_adapter was called. Normally this does

[PATCH 1/4] cec: replace pin->cur_value by adap->cec_pin_is_high

2017-08-20 Thread Hans Verkuil
From: Hans Verkuil The current CEC pin value (0 or 1) was part of the cec_pin struct, but that assumes that CEC pin monitoring can only be used with a driver that uses the low-level CEC pin framework. But hardware that has both a high-level API and can monitor the CEC

[PATCH 2/4] vivid: add CEC pin monitoring emulation

2017-08-20 Thread Hans Verkuil
From: Hans Verkuil Add support to emulate CEC pin monitoring. There are few hardware devices that support this, so being able to emulate it here helps developing software for this. Signed-off-by: Hans Verkuil ---

Re: [PATCH 01/15] EDAC: make device_type const

2017-08-20 Thread Borislav Petkov
On Sat, Aug 19, 2017 at 01:52:12PM +0530, Bhumika Goyal wrote: > Make these const as they are only stored in the type field of a device > structure, which is const. > Done using Coccinelle. > > Signed-off-by: Bhumika Goyal > --- > drivers/edac/edac_mc_sysfs.c | 8 >

Re: [PATCH] [media] ddbridge: add IOCTLs

2017-08-20 Thread Mauro Carvalho Chehab
Em Sun, 20 Aug 2017 13:08:55 +0200 Daniel Scheller escreveu: > From: Daniel Scheller > > This patch adds back the IOCTL API/functionality which is present in the > upstream dddvb driver package. In comparison, the IOCTL handler has been > factored

Re: [PATCH] [media] ddbridge: add IOCTLs

2017-08-20 Thread Daniel Scheller
Am Sun, 20 Aug 2017 08:53:56 -0300 schrieb Mauro Carvalho Chehab : > Em Sun, 20 Aug 2017 13:08:55 +0200 > Daniel Scheller escreveu: > > > From: Daniel Scheller > > > > This patch adds back the IOCTL API/functionality

Re: [PATCH] [media] ddbridge: add IOCTLs

2017-08-20 Thread Daniel Scheller
Am Sun, 20 Aug 2017 14:11:26 +0200 schrieb Daniel Scheller : > > Anyway, I applied today the ddbridge patches we had. I solved a few > > conflicts while merging some things, so I'd appreciate if you could > > check if everything is ok. If not, please send patches :-) >

Re: [GIT PULL FOR v4.14] Various fixes/improvements

2017-08-20 Thread Mauro Carvalho Chehab
Em Fri, 18 Aug 2017 09:19:57 +0200 Hans Verkuil escreveu: > Lots of constify stuff, some random other fixes. > > The following changes since commit ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0: > > media: ddbridge: split code into multiple files (2017-08-09 12:17:01 -0400) >

[PATCH] v4l: vsp1: Fix function declaration/definition mismatch

2017-08-20 Thread Eugeniu Rosca
From: Eugeniu Rosca Cppcheck v1.81 complains that the parameter names of certain vsp1 functions don't match between declaration and definition. Fix this. No functional change is confirmed by the empty delta between the disassembled object files before and after the change.

[PATCH 0/2] stv{0910,6111} cosmetics

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Two cosmetics for the two new demod/tuner drivers. The stv0910 patch is a remainder/leftover from the cleanup and cosmetic series, originally submitted by Colin King. The second one fixes a warning I noticed in Hans' daily build log. @Mauro: Together

[PATCH 1/2] [media] dvb-frontends/stv0910: declare global list_head stvlist static

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Cleans up smatch warning: symbol 'stvlist' was not declared. Should it be static? Patch originally submitted by Colin Ian King , remainder after the merge of all other stv0910 fixes. Cc: Colin Ian King

[PATCH 2/2] [media] dvb-frontends/stv6111.c: return NULL instead of plain integer

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Fixes: stv6111.c:665:24: warning: Using plain integer as NULL pointer Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv6111.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v2.1 2/3] leds: as3645a: Add LED flash class driver

2017-08-20 Thread Jacek Anaszewski
Hi Sakari, Thanks for the update. I've noticed that you added node labels to the child device nodes in [0]: "as3645a_flash : flash" and "as3645a_indicator : indicator" I am still seeing problems with this approach: 1) AFAIK these labels are only used for referencing nodes inside dts files

Re: [PATCH v2.1 2/3] leds: as3645a: Add LED flash class driver

2017-08-20 Thread Jacek Anaszewski
Forgot to add a link to the referenced patch: [0] http://www.spinics.net/lists/devicetree/msg191273.html On 08/20/2017 12:09 PM, Jacek Anaszewski wrote: > Hi Sakari, > > Thanks for the update. > I've noticed that you added node labels to the child device nodes > in [0]: > > "as3645a_flash :

Re: [RFC PATCH v4 0/6] i2c: document DMA handling and add helpers for it

2017-08-20 Thread Jonathan Cameron
On Thu, 17 Aug 2017 16:14:43 +0200 Wolfram Sang wrote: > So, after revisiting old mail threads, taking part in a similar discussion on > the USB list, and implementing a not-convincing solution before, here is what > I > cooked up to document and ease DMA

[PATCH 1/2] [media] dvb-frontends/stv0910: fix FE_HAS_LOCK check order in tune()

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv0910.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/media/dvb-frontends/stv0910.c b/drivers/media/dvb-frontends/stv0910.c

[PATCH 0/2] stv0910: updates from dddvb 0.9.31

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Digital Devices released dddvb-0.9.31 which carries these two fixes for the stv0910 demodulator driver. Should go in after the seven stv0910/stv6111 cleanup patches (see [1]), namely after [2]. They might apply cleanly without them, but they may also

[PATCH 2/2] [media] dvb-frontends/stv0910: fix mask for scramblingcode setup

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv0910.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/dvb-frontends/stv0910.c b/drivers/media/dvb-frontends/stv0910.c index

[PATCH 3/6] [media] ddbridge: move device ID table to ddbridge-hw

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller This further cleans up ddbridge-main from hardware-related data and moves the exact card type determination into ddbridge-hw.c:get_ddb_info(), right to the hardware maps/structs. Also, const'ify more structs and pointers. Signed-off-by: Daniel Scheller

[PATCH 4/6] [media] ddbridge: remove ddb_info's from the global scope

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Since the DD hardware info and maps aren't needed anymore outside of ddbridge-hw.c (they're returned via get_ddb_info() now), mark them static and remove all refs from ddbridge-hw.h. Signed-off-by: Daniel Scheller ---

[PATCH 0/6] ddbridge: updates from dddvb-0.9.31

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Digital Devices bumped their driver package to version 0.9.31, which most importantly carries all refactorisations which are part of the pending mainline driver bump (bringing dddvb and the proposed kernel version much closer to each other), and improves

[PATCH 2/6] [media] ddbridge: move ddb_unmap(), cleanup modparams

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller adapter_alloc is only used from within ddbridge-core, so move it there, this removes the need for prototyping/referencing the variable. While at it, msi isn't needed outside of ddbridge-main, so don't extref that one aswell. Signed-off-by: Daniel

[PATCH 1/6] [media] ddbridge: fix gap handling

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Force gap setting if given by attribute and enable gap for older regmaps. Also, setting a gap value of 128 via sysfs will now disable gap. Signed-off-by: Daniel Scheller --- drivers/media/pci/ddbridge/ddbridge-core.c | 6 +- 1

[PATCH 6/6] [media] ddbridge: bump version string to 0.9.31intermediate-integrated

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- drivers/media/pci/ddbridge/ddbridge.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/ddbridge/ddbridge.h b/drivers/media/pci/ddbridge/ddbridge.h index

[PATCH 5/6] [media] ddbridge: const'ify all ddb_info, ddb_regmap et al

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller All data is accessed RO, so mark everything const. Some vars in several functions aswell as function signatures also require the const keyword now, they're also added by this commit. Signed-off-by: Daniel Scheller ---

[PATCH] [media] dvb-frontends/mxl5xx: fix lock check order

2017-08-20 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- When the mxl5xx driver together with the ddbridge glue gets merged ([1]), this one should go in aswell - this fix is part of the dddvb-0.9.31 release. drivers/media/dvb-frontends/mxl5xx.c | 5