On Tue 2016-07-12 20:32:01, Mauro Carvalho Chehab wrote:
1;2802;0c> 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 eithe
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 versi
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
>> > convert it to a format tha
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 05:00:28 2016 +0300
Fix checkpatch
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 documentation
> >> >
> >> >
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 i
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 you
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
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
b/drivers/media/platform/s5p-mfc/s5p_mfc.c
index b6fde2
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 like a bit of ove
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
> Cc: Kamil Debski
> Signed-off-by: Krzysztof Ko
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
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
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 f
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.
Signed-off-by: Hans Verkuil
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 +++
.../media/uapi/cec/cec-ioc-adap-g-caps.rst | 18 ++--
.../media
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
Documentation/media/uapi/cec/cec-func-ioctl.rst| 2 +-
Docu
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(&sd->work) and hence
> doesn't require ordering. Also, it is not being used on a memory
> reclaim path. Hence, the single
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
Review
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,
>
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 unit of the SoC
>> a
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: dec
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 misspelle
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 conditi
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 works:
> > >
> > >
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 a/drivers/medi
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 missing.
>
> I'm assuming that this patch will
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: su
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
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
---
drivers/media/platform/soc_ca
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 a/drivers/media/platform/s5p-mfc/s5p_mfc.c
> b/drivers/media/platf
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 &&
> [...]
> + m
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 +-
Documentation/medi
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
>> b/drivers/media/platform/s5p-mf
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
+++ b/Documentation/media/uapi/v4
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
+++ b/Documentation/media/uapi/v4l/pixfmt-006.rst
@@ -240,6 +
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 i
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:
> > > > On Tue, Jul 12, 2016 at
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
b/drivers/staging/media/cec/cec-
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 in
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 follow
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-
> > > > -#defin
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, __u32)
> >
> >
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.
> >
> > Signed-off-by
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 mainstream).
> >
> >
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
> anywhere
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
> ---
> Documentation/media/uapi/rc/lirc-get
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 Cheha
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 +
Documentation/media/uapi/rc/lirc_ioctl.rst | 9
3
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 +
Documentation/media/uapi/rc/lirc_ioctl.rst
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| 1 +
Documentation/media/
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 | 49 ++
.
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 | 9
3 files changed, 5
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 +
Documentation/media/uapi/rc/lirc_ioctl.rst
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 | 14
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
---
.../media/uapi/rc/lirc-set-rec-carrier-range.rst | 49
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, 60 insertions(+), 39 deleti
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 +
Documentation/media/uapi/rc/lirc_ioctl.rst | 11 --
3
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 +
Documentation/media/uapi/rc/lirc_ioctl.rst
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 ++
.../media/uapi/rc/lirc_device_interface.r
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 ++-
2 files changed, 9 insertio
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 | 55 +++
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 ++
.../media/uapi/rc/lirc_device_interface.r
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
---
Documentation/media/uapi/rc/lirc-get-rec-mode.rst | 32
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 Chehab
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
---
Documentation/media/uapi/rc/lirc-header.rst | 10 ++
1 fi
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 | 35 -
Documentation/m
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 ++---
Documentation/media/uapi/rc/lirc-get-send-mode.rs
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
Hello, Wei Yongjun.
Thanks for the patch!
Acked-by: Mikhail Ulyanov
--
W.B.R, M
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 r
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 when
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
---
drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers
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
1 file changed, 4 deletions(-)
diff --git a/drivers/media/
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.
>
> Signed-off-by: Wei Yongjun
> ---
> drivers/media/pla
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 insertions(+), 1 deletion(-)
diff --git a/drivers/media/platform/rcar
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
> should be replaced with IS_ERR().
>
> Signed-off-by: Wei
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
---
drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_pm.c | 16
1 file c
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 --
1 file changed, 2 deletions(-)
diff --git a/drivers/media
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
---
drivers/media/platform/mtk-vpu/mtk_vpu.c | 4 ++--
1 file changed, 2 insertions(+), 2
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 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/driv
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 Verku
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
>
From: Colin Ian King
The result of mb86a20s_readreg(state, 0x0a) & 0xf is always in the range
0x00 to 0x0f and can never be negative, so remove the redundant check
of the result being less than zero.
Signed-off-by: Colin Ian King
---
drivers/media/dvb-frontends/mb86a20s.c | 2 --
1 file change
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
>> > ---
>> > Documentation/DocBook/media/v4l/pix
84 matches
Mail list logo