Re: [RFC] Support for 'Coda' video codec IP.

2012-06-20 Thread Shawn Guo
On Wed, Jun 20, 2012 at 11:00:15AM +0100, Mark Brown wrote: The approach a lot of platforms have been taking is that it's OK to keep on maintaining existing boards using board files (especially for trivial things like adding new devices). I think that's the approach being taken during the

Re: [RFC] Support for 'Coda' video codec IP.

2012-06-20 Thread Shawn Guo
On Wed, Jun 20, 2012 at 02:31:48PM +0100, Mark Brown wrote: I think that's the approach being taken during the transition to device tree. But it's definitely a desirable thing to remove those board files with device tree support at some point, because not having non-DT users will ease

Re: [RFC] Support for 'Coda' video codec IP.

2012-06-20 Thread Shawn Guo
On Wed, Jun 20, 2012 at 11:00:15AM +0100, Mark Brown wrote: The approach a lot of platforms have been taking is that it's OK to keep on maintaining existing boards using board files (especially for trivial things like adding new devices). If the device is added without introducing any

Re: [RFC] Support for 'Coda' video codec IP.

2012-06-20 Thread Shawn Guo
On Wed, Jun 20, 2012 at 04:10:27PM +0200, javier Martin wrote: If I drop this platform data it is OK with you if I don't add device tree support by now? I'm fine. Sascha? -- Regards, Shawn -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to

Re: [RFC] Support for 'Coda' video codec IP.

2012-06-20 Thread Shawn Guo
On Wed, Jun 20, 2012 at 04:25:54PM +0200, javier Martin wrote: Do you plan to add pinctrl support for i.MX27 and i.MX21? We will have to when we are there to convert these platforms over to DT. -- Regards, Shawn -- To unsubscribe from this list: send the line unsubscribe linux-media in the

[PATCH 00/34] i.MX multi-platform support

2012-09-16 Thread Shawn Guo
are good. Thanks. Shawn Guo (34): ARM: imx: include board headers in the same folder ASoC: mx27vis: retrieve gpio numbers from platform_data ARM: imx: move iomux drivers and headers into mach-imx ARM: imx: remove unnecessary inclusion from device-imx*.h ARM: imx: move platform device code

[PATCH 12/34] media: mx1_camera: remove the driver

2012-09-16 Thread Shawn Guo
it becomes a piece of unmaintained code, so let's remove it. [1] https://lkml.org/lkml/2012/2/9/171 Signed-off-by: Shawn Guo shawn@linaro.org Cc: Paulius Zaleckas paulius.zalec...@teltonika.lt Cc: Guennadi Liakhovetski g.liakhovet...@gmx.de Cc: linux-media@vger.kernel.org --- arch/arm/mach-imx

[PATCH 14/34] dma: ipu: rename mach/ipu.h to include/linux/dma/ipu-dma.h

2012-09-16 Thread Shawn Guo
The header ipu.h really belongs to dma subsystem rather than imx platform. Rename it to ipu-dma.h and put it into include/linux/dma/. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Vinod Koul vinod.k...@intel.com Cc: Guennadi Liakhovetski g.liakhovet...@gmx.de Cc: Florian Tobias Schandinat

[PATCH 26/34] media: mx2_camera: remove dead code in mx2_camera_add_device

2012-09-16 Thread Shawn Guo
This is a piece of code becoming dead since commit 2c9ba37 ([media] V4L: mx2_camera: remove unsupported i.MX27 DMA mode, make EMMA mandatory). It should have been removed together with the commit. Remove it now. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Guennadi Liakhovetski g.liakhovet

[PATCH 27/34] media: mx2_camera: use managed functions to clean up code

2012-09-16 Thread Shawn Guo
Use managed functions to clean up the error handling code and function mx2_camera_remove(). Along with the change, a few variables get removed from struct mx2_camera_dev. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Guennadi Liakhovetski g.liakhovet...@gmx.de Cc: linux-media@vger.kernel.org

[PATCH 28/34] media: mx2_camera: remove mach/hardware.h inclusion

2012-09-16 Thread Shawn Guo
It changes the driver to use platform_device_id rather than cpu_is_xxx to determine the controller type, and updates the platform code accordingly. As the result, mach/hardware.h inclusion gets removed from the driver. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Guennadi Liakhovetski

Re: [PATCH 12/34] media: mx1_camera: remove the driver

2012-09-17 Thread Shawn Guo
On Mon, Sep 17, 2012 at 10:33:25AM +0200, Guennadi Liakhovetski wrote: Ok, it used to compile not-so-long-ago, but it doesn't seem to be cared for a lot lately. Let's give Paulius a bit more time to react to this mail, otherwise I'll have no objections. Just as an idea, to make it a bit

Re: [PATCH 26/34] media: mx2_camera: remove dead code in mx2_camera_add_device

2012-09-17 Thread Shawn Guo
as a whole. On Mon, 17 Sep 2012, Shawn Guo wrote: This is a piece of code becoming dead since commit 2c9ba37 ([media] V4L: mx2_camera: remove unsupported i.MX27 DMA mode, make EMMA mandatory). It should have been removed together with the commit. Remove it now. Signed-off-by: Shawn Guo

Re: [PATCH 27/34] media: mx2_camera: use managed functions to clean up code

2012-09-18 Thread Shawn Guo
On Mon, Sep 17, 2012 at 03:36:07PM +0200, javier Martin wrote: This patch breaks the driver: Javier, Can you please apply the following change to see if it fixes the problem? Shawn @@ -1783,6 +1783,8 @@ static int __devinit mx2_camera_probe(struct platform_device *pdev)

Re: [alsa-devel] [PATCH 00/34] i.MX multi-platform support

2012-09-18 Thread Shawn Guo
On Tue, Sep 18, 2012 at 09:52:13AM +0200, Sascha Hauer wrote: I just had a look at the remaining initcalls in arch-imx. Most of them are protected with a cpu_is_*, but this one should be fixed before i.MX is enabled for multi platform:

Re: [PATCH 00/34] i.MX multi-platform support

2012-09-18 Thread Shawn Guo
On Mon, Sep 17, 2012 at 09:51:38AM +0200, Sascha Hauer wrote: I gave it a test on i.MX1, i.MX27, i.MX31 and i.MX35. All run fine, but the last patch breaks the imx_v4_v5_defconfig: Somehow it now defaults to ARMv7 based machines. I haven't looked into it, just reenabled ARMv4/ARMv5 and the

Re: [PATCH 12/34] media: mx1_camera: remove the driver

2012-09-19 Thread Shawn Guo
On Tue, Sep 18, 2012 at 09:13:54AM +, Arnd Bergmann wrote: On Tuesday 18 September 2012, Shawn Guo wrote: On Mon, Sep 17, 2012 at 10:33:25AM +0200, Guennadi Liakhovetski wrote: Ok, it used to compile not-so-long-ago, but it doesn't seem to be cared for a lot lately. Let's give

[PATCH v2 00/34] i.MX multi-platform support

2012-09-20 Thread Shawn Guo
-data arm-soc/multiplatform/smp_ops arm-soc/imx/cleanup arm-soc/imx/dt sound/for-3.7 Subsystem maintainers, I plan to send the whole series for 3.7 via arm-soc tree. Please let me know if you have problem with that. Thanks. Shawn --- Shawn Guo (34): ARM: imx: include board headers

[PATCH v2 11/34] media: mx1_camera: remove the driver

2012-09-20 Thread Shawn Guo
it becomes a piece of unmaintained code, so let's remove it. [1] https://lkml.org/lkml/2012/2/9/171 Signed-off-by: Shawn Guo shawn@linaro.org Cc: Paulius Zaleckas paulius.zalec...@teltonika.lt Cc: Guennadi Liakhovetski g.liakhovet...@gmx.de Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: linux-media

[PATCH v2 13/34] dma: ipu: rename mach/ipu.h to include/linux/dma/ipu-dma.h

2012-09-20 Thread Shawn Guo
The header ipu.h really belongs to dma subsystem rather than imx platform. Rename it to ipu-dma.h and put it into include/linux/dma/. Signed-off-by: Shawn Guo shawn@linaro.org Acked-by: Guennadi Liakhovetski g.liakhovet...@gmx.de Acked-by: Sascha Hauer s.ha...@pengutronix.de Acked-by: Arnd

[PATCH v2 25/34] media: mx2_camera: remove dead code in mx2_camera_add_device

2012-09-20 Thread Shawn Guo
This is a piece of code becoming dead since commit 2c9ba37 ([media] V4L: mx2_camera: remove unsupported i.MX27 DMA mode, make EMMA mandatory). It should have been removed together with the commit. Remove it now. Signed-off-by: Shawn Guo shawn@linaro.org Acked-by: Sascha Hauer s.ha

[PATCH v2 26/34] media: mx2_camera: use managed functions to clean up code

2012-09-20 Thread Shawn Guo
Use managed functions to clean up the error handling code and function mx2_camera_remove(). Along with the change, a few variables get removed from struct mx2_camera_dev. Signed-off-by: Shawn Guo shawn@linaro.org Acked-by: Sascha Hauer s.ha...@pengutronix.de Acked-by: Arnd Bergmann

[PATCH v2 27/34] media: mx2_camera: remove cpu_is_xxx by using platform_device_id

2012-09-20 Thread Shawn Guo
It changes the driver to use platform_device_id rather than cpu_is_xxx to determine the controller type, and updates the platform code accordingly. As the result, mach/hardware.h inclusion gets removed from the driver. Signed-off-by: Shawn Guo shawn@linaro.org Acked-by: Sascha Hauer s.ha

Re: [PATCH v2 00/34] i.MX multi-platform support

2012-09-20 Thread Shawn Guo
On Thu, Sep 20, 2012 at 07:41:50AM -0400, Mark Brown wrote: It's usually pretty early but Takashi will be on holiday this time so I'm not sure if things might be different (he was going to send the pull request from holiday). I also didn't guarantee that it'll be stable yet, can someone

Re: [PATCH v2 00/34] i.MX multi-platform support

2012-09-20 Thread Shawn Guo
On Thu, Sep 20, 2012 at 08:47:10AM -0400, Mark Brown wrote: On Thu, Sep 20, 2012 at 07:52:15PM +0800, Shawn Guo wrote: On Thu, Sep 20, 2012 at 07:41:50AM -0400, Mark Brown wrote: It's usually pretty early but Takashi will be on holiday this time so I'm not sure if things might

Re: [PATCH v2 11/34] media: mx1_camera: remove the driver

2012-09-20 Thread Shawn Guo
On Thu, Sep 20, 2012 at 09:56:34AM -0300, Mauro Carvalho Chehab wrote: It is better to mark it as BROKEN for the next Kernel, and then to move it to staging, before dropping a broken driver. That gives people some time to fix it, if someone has interests on fixing the issues. Ok. The driver

Re: [PATCH v2 00/34] i.MX multi-platform support

2012-09-20 Thread Shawn Guo
On Thu, Sep 20, 2012 at 07:39:34AM +, Arnd Bergmann wrote: The first five branches are scheduled to go through the arm-soc tree, so I'm fine with that. For the sound/for-3.7 branch, I'd like to know when to expect that hitting mainline. If it always gets in very early during the merge

[PATCH] media: mx1_camera: mark the driver BROKEN

2012-09-20 Thread Shawn Guo
. [1] https://lkml.org/lkml/2012/2/9/171 Signed-off-by: Shawn Guo shawn@linaro.org --- drivers/media/platform/soc_camera/Kconfig |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/soc_camera/Kconfig b/drivers/media/platform/soc_camera/Kconfig index 9afe1e7..cb6791e

Re: [alsa-devel] [PATCH v2 00/34] i.MX multi-platform support

2012-09-21 Thread Shawn Guo
On Thu, Sep 20, 2012 at 03:56:56PM +, Arnd Bergmann wrote: Ok, fair enough. I think we can put it in arm-soc/for-next as a staging branch anyway to give it some exposure to linux-next, and then we can decide whether a rebase is necessary before sending it to Linus. I just saw the

Re: [alsa-devel] [PATCH v2 00/34] i.MX multi-platform support

2012-09-21 Thread Shawn Guo
Hi Olof, On Fri, Sep 21, 2012 at 01:26:43AM -0700, Olof Johansson wrote: I'll take a look at merging it tomorrow after I've dealt with smp_ops; if it looks reasonably conflict-free I'll pull it in. We need the sound dependency sorted out (or agreed upon) first though. I just published the

Re: [alsa-devel] [PATCH v2 00/34] i.MX multi-platform support

2012-09-21 Thread Shawn Guo
On Sat, Sep 22, 2012 at 12:46:26AM +0800, Shawn Guo wrote: I just published the branch below with this series rebased on top of the necessary dependant branches. git://git.linaro.org/people/shawnguo/linux-2.6.git staging/imx-multiplatform The dependant branches include: Forgot

Re: [alsa-devel] [PATCH v2 00/34] i.MX multi-platform support

2012-09-22 Thread Shawn Guo
On Sat, Sep 22, 2012 at 01:09:27AM -0700, Olof Johansson wrote: I've pulled this in now as staging/imx-multiplatform. As you mention, it might or might not make sense to send this up. It also accrued a few more merge conflicts with other branches in arm-soc, so we'll see how things play

Re: [PATCH 28/34] media: mx2_camera: remove mach/hardware.h inclusion

2012-10-06 Thread Shawn Guo
On Thu, Sep 27, 2012 at 04:36:56AM -0300, Mauro Carvalho Chehab wrote: I'm understanding that this patch will flow through arm tree[1]. So: Yes, it will go through arm-soc tree for 3.8. Acked-by: Mauro Carvalho Chehab mche...@redhat.com Thanks, Mauro. Shawn -- To unsubscribe from this list:

Re: [PATCH v2 27/34] media: mx2_camera: remove cpu_is_xxx by using platform_device_id

2012-10-06 Thread Shawn Guo
On Thu, Sep 27, 2012 at 04:03:21PM -0300, Mauro Carvalho Chehab wrote: It seems that it depends on some stuff that got merged via the arm tree. Not sure what would the better way to handle that, as applying it via -arm will likely generate conflicts when merging from both trees upstream.

Re: [PATCHv7][ 1/7] [media] v4l2: add new V4L2_PIX_FMT_RGB666 pixel format.

2014-02-23 Thread Shawn Guo
On Fri, Feb 21, 2014 at 02:59:58PM +0100, Denis Carikli wrote: That new macro is needed by the imx_drm staging driver for supporting the QVGA display of the eukrea-cpuimx51 board. Cc: Eric Bénard e...@eukrea.com CC: Troy Kisky troy.ki...@boundarydevices.com Cc: linux-media@vger.kernel.org

Re: [PATCH v12][ 06/12] ARM: dts: imx5*, imx6*: correct display-timings nodes.

2014-04-08 Thread Shawn Guo
On Mon, Apr 07, 2014 at 02:44:45PM +0200, Denis Carikli wrote: The imx-drm driver can't use the de-active and pixelclk-active display-timings properties yet. Instead the data-enable and the pixel data clock polarity are hardcoded in the imx-drm driver. So theses properties are now set to

Re: [PATCH v12][ 12/12] ARM: imx_v6_v7_defconfig: Add more drm drivers.

2014-04-08 Thread Shawn Guo
On Mon, Apr 07, 2014 at 02:44:51PM +0200, Denis Carikli wrote: The DRM_PANEL_SIMPLE is needed by the eukrea mbimxsd51's displays. Signed-off-by: Denis Carikli de...@eukrea.com Applied, thanks. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message

Re: [PATCH] ARM: i.MX: Remove excess symbols ARCH_MX1, ARCH_MX25 and MACH_MX27

2014-05-12 Thread Shawn Guo
On Sun, May 11, 2014 at 12:50:06PM +0400, Alexander Shiyan wrote: This patch removes excess symbols ARCH_MX1, ARCH_MX25 and MACH_MX27. Instead we use SOC_IMX*. Signed-off-by: Alexander Shiyan shc_w...@mail.ru --- arch/arm/mach-imx/Kconfig | 12

Re: [PATCH] media: mx1_camera: Remove driver

2014-05-12 Thread Shawn Guo
On Sun, May 11, 2014 at 10:09:11AM +0400, Alexander Shiyan wrote: That driver hasn't been really maintained for a long time. It doesn't compile in any way, it includes non-existent headers, has no users, and marked as broken more than year. Due to these factors, mx1_camera is now removed from

Re: [PATCH 3/3] media: mx2-emmaprp: Add devicetree support

2014-05-14 Thread Shawn Guo
On Wed, May 14, 2014 at 08:59:54PM +0400, Alexander Shiyan wrote: Wed, 14 May 2014 12:49:03 +0200 от Sylwester Nawrocki s.nawro...@samsung.com: On 13/05/14 19:23, Alexander Shiyan wrote: Tue, 13 May 2014 19:09:30 +0200 от Sylwester Nawrocki s.nawro...@samsung.com: Hi, On

Re: [PATCH 2/2] media: mx2_camera: Change Kconfig dependency

2014-05-25 Thread Shawn Guo
On Sat, May 24, 2014 at 08:56:23AM +0400, Alexander Shiyan wrote: This patch change MACH_MX27 dependency to SOC_IMX27 for MX2 camera driver, since MACH_MX27 symbol is scheduled for removal. Signed-off-by: Alexander Shiyan shc_w...@mail.ru Acked-by: Shawn Guo shawn@freescale.com

Re: [PATCH v14 05/10] ARM: dts: imx5*, imx6*: correct display-timings nodes.

2014-06-24 Thread Shawn Guo
On Tue, Jun 24, 2014 at 04:01:58PM +0100, Russell King - ARM Linux wrote: On Mon, Jun 16, 2014 at 12:11:19PM +0200, Denis Carikli wrote: The imx-drm driver can't use the de-active and pixelclk-active display-timings properties yet. Instead the data-enable and the pixel data clock

Re: [PATCH v3 06/32] [media] coda: Add encoder/decoder support for CODA960

2014-07-29 Thread Shawn Guo
Hi Philipp, On Tue, Jul 22, 2014 at 02:50:33PM +0200, Philipp Zabel wrote: The firmware-imx packages referenced in the Freescale meta-fsl-arm repository on github.com contain VPU firmware files. Their use is restricted by an EULA. For example:

Re: [PATCH v3 06/32] [media] coda: Add encoder/decoder support for CODA960

2014-07-30 Thread Shawn Guo
On Tue, Jul 29, 2014 at 07:06:25PM +0200, Philipp Zabel wrote: I followed the step to generate the firmware v4l-coda960-imx6q, and tested it on next-20140725 with patch 'ARM: dts: imx6qdl: Enable CODA960 VPU' applied on top of it. But I got the error of 'Wrong firmwarel' as below. [

Re: [PATCH 01/28] ARM: dts: imx6qdl: Add ipu aliases

2014-08-01 Thread Shawn Guo
On Thu, Jul 31, 2014 at 05:28:06PM +0200, Philipp Zabel wrote: Am Mittwoch, den 25.06.2014, 18:05 -0700 schrieb Steve Longerbeam: Add ipu0 (and ipu1 for quad) aliases to ipu1/ipu2 nodes respectively. Signed-off-by: Steve Longerbeam steve_longerb...@mentor.com Acked-by: Philipp Zabel

Re: [PATCH v2] [media] move media platform data to linux/platform_data/media

2015-11-23 Thread Shawn Guo
$i | \\"; > ( > cd include/$MAIN_DIR; > for j in $DIRS; do > for i in $(ls $j); do > echo " perl -ne > 's,include/$PREV_DIR($i)\b,include/$MAIN_DIR$j\1,; print \$_' |\\"; >

Re: [PATCH 2/2] rc: add zx-irdec remote control driver

2017-07-30 Thread Shawn Guo
Hi Sean, On Sat, Jul 29, 2017 at 10:51:13AM +0100, Sean Young wrote: > Hi Shawn, > > The driver looks great! Just a minor point, see below. Thanks for taking time to look at the patch. I appreciate your review comments, and will post v2 that addressed them shortly. Thanks. Shawn

[PATCH v2 1/3] rc: ir-nec-decoder: move scancode composing code into a shared function

2017-07-30 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> The NEC scancode composing and protocol type detection in ir_nec_decode() is generic enough to be a shared function. Let's create an inline function in rc-core.h, so that other remote control drivers can reuse this function to save some code. Sign

[PATCH v2 2/3] dt-bindings: add bindings document for zx-irdec

2017-07-30 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> It adds the dt-bindings document for ZTE ZX IRDEC remote control block. Signed-off-by: Shawn Guo <shawn@linaro.org> --- Documentation/devicetree/bindings/media/zx-irdec.txt | 14 ++ 1 file changed, 14 insertions(+) create

[PATCH v2 3/3] rc: add zx-irdec remote control driver

2017-07-30 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> It adds the remote control driver and corresponding keymap file for IRDEC block found on ZTE ZX family SoCs. Signed-off-by: Shawn Guo <shawn@linaro.org> --- drivers/media/rc/Kconfig | 11 ++ drivers/media/rc/Makefile

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

2017-07-30 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> The series adds dt-bindings and remote control driver for IRDEC block found on ZTE ZX family SoCs. Changes for v2: - Add one patch to move generic NEC scancode composing and protocol type detection code from ir_nec_decode() into an inline

[PATCH 0/2] Add ZTE zx-irdec remote control driver

2017-07-29 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> The series adds dt-bindings and remote control driver for IRDEC block found on ZTE ZX family SoCs. Shawn Guo (2): dt-bindings: add bindings document for zx-irdec rc: add zx-irdec remote control driver .../devicetree/bindings/media/zx-ird

[PATCH 2/2] rc: add zx-irdec remote control driver

2017-07-29 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> It adds the remote control driver and corresponding keymap file for IRDEC block found on ZTE ZX family SoCs. Signed-off-by: Shawn Guo <shawn@linaro.org> --- drivers/media/rc/Kconfig | 11 ++ drivers/media/rc/Makefile

[PATCH 1/2] dt-bindings: add bindings document for zx-irdec

2017-07-29 Thread Shawn Guo
From: Shawn Guo <shawn@linaro.org> It adds the dt-bindings document for ZTE ZX IRDEC remote control block. Signed-off-by: Shawn Guo <shawn@linaro.org> --- Documentation/devicetree/bindings/media/zx-irdec.txt | 14 ++ 1 file changed, 14 insertions(+) create

Re: [PATCH v2 3/3] rc: add zx-irdec remote control driver

2017-08-09 Thread Shawn Guo
Hi Sean, On Wed, Aug 09, 2017 at 02:00:29PM +0100, Sean Young wrote: > On Sun, Jul 30, 2017 at 09:23:13PM +0800, Shawn Guo wrote: > > From: Shawn Guo <shawn@linaro.org> > > > > It adds the remote control driver and corresponding keymap file for > > IRDEC

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

2017-08-18 Thread Shawn Guo
On Sun, Jul 30, 2017 at 09:23:10PM +0800, Shawn Guo wrote: > From: Shawn Guo <shawn@linaro.org> > > The series adds dt-bindings and remote control driver for IRDEC block > found on ZTE ZX family SoCs. > > Changes for v2: > - Add one patch to move generic NEC scan

Re: [PATCH v4 4/5] ARM: dts: imx: Add TDA19971 HDMI Receiver to GW54xx

2017-11-29 Thread Shawn Guo
On Wed, Nov 29, 2017 at 01:19:56PM -0800, Tim Harvey wrote: > The GW54xx has a front-panel microHDMI connector routed to a TDA19971 > which is connected the the IPU CSI when using IMX6Q. > > Signed-off-by: Tim Harvey > --- > v2: > - add HDMI audio input support > --- >

Re: [PATCH v2 03/15] clk: imx7d: fix mipi dphy div parent

2018-05-02 Thread Shawn Guo
Anson, Please have a look at this change. Shawn On Mon, Apr 23, 2018 at 02:47:38PM +0100, Rui Miguel Silva wrote: > Fix the mipi dphy root divider to mipi_dphy_pre_div, this would remove a > orphan > clock and set the correct parent. > > before: > cat clk_orphan_summary >

Re: [PATCH v2 04/15] clk: imx7d: reset parent for mipi csi root

2018-05-02 Thread Shawn Guo
over 80 columns warning to make the file easier for read. So I would suggest you keep it on a single line to stay consistent with other clk_set_parent() calls. Other than that, Acked-by: Shawn Guo <shawn...@kernel.org> > /* use old gpt clk setting, gpt1 root clk must

Re: [PATCH v2 07/15] ARM: dts: increase default cma size to 40MB

2018-05-02 Thread Shawn Guo
On Mon, Apr 23, 2018 at 02:47:42PM +0100, Rui Miguel Silva wrote: > To support camera in i.MX7 the cma heap is used to allocate frame buffers. The > default size of CMA is 16MB which is not enough for higher resolutions (ex: > 1600x1200). > > So, increase the default CMA size to 40MB. > >

Re: [PATCH v2 10/15] ARM: dts: imx7s: add multiplexer controls

2018-05-02 Thread Shawn Guo
On Mon, Apr 23, 2018 at 02:47:45PM +0100, Rui Miguel Silva wrote: > The IOMUXC General Purpose Register has bitfield to control video bus > multiplexer to control the CSI input between the MIPI-CSI2 and parallel > interface. Add that register and mask. > > Signed-off-by: Rui Miguel Silva

[PATCH] dma-buf/sw_sync: fix document of sw_sync_create_fence_data

2018-01-14 Thread Shawn Guo
The structure should really be sw_sync_create_fence_data rather than sw_sync_ioctl_create_fence which is the function name. Signed-off-by: Shawn Guo <shawn@linaro.org> --- drivers/dma-buf/sw_sync.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/d

Re: [PATCH v13 7/8] ARM: dts: imx: Add TDA19971 HDMI Receiver to GW54xx

2018-02-21 Thread Shawn Guo
On Thu, Feb 15, 2018 at 01:19:17PM -0800, Tim Harvey wrote: > On Thu, Feb 15, 2018 at 10:36 AM, Hans Verkuil wrote: > > On 15/02/18 18:55, Tim Harvey wrote: > >> The GW54xx has a front-panel microHDMI connector routed to a TDA19971 > >> which is connected the the IPU CSI when

Re: [PATCH 2/3] ARM: dts: imx6ull: add pxp support

2018-08-27 Thread Shawn Guo
On Fri, Aug 10, 2018 at 05:18:21PM +0200, Philipp Zabel wrote: > Add the device node for the i.MX6ULL Pixel Pipeline (PXP). > > Signed-off-by: Philipp Zabel > --- > arch/arm/boot/dts/imx6ul.dtsi | 8 > arch/arm/boot/dts/imx6ull.dtsi | 6 ++ > 2 files changed, 14 insertions(+) > >