Re: [PATCH 2/2] [media] s5p-g2d: Replace old driver with DRM version

I suspect that you'll be applying this one via DRM tree, so: Em Tue, 24 May 2016 15:28:13 +0200 Krzysztof Kozlowski escreveu: > Remove the old non-DRM driver because it is now entirely supported by > exynos_drm_g2d driver. > > Cc: Kyungmin Park

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

On Mon, Jul 11, 2016 at 10:56 AM, tiffany lin wrote: > Hi Hans, > > On Fri, 2016-07-08 at 12:23 +0200, Hans Verkuil wrote: >> On 05/30/2016 02:29 PM, Tiffany Lin wrote: >> > Add V4L2_PIX_FMT_MT21 documentation >> > >> > Signed-off-by: Tiffany Lin

Re: [PATCHv2 3/5] pulse8-cec: new driver for the Pulse-Eight USB-CEC Adapter

On 07/11/16 12:17, Lars Op den Kamp wrote: > Hi Hans, > > On 11-07-16 12:02, Hans Verkuil wrote: >> Hi Lars, >> >> On 07/11/2016 11:41 AM, Lars Op den Kamp wrote: >>> Hi Hans, >>> >>> just did a quick scan of this patch. >>> >>> The code should work on any firmware >= v2 revision 8, though older

Re: [PATCH v4] [media] pci: Add tw5864 driver - fixed few style nits, going to resubmit soon

Found and fixed few very minor coding style nits, will resubmit in few days, now still waiting for comments to v4. https://github.com/bluecherrydvr/linux/commits/tw5864 commit 31f7c98a144cb3fb8a94662f002d9b6142d1f390 Author: Andrey Utkin Date: Wed Jul 13

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

On Tue, 2016-07-12 at 16:16 +0800, Wu-Cheng Li (李務誠) wrote: > On Mon, Jul 11, 2016 at 10:56 AM, tiffany lin > wrote: > > Hi Hans, > > > > On Fri, 2016-07-08 at 12:23 +0200, Hans Verkuil wrote: > >> On 05/30/2016 02:29 PM, Tiffany Lin wrote: > >> > Add V4L2_PIX_FMT_MT21

Re: [PATCH] userspace API definitions for auto-focus coil

Em Sat, 18 Jun 2016 17:38:46 +0200 Pavel Machek escreveu: > Hi! > > > > Not V4L2_CID_USER_AD5820...? > > > > The rest of the controls have no USER as part of the macro name, so I > > wouldn't use it here either. > > Ok. > > > > Ok, separate header file for 2 lines seemed

cron job: media_tree daily build: ERRORS

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 Jul 13 04:00:22 CEST 2016 git branch: test git hash: 25f1ec2462f4afe544c9374ef4dbb4f5f0a3b701 gcc

[PATCH] media: s5p-mfc remove unnecessary error messages

Removing unnecessary error messages as appropriate error code is returned. Signed-off-by: Shuah Khan --- drivers/media/platform/s5p-mfc/s5p_mfc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

Hi Nicolas, On Tue, 2016-07-12 at 15:14 -0400, Nicolas Dufresne wrote: > Le mardi 12 juillet 2016 à 15:08 -0400, Nicolas Dufresne a écrit : > > Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : > > > Decoder hardware produces MT21 (compressed). Image processor can > > > convert

Re: [PATCH -next] [media] mtk-vcodec: remove redundant dev_err call in mtk_vcodec_probe()

Reviewed-by: Tiffany Lin On Tue, 2016-07-12 at 11:02 +, weiyj...@163.com wrote: > From: Wei Yongjun > > There is a error message within devm_ioremap_resource > already, so remove the dev_err call to avoid redundant > error message. >

[PATCH -next] [media] VPU: mediatek: fix return value check in mtk_vpu_probe()

From: Wei Yongjun In case of error, the function devm_clk_get() returns ERR_PTR() and not returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun ---

[PATCH -next] [media] VPU: mediatek: remove redundant dev_err call in mtk_vpu_probe()

From: Wei Yongjun There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/media/platform/mtk-vpu/mtk_vpu.c | 8 ++-- 1

Re: [PATCH v5 00/44] dma-mapping: Use unsigned long for dma_attrs

On 07/12/2016 02:16 PM, Daniel Vetter wrote: > On Thu, Jun 30, 2016 at 10:23:39AM +0200, Krzysztof Kozlowski wrote: >> Hi, >> >> >> This is fifth approach for replacing struct dma_attrs with unsigned >> long. >> >> The main patch (1/44) doing the change is split into many subpatches >> for easier

Re: [PATCH -next] [media] rcar_jpu: Add missing clk_disable_unprepare() on error in jpu_open()

On Tue, Jul 12, 2016 at 2:21 PM, wrote: > From: Wei Yongjun > > Add the missing clk_disable_unprepare() before return from > jpu_open() in the software reset error handling case. > > Signed-off-by: Wei Yongjun

Re: [PATCH 08/20] [media] doc-rst: document ioctl LIRC_GET_REC_MODE

On Tue, Jul 12, 2016 at 09:42:02AM -0300, Mauro Carvalho Chehab wrote: > Move the documentation of this ioctl from lirc_ioctl to its > own file, and add a short description about the pulse mode > used by IR RX. > > Signed-off-by: Mauro Carvalho Chehab > --- >

Re: [PATCH 03/20] [media] lirc.h: remove several unused ioctls

On Tue, Jul 12, 2016 at 09:41:57AM -0300, Mauro Carvalho Chehab wrote: > While reviewing the documentation gaps on LIRC, it was > noticed that several ioctls aren't used by any LIRC drivers > (nor at staging or mainstream). > > It doesn't make sense to document them, as they're not used >

Re: [PATCH v5 00/44] dma-mapping: Use unsigned long for dma_attrs

On Thu, Jun 30, 2016 at 10:23:39AM +0200, Krzysztof Kozlowski wrote: > Hi, > > > This is fifth approach for replacing struct dma_attrs with unsigned > long. > > The main patch (1/44) doing the change is split into many subpatches > for easier review (2-42). They should be squashed together

Re: [PATCH 08/20] [media] doc-rst: document ioctl LIRC_GET_REC_MODE

Em Tue, 12 Jul 2016 14:01:59 +0100 Sean Young escreveu: > On Tue, Jul 12, 2016 at 09:42:02AM -0300, Mauro Carvalho Chehab wrote: > > Move the documentation of this ioctl from lirc_ioctl to its > > own file, and add a short description about the pulse mode > > used by IR RX. > > >

[PATCH -next] [media] rcar_jpu: Add missing clk_disable_unprepare() on error in jpu_open()

From: Wei Yongjun Add the missing clk_disable_unprepare() before return from jpu_open() in the software reset error handling case. Signed-off-by: Wei Yongjun --- drivers/media/platform/rcar_jpu.c | 4 +++- 1 file changed, 3

Re: [PATCH 03/20] [media] lirc.h: remove several unused ioctls

Em Tue, 12 Jul 2016 14:14:06 +0100 Sean Young escreveu: > On Tue, Jul 12, 2016 at 09:41:57AM -0300, Mauro Carvalho Chehab wrote: > > While reviewing the documentation gaps on LIRC, it was > > noticed that several ioctls aren't used by any LIRC drivers > > (nor at staging or

[PATCH -next] [media] s5p-mfc: remove redundant return value check of platform_get_resource()

From: Wei Yongjun Remove unneeded error handling on the result of a call to platform_get_resource() when the value is passed to devm_ioremap_resource(). Signed-off-by: Wei Yongjun --- drivers/media/platform/s5p-mfc/s5p_mfc.c | 4

[PATCH 08/20] [media] doc-rst: document ioctl LIRC_GET_REC_MODE

Move the documentation of this ioctl from lirc_ioctl to its own file, and add a short description about the pulse mode used by IR RX. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-get-rec-mode.rst | 59 ++

[PATCH 04/20] [media] doc-rst: remove not used ioctls from documentation

As we removed those ioctls from the header file, do the same at the documentation side. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-get-features.rst | 9 +-- Documentation/media/uapi/rc/lirc_ioctl.rst| 74 ++-

[PATCH 11/20] [media] doc-rst: document LIRC_GET_*_TIMEOUT ioctls

Improve the documentation for those ioctls, adding them to a separate file, in order to look like the rest of the book, and to later allow to generate a man page for those ioctls. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-get-timeout.rst

[PATCH 06/20] [media] doc-rst: document ioctl LIRC_GET_SEND_MODE

Move the documentation of this ioctl from lirc_ioctl to its own file, and add a short description about the pulse mode used by IR TX. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-get-send-mode.rst | 48 ++

[PATCH 16/20] [media] doc-rst: document LIRC_SET_REC_TIMEOUT_REPORTS

Add a separate page for this ioctl and improve its documentation. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/rc/lirc-set-rec-timeout-reports.rst | 49 ++ .../media/uapi/rc/lirc_device_interface.rst| 1 +

[PATCH 14/20] [media] doc-rst: document LIRC_SET_TRANSMITTER_MASK

Add proper documentation for this ioctl, providing some additional information about its usage. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/rc/lirc-set-transmitter-mask.rst| 53 ++ .../media/uapi/rc/lirc_device_interface.rst|

[PATCH 09/20] [media] doc-rst: document LIRC_GET_REC_RESOLUTION

Improve the documentation for this ioctl, adding it to a separate file, in order to look like the rest of the book, and to later allow to generate a man page for this ioctl. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/rc/lirc-get-rec-resolution.rst |

[PATCH 10/20] [media] doc-rst: document LIRC_SET_SEND_DUTY_CYCLE

Add a separate page for this ioctl. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/rc/lirc-set-send-duty-cycle.rst | 49 ++ .../media/uapi/rc/lirc_device_interface.rst| 1 + Documentation/media/uapi/rc/lirc_ioctl.rst |

[PATCH 18/20] [media] doc-rst: document LIRC_SET_WIDEBAND_RECEIVER

Put documentation for this ioctl on a separate page and improve it. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/rc/lirc-set-wideband-receiver.rst | 56 ++ .../media/uapi/rc/lirc_device_interface.rst| 1 +

[PATCH 03/20] [media] lirc.h: remove several unused ioctls

While reviewing the documentation gaps on LIRC, it was noticed that several ioctls aren't used by any LIRC drivers (nor at staging or mainstream). It doesn't make sense to document them, as they're not used anywhere. So, let's remove those from the lirc header. Signed-off-by: Mauro Carvalho

[PATCH 20/20] [media] doc-rst: reorganize LIRC ReST files

Reorganize the LIRC rst files, using "-" instead of "_" on their names, and creating a separate chapter for syscalls. Signed-off-by: Mauro Carvalho Chehab --- .../uapi/rc/{lirc_dev_intro.rst => lirc-dev-intro.rst} | 0 Documentation/media/uapi/rc/lirc-dev.rst

[PATCH 13/20] [media] doc-rst: document LIRC set carrier ioctls

Put each ioctl on its own page and improve documentation, adding cross-references for LIRC_SET_REC_CARRIER_RANGE and LIRC_SET_REC_CARRIER, with can be used together to set a carrier frequency range. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 05/20] [media] doc-rst: Fix LIRC_GET_FEATURES references

The references pointed by LIRC_GET_FEATURES ioctl are broken. Fix them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/lirc.h.rst.exceptions | 4 + Documentation/media/uapi/rc/lirc-get-features.rst | 95 +-- 2 files changed,

[PATCH 19/20] [media] doc-rst: Document LIRC set mode ioctls

Add LIRC_SET_[REC|SEND]_MODE ioctls to the corresponding GET functions, and put all LIRC modes altogether. As now everything is already documented on its own ioctl pages, get rid of lirc_ioctl.rst. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 00/20] Improve LIRC documentation

The LIRC documentation doesn't follow the remaining of the media book. There's just a single page with all ioctls inside, and, IMHO, not very clear. Also, the LIRC_CAN flags were not described. This patchset address these. Also, it removes some LIRC ioctls that aren't used. Mauro Carvalho

[PATCH 12/20] [media] doc-rst: Document LIRC_GET_LENGTH ioctl

Put documentation for this ioctl on its own page. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-get-length.rst| 45 ++ .../media/uapi/rc/lirc_device_interface.rst| 1 +

[PATCH 17/20] [media] doc-rst: add documentation for LIRC_SET_MEASURE_CARRIER_MODE

Place documentation for this ioctl on its own page. Signed-off-by: Mauro Carvalho Chehab --- .../uapi/rc/lirc-set-measure-carrier-mode.rst | 48 ++ .../media/uapi/rc/lirc_device_interface.rst| 1 +

[PATCH 15/20] [media] doc-rst: document LIRC_SET_REC_TIMEOUT

Add a separate page for this ioctl and adds the cross-references. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/rc/lirc-set-rec-timeout.rst | 52 ++ .../media/uapi/rc/lirc_device_interface.rst| 1 +

Re: [PATCH 03/20] [media] lirc.h: remove several unused ioctls

On Tue, Jul 12, 2016 at 10:23:00AM -0300, Mauro Carvalho Chehab wrote: > Em Tue, 12 Jul 2016 14:14:06 +0100 > Sean Young escreveu: > > On Tue, Jul 12, 2016 at 09:41:57AM -0300, Mauro Carvalho Chehab wrote: -snip- > > > -#define LIRC_SET_REC_DUTY_CYCLE_IOW('i', 0x0016,

Re: [PATCH v4] [media] pci: Add tw5864 driver

On Mon, Jul 11, 2016 at 09:40:53AM -0700, Joe Perches wrote: > Each of these blocks will start with the dev_ prefix > and the subsequent lines will not have the same prefix Yes. I have checked how it looks before submitting, but I didn't see this as a problem. I don't mind changing that (anyway I

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

Hi Nicolas, On Tue, 2016-07-12 at 15:08 -0400, Nicolas Dufresne wrote: > Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : > > Decoder hardware produces MT21 (compressed). Image processor can > > convert it to a format that can be input of display driver. Tiffany. > > When do

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

On Wed, Jul 13, 2016 at 3:14 AM, Nicolas Dufresne wrote: > Le mardi 12 juillet 2016 à 15:08 -0400, Nicolas Dufresne a écrit : >> Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : >> > Decoder hardware produces MT21 (compressed). Image processor can >>

[PATCH 0/2] CEC improvements

Two CEC patches: one adds a sanity check, the other splits the timestamp into two: one for rx and one for tx. The work on the CEC compliance test made us realize that you need this information it order to diagnose problems like reply messages that take too long. A documentation patch will

[PATCH 1/2] cec: add sanity check for msg->len

Check (and warn) if the msg->len is too long or if it is 0. Should never happen, but just in case... Signed-off-by: Hans Verkuil --- drivers/staging/media/cec/cec-adap.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/media/cec/cec-adap.c

[PATCH 2/2] cec: split the timestamp into an rx and tx timestamp

When transmitting a message and waiting for a reply it would be good to know the time between when the message was transmitted and when the reply arrived. With only one timestamp field it was set to when the reply arrived and the original transmit time was overwritten. Just taking the timestamp

Re: [PATCH 03/20] [media] lirc.h: remove several unused ioctls

Em Tue, 12 Jul 2016 14:54:38 +0100 Sean Young escreveu: > On Tue, Jul 12, 2016 at 02:35:56PM +0100, Sean Young wrote: > > On Tue, Jul 12, 2016 at 10:23:00AM -0300, Mauro Carvalho Chehab wrote: > > > Em Tue, 12 Jul 2016 14:14:06 +0100 > > > Sean Young escreveu:

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

On 07/12/2016 08:06 AM, Joe Perches wrote: > On Mon, 2016-07-11 at 16:39 -0600, Shuah Khan wrote: >> Fix misspelled error message and existing checkpatch errors in the >> error message conditional. > [] >> diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c >>

vidioc-g-dv-timings.rst: document interlaced defines

Fix missing references. Signed-off-by: Hans Verkuil diff --git a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst index e19d64e..f7bf21f 100644 --- a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst +++

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

On Mon, 2016-07-11 at 16:39 -0600, Shuah Khan wrote: > Fix misspelled error message and existing checkpatch errors in the > error message conditional. [] > diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c > b/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c [] > @@ -775,11 +775,11 @@ static

[PATCH] pixfmt-006.rst: add missing V4L2_YCBCR_ENC_SMPTE240M

Fix missing reference. Signed-off-by: Hans Verkuil diff --git a/Documentation/media/uapi/v4l/pixfmt-006.rst b/Documentation/media/uapi/v4l/pixfmt-006.rst index 9a0f494..987b9a8 100644 --- a/Documentation/media/uapi/v4l/pixfmt-006.rst +++

[PATCH] [media] doc-rst: fix a missing reference for V4L2_BUF_FLAG_LAST

Fix it by adding a header to the flat-table to match to the list of define symbols. As a side-effect, it also removes some exceptions from videodev2.h.rst. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/buffer.rst| 38

Re: [PATCH 03/20] [media] lirc.h: remove several unused ioctls

On Tue, Jul 12, 2016 at 02:35:56PM +0100, Sean Young wrote: > On Tue, Jul 12, 2016 at 10:23:00AM -0300, Mauro Carvalho Chehab wrote: > > Em Tue, 12 Jul 2016 14:14:06 +0100 > > Sean Young escreveu: > > > On Tue, Jul 12, 2016 at 09:41:57AM -0300, Mauro Carvalho Chehab wrote: > -snip-

[PATCH 01/20] [media] doc-rst: Document ioctl LIRC_GET_FEATURES

The documentation for this ioctl was really crappy. Add a better documentation, using the lirc.4 man pages as a reference, plus what was written originally at the lirc-ioctl. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/lirc.h.rst.exceptions |

[PATCH 07/20] [media] doc-rst: fix some lirc cross-references

Some references were broken. It was also mentioning LIRC_MODE_RAW, with it is not implemented on current LIRC drivers. So, fix the references. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-get-features.rst | 5 ++---

[PATCH 02/20] [media] doc-rst: add media/uapi/rc/lirc-header.rst

changeset 68cd5e0bed99 ("[media] doc-rst: add LIRC header to the book") did everything but adding the lirc-reader.rst :-p My fault: I forgot to do a git add for this guy on such changeset. Signed-off-by: Mauro Carvalho Chehab ---

Re: [PATCH 06/11] media: adv7180: add bt.656-4 OF property

This conversation has drifted off topic, sorry. It now relates to code in drivers/gpu/ipu-v3/ipu-csi.c, so adding Philipp Zabel. On 11/07/16 23:03, Steve Longerbeam wrote: On 07/11/2016 12:06 AM, Ian Arkver wrote: On 10/07/16 23:34, Steve Longerbeam wrote: On 07/10/2016 07:30 AM, Hans

[PATCH -next] [media] mtk-vcodec: remove redundant dev_err call in mtk_vcodec_probe()

From: Wei Yongjun There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c | 2

[PATCH -next] [media] vcodec: mediatek: Fix return value check in mtk_vcodec_init_enc_pm()

From: Wei Yongjun In case of error, the function devm_clk_get() returns ERR_PTR() and not returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun ---

Re: [PATCH -next] [media] vcodec: mediatek: Fix return value check in mtk_vcodec_init_enc_pm()

Reviewed-by:Tiffany Lin On Tue, 2016-07-12 at 11:02 +, weiyj...@163.com wrote: > From: Wei Yongjun > > In case of error, the function devm_clk_get() returns ERR_PTR() > and not returns NULL. The NULL test in the return value check >

[PATCH -next] [media] mtk-vcodec: remove .owner field for driver

From: Wei Yongjun Remove .owner field if calls are used which set it automatically. Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Wei Yongjun ---

Re: [PATCH v2] zc3xx: Remove deprecated create_singlethread_workqueue

On Sat, Jul 09, 2016 at 01:22:29PM +0530, Bhaktipriya Shridhar wrote: > The workqueue "work_thread" is involved in updating parameters for > transfers. It has a single work item(>work) and hence > doesn't require ordering. Also, it is not being used on a memory > reclaim path. Hence, the

[PATCH 0/2] doc-rst: CEC improvements

From: Hans Verkuil Note: patch 1 assumes this pull request has been merged: https://patchwork.linuxtv.org/patch/35377/ Regards, Hans Hans Verkuil (2): doc-rst: update CEC_RECEIVE doc-rst: improve CEC documentation

[PATCH 1/2] doc-rst: update CEC_RECEIVE

From: Hans Verkuil The timestamp field was split into rx_ts and tx_ts, and the rx/tx_status fields were moved. Update the doc accordingly. Also fix a bug that stated that a non-zero tx_status field signaled an error. That's not true, since TX_STATUS_OK is 1, not 0.

[PATCH 2/2] doc-rst: improve CEC documentation

From: Hans Verkuil Lots of fixups relating to references. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/cec/cec-func-ioctl.rst| 2 +- Documentation/media/uapi/cec/cec-func-open.rst | 10 +++

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

Hello Shuah, On 07/11/2016 06:39 PM, Shuah Khan wrote: > Fix misspelled error message and existing checkpatch errors in the > error message conditional. > > WARNING: suspect code indent for conditional statements (8, 24) > if (ctx->state != MFCINST_HEAD_PARSED && > [...] > +

[PATCH -next] [media] rcar_vin: remove redundant return value check of platform_get_resource()

From: Wei Yongjun Remove unneeded error handling on the result of a call to platform_get_resource() when the value is passed to devm_ioremap_resource(). And move those two call together to make the connection between them more clear. Signed-off-by: Wei Yongjun

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

On Tue, 2016-07-12 at 08:42 -0600, Shuah Khan wrote: > On 07/12/2016 08:06 AM, Joe Perches wrote: > > On Mon, 2016-07-11 at 16:39 -0600, Shuah Khan wrote: > > > Fix misspelled error message and existing checkpatch errors in the > > > error message conditional. > > [] > > > diff --git

Re: [PATCH] media: s5p-mfc remove void function return statement

Hello Shuah On 07/11/2016 08:29 PM, Shuah Khan wrote: > Remove void function return statement > > Signed-off-by: Shuah Khan > --- > drivers/media/platform/s5p-mfc/s5p_mfc.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git

Re: [PATCH 28/54] MAINTAINERS: Add file patterns for media device tree bindings

Hi Mauro, On Sun, Jul 10, 2016 at 5:27 PM, Mauro Carvalho Chehab wrote: > Em Sun, 22 May 2016 11:06:05 +0200 > Geert Uytterhoeven escreveu: > >> Submitters of device tree binding documentation may forget to CC >> the subsystem maintainer if this is

Re: mceusb xhci issue?

On Sat, 9 Jul 2016, Mauro Carvalho Chehab wrote: > C/C linux-usb Mailing list: > > > Em Wed, 18 May 2016 08:52:28 -0600 > Wade Berrier escreveu: ... > > > That message above links to some other threads describing the issue. > > > Here's a post with a patch that supposedly

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

On 07/12/2016 09:03 AM, Javier Martinez Canillas wrote: > Hello Shuah, > > On 07/11/2016 06:39 PM, Shuah Khan wrote: >> Fix misspelled error message and existing checkpatch errors in the >> error message conditional. >> >> WARNING: suspect code indent for conditional statements (8, 24) >> if

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

Hello Shuah, On 07/12/2016 11:07 AM, Shuah Khan wrote: > On 07/12/2016 09:03 AM, Javier Martinez Canillas wrote: >> Hello Shuah, >> >> On 07/11/2016 06:39 PM, Shuah Khan wrote: >>> Fix misspelled error message and existing checkpatch errors in the >>> error message conditional. >>> >>> WARNING:

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

On Tue, 2016-07-12 at 10:08 -0600, Shuah Khan wrote: > On 07/12/2016 09:51 AM, Joe Perches wrote: > > On Tue, 2016-07-12 at 08:42 -0600, Shuah Khan wrote: > > > On 07/12/2016 08:06 AM, Joe Perches wrote: > > > > On Mon, 2016-07-11 at 16:39 -0600, Shuah Khan wrote: > > > > > > > > > > Fix

[PATCH v2] media: s5p-mfc Fix misspelled error message and checkpatch errors

Fix misspelled error message and existing checkpatch errors in the error message conditional. WARNING: suspect code indent for conditional statements (8, 24) if (ctx->state != MFCINST_HEAD_PARSED && [...] mfc_err("Can not get crop information\n"); Signed-off-by: Shuah Khan

Re: [PATCH] media: s5p-mfc Fix misspelled error message and checkpatch errors

On 07/12/2016 09:51 AM, Joe Perches wrote: > On Tue, 2016-07-12 at 08:42 -0600, Shuah Khan wrote: >> On 07/12/2016 08:06 AM, Joe Perches wrote: >>> On Mon, 2016-07-11 at 16:39 -0600, Shuah Khan wrote: Fix misspelled error message and existing checkpatch errors in the error message

[GIT PULL FOR v4.8] Various fixes

A bunch of vcodec code improvements and two cec changes: one adds a sanity check, the other improves timestamping. A patch updating the doc-rst will be posted separately. Regards, Hans The following changes since commit 9d01315d132469fd0a92f5a13c0a605d6ce96b21: [media] pulse8-cec:

Re: [PATCH RFC v2 1/2] media: platform: transfer format translations to soc_mediabus

Hans Verkuil writes: > On 04/02/2016 04:26 PM, Robert Jarzmik wrote: >> Transfer the formats translations to soc_mediabus. Even is soc_camera >> was to be deprecated, soc_mediabus will survive, and should describe all >> that happens on the bus connecting the image processing

Re: [PATCH 06/11] media: adv7180: add bt.656-4 OF property

Hi Ian, On 07/12/2016 03:25 AM, Ian Arkver wrote: > This conversation has drifted off topic, sorry. > It now relates to code in drivers/gpu/ipu-v3/ipu-csi.c, so adding Philipp > Zabel. > > On 11/07/16 23:03, Steve Longerbeam wrote: >> On 07/11/2016 12:06 AM, Ian Arkver wrote: >>> >>> Hi Steve,

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : > Decoder hardware produces MT21 (compressed). Image processor can > convert it to a format that can be input of display driver. Tiffany. > When do you plan to upstream image processor (mtk-mdp)? > > > > It can be as input format

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

Le mardi 12 juillet 2016 à 15:08 -0400, Nicolas Dufresne a écrit : > Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : > > Decoder hardware produces MT21 (compressed). Image processor can > > convert it to a format that can be input of display driver. > > Tiffany. > > When do you

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

On 12/07/16 20:14, Nicolas Dufresne wrote: Le mardi 12 juillet 2016 à 15:08 -0400, Nicolas Dufresne a écrit : Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : Decoder hardware produces MT21 (compressed). Image processor can convert it to a format that can be input of display