Re: [PATCH v7 1/3] media: adv748x: Add adv7481, adv7482 bindings

2017-07-09 Thread Rob Herring
On Thu, Jul 06, 2017 at 12:01:15PM +0100, Kieran Bingham wrote: > From: Kieran Bingham > > Create device tree bindings documentation for the ADV748x. > The ADV748x supports both the ADV7481 and ADV7482 chips which > provide analogue decoding and HDMI

Re: [PATCH 2/2] dt-bindings: media: Add Amlogic Meson AO-CEC bindings

2017-07-09 Thread Rob Herring
On Thu, Jul 06, 2017 at 12:27:50PM +0200, Neil Armstrong wrote: > The Amlogic SoCs embeds a standalone CEC Controller, this patch adds this > device bindings. > > Signed-off-by: Neil Armstrong > --- > .../devicetree/bindings/media/meson-ao-cec.txt | 28 >

cron job: media_tree daily build: ERRORS

2017-07-09 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 Jul 10 05:00:20 CEST 2017 media-tree git hash:2748e76ddb2967c4030171342ebdd3faa6a5e8e8 media_build

[PATCH] ngene: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] mantis: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] dm1105: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] ddbridge: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] cx24123: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] zd1301_demod: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] dib8000: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] s5h1420: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[no subject]

2017-07-09 Thread Corporate Lenders
Schönen Tag, Ich bin Thomas Walter, der Finanzagent dieser Firma, bekannt als Corporate Lenders. Wir leihen Geld für Einzelpersonen und Unternehmen, die finanzielle Hilfe benötigen. Hast du einen schlechten Kredit oder du brauchst Geld, um deine Rechnungen zu bezahlen? Wir verwenden dieses

[PATCH] dib7000p: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] marvell-ccic: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] saa7146: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] dib9000: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] usbvision: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH] dvb-ttusb-budget: constify i2c_algorithm structure

2017-07-09 Thread Gustavo A. R. Silva
Check for i2c_algorithm structures that are only stored in the algo field of an i2c_adapter structure. This field is declared const, so i2c_algorithm structures that have this property can be declared as const also. This issue was identified using Coccinelle and the following semantic patch: @r

[PATCH]

2017-07-09 Thread armetallica
From 043428d63637a6dd8e52449b73dbb8341885d7e4 Mon Sep 17 00:00:00 2001 From: Armin Schoenlieb Date: Mon, 10 Jul 2017 01:12:52 +0200 Subject: [PATCH] Staging: media: atomisp2: fixed trailing whitespace error in atomisp_v4l2.c This is a patch to the atomisp_v4l2.c file that

Re: [PATCH] media: venus: hfi: fix error handling in hfi_sys_init_done()

2017-07-09 Thread Rob Clark
On Sun, Jul 9, 2017 at 3:49 PM, Stanimir Varbanov wrote: > Hi Rob, > > On 07/09/2017 04:19 PM, Rob Clark wrote: >> Not entirely sure what triggers it, but with venus build as kernel >> module and in initrd, we hit this crash: > > Is it happens occasionally or

Re: [PATCH v2] staging: atomisp: use kstrdup to replace kmalloc and memcpy

2017-07-09 Thread Sakari Ailus
On Sun, Jul 09, 2017 at 05:56:15PM +0530, hari prasath wrote: > On 8 July 2017 at 16:31, Sakari Ailus wrote: > > Hi Hari, > > > > On Fri, Jul 07, 2017 at 08:15:21PM +0530, Hari Prasath wrote: > >> kstrdup kernel primitive can be used to replace kmalloc followed by > >> string

Re: [PATCH] media: venus: hfi: fix error handling in hfi_sys_init_done()

2017-07-09 Thread Stanimir Varbanov
Hi Rob, On 07/09/2017 04:19 PM, Rob Clark wrote: > Not entirely sure what triggers it, but with venus build as kernel > module and in initrd, we hit this crash: Is it happens occasionally or everytime in the initrd? And also with your patch it will bail out on venus_probe, does it crash again on

[PATCH 10/14] [media] ddbridge: remove unreachable code

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller >From smatch: drivers/media/pci/ddbridge/ddbridge-core.c:3490 snr_store() info: ignoring unreachable code. In fact, the function immediately returns zero, so remove it and update ddb_attrs_snr[] to not reference it anymore. Cc: Ralph Metzler

[PATCH 05/14] [media] ddbridge: split off IRQ handling

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller This not only helps keep the ddbridge-core tidy, but also gets rid of defined-but-unused-function warnings which might be triggered depending of CONFIG_PCI_MSI, without having to clutter the code with #ifdef'ery. Signed-off-by: Daniel Scheller

[PATCH 09/14] [media] ddbridge: fix possible buffer overflow in ddb_ports_init()

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Report from smatch: drivers/media/pci/ddbridge/ddbridge-core.c:2659 ddb_ports_init() error: buffer overflow 'dev->port' 32 <= u32max Fix by making sure "p" is greater than zero before checking for "dev->port[].type == DDB_CI_EXTERNAL_XO2". Cc:

[PATCH 3/4] [media] ddbridge: fix buffer overflow in max_set_input_unlocked()

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Picked up code parts introduced one smatch error: drivers/media/pci/ddbridge/ddbridge-maxs8.c:163 max_set_input_unlocked() error: buffer overflow 'dev->link[port->lnr].lnb.voltage' 4 <= 255 Fix this by clamping the .lnb.voltage array access to 0-3

[PATCH 1/4] [media] dvb-frontends: MaxLinear MxL5xx DVB-S/S2 tuner-demodulator driver

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller This adds the frontend driver for the MaxLinear MxL5xx family of tuner- demodulators, as used on Digital Devices MaxS4/8 four/eight-tuner cards. The driver was picked from the dddvb vendor driver package and - judging solely from the diff - has

[PATCH 12/14] [media] ddbridge: fix dereference before check

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Both ts_release() and ts_open() can use "output" before check (smatch): drivers/media/pci/ddbridge/ddbridge-core.c:816 ts_release() warn: variable dereferenced before check 'output' (see line 809) drivers/media/pci/ddbridge/ddbridge-core.c:836

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

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Target: 4.14 seems unrealistic, so: 4.14+ 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 frontend) as

[PATCH 4/4] [media] MAINTAINERS: add entry for mxl5xx

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6abb534c69c7..685886472d8a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8236,6 +8236,14 @@

[PATCH 07/14] [media] ddbridge: check pointers before dereferencing

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Fixes two warnings reported by smatch: drivers/media/pci/ddbridge/ddbridge-core.c:240 ddb_redirect() warn: variable dereferenced before check 'idev' (see line 238) drivers/media/pci/ddbridge/ddbridge-core.c:240 ddb_redirect() warn: variable

[PATCH 06/14] [media] ddbridge: split off hardware definitions and mappings

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Further cleanup of ddbridge-core and ddbridge-main, and moves all such hw definitions into one single place, making things easier to maintain. Signed-off-by: Daniel Scheller --- drivers/media/pci/ddbridge/Makefile| 4 +-

[PATCH 2/4] [media] ddbridge: support MaxLinear MXL5xx based cards (MaxS4/8)

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller This enables MaxS4/S8 and Octopus Max card support in ddbridge by adding glue code into ddbridge-core, having another PCI ID, and have the LNB IC control code (and all other MaxS4/8 related code) in ddbridge-maxs8.c (rather than another ~400 LoC in

[PATCH 13/14] [media] ddbridge: Kconfig option to control the MSI modparam default

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller It is known that MSI interrupts - while working quite well so far - can still cause issues on some hardware platforms (causing I2C timeouts due to unhandled interrupts). The msi variable/option is set to 1 by default. So, add a Kconfig option prefixed

[PATCH 08/14] [media] ddbridge: only register frontends in fe2 if fe is not NULL

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Smatch reported: drivers/media/pci/ddbridge/ddbridge-core.c:1602 dvb_input_attach() error: we previously assumed 'dvb->fe' could be null (see line 1595) dvb->fe2 will ever only be populated when dvb->fe is set. So only handle registration of

[PATCH 11/14] [media] ddbridge: fix impossible condition warning

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Smatch and gcc complained: drivers/media/pci/ddbridge/ddbridge-core.c:3491 bpsnr_show() warn: impossible condition '(snr[0] == 255) => ((-128)-127 == 255)' drivers/media/pci/ddbridge/ddbridge-core.c: In function ‘bpsnr_show’:

[PATCH 14/14] [media] MAINTAINERS: add entry for ddbridge

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Signed-off-by: Daniel Scheller --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7b85e578d238..6abb534c69c7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8262,6 +8262,14 @@

[PATCH 04/14] [media] ddbridge: split I/O related functions off from ddbridge.h

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller While it seems valid that headers can carry simple oneline static inline annotated functions, move them into their own header file to have the overall code more readable. Also, keep them as header (and don't put in a separate object) and static inline to

[PATCH 02/14] [media] ddbridge: split code into multiple files

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller As of 0.9.9b, the ddbridge code has been split from one single file (ddbridge-core.c) into multiple files, with the purpose of taking care of different topics, and to be able to reuse code in different kernel modules (ddbridge.ko and octonet.ko). This

[PATCH 01/14] [media] ddbridge: move/reorder functions

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller The functions in ddbridge-core.c have been moved to different positions in newer versions of the dddvb vendor driver package (most notably in version 0.9.9b). Perform the same code move to keep the diff of the upcoming code bump simpler. Signed-off-by:

[PATCH 00/14] ddbridge: bump to ddbridge-0.9.29

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Preferrably for Linux 4.14 (to get things done). Hard-depends on the STV0910/STV6111 driver patchset as the diff and the updated code depends on the driver and the changes involved with the glue code of the STV/DDCineS2V7 series [1]. Mauro/Media

Re: [PATCH] checkpatch: fixed alignment and comment style

2017-07-09 Thread Joe Perches
On Sun, 2017-07-09 at 19:39 +0200, Philipp Guendisch wrote: > This patch fixed alignment, comment style and one appearance of > misordered constant in an if comparison. > Semantic should not be affected by this patch. Your email subject is wrong. This is not a checkpatch patch. Your subject

[PATCH] checkpatch: fixed alignment and comment style

2017-07-09 Thread Philipp Guendisch
This patch fixed alignment, comment style and one appearance of misordered constant in an if comparison. Semantic should not be affected by this patch. There are still a few warnings and even errors left which require a semantic change of the code to fix. e.g. use of in_atomic in drivers There

[PATCH] [media] dvb-frontends/stv0367: DDB frontend status inquiry fixup

2017-07-09 Thread Daniel Scheller
From: Daniel Scheller Return 0 instead of -EINVAL in get_frontend if no demod mode is active. This fixes ie. dvb-fe-tool getting confused and assuming a DVBv3 FE on idle frontends when the FE has been put to sleep using sleep(). Also, in read_status(), don't immediately

Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

2017-07-09 Thread Sylwester Nawrocki
Hi Hugues, On 07/06/2017 09:51 AM, Hugues FRUCHET wrote: Hi Sylwester, Do you have the possibility to check for non-regression of this patchset on 9650/52 camera ? I will try to test your patch set once I find the camera module for my Micro2440SDK board. I've spent already a day on setting

Dein Geld

2017-07-09 Thread Qatar Foundation
Liebe Begünstigte, Du hast gewonnen (€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: qcharit...@gmail.com Präsident der Qatar

[PATCH] media: venus: hfi: fix error handling in hfi_sys_init_done()

2017-07-09 Thread Rob Clark
Not entirely sure what triggers it, but with venus build as kernel module and in initrd, we hit this crash: Unable to handle kernel paging request at virtual address 80003c039000 pgd = 0a14f000 [80003c039000] *pgd=bd9f7003, *pud=bd9f6003,

Re: [PATCH v2] staging: atomisp: use kstrdup to replace kmalloc and memcpy

2017-07-09 Thread hari prasath
On 8 July 2017 at 16:31, Sakari Ailus wrote: > Hi Hari, > > On Fri, Jul 07, 2017 at 08:15:21PM +0530, Hari Prasath wrote: >> kstrdup kernel primitive can be used to replace kmalloc followed by >> string copy. This was reported by coccinelle tool >> >> Signed-off-by: Hari

Re: [PATCH RFC 1/2] app: kaffeine: Fix missing PCR on live streams.

2017-07-09 Thread Malcolm Priestley
On 09/07/17 12:14, Mauro Carvalho Chehab wrote: Hi Malcolm, Em Sun, 9 Jul 2017 10:43:50 +0100 Malcolm Priestley escreveu: The ISO/IEC standard 13818-1 or ITU-T Rec. H.222.0 standard allow transport vendors to place PCR (Program Clock Reference) on a different PID. If

Re: [PATCH RFC 1/2] app: kaffeine: Fix missing PCR on live streams.

2017-07-09 Thread Mauro Carvalho Chehab
Hi Malcolm, Em Sun, 9 Jul 2017 10:43:50 +0100 Malcolm Priestley escreveu: > The ISO/IEC standard 13818-1 or ITU-T Rec. H.222.0 standard allow transport > vendors to place PCR (Program Clock Reference) on a different PID. > > If the PCR is unset the value is 0x1fff, most

[PATCH RFC 1/2] app: kaffeine: Fix missing PCR on live streams.

2017-07-09 Thread Malcolm Priestley
The ISO/IEC standard 13818-1 or ITU-T Rec. H.222.0 standard allow transport vendors to place PCR (Program Clock Reference) on a different PID. If the PCR is unset the value is 0x1fff, most vendors appear to set it the same as video pid in which case it need not be set. The PCR PID is at an

[PATCH RFC 2/2] app: kaffeine: Fix missing PCR on stream recordings.

2017-07-09 Thread Malcolm Priestley
The ISO/IEC standard 13818-1 or ITU-T Rec. H.222.0 standard allow transport vendors to place PCR (Program Clock Reference) on a different PID. This patch adds it recording to file. Signed-off-by: Malcolm Priestley --- src/dvb/dvbrecording.cpp | 8 1 file changed, 8