Lad, Prabhakar | 18 Nov 14:00 2014

[PATCH] media: ti-vpe: Use mem-to-mem ioctl helpers

This patch does the following:

1: Simplify the vpe mem-to-mem driver by using the m2m ioctl
   and vb2 helpers.
2: Minor code cleanup

Signed-off-by: Lad, Prabhakar <prabhakar.csengg <at>>
 drivers/media/platform/ti-vpe/vpe.c | 152 +++++++++++-------------------------
 1 file changed, 47 insertions(+), 105 deletions(-)

diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c
index 74b858d..faf38d4 100644
--- a/drivers/media/platform/ti-vpe/vpe.c
+++ b/drivers/media/platform/ti-vpe/vpe.c
 <at>  <at>  -21,6 +21,7  <at>  <at> 
 #include <linux/dma-mapping.h>
 #include <linux/err.h>
 #include <linux/fs.h>
+#include <linux/of.h>
 #include <linux/interrupt.h>
 #include <linux/io.h>
 #include <linux/ioctl.h>
 <at>  <at>  -377,7 +378,6  <at>  <at>  struct vpe_dev {
 struct vpe_ctx {
 	struct v4l2_fh		fh;
 	struct vpe_dev		*dev;
-	struct v4l2_m2m_ctx	*m2m_ctx;
 	struct v4l2_ctrl_handler hdl;

(Continue reading)

Sudip Mukherjee | 18 Nov 13:59 2014

[PATCH] usbnet: rtl8150: remove unused variable

we were just returning the initial value of res, instead now
we are returning the value directly.

Signed-off-by: Sudip Mukherjee <sudip <at>>
 drivers/net/usb/rtl8150.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/net/usb/rtl8150.c b/drivers/net/usb/rtl8150.c
index 6e87e57..d37b7dc 100644
--- a/drivers/net/usb/rtl8150.c
+++ b/drivers/net/usb/rtl8150.c
 <at>  <at>  -753,14 +753,13  <at>  <at>  static int rtl8150_open(struct net_device *netdev)
 static int rtl8150_close(struct net_device *netdev)
 	rtl8150_t *dev = netdev_priv(netdev);
-	int res = 0;

 	if (!test_bit(RTL8150_UNPLUG, &dev->flags))

-	return res;
+	return 0;

 static void rtl8150_get_drvinfo(struct net_device *netdev, struct ethtool_drvinfo *info)

(Continue reading)

Sudip Mukherjee | 18 Nov 13:12 2014

[PATCH] ASoC: rt286: build warning of section mismatch

From: Sudip Mukherjee <sudip <at>>

while building we were getting the following build warning:

Section mismatch in reference from the function rt286_i2c_probe()
to the variable
The function rt286_i2c_probe() references
the variable __initdata force_combo_jack_table.
This is often because rt286_i2c_probe lacks a __initdata
annotation or the annotation of force_combo_jack_table is wrong.

we were getting the warning as force_combo_jack_table was marked
with __initdata

Signed-off-by: Sudip Mukherjee <sudip <at>>
 sound/soc/codecs/rt286.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/rt286.c b/sound/soc/codecs/rt286.c
index 2e818aa..2cd4fe4 100644
--- a/sound/soc/codecs/rt286.c
+++ b/sound/soc/codecs/rt286.c
 <at>  <at>  -1206,7 +1206,7  <at>  <at>  static const struct acpi_device_id rt286_acpi_match[] = {
 MODULE_DEVICE_TABLE(acpi, rt286_acpi_match);

-static struct dmi_system_id force_combo_jack_table[] __initdata = {
+static struct dmi_system_id force_combo_jack_table[] = {
(Continue reading)

Kiran Kumar Raparthy | 18 Nov 13:08 2014

[RFC] debug: add parameters to prevent entering debug mode on errors

From: Colin Cross <ccross <at>>

debug: add parameters to prevent entering debug mode on errors

On non-developer devices kgdb prevents CONFIG_PANIC_TIMEOUT from rebooting the
device after a panic. Add module parameters debug_core.break_on_exception and
debug_core.break_on_panic to allow skipping debug on panics and exceptions
respectively.  Both default to true to preserve existing behavior.

Cc: Jason Wessel <jason.wessel <at>>
Cc: kgdb-bugreport <at>
Cc: linux-kernel <at>
Cc: Android Kernel Team <kernel-team <at>>
Cc: John Stultz <john.stultz <at>>
Cc: Sumit Semwal <sumit.semwal <at>>
Signed-off-by: Colin Cross <ccross <at>>
[Kiran: Added context to commit message]
Signed-off-by: Kiran Raparthy <kiran.kumar <at>>
This is one of the number of patches from the Android AOSP common.git tree,
which is used on almost all Android devices.  I wanted to submit it for review
to see if it should go upstream.

 kernel/debug/debug_core.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/kernel/debug/debug_core.c b/kernel/debug/debug_core.c
index 1adf62b..af06122 100644
--- a/kernel/debug/debug_core.c
+++ b/kernel/debug/debug_core.c
(Continue reading)

Lad, Prabhakar | 18 Nov 12:23 2014

[PATCH 00/12] media: use vb2_ops_wait_prepare/finish helper

Hi All,

This patch set uses the vb2 ops helpers for wait_prepare and
wait_finish callbacks for drivers which using vb2 helpers.
This patchset is compile tested only.

I am not sure on how mem2mem drivers work because the have
2 queues sharing the same lock, so not sure if these
changes to mem2mem drivers are valid.

Lad, Prabhakar (12):
  media: s3c-camif: use vb2_ops_wait_prepare/finish helper
  media: ti-vpe: use vb2_ops_wait_prepare/finish helper
  media: exynos-gsc: use vb2_ops_wait_prepare/finish helper
  media: soc_camera: use vb2_ops_wait_prepare/finish helper
  media: sh_veu: use vb2_ops_wait_prepare/finish helper
  media: marvell-ccic: use vb2_ops_wait_prepare/finish helper
  media: s5p-tv: use vb2_ops_wait_prepare/finish helper
  media: blackfin: use vb2_ops_wait_prepare/finish helper
  media: s5p-mfc: use vb2_ops_wait_prepare/finish helper
  media: vivid: use vb2_ops_wait_prepare/finish helper
  media: davinci: vpif_capture: use vb2_ops_wait_prepare/finish helper
  media: usb: uvc: use vb2_ops_wait_prepare/finish helper

 drivers/media/platform/blackfin/bfin_capture.c     | 17 ++---------
 drivers/media/platform/davinci/vpif_capture.c      |  2 ++
 drivers/media/platform/exynos-gsc/gsc-core.h       | 12 --------
 drivers/media/platform/exynos-gsc/gsc-m2m.c        |  6 ++--
 drivers/media/platform/marvell-ccic/mcam-core.c    | 29 ++++--------------
 drivers/media/platform/s3c-camif/camif-capture.c   | 17 ++---------
(Continue reading)

SF Markus Elfring | 18 Nov 12:20 2014

[PATCH 1/1] fs-namespace: Deletion of unnecessary checks before the function call "mntput"

From: Markus Elfring <elfring <at>>
Date: Tue, 18 Nov 2014 12:10:43 +0100

The mntput() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring <at>>
 fs/namespace.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/fs/namespace.c b/fs/namespace.c
index 550dbff..3b3710e 100644
--- a/fs/namespace.c
+++ b/fs/namespace.c
 <at>  <at>  -2615,10 +2615,8  <at>  <at>  struct mnt_namespace *copy_mnt_ns(unsigned long flags, struct mnt_namespace *ns,

-	if (rootmnt)
-		mntput(rootmnt);
-	if (pwdmnt)
-		mntput(pwdmnt);
+	mntput(rootmnt);
+	mntput(pwdmnt);

 	return new_ns;
(Continue reading)

pang.xunlei | 18 Nov 12:15 2014

[RFC PATCH v3 0/5] y2038 in-kernel interface changes for drivers/rtc

On 32bit systems, the kernel uses a 32bit signed time_t value
for seconds since 1970-01-01:00:00:00. This will overflow at
2038-01-19 03:14:08, and is widely known as the y2038 problem.

To address this, the plan is to create 64bit internal interfaces
which are 2038 safe, mark the unsafe versions as deprecated and 
then convert subsystems one by one over to the new interfaces.
Once all users are converted, the deprecated internal functions
will be removed.

NOTE: In some cases, there are issues with 32bit values that are
unsigned, and thus become y2106 issues. Since its somewhat simpler
to unify the time types, we'll try to convert those usage to 64bit
where it can be easily done. In those cases where a fix would be 
overly complicated, we'll simply explicitly mark the limitation.

This patchset introduces the 64bit interfaces needed to convert
the drivers/rtc subsystem, and to keep it relatively short some
patches to demonstrate how the conversion will be done. If the 
feedback is positive on this series, we'll provide the full 
conversion series as well.

pang.xunlei (5):
  time: Provide y2038 safe do_settimeofday() replacement
  time: Provide y2038 safe timekeeping_inject_sleeptime() replacement
  time: Provide y2038 safe mktime() replacement
  rtc/lib: Provide y2038 safe rtc_tm_to_time()/rtc_time_to_tm()
  rtc/mc13xxx: Eliminate time problems

(Continue reading)

Julien CHAUVEAU | 18 Nov 12:10 2014

[PATCH] clk: rockchip: fix parent clock for rk3188 hclk_lcdc1

The parent clock for hclk_lcdc1 was set to aclk_cpu instead of hclk_cpu.

Signed-off-by: Julien CHAUVEAU <julien.chauveau <at>>
 drivers/clk/rockchip/clk-rk3188.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/rockchip/clk-rk3188.c b/drivers/clk/rockchip/clk-rk3188.c
index f88eb7d..e6cd483 100644
--- a/drivers/clk/rockchip/clk-rk3188.c
+++ b/drivers/clk/rockchip/clk-rk3188.c
 <at>  <at>  -410,7 +410,7  <at>  <at>  static struct rockchip_clk_branch common_clk_branches[] __initdata = {
 	/* hclk_ahb2apb is part of a clk branch */
 	GATE(0, "hclk_vio_bus", "hclk_cpu", 0, RK2928_CLKGATE_CON(6), 12, GFLAGS),
 	GATE(HCLK_LCDC0, "hclk_lcdc0", "hclk_cpu", 0, RK2928_CLKGATE_CON(6), 1, GFLAGS),
-	GATE(HCLK_LCDC1, "hclk_lcdc1", "aclk_cpu", 0, RK2928_CLKGATE_CON(6), 2, GFLAGS),
+	GATE(HCLK_LCDC1, "hclk_lcdc1", "hclk_cpu", 0, RK2928_CLKGATE_CON(6), 2, GFLAGS),
 	GATE(HCLK_CIF0, "hclk_cif0", "hclk_cpu", 0, RK2928_CLKGATE_CON(6), 4, GFLAGS),
 	GATE(HCLK_IPP, "hclk_ipp", "hclk_cpu", 0, RK2928_CLKGATE_CON(6), 9, GFLAGS),
 	GATE(HCLK_RGA, "hclk_rga", "hclk_cpu", 0, RK2928_CLKGATE_CON(6), 10, GFLAGS),


Rogelio Serrano | 18 Nov 12:05 2014

mysterious udev segfault

on boot i get this half of the time:

[    6.349867] general protection fault: 0000 [#1] SMP
[    6.352177] Modules linked in: zram lz4_decompress lz4_compress unix
[    6.354483] CPU: 1 PID: 112 Comm: udevd Not tainted 3.16.3-grsec-26 #2
[    6.356583] Hardware name: Matsushita Electric Industrial Co.,Ltd. CF-W7DWM0$
[    6.356583] task: ffff8800ba589e00 ti: ffff8800ba58a5d8 task.ti: ffff8800ba5$
[    6.356583] RIP: 0010:[<ffffffff812f9a2c>]  [<ffffffff812f9a2c>] ffffffff812$
[    6.356583] RSP: 0018:ffffc9001555b9c8  EFLAGS: 00010246
[    6.356583] RAX: 0000000000000000 RBX: fefefefefefefefe RCX: ffff8800bb6d0a68
[    6.356583] RDX: ffffc9001555ba18 RSI: 0000000000000036 RDI: fefefefefefefefe
[    6.356583] RBP: ffffc9001555b9f8 R08: 0000000000015660 R09: ffff8800bb71d890
[    6.356583] R10: ffff8800b9087000 R11: 000000000000000a R12: 0000000000000036
[    6.356583] R13: ffffc9001555ba18 R14: ffff8800bb145b58 R15: ffff8800bab10000
[    6.356583] FS:  000002c2ecf717c0(0000) GS:ffff8800bf500000(0000) knlGS:0000$
[    6.356583] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[    6.356583] CR2: 000006c2ecf7b000 CR3: 000000000194d000 CR4: 00000000000007f0
[    6.356583] Stack:
[    6.356583]  ffffc9001555b9f8 f4b9ed2e7f08ef51 000000000003602f ffff8800bb71$
[    6.356583]  ffff8800bb6d0c70 0000000000000036 ffffc9001555ba58 ffffffff812f$
[    6.356583]  ffff8800bb6d0a68 ffff8800bb71d680 ffffc9001555ba58 f4b9ed2e7f08$
[    6.356583] Call Trace:
[    6.356583]  [<ffffffff812fa70e>] diRead+0x7e/0x250
[    6.356583]  [<ffffffff812f06c0>] jfs_iget+0x70/0x190
[    6.356583]  [<ffffffff812f1034>] jfs_lookup+0xd4/0xe0
[    6.356583]  [<ffffffff8119e8dc>] ? kmem_cache_alloc+0x3c/0x160
[    6.356583]  [<ffffffff811cf315>] ? __d_alloc+0x35/0x1d0
[    6.356583]  [<ffffffff813b643a>] ? __list_add+0x3a/0x80
[    6.356583]  [<ffffffff811cf7ec>] ? d_alloc+0x6c/0xa0
[    6.356583]  [<ffffffff811bbe28>] lookup_real+0x58/0xb0
(Continue reading)

Lad, Prabhakar | 18 Nov 11:57 2014

[PATCH] media: exynos-gsc: fix build warning

this patch fixes following build warning:

gsc-core.c:350:17: warning: 'low_plane' may be used uninitialized
gsc-core.c:371:31: warning: 'high_plane' may be used uninitialized

Signed-off-by: Lad, Prabhakar <prabhakar.csengg <at>>
 drivers/media/platform/exynos-gsc/gsc-core.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/exynos-gsc/gsc-core.c b/drivers/media/platform/exynos-gsc/gsc-core.c
index 91d226b..6c71b17 100644
--- a/drivers/media/platform/exynos-gsc/gsc-core.c
+++ b/drivers/media/platform/exynos-gsc/gsc-core.c
 <at>  <at>  -347,8 +347,8  <at>  <at>  void gsc_set_prefbuf(struct gsc_dev *gsc, struct gsc_frame *frm)
 		s_chk_addr = frm->addr.cb;
 		s_chk_len = frm->payload[1];
 	} else if (frm->fmt->num_planes == 3) {
-		u32 low_addr, low_plane, mid_addr, mid_plane;
-		u32 high_addr, high_plane;
+		u32 low_addr, low_plane = 0, mid_addr, mid_plane;
+		u32 high_addr, high_plane = 0;
 		u32 t_min, t_max;

 		t_min = min3(frm->addr.y, frm->addr.cb, frm->;


Yuan Yao | 18 Nov 11:31 2014

[PATCH v11 0/3] i2c: imx: add DMA support for freescale i2c driver

Changed in v11: 
- Fix the bug for 'orig_jiffies' uninitialized.
- Use wait_for_completion_timeout() instead of wait_for_completion_interruptible_timeout(). 

Changed in v10: 
- Rebase to the latest code. 
- Add dma_submit_error for dmaengine_submit handling
- Some minor fixes for coding style. 

Changed in v9: 
- seperate a patch for sort include headers alphabetically 
- some minor fixes for coding style. 

Changed in v8: 
- some minor fixes for coding style. 
- unsetting I2CR_DMAEN immediatelly when DMA failed. 

Changed in v7: 
- when waiting for transfer complete use schedule() instead of udelay(). 

Changed in v6: 
- changed the inappropriate print message. 
- rebase to the latest code. 

Changed in v5: 
- add "*chan_dev = dma->chan_using->device->dev" for code cleanup. 
- add the test logs. 

Changed in v4: 
(Continue reading)