[PATCH v4 7/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3

[PATCH v4 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Philipp Zabel
In preparation for a following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH v4 2/8] imx-drm: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Philipp Zabel
In preparation for the following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH v4 4/8] of: Add for_each_endpoint_of_node helper macro

2014-09-29 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- include/linux/of_graph.h | 11 +++ 1

[PATCH v4 5/8] of: Add of_graph_get_port_by_id function

2014-09-29 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/of/base.c| 26 ++ include/linux/of_graph.h | 7 +++ 2 files changed, 33 insertions(+) diff

[PATCH v4 0/8] Add of-graph helpers to loop over endpoints and find ports by id

2014-09-29 Thread Philipp Zabel
696 regards Philipp Philipp Zabel (8): [media] soc_camera: Do not decrement endpoint node refcount in the loop imx-drm: Do not decrement endpoint node refcount in the loop of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint of: Add for_each_endpoint_of_node he

Re: [PATCH v3 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-25 Thread Philipp Zabel
Hi Guennadi, Am Samstag, den 20.09.2014, 09:24 +0200 schrieb Guennadi Liakhovetski: > Hi Philippe, > > On Thu, 11 Sep 2014, Philipp Zabel wrote: > > > In preparation for a following patch, stop decrementing the endpoint node > > refcount in the loop. This temporarily

[PATCH v3 0/8] Add of-graph helpers to loop over endpoints and find ports by id

2014-09-11 Thread Philipp Zabel
macro instead of of_graph_get_next_endpoint. Changes since v2: - Fixed and simplified of_graph_get_port_by_id function - More acks The previous version can be found here: https://lkml.org/lkml/2014/9/10/252 regards Philipp Philipp Zabel (8): [media] soc_camera: Do not decrement endpoint

[PATCH v3 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-11 Thread Philipp Zabel
In preparation for a following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH v3 3/8] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-09-11 Thread Philipp Zabel
prev argument's refcount themselves. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/of/base.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/of/base.c b/drivers/of/base.c index d8574ad..a49b5628 100644 --- a/drivers/of/base.c +++ b/dr

[PATCH v3 8/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_parse_of

2014-09-11 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/staging/imx-drm/imx-drm-core.c b/drivers

[PATCH v3 7/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-09-11 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3

[PATCH v3 6/8] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-09-11 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm

[PATCH v3 2/8] imx-drm: Do not decrement endpoint node refcount in the loop

2014-09-11 Thread Philipp Zabel
In preparation for the following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH v3 5/8] of: Add of_graph_get_port_by_id function

2014-09-11 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel --- Changes since v2: - Fixed and simplified of_graph_get_port_by_id function --- drivers/of/base.c| 26 ++ include/linux/of_graph.h | 7

[PATCH v3 4/8] of: Add for_each_endpoint_of_node helper macro

2014-09-11 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- include/linux/of_graph.h | 11 +++ 1

Re: [GIT PULL] IPUv3 CPMEM API change across staging and drm

2014-09-11 Thread Philipp Zabel
Am Mittwoch, den 10.09.2014, 11:32 -0700 schrieb Greg Kroah-Hartman: > On Wed, Sep 10, 2014 at 11:35:29AM +0200, Philipp Zabel wrote: > > Hi Greg, > > > > I have asked Dave to pull Steve Longerbeam's preparation series for > > capture support. It starts off wit

[PATCH v2 2/8] imx-drm: Do not decrement endpoint node refcount in the loop

2014-09-10 Thread Philipp Zabel
In preparation for the following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH v2 6/8] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-09-10 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm

[PATCH v2 4/8] of: Add for_each_endpoint_of_node helper macro

2014-09-10 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel --- Changes since v1: - Added a comment about the child node reference count

[PATCH v2 8/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_parse_of

2014-09-10 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/staging/imx-drm/imx-drm-core.c b/drivers

[PATCH v2 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-10 Thread Philipp Zabel
In preparation for a following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH v2 0/8] Add of-graph helpers to loop over endpoints and find ports by id

2014-09-10 Thread Philipp Zabel
Laurent, making use of port id defaulting to 0 if no "reg" property is given. - Added Laurent's ack to drm_of_find_possible_crtcs patch The previous version can be found here: https://lkml.org/lkml/2014/8/19/280 regards Philipp Philipp Zabel (8): [media] soc_camera: Do not dec

[PATCH v2 5/8] of: Add of_graph_get_port_by_id function

2014-09-10 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel --- Changes since v1: - Fixed whitespace in comment - Changed id parameter to of_graph_get_port_by_id to u32 - Simplified of_graph_get_port_by_id as suggested by Laurent

[PATCH v2 7/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-09-10 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3

[PATCH v2 3/8] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-09-10 Thread Philipp Zabel
prev argument's refcount themselves. Signed-off-by: Philipp Zabel --- drivers/of/base.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/of/base.c b/drivers/of/base.c index d8574ad..a49b5628 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -2058,8 +2

[GIT PULL] IPUv3 CPMEM API change across staging and drm

2014-09-10 Thread Philipp Zabel
Hi Greg, I have asked Dave to pull Steve Longerbeam's preparation series for capture support. It starts off with the following two commits that also touch the staging driver. Could you pull this into the staging tree to avoid conflicts later on? The following changes since commit 7d1311b93e58ed55

[PATCH v2 2/4] imx-drm: ipuv3-plane: move stride setting out of base setup

2014-09-09 Thread Philipp Zabel
Setting the stride can only be done on inactive channels, while the buffer base address can also be updated for running channels using the hardware double buffering feature. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH v2 3/4] imx-drm: ipuv3-plane: enable double buffering

2014-09-09 Thread Philipp Zabel
This allows to update the buffer base address while the DMA channel is running. It is needed to flip the frame buffer of an active plane. Signed-off-by: Philipp Zabel --- Change since v1: - Added missing ipu_idmac_select_buffer call to make the hardware double buffering actually consider the

[PATCH v2 1/4] imx-drm: ipuv3-plane: allow local alpha in ipu_plane_mode_set()

2014-09-09 Thread Philipp Zabel
For the overlay plane scanning out a framebuffer with an alpha component, enable the DP local alpha feature on the partial plane. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH v2 4/4] imx-drm: ipuv3-plane: fix plane updates for active planes

2014-09-09 Thread Philipp Zabel
While the DMA channel is running, it is not allowed to change anything but the inactive (double) buffer base address, so resizing a plane or changing to a frame buffer with different pixel format is not possible. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 15

Re: [PATCH 1/4] imx-drm: ipuv3-plane: allow local alpha in ipu_plane_mode_set()

2014-09-09 Thread Philipp Zabel
Am Montag, den 08.09.2014, 12:50 -0700 schrieb Greg Kroah-Hartman: > On Tue, Jul 29, 2014 at 11:57:06AM +0200, Philipp Zabel wrote: > > For the overlay plane scanning out a framebuffer with an alpha component, > > enable the DP local alpha feature on the partial plane. > &g

Re: [PATCH 5/8] of: Add of_graph_get_port_by_id function

2014-08-22 Thread Philipp Zabel
Hi Laurent, Thank you for the comments. Am Mittwoch, den 20.08.2014, 22:13 +0200 schrieb Laurent Pinchart: [...] > > + struct device_node *port = NULL; > > + int port_id; > > + > > + while (true) { > > + port = of_get_next_child(node, port); > > + if (!port) > > +

Re: [PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Philipp Zabel
>plane[0]; > + > ipu_crtc->newfb = NULL; > - ipu_plane_set_base(ipu_crtc->plane[0], > ipu_crtc->base.primary->fb, > - ipu_crtc->plane[0]->x, ipu_crtc->plane[0]->y); > + ipu_plane_set_base(plane, ipu_crtc->ba

Re: [PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

2014-08-19 Thread Philipp Zabel
Am Dienstag, den 19.08.2014, 16:59 +0300 schrieb Yannis Damigos: > This is a patch to the ipuv3-crtc.c file that fixes up two "lines > over 80 characters" warnings found by the checkpatch.pl tool. > > Signed-off-by: Yannis Damigos > --- > drivers/staging/imx-drm/ipuv3-crtc.c | 9 + > 1 f

[PATCH 8/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_parse_of

2014-08-19 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/staging/imx-drm/imx-drm-core.c b/drivers

[PATCH 7/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-08-19 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3

[PATCH 2/8] imx-drm: Do not decrement endpoint node refcount in the loop

2014-08-19 Thread Philipp Zabel
In preparation for the following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH 5/8] of: Add of_graph_get_port_by_id function

2014-08-19 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel --- drivers/of/base.c| 30 ++ include/linux/of_graph.h | 7 +++ 2 files changed, 37 insertions(+) diff --git a/drivers/of/base.c b

[PATCH 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-08-19 Thread Philipp Zabel
In preparation for a following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel

[PATCH 6/8] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-08-19 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.c index 16150a0..024fa77

[PATCH 3/8] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-08-19 Thread Philipp Zabel
prev argument's refcount themselves. Signed-off-by: Philipp Zabel --- drivers/of/base.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/of/base.c b/drivers/of/base.c index d8574ad..a49b5628 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -2058,8 +2

[PATCH 4/8] of: Add for_each_endpoint_of_node helper macro

2014-08-19 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel --- include/linux/of_graph.h | 4 1 file changed, 4 insertions(+) diff

[PATCH 0/5] Add of-graph helpers to loop over endpoints and find ports by id

2014-08-19 Thread Philipp Zabel
macro instead of of_graph_get_next_endpoint. regards Philipp Philipp Zabel (8): [media] soc_camera: Do not decrement endpoint node refcount in the loop imx-drm: Do not decrement endpoint node refcount in the loop of: Decrement refcount of previous endpoint in

[PATCH 1/4] imx-drm: ipuv3-plane: allow local alpha in ipu_plane_mode_set()

2014-07-29 Thread Philipp Zabel
For the overlay plane scanning out a framebuffer with an alpha component, enable the DP local alpha feature on the partial plane. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 3/4] imx-drm: ipuv3-plane: enable double buffering

2014-07-29 Thread Philipp Zabel
This allows to update the buffer base address while the DMA channel is running. It is needed to flip the frame buffer of an active plane. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a

[PATCH 4/4] imx-drm: ipuv3-plane: fix plane updates for active planes

2014-07-29 Thread Philipp Zabel
While the DMA channel is running, it is not allowed to change anything but the inactive (double) buffer base address, so resizing a plane or changing to a frame buffer with different pixel format is not possible. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 15

[PATCH 2/4] imx-drm: ipuv3-plane: move stride setting out of base setup

2014-07-29 Thread Philipp Zabel
Setting the stride can only be done on inactive channels, while the buffer base address can also be updated for running channels using the hardware double buffering feature. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-plane.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH 2/5] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-07-04 Thread Philipp Zabel
Decreasing the reference count of the previous endpoint node allows to use the of_graph_get_next_endpoint function in a for_each_... style macro. Signed-off-by: Philipp Zabel --- drivers/of/base.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/of/base.c b

[PATCH 4/5] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-07-04 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.c index 46d9678..1b315b6

[PATCH 1/5] imx-drm: Drop imx_drm_of_get_next_endpoint wrapper

2014-07-04 Thread Philipp Zabel
We will decrease the prev node reference count in of_graph_get_next_endpoint instead. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/staging/imx-drm/imx-drm-core.c b/drivers/staging

[PATCH 3/5] of: Add for_each_endpoint_of_node helper macro

2014-07-04 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel --- include/linux/of_graph.h | 4 1 file changed, 4 insertions(+) diff

[PATCH 0/5] Iterate of_graph endpoints using a for_each_... style macro

2014-07-04 Thread Philipp Zabel
builds upon Russell's patches: "drm: add of_graph endpoint helper to find possible CRTCs" and "imx-drm: convert imx-drm to use the generic DRM OF helper" regards Philipp Philipp Zabel (5): imx-drm: Drop imx_drm_of_get_next_endpoint wrapper of: Decrement refcoun

[PATCH 5/5] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-07-04 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3

Re: [PATCH RFC] imx-drm: convert imx-drm to use the generic DRM OF helper

2014-07-04 Thread Philipp Zabel
7;s a reference counting issue in the patch mentioned above, but I'd like to fix this in of_graph_get_next_endpoint instead. For both patches, Acked-by: Philipp Zabel regards Philipp ___ devel mailing list de...@linuxdriverproject.org http://d

Re: [PATCH RFC v2 3/8] component: add support for component match array

2014-06-26 Thread Philipp Zabel
Hi Russell, On Tue, Jun 24, 2014 at 9:29 PM, Russell King wrote: [...] > +/* > + * Add a component to be matched. > + * > + * The match array is first created or extended if necessary. > + */ > +void component_match_add(struct device *dev, struct component_match > **matchptr, > + int (*com

Re: [PATCH] imx-drm: imx-hdmi: fix hdmi hotplug detection initial state

2014-06-11 Thread Philipp Zabel
Hi Russell, Am Mittwoch, den 11.06.2014, 09:17 +0100 schrieb Russell King - ARM Linux: > On Tue, Jun 10, 2014 at 10:32:25AM -0300, Fabio Estevam wrote: > > On Tue, Jun 10, 2014 at 9:58 AM, Fabio Estevam wrote: > > > On Mon, Jun 9, 2014 at 5:09 PM, Russell King - ARM Linux > > > wrote: > > > > >

[PULL] Move IPUv3 core out of staging, add CSI support

2014-06-04 Thread Philipp Zabel
V4L2 support in parallel. I have rebased the series back to v3.15-rc3, the tag closest to the common ancestor of drm-next and staging-next, and verified that merging the two produces the correct result. regards Philipp Philipp Zabel (5

Re: [PATCH v2 1/5] gpu: ipu-v3: Move i.MX IPUv3 core driver out of staging

2014-06-02 Thread Philipp Zabel
Am Mittwoch, den 28.05.2014, 14:13 -0700 schrieb Greg Kroah-Hartman: > On Mon, May 26, 2014 at 04:19:39PM +0200, Philipp Zabel wrote: > > The i.MX Image Processing Unit (IPU) contains a number of image processing > > blocks that sit right in the middle between DRM and V4

Re: [PATCH v2 0/5] Move IPUv3 core out of staging, add CSI support

2014-05-26 Thread Philipp Zabel
On Mon, May 26, 2014 at 4:19 PM, Philipp Zabel wrote: [...] > Cc: drm-de...@vger.kernel.org Sorry about this unfortunate combination of me misremembering dri-devel, noticing something is wrong, looking up and inserting the correct address, and then, after a context switch, failing to remove

[PATCH v2 1/5] gpu: ipu-v3: Move i.MX IPUv3 core driver out of staging

2014-05-26 Thread Philipp Zabel
framework, but others, such as the dma controller (IDMAC) and image converter (IC) can be used by both. The IPUv3 core driver provides an internal API to access the modules, to be used by both DRM and V4L2 IPUv3 drivers. Signed-off-by: Lucas Stach Signed-off-by: Philipp Zabel --- Changes since RFC

[PATCH v2 4/5] gpu: ipu-v3: Add CSI and SMFC module enable wrappers

2014-05-26 Thread Philipp Zabel
IPU_CONF_..._EN bits are implementation details, not to be made public. Add wrappers around ipu_module_enable/disable, so the CSI V4L2 driver can enable/disable the CSI and SMFC modules. Signed-off-by: Philipp Zabel --- drivers/gpu/ipu-v3/ipu-common.c | 24 include

[PATCH v2 3/5] gpu: ipu-v3: Add ipu_idmac_get_current_buffer function

2014-05-26 Thread Philipp Zabel
This function returns the currently active buffer (0 or 1) of a double buffered IDMAC channel. It is to be used by the CSI driver. Signed-off-by: Philipp Zabel --- drivers/gpu/ipu-v3/ipu-common.c | 9 + include/video/imx-ipu-v3.h | 1 + 2 files changed, 10 insertions(+) diff --git

[PATCH v2 0/5] Move IPUv3 core out of staging, add CSI support

2014-05-26 Thread Philipp Zabel
se nearly unchanged. I hope we can get this move in before the approaching merge window so that we have a base for submitting the CSI V4L2 patches in the following round. regards Philipp --- Changes since RFC: - Rebased onto current staging-next - Streamlined destaging patch a bit --- Philipp Zabel (

[PATCH v2 2/5] gpu: ipu-v3: Add SMFC code

2014-05-26 Thread Philipp Zabel
The Sensor Multi Fifo Controller (SMFC) is used as a buffer between the two CSIs (writing simultaneously) and up to four IDMAC channels. Signed-off-by: Philipp Zabel --- drivers/gpu/ipu-v3/Makefile | 2 +- drivers/gpu/ipu-v3/ipu-common.c | 10 + drivers/gpu/ipu-v3/ipu-prv.h| 6

[PATCH v2 5/5] gpu: ipu-v3: Register the CSI modules

2014-05-26 Thread Philipp Zabel
This patch registers the two CSI platform devices per IPU. Signed-off-by: Lucas Stach Signed-off-by: Philipp Zabel --- Changes since RFC: - Rebased onto current staging-next --- drivers/gpu/ipu-v3/ipu-common.c | 37 - include/video/imx-ipu-v3.h | 1

[PATCH] imx-drm: imx-hdmi: move memory and resource allocation into probe function

2014-05-14 Thread Philipp Zabel
the irq only after the interrupt handler mutes are set up, to avoid spurious interrupts. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-hdmi.c | 163 +++-- 1 file changed, 83 insertions(+), 80 deletions(-) diff --git a/drivers/staging/imx-drm/imx

[PATCH v2 1/8] imx-drm: ipu-common: add ipu_map_irq to request non-IDMAC interrupts

2014-04-14 Thread Philipp Zabel
This allows to request the DC related interrupts. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h | 1 + drivers/staging/imx-drm/ipu-v3/ipu-common.c | 19 +-- 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/drivers/staging/imx-drm

[PATCH v2 3/8] imx-drm: ipu-dmfc: Wait for FIFOs to run empty before disabling

2014-04-14 Thread Philipp Zabel
Disabling the DMFC module while there is still data in the FIFOs could cause the "new frame before end of frame" error state when the DMFC is enabled again. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-dmfc.c | 25 +++-- 1 file changed, 23

[PATCH v2 2/8] imx-drm: ipu-common: Add helpers to check for a busy IDMAC channel and to busywait for an interrupt

2014-04-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-common.c | 22 ++ drivers/staging/imx-drm/ipu-v3/ipu-prv.h| 3 +++ 2 files changed, 25 insertions(+) diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-common.c b/drivers/staging/imx-drm/ipu-v3/ipu

[PATCH v2 6/8] imx-drm: imx-dp: When disabling the DP foreground channel, wait until the DP background channel is finished before disabling the IDMAC channel

2014-04-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-dp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-dp.c b/drivers/staging/imx-drm/ipu-v3/ipu-dp.c index 6980fa1..d90f82a 100644 --- a/drivers/staging/imx-drm/ipu-v3/ipu-dp.c +++ b

[PATCH v2 4/8] imx-drm: ipu-dc: Wait for DC_FC_1 / DP_SF_END interrupt

2014-04-14 Thread Philipp Zabel
Wait for the DC Frame Complete or DP Sync Flow End interrupts before disabling DC channels. Signed-off-by: Philipp Zabel --- Changes since v1: - Moved disable_irq() out of dc_irq_handler() --- drivers/staging/imx-drm/ipu-v3/ipu-dc.c | 71 +++-- 1 file changed, 50

[PATCH v2 5/8] imx-drm: ipu-dp: Split disabling the DP foreground channel from disabling the DP module

2014-04-14 Thread Philipp Zabel
The former has to be done before disabling the DMFC, the latter has to be done afterwards. Otherwise the DMFC FIFOs never get cleared properly. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h | 2 + drivers/staging/imx-drm/ipu-v3/ipu-dp.c | 68

[PATCH v2 8/8] imx-drm: ipu-dc: Disable DC module when not in use

2014-04-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h | 2 ++ drivers/staging/imx-drm/ipu-v3/ipu-dc.c | 14 -- drivers/staging/imx-drm/ipuv3-crtc.c| 8 ++-- 3 files changed, 20 insertions(+), 4 deletions(-) diff --git a/drivers/staging/imx

[PATCH v2 0/8] Reorder i.MX IPU display enable/disable sequence

2014-04-14 Thread Philipp Zabel
odule regards Philipp Philipp Zabel (8): imx-drm: ipu-common: add ipu_map_irq to request non-IDMAC interrupts imx-drm: ipu-common: Add helpers to check for a busy IDMAC channel and to busywait for an interrupt imx-drm: ipu-dmfc: Wait for FIFOs to run empty before disabling imx-drm: i

[PATCH v2 7/8] imx-drm: ipuv3-crtc: Change display enable/disable order

2014-04-14 Thread Philipp Zabel
would not correctly start, leading to non-working LVDS displays. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-crtc.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/staging/imx-drm/ipuv3-crtc.c b/drivers/staging/imx-drm/ipuv3-crtc.c index

[PATCH 2/7] imx-drm: ipu-common: Add helpers to check for a busy IDMAC channel and to busywait for an interrupt

2014-04-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-common.c | 22 ++ drivers/staging/imx-drm/ipu-v3/ipu-prv.h| 3 +++ 2 files changed, 25 insertions(+) diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-common.c b/drivers/staging/imx-drm/ipu-v3/ipu

[PATCH 3/7] imx-drm: ipu-dmfc: Wait for FIFOs to run empty before disabling

2014-04-14 Thread Philipp Zabel
Disabling the DMFC module while there is still data in the FIFOs could cause the "new frame before end of frame" error state when the DMFC is enabled again. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-dmfc.c | 25 +++-- 1 file changed, 23

[PATCH 6/7] imx-drm: imx-dp: When disabling the DP foreground channel, wait until the DP background channel is finished before disabling the IDMAC channel

2014-04-14 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-dp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-dp.c b/drivers/staging/imx-drm/ipu-v3/ipu-dp.c index 6980fa1..d90f82a 100644 --- a/drivers/staging/imx-drm/ipu-v3/ipu-dp.c +++ b

[PATCH 1/7] imx-drm: ipu-common: add ipu_map_irq to request non-IDMAC interrupts

2014-04-14 Thread Philipp Zabel
This allows to request the DC related interrupts. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h | 1 + drivers/staging/imx-drm/ipu-v3/ipu-common.c | 19 +-- 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/drivers/staging/imx-drm

[PATCH 0/7] Reorder i.MX IPU display enable/disable sequence

2014-04-14 Thread Philipp Zabel
DP to finish processing the current frame, then stop the DI and IDMAC, and only then disable clocks to the submodules. Also from now on we disable the DC when it is not in use. regards Philipp Philipp Zabel (7): imx-drm: ipu-common: add ipu_map_irq to request non-IDMAC interrupts imx-drm: ipu-c

[PATCH 5/7] imx-drm: ipu-dp: Split disabling the DP foreground channel from disabling the DP module

2014-04-14 Thread Philipp Zabel
The former has to be done before disabling the DMFC, the latter has to be done afterwards. Otherwise the DMFC FIFOs never get cleared properly. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/imx-ipu-v3.h | 2 + drivers/staging/imx-drm/ipu-v3/ipu-dp.c | 68

[PATCH 7/7] imx-drm: ipuv3-crtc: Change display enable/disable order

2014-04-14 Thread Philipp Zabel
would not correctly start, leading to non-working LVDS displays. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipuv3-crtc.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/staging/imx-drm/ipuv3-crtc.c b/drivers/staging/imx-drm/ipuv3-crtc.c index

[PATCH 4/7] imx-drm: ipu-dc: Wait for DC_FC_1 / DP_SF_END interrupt

2014-04-14 Thread Philipp Zabel
Wait for the DC Frame Complete or DP Sync Flow End interrupts before disabling DC channels. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/ipu-v3/ipu-dc.c | 71 +++-- 1 file changed, 50 insertions(+), 21 deletions(-) diff --git a/drivers/staging/imx-drm

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-04-07 Thread Philipp Zabel
Am Montag, den 07.04.2014, 12:05 +0200 schrieb Philipp Zabel: > Hi Shawn, > > Am Montag, den 07.04.2014, 12:23 +0800 schrieb Shawn Guo: > > On Tue, Mar 11, 2014 at 11:46:11AM +0800, Shawn Guo wrote: > > > I just came across a couple problems when testing the series on

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-04-07 Thread Philipp Zabel
Hi Shawn, Am Montag, den 07.04.2014, 12:23 +0800 schrieb Shawn Guo: > On Tue, Mar 11, 2014 at 11:46:11AM +0800, Shawn Guo wrote: > > I just came across a couple problems when testing the series on > > my imx6dl-sabresd board in dual display case - HDMI + LVDS. I tested it > > using Russell's bran

[PATCH] imx-drm: imx-drm-core: Fix imx_drm_encoder_get_mux_id

2014-04-07 Thread Philipp Zabel
The decoder mux id is equal to the port id of the encoder's input port that is connected to the given crtc, not to the endpoint id (which is arbitrary and usually zero). Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 2 +- 1 file changed, 1 insertion(+), 1 del

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-03-11 Thread Philipp Zabel
Hi Shawn, Am Dienstag, den 11.03.2014, 11:46 +0800 schrieb Shawn Guo: > On Wed, Mar 05, 2014 at 10:20:51AM +0100, Philipp Zabel wrote: > > Hi, > > > > this latest version of the imx-drm DT binding patches applies > > on top of staging-next and also depends on the OF

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-03-07 Thread Philipp Zabel
[Added Shawn to Cc:] On Fri, Mar 7, 2014 at 7:28 PM, Greg Kroah-Hartman wrote: > On Fri, Mar 07, 2014 at 05:56:12PM +, Russell King - ARM Linux wrote: >> On Wed, Mar 05, 2014 at 10:20:51AM +0100, Philipp Zabel wrote: >> > Hi, >> > >> > this latest versi

Re: [PATCH 1/2] imx-drm: imx-ldb: Add drm_panel support

2014-03-07 Thread Philipp Zabel
Hi Russell, Am Freitag, den 07.03.2014, 17:22 + schrieb Russell King - ARM Linux: > On Thu, Mar 06, 2014 at 02:54:39PM +0100, Philipp Zabel wrote: > > This patch allows to optionally attach the lvds-channel to a panel > > supported by a drm_panel driver instead of supplyin

Re: [PATCH v9][ 3/8] staging: imx-drm: Correct BGR666 and the board's dts that use them.

2014-03-06 Thread Philipp Zabel
Hi Denis, Am Donnerstag, den 06.03.2014, 17:01 +0100 schrieb Denis Carikli: > The current BGR666 is not consistent with the other color mapings like BGR24. > BGR666 should be in the same byte order than BGR24. > > Signed-off-by: Denis Carikli patches 1 to 3 Acked-by: Philipp Zabel

Re: [PATCH v9][ 6/8] staging: imx-drm: parallel display: add regulator support.

2014-03-06 Thread Philipp Zabel
Hi Denis, Am Donnerstag, den 06.03.2014, 17:04 +0100 schrieb Denis Carikli: > diff --git > a/Documentation/devicetree/bindings/staging/imx-drm/fsl-imx-drm.txt > b/Documentation/devicetree/bindings/staging/imx-drm/fsl-imx-drm.txt > index 2d24425..4dd7ce5 100644 > --- a/Documentation/devicetree/bi

Re: [PATCH v9][ 7/8] ARM: dts: mbimx51sd: Add display support.

2014-03-06 Thread Philipp Zabel
Hi Denis, Am Donnerstag, den 06.03.2014, 17:04 +0100 schrieb Denis Carikli: [...] > +&display { > + status = "okay"; > + display-timings { > + model = "DVI-SVGA"; Does this board really have a DVI connector and no I2C bus routed to the DDC pins? > + bits-per-pixel

Re: [PATCH 2/2] imx-drm: imx-ldb: Use OF graph to find connected panel

2014-03-06 Thread Philipp Zabel
Am Donnerstag, den 06.03.2014, 14:54 +0100 schrieb Philipp Zabel: > This patch depends on the OF graph parsing helper and imx-drm-dt series. > It allows to connect the panel to the output port (port@1 on i.MX53, port@4 > on i.MX6) via the OF graph bindings. If we postpone this until th

[PATCH 2/2] imx-drm: imx-ldb: Use OF graph to find connected panel

2014-03-06 Thread Philipp Zabel
This patch depends on the OF graph parsing helper and imx-drm-dt series. It allows to connect the panel to the output port (port@1 on i.MX53, port@4 on i.MX6) via the OF graph bindings. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-ldb.c | 19 ++- 1 file changed

[PATCH 1/2] imx-drm: imx-ldb: Add drm_panel support

2014-03-06 Thread Philipp Zabel
}; panel: panel { compatible = "edt,etm0700g0dh6", "simple-panel"; }; Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-ldb.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/staging/imx-drm/imx-

Re: [PATCH v5 06/11] imx-drm: imx-tve: Fix DDC I2C bus property

2014-03-06 Thread Philipp Zabel
Hi Russell, Am Donnerstag, den 06.03.2014, 13:03 + schrieb Russell King - ARM Linux: > On Wed, Mar 05, 2014 at 10:20:57AM +0100, Philipp Zabel wrote: > > This patch fixes the TV Encoder DDC I2C bus property to use the common > > 'ddc-i2c-bus' property name instead

Re: [PATCH v5 01/11] staging: imx-drm-core: Use OF graph to find components and connections between encoder and crtcs

2014-03-05 Thread Philipp Zabel
Am Mittwoch, den 05.03.2014, 10:05 + schrieb Russell King - ARM Linux: > On Wed, Mar 05, 2014 at 10:20:52AM +0100, Philipp Zabel wrote: > > +struct imx_drm_component { > > + struct device_node *of_node; > > + struct list_head list; > > +}; > > + > >

[PATCH v2] gpu: ipu-v3: Move i.MX IPUv3 core driver out of staging

2014-03-05 Thread Philipp Zabel
framework, but others, such as the dma controller (IDMAC) and image converter (IC) can be used by both. The IPUv3 core driver provides an internal API to access the modules, to be used by both DRM and V4L2 IPUv3 drivers. Signed-off-by: Lucas Stach Signed-off-by: Philipp Zabel --- This is the same as

[PATCH v5 06/11] imx-drm: imx-tve: Fix DDC I2C bus property

2014-03-05 Thread Philipp Zabel
This patch fixes the TV Encoder DDC I2C bus property to use the common 'ddc-i2c-bus' property name instead of 'ddc'. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-tve.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/

<    1   2   3   4   >