Re: [PATCH 1/2] drm/exynos: remove legacy -suspend()/resume()

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 12:50:44PM -0300, Gustavo Padovan wrote: Hi, What about this patch? We need it to avoid the WARN_ON added by patch 2/2 that was already picked up by Daniel. That patch is only for 4.4, so not too time critical to get the exynos one in. But might be good to get it into

Re: [PATCH] ARM: exynos_defconfig: Enable DWC2 USB driver and USB ethernet gadget

2015-08-26 Thread Marek Szyprowski
Hello, On 2015-08-26 05:43, Krzysztof Kozlowski wrote: On 26.08.2015 10:58, Tyler Baker wrote: Hi Krzysztof, On 25 August 2015 at 18:44, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: On 26.08.2015 06:00, Tyler Baker wrote: Hi Marek/Krzysztof, On 18 August 2015 at 02:36, Krzysztof

Re: [PATCH] ARM: exynos_defconfig: Enable DWC2 USB driver and USB ethernet gadget

2015-08-26 Thread Krzysztof Kozlowski
On 26.08.2015 15:03, Marek Szyprowski wrote: Hello, On 2015-08-26 05:43, Krzysztof Kozlowski wrote: On 26.08.2015 10:58, Tyler Baker wrote: Hi Krzysztof, On 25 August 2015 at 18:44, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: On 26.08.2015 06:00, Tyler Baker wrote: Hi

[PATCH v2 10/11] drm/exynos: remove wait queue for pending page flip

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Exynos atomic commit procedures already does this job of waiting for pending updates to finish, that means using pending_flip_queue is pointless now because the disable CRTC procedure will never happen during a page_flip. Signed-off-by:

[PATCH v2 11/11] drm/exynos: Enable atomic modesetting feature

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Now that atomic modesetting is implemented for exynos enable the DRIVER_ATOMIC flag on the driver's features. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 3 ++- 1 file changed,

Re: [PATCH 1/2] drm/exynos: remove legacy -suspend()/resume()

2015-08-26 Thread Gustavo Padovan
Hi, What about this patch? We need it to avoid the WARN_ON added by patch 2/2 that was already picked up by Daniel. Gustavo 2015-08-13 Gustavo Padovan gust...@padovan.org: From: Gustavo Padovan gustavo.pado...@collabora.co.uk These legacy helpers should only be used by

Re: [PATCH 03/11] drm/exynos: add prepare and cleanup phases for planes

2015-08-26 Thread Gustavo Padovan
Hi Inki, 2015-08-24 Inki Dae inki@samsung.com: On 2015년 08월 16일 01:26, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk .prepare_plane() and .cleanup_plane() allows to perform extra operations before and after the update of planes. For FIMD for example

[PATCH v2 09/11] drm/exynos: wait all planes updates to finish

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Add infrastructure to wait for all planes updates to finish by using an atomic_t variable to track how many pending updates we are waiting plus a wait_queue for the wait part. It also changes vblank behaviour and keeps it enabled for all

[PATCH v2 08/11] drm/exynos: add atomic asynchronous commit

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk The atomic modesetting interfaces supports async commits that should be implemented by the drivers. If drm core requests an async commit exynos_atomic_commit() will now schedule a work task to run the update later. It also serializes commits

[PATCH v2 02/11] drm/exynos: fimd: unify call to exynos_drm_crtc_finish_pageflip()

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Unify handling of finished plane update to prepare for a following patch that will check for the START and START_S regs to really make sure that the plane was updated. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk ---

[PATCH v2 04/11] drm/exynos: fimd: move window protect code to atomic_begin/flush

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Only set/clear the update bit in the CRTC's .atomic_begin()/flush() so all planes are really committed at the same time. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- v2: rename prepare_plane/cleanup_plane to

[PATCH v2 03/11] drm/exynos: add begin and flush phases for planes

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk .atomic_begin() and .atomic_flush() allows to perform extra operations before and after the update of planes. For FIMD for example this will be used to enable disable the shadow protection bit. Signed-off-by: Gustavo Padovan

[PATCH v2 00/11] drm/exynos: improve atomic modesetting

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Hi, This patchset adds a couple of changes to improve atomic modesetting: * add check for the START shadow register for FIMD to only finish the update when the screen was actually updated. * add asynchronous atomic commit, so now page

[PATCH v2 06/11] drm/exynos: add macro to get the address of START_S reg

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk This macro is need to get the value of the START shadow register, that will tell if an framebuffer is currently displayed on the screen or not. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- include/video/samsung_fimd.h |

[PATCH] drm/exynos: fimd: move window protect code to prepare/atomic_flush

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Only set/clear the update bit in the CRTC's .atomic_begin()/flush() so all planes are really committed at the same time. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 57

[PATCH v2 05/11] drm/exynos: check for pending fb before finish update

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk The current code was ignoring the end of update for all overlay planes, caring only for the primary plane update in case of pageflip. This change adds a change to start to check for pending updates for all planes through

[PATCH v2 01/11] drm/exynos: don't track enabled state at exynos_crtc

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk struct drm_crtc already stores the enabled state of the crtc thus we don't need to replicate enabled in exynos_drm_crtc. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 16

Re: [PATCH V2] Watchdog: Fix parent of watchdog_devices

2015-08-26 Thread Linus Walleij
On Tue, Aug 18, 2015 at 6:34 PM, Pratyush Anand pan...@redhat.com wrote: /sys/class/watchdog/watchdogn/device/modalias can help to identify the driver/module for a given watchdog node. However, many wdt devices does not set its parent and so, we do not see an entry for device in sysfs for

Re: [PATCH v3] irqchip: kill off set_irq_flags usage

2015-08-26 Thread Rob Herring
On Mon, Jul 27, 2015 at 3:55 PM, Rob Herring r...@kernel.org wrote: set_irq_flags is ARM specific with custom flags which have genirq equivalents. Convert drivers to use the genirq interfaces directly, so we can kill off set_irq_flags. The translation of flags is as follows: IRQF_VALID -

Re: [PATCH v7 11/44] [media] media: use entity.graph_obj.mdev instead of .parent

2015-08-26 Thread Mauro Carvalho Chehab
Em Tue, 25 Aug 2015 13:25:15 -0600 Shuah Khan shuahk...@gmail.com escreveu: On Tue, Aug 25, 2015 at 12:36 AM, Hans Verkuil hverk...@xs4all.nl wrote: On 08/23/2015 10:17 PM, Mauro Carvalho Chehab wrote: From: Javier Martinez Canillas jav...@osg.samsung.com The struct media_entity has a

Re: [PATCH v7 10/44] [media] media: rename the function that create pad links

2015-08-26 Thread Mauro Carvalho Chehab
Em Tue, 25 Aug 2015 12:55:41 -0600 Shuah Khan shua...@osg.samsung.com escreveu: On 08/23/2015 02:17 PM, Mauro Carvalho Chehab wrote: Now that a link can be either between two different graph objects, we'll need to add more functions to create links. Is this an incomplete sentence. Should