[PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-25 Thread Jose Abreu
: Not even compiled tested Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Daniel Vetter Cc: Liviu Dudau Cc: Brian Starkey Cc: David Airlie --- drivers/gpu/drm/arm/malidp_crtc.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/arm/malidp_crtc.c b

[PATCH v5 02/10] drm: Introduce drm_bridge_mode_valid()

2017-05-25 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu <joab...@synopsys.com> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Ville Syrjälä <ville.sy

[PATCH v5 02/10] drm: Introduce drm_bridge_mode_valid()

2017-05-25 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu Reviewed-by: Daniel Vetter Cc: Carlos Palminha Cc: Ville Syrjälä Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc: Laurent Pinchart --- drivers/gpu/drm

Re: [RFC 2/2] [media] platform: Add Synopsys Designware HDMI RX Controller Driver

2017-05-23 Thread Jose Abreu
Hi Hans, Thanks for the review! On 22-05-2017 11:36, Hans Verkuil wrote: > On 04/21/2017 11:53 AM, Jose Abreu wrote: >> This is an initial submission for the Synopsys Designware HDMI RX >> Controller Driver. This driver interacts with a phy driver so that >> a commu

Re: [RFC 2/2] [media] platform: Add Synopsys Designware HDMI RX Controller Driver

2017-05-23 Thread Jose Abreu
Hi Hans, Thanks for the review! On 22-05-2017 11:36, Hans Verkuil wrote: > On 04/21/2017 11:53 AM, Jose Abreu wrote: >> This is an initial submission for the Synopsys Designware HDMI RX >> Controller Driver. This driver interacts with a phy driver so that >> a commu

Re: [PATCH v4 00/10] Introduce new mode validation callbacks

2017-05-23 Thread Jose Abreu
Hi Daniel, On 22-05-2017 16:31, Daniel Vetter wrote: > On Mon, May 22, 2017 at 09:56:00AM +0200, Daniel Vetter wrote: >> On Fri, May 19, 2017 at 01:52:09AM +0100, Jose Abreu wrote: >>> This series is a follow up from the discussion at [1]. We start by >>> introducing

Re: [PATCH v4 00/10] Introduce new mode validation callbacks

2017-05-23 Thread Jose Abreu
Hi Daniel, On 22-05-2017 16:31, Daniel Vetter wrote: > On Mon, May 22, 2017 at 09:56:00AM +0200, Daniel Vetter wrote: >> On Fri, May 19, 2017 at 01:52:09AM +0100, Jose Abreu wrote: >>> This series is a follow up from the discussion at [1]. We start by >>> introducing

Re: [RFC 1/2] [media] platform: Add Synopsys Designware HDMI RX PHY e405 Driver

2017-05-22 Thread Jose Abreu
Hi Hans, Thanks for the review! On 22-05-2017 11:04, Hans Verkuil wrote: > On 04/21/2017 11:53 AM, Jose Abreu wrote: >> This adds support for the Synopsys Designware HDMI RX PHY e405. This >> phy receives and decodes HDMI video that is delivered to a controller. >> >

Re: [RFC 1/2] [media] platform: Add Synopsys Designware HDMI RX PHY e405 Driver

2017-05-22 Thread Jose Abreu
Hi Hans, Thanks for the review! On 22-05-2017 11:04, Hans Verkuil wrote: > On 04/21/2017 11:53 AM, Jose Abreu wrote: >> This adds support for the Synopsys Designware HDMI RX PHY e405. This >> phy receives and decodes HDMI video that is delivered to a controller. >> >

Re: [PATCH v4 00/10] Introduce new mode validation callbacks

2017-05-22 Thread Jose Abreu
Hi Daniel, On 22-05-2017 08:56, Daniel Vetter wrote: > On Fri, May 19, 2017 at 01:52:09AM +0100, Jose Abreu wrote: >> This series is a follow up from the discussion at [1]. We start by >> introducing crtc->mode_valid(), encoder->mode_valid() and >> bridge-&g

Re: [PATCH v4 00/10] Introduce new mode validation callbacks

2017-05-22 Thread Jose Abreu
Hi Daniel, On 22-05-2017 08:56, Daniel Vetter wrote: > On Fri, May 19, 2017 at 01:52:09AM +0100, Jose Abreu wrote: >> This series is a follow up from the discussion at [1]. We start by >> introducing crtc->mode_valid(), encoder->mode_valid() and >> bridge-&g

Re: [RFC 0/2] Synopsys Designware HDMI Video Capture Controller + PHY

2017-05-22 Thread Jose Abreu
Hi Hans, On 21-04-2017 10:53, Jose Abreu wrote: > Hi All, > > This is a RFC series that is intended to collect comments regarding the > Synopsys Designware HDMI RX controller and Synopsys Designware HDMI RX e405 > PHY > drivers. > > The Synopsys Designware HDMI RX contro

Re: [RFC 0/2] Synopsys Designware HDMI Video Capture Controller + PHY

2017-05-22 Thread Jose Abreu
Hi Hans, On 21-04-2017 10:53, Jose Abreu wrote: > Hi All, > > This is a RFC series that is intended to collect comments regarding the > Synopsys Designware HDMI RX controller and Synopsys Designware HDMI RX e405 > PHY > drivers. > > The Synopsys Designware HDMI RX contro

[PATCH v4 02/10] drm: Introduce drm_bridge_mode_valid()

2017-05-18 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu <joab...@synopsys.com> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abro

[PATCH v4 09/10] drm/atmel-hlcdc: Use crtc->mode_valid() callback

2017-05-18 Thread Jose Abreu
: Not even compiled tested Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Dav

[PATCH v4 02/10] drm: Introduce drm_bridge_mode_valid()

2017-05-18 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu Reviewed-by: Daniel Vetter Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc

[PATCH v4 09/10] drm/atmel-hlcdc: Use crtc->mode_valid() callback

2017-05-18 Thread Jose Abreu
: Not even compiled tested Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc: Laurent Pinchart --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 9 - 1 file changed, 4

[PATCH v4 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-18 Thread Jose Abreu
: Not even compiled tested Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Dave Airlie

[PATCH v4 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-18 Thread Jose Abreu
: Not even compiled tested Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc: Laurent Pinchart --- drivers/gpu/drm/arm/malidp_crtc.c | 11 +-- 1 file changed, 5 insertions(+), 6

[PATCH v4 03/10] drm: Use new mode_valid() helpers in connector probe helper

2017-05-18 Thread Jose Abreu
ode loops through all the connector's encodersXbridgesXcrtcs and calls the callback. If at least a valid encoderXbridgeXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synops

[PATCH v4 03/10] drm: Use new mode_valid() helpers in connector probe helper

2017-05-18 Thread Jose Abreu
ode loops through all the connector's encodersXbridgesXcrtcs and calls the callback. If at least a valid encoderXbridgeXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä C

[PATCH v4 06/10] drm/bridge: analogix-anx78xx: Use bridge->mode_valid() callback

2017-05-18 Thread Jose Abreu
: Only compile tested. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Dave Airlie

[PATCH v4 06/10] drm/bridge: analogix-anx78xx: Use bridge->mode_valid() callback

2017-05-18 Thread Jose Abreu
: Only compile tested. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc: Laurent Pinchart --- drivers/gpu/drm/bridge/analogix-anx78xx.c | 13 ++--- 1 file changed, 6 insertions

[PATCH v4 10/10] drm: vc4: Use crtc->mode_valid() and encoder->mode_valid() callbacks

2017-05-18 Thread Jose Abreu
. NOTE: Not even compiled tested Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Dav

[PATCH v4 05/10] drm: arc: Use crtc->mode_valid() callback

2017-05-18 Thread Jose Abreu
this clock does not support all the needed ranges. Also, remove the atomic_check() callback as mode_valid() callback will be called before. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com>

[PATCH v4 10/10] drm: vc4: Use crtc->mode_valid() and encoder->mode_valid() callbacks

2017-05-18 Thread Jose Abreu
. NOTE: Not even compiled tested Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc: Laurent Pinchart --- drivers/gpu/drm/vc4/vc4_crtc.c | 13 ++--- drivers/gpu/drm/vc4/vc4_dpi.c

[PATCH v4 05/10] drm: arc: Use crtc->mode_valid() callback

2017-05-18 Thread Jose Abreu
this clock does not support all the needed ranges. Also, remove the atomic_check() callback as mode_valid() callback will be called before. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit

[PATCH v4 01/10] drm: Add drm_{crtc/encoder/connector}_mode_valid()

2017-05-18 Thread Jose Abreu
Add a new helper to call crtc->mode_valid, connector->mode_valid and encoder->mode_valid callbacks. Suggested-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Signed-off-by: Jose Abreu <joab...@synopsys.com> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> Revie

[PATCH v4 07/10] drm/bridge/synopsys: dw-hdmi: Use bridge->mode_valid() callback

2017-05-18 Thread Jose Abreu
validation. NOTE: Only compile tested NOTE 2: I also had to change the pdata declaration of mode_valid custom callback so that the passed modes are const. I also changed in the platforms I found. Not even compiled it though. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha

[PATCH v4 04/10] drm: Use mode_valid() in atomic modeset

2017-05-18 Thread Jose Abreu
be accepted in every components. Signed-off-by: Jose Abreu <joab...@synopsys.com> Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> Reviewed-by: Andrzej Hajda <a.ha...@samsung.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@syno

[PATCH v4 01/10] drm: Add drm_{crtc/encoder/connector}_mode_valid()

2017-05-18 Thread Jose Abreu
Add a new helper to call crtc->mode_valid, connector->mode_valid and encoder->mode_valid callbacks. Suggested-by: Ville Syrjälä Signed-off-by: Jose Abreu Reviewed-by: Daniel Vetter Reviewed-by: Andrzej Hajda Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel V

[PATCH v4 07/10] drm/bridge/synopsys: dw-hdmi: Use bridge->mode_valid() callback

2017-05-18 Thread Jose Abreu
validation. NOTE: Only compile tested NOTE 2: I also had to change the pdata declaration of mode_valid custom callback so that the passed modes are const. I also changed in the platforms I found. Not even compiled it though. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville

[PATCH v4 04/10] drm: Use mode_valid() in atomic modeset

2017-05-18 Thread Jose Abreu
be accepted in every components. Signed-off-by: Jose Abreu Reviewed-by: Daniel Vetter Reviewed-by: Andrzej Hajda Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja Cc: Laurent Pinchart --- drivers/gpu/drm/drm_ato

[PATCH v4 00/10] Introduce new mode validation callbacks

2017-05-18 Thread Jose Abreu
ment it. [1] https://patchwork.kernel.org/patch/9702233/ [2] https://lists.freedesktop.org/archives/dri-devel/2017-May/141761.html Jose Abreu (10): drm: Add drm_{crtc/encoder/connector}_mode_valid() drm: Introduce drm_bridge_mode_valid() drm: Use new mode_valid() helpers in connector probe

[PATCH v4 00/10] Introduce new mode validation callbacks

2017-05-18 Thread Jose Abreu
ment it. [1] https://patchwork.kernel.org/patch/9702233/ [2] https://lists.freedesktop.org/archives/dri-devel/2017-May/141761.html Jose Abreu (10): drm: Add drm_{crtc/encoder/connector}_mode_valid() drm: Introduce drm_bridge_mode_valid() drm: Use new mode_valid() helpers in connector probe

Re: [PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-15 Thread Jose Abreu
Hi Laurent, On 15-05-2017 08:05, Laurent Pinchart wrote: > On Monday 15 May 2017 08:47:49 Daniel Vetter wrote: >> On Sun, May 14, 2017 at 02:04:24PM +0300, Laurent Pinchart wrote: >>> On Friday 12 May 2017 17:06:14 Jose Abreu wrote: >>>> On 12-05-2017

Re: [PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-15 Thread Jose Abreu
Hi Laurent, On 15-05-2017 08:05, Laurent Pinchart wrote: > On Monday 15 May 2017 08:47:49 Daniel Vetter wrote: >> On Sun, May 14, 2017 at 02:04:24PM +0300, Laurent Pinchart wrote: >>> On Friday 12 May 2017 17:06:14 Jose Abreu wrote: >>>> On 12-05-2017

Re: [PATCH v3 6/6] drm: arc: Use crtc->mode_valid() callback

2017-05-15 Thread Jose Abreu
Hi Daniel, On 15-05-2017 16:52, Daniel Vetter wrote: > On Mon, May 15, 2017 at 10:53:25AM +0200, Daniel Vetter wrote: >> On Thu, May 11, 2017 at 10:06:02AM +0100, Jose Abreu wrote: >>> Now that we have a callback to check if crtc supports a given mode >>> we can

Re: [PATCH v3 6/6] drm: arc: Use crtc->mode_valid() callback

2017-05-15 Thread Jose Abreu
Hi Daniel, On 15-05-2017 16:52, Daniel Vetter wrote: > On Mon, May 15, 2017 at 10:53:25AM +0200, Daniel Vetter wrote: >> On Thu, May 11, 2017 at 10:06:02AM +0100, Jose Abreu wrote: >>> Now that we have a callback to check if crtc supports a given mode >>> we can

Re: [PATCH v2 8/8] drm: arc: Use crtc->mode_valid() callback

2017-05-14 Thread Jose Abreu
Hi Laurent, Sorry for the late reply. On 12-05-2017 10:57, Laurent Pinchart wrote: > Hi Jose, > > Thank you for the patch. > > On Tuesday 09 May 2017 18:00:15 Jose Abreu wrote: >> Now that we have a callback to check if crtc supports a given mode >> we can use it in

Re: [PATCH v2 8/8] drm: arc: Use crtc->mode_valid() callback

2017-05-14 Thread Jose Abreu
Hi Laurent, Sorry for the late reply. On 12-05-2017 10:57, Laurent Pinchart wrote: > Hi Jose, > > Thank you for the patch. > > On Tuesday 09 May 2017 18:00:15 Jose Abreu wrote: >> Now that we have a callback to check if crtc supports a given mode >> we can use it in

Re: [PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-12 Thread Jose Abreu
Hi Laurent, On 12-05-2017 10:35, Laurent Pinchart wrote: > Hi Jose, > > Thank you for the patch. > > On Tuesday 09 May 2017 18:00:12 Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc-

Re: [PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-12 Thread Jose Abreu
Hi Laurent, On 12-05-2017 10:35, Laurent Pinchart wrote: > Hi Jose, > > Thank you for the patch. > > On Tuesday 09 May 2017 18:00:12 Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc-

[PATCH v3 5/6] drm: Use mode_valid() in atomic modeset

2017-05-11 Thread Jose Abreu
omponents. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Dave Airlie <airl...@linux.

[PATCH v3 5/6] drm: Use mode_valid() in atomic modeset

2017-05-11 Thread Jose Abreu
omponents. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- Changes v1->v2: - Removed call to connector->mode_valid (Ville, Daniel) - Change function name (Ville

[PATCH v3 6/6] drm: arc: Use crtc->mode_valid() callback

2017-05-11 Thread Jose Abreu
this clock does not support all the needed ranges. Also, remove the atomic_check() callback as mode_valid() callback will be called before. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com>

[PATCH v3 4/6] drm: Use new mode_valid() helpers in connector probe helper

2017-05-11 Thread Jose Abreu
ode loops through all the connector's encodersXbridgesXcrtcs and calls the callback. If at least a valid encoderXbridgeXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synops

[PATCH v3 3/6] drm: Introduce drm_bridge_mode_valid()

2017-05-11 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj..

[PATCH v3 6/6] drm: arc: Use crtc->mode_valid() callback

2017-05-11 Thread Jose Abreu
this clock does not support all the needed ranges. Also, remove the atomic_check() callback as mode_valid() callback will be called before. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit

[PATCH v3 4/6] drm: Use new mode_valid() helpers in connector probe helper

2017-05-11 Thread Jose Abreu
ode loops through all the connector's encodersXbridgesXcrtcs and calls the callback. If at least a valid encoderXbridgeXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä C

[PATCH v3 3/6] drm: Introduce drm_bridge_mode_valid()

2017-05-11 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- drivers/gpu/drm/drm_bridge.c

[PATCH v3 0/6] Introduce new mode validation callbacks

2017-05-11 Thread Jose Abreu
g/patch/9702233/ Jose Abreu (6): drm: Add crtc/encoder/bridge->mode_valid() callbacks drm: Add drm_{crtc/encoder/connector}_mode_valid() drm: Introduce drm_bridge_mode_valid() drm: Use new mode_valid() helpers in connector probe helper drm: Use mode_valid() in atomic modeset

[PATCH v3 0/6] Introduce new mode validation callbacks

2017-05-11 Thread Jose Abreu
g/patch/9702233/ Jose Abreu (6): drm: Add crtc/encoder/bridge->mode_valid() callbacks drm: Add drm_{crtc/encoder/connector}_mode_valid() drm: Introduce drm_bridge_mode_valid() drm: Use new mode_valid() helpers in connector probe helper drm: Use mode_valid() in atomic modeset

[PATCH v3 1/6] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-11 Thread Jose Abreu
also change the documentation so that the new and old callbacks are correctly documented. Only the callbacks were implemented to simplify review process, following patches will make use of them. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsy

[PATCH v3 1/6] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-11 Thread Jose Abreu
also change the documentation so that the new and old callbacks are correctly documented. Only the callbacks were implemented to simplify review process, following patches will make use of them. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel

[PATCH v3 2/6] drm: Add drm_{crtc/encoder/connector}_mode_valid()

2017-05-11 Thread Jose Abreu
Add a new helper to call crtc->mode_valid, connector->mode_valid and encoder->mode_valid callbacks. Suggested-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Bro

[PATCH v3 2/6] drm: Add drm_{crtc/encoder/connector}_mode_valid()

2017-05-11 Thread Jose Abreu
Add a new helper to call crtc->mode_valid, connector->mode_valid and encoder->mode_valid callbacks. Suggested-by: Ville Syrjälä Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Arch

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

2017-05-10 Thread Jose Abreu
Hi Ville, On 10-05-2017 15:01, Jose Abreu wrote: > Hi Ville, > > > On 10-05-2017 14:41, Ville Syrjälä wrote: >> On Tue, May 09, 2017 at 06:00:13PM +0100, Jose Abreu wrote: >>> Introduce a new helper function which calls mode_valid() callback >>>

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

2017-05-10 Thread Jose Abreu
Hi Ville, On 10-05-2017 15:01, Jose Abreu wrote: > Hi Ville, > > > On 10-05-2017 14:41, Ville Syrjälä wrote: >> On Tue, May 09, 2017 at 06:00:13PM +0100, Jose Abreu wrote: >>> Introduce a new helper function which calls mode_valid() callback >>>

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

2017-05-10 Thread Jose Abreu
Hi Ville, On 10-05-2017 14:41, Ville Syrjälä wrote: > On Tue, May 09, 2017 at 06:00:13PM +0100, Jose Abreu wrote: >> Introduce a new helper function which calls mode_valid() callback >> for all bridges in an encoder chain. >> >> Signed-off-by: Jose Abreu <joab.

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

2017-05-10 Thread Jose Abreu
Hi Ville, On 10-05-2017 14:41, Ville Syrjälä wrote: > On Tue, May 09, 2017 at 06:00:13PM +0100, Jose Abreu wrote: >> Introduce a new helper function which calls mode_valid() callback >> for all bridges in an encoder chain. >> >> Signed-off-by: Jose Abreu >> C

Re: [PATCH v2 1/8] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-10 Thread Jose Abreu
Hi Daniel, On 10-05-2017 09:03, Daniel Vetter wrote: > On Tue, May 09, 2017 at 06:00:08PM +0100, Jose Abreu wrote: >> This adds a new callback to crtc, encoder and bridge helper functions >> called mode_valid(). This callback shall be implemented if the >> corresponding co

Re: [PATCH v2 1/8] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-10 Thread Jose Abreu
Hi Daniel, On 10-05-2017 09:03, Daniel Vetter wrote: > On Tue, May 09, 2017 at 06:00:08PM +0100, Jose Abreu wrote: >> This adds a new callback to crtc, encoder and bridge helper functions >> called mode_valid(). This callback shall be implemented if the >> corresponding co

Re: [PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-10 Thread Jose Abreu
Hi Daniel, On 10-05-2017 09:01, Daniel Vetter wrote: > On Tue, May 09, 2017 at 06:00:12PM +0100, Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc->mode_valid() helper callbacks to >> validate the mod

Re: [PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-10 Thread Jose Abreu
Hi Daniel, On 10-05-2017 09:01, Daniel Vetter wrote: > On Tue, May 09, 2017 at 06:00:12PM +0100, Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc->mode_valid() helper callbacks to >> validate the mod

Re: [PATCH v2 2/8] drm: Add drm_crtc_mode_valid()

2017-05-10 Thread Jose Abreu
Hi Daniel, On 10-05-2017 08:59, Daniel Vetter wrote: > On Tue, May 09, 2017 at 06:00:09PM +0100, Jose Abreu wrote: >> Add a new helper to call crtc->mode_valid callback. >> >> Suggested-by: Ville Syrjälä <ville.syrj...@linux.intel.com> >> Signed-off-by: Jos

Re: [PATCH v2 2/8] drm: Add drm_crtc_mode_valid()

2017-05-10 Thread Jose Abreu
Hi Daniel, On 10-05-2017 08:59, Daniel Vetter wrote: > On Tue, May 09, 2017 at 06:00:09PM +0100, Jose Abreu wrote: >> Add a new helper to call crtc->mode_valid callback. >> >> Suggested-by: Ville Syrjälä >> Signed-off-by: Jose Abreu >> Cc: Carlos Palminha

[PATCH v2 2/8] drm: Add drm_crtc_mode_valid()

2017-05-09 Thread Jose Abreu
Add a new helper to call crtc->mode_valid callback. Suggested-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <

[PATCH v2 2/8] drm: Add drm_crtc_mode_valid()

2017-05-09 Thread Jose Abreu
Add a new helper to call crtc->mode_valid callback. Suggested-by: Ville Syrjälä Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- drivers/gpu/drm/drm_crtc.c |

[PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-09 Thread Jose Abreu
gh all the connector's encodersXcrtcs and calls the callback. If at least a valid encoderXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abro

[PATCH v2 5/8] drm: Use new mode_valid() helpers in connector probe helper

2017-05-09 Thread Jose Abreu
gh all the connector's encodersXcrtcs and calls the callback. If at least a valid encoderXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej

[PATCH v2 8/8] drm: arc: Use crtc->mode_valid() callback

2017-05-09 Thread Jose Abreu
this clock does not support all the needed ranges. Also, remove the atomic_check() callback as mode_valid() callback will be called before. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com>

[PATCH v2 8/8] drm: arc: Use crtc->mode_valid() callback

2017-05-09 Thread Jose Abreu
this clock does not support all the needed ranges. Also, remove the atomic_check() callback as mode_valid() callback will be called before. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit

[PATCH v2 3/8] drm: Add drm_encoder_mode_valid()

2017-05-09 Thread Jose Abreu
Add a new helper to call encoder->mode_valid callback. Suggested-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc

[PATCH v2 3/8] drm: Add drm_encoder_mode_valid()

2017-05-09 Thread Jose Abreu
Add a new helper to call encoder->mode_valid callback. Suggested-by: Ville Syrjälä Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- drivers/gpu/drm/drm_crtc_internal.h |

[PATCH v2 4/8] drm: Add drm_connector_mode_valid()

2017-05-09 Thread Jose Abreu
Add a new helper to call connector->mode_valid callback. Suggested-by: Ville Syrjälä <ville.syrj...@linux.intel.com> Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc

[PATCH v2 0/8] Introduce new mode validation callbacks

2017-05-09 Thread Jose Abreu
e, except the connector->mode_valid one, so that the mode is validated. This is done before calling mode_fixup(). Finally, at 8/8 we use the new crtc->mode_valid() callback in arcpgu and remove the atomic_check() callback. [1] https://patchwork.kernel.org/patch/9702233/ Jose Abreu (8): drm: A

[PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

2017-05-09 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj..

[PATCH v2 7/8] drm: Use mode_valid() in atomic modeset

2017-05-09 Thread Jose Abreu
be accepted in every components. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc:

[PATCH v2 4/8] drm: Add drm_connector_mode_valid()

2017-05-09 Thread Jose Abreu
Add a new helper to call connector->mode_valid callback. Suggested-by: Ville Syrjälä Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- TODO: function prototype should rece

[PATCH v2 0/8] Introduce new mode validation callbacks

2017-05-09 Thread Jose Abreu
e, except the connector->mode_valid one, so that the mode is validated. This is done before calling mode_fixup(). Finally, at 8/8 we use the new crtc->mode_valid() callback in arcpgu and remove the atomic_check() callback. [1] https://patchwork.kernel.org/patch/9702233/ Jose Abreu (8): drm: A

[PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

2017-05-09 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- drivers/gpu/drm/drm_bridge.c

[PATCH v2 7/8] drm: Use mode_valid() in atomic modeset

2017-05-09 Thread Jose Abreu
be accepted in every components. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- Changes v1->v2: - Removed call to connector->mode_valid (Ville, Daniel) - Ch

[PATCH v2 1/8] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-09 Thread Jose Abreu
also change the description of connector->mode_valid() callback so that it matches the existing behaviour: It is never called in atomic check phase. Only the callbacks were implemented to simplify review process, following patches will make use of them. Signed-off-by: Jose Abreu &l

[PATCH v2 1/8] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-09 Thread Jose Abreu
also change the description of connector->mode_valid() callback so that it matches the existing behaviour: It is never called in atomic check phase. Only the callbacks were implemented to simplify review process, following patches will make use of them. Signed-off-by: Jose Abreu Cc: Car

Re: [PATCH 2/5] drm: Use new mode_valid() helpers in connector probe helper

2017-05-08 Thread Jose Abreu
Hi Daniel, On 08-05-2017 08:50, Daniel Vetter wrote: > On Thu, May 04, 2017 at 03:11:39PM +0100, Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc->mode_valid() helper callbacks to >> validate the mod

Re: [PATCH 2/5] drm: Use new mode_valid() helpers in connector probe helper

2017-05-08 Thread Jose Abreu
Hi Daniel, On 08-05-2017 08:50, Daniel Vetter wrote: > On Thu, May 04, 2017 at 03:11:39PM +0100, Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc->mode_valid() helper callbacks to >> validate the mod

Re: [PATCH 4/5] drm: Use mode_valid() in atomic modeset

2017-05-04 Thread Jose Abreu
Hi Ville, On 04-05-2017 15:40, Ville Syrjälä wrote: > On Thu, May 04, 2017 at 03:11:41PM +0100, Jose Abreu wrote: >> This patches makes use of the new mode_valid() callbacks introduced >> previously to validate the full video pipeline when modesetting. >> >> This call

Re: [PATCH 4/5] drm: Use mode_valid() in atomic modeset

2017-05-04 Thread Jose Abreu
Hi Ville, On 04-05-2017 15:40, Ville Syrjälä wrote: > On Thu, May 04, 2017 at 03:11:41PM +0100, Jose Abreu wrote: >> This patches makes use of the new mode_valid() callbacks introduced >> previously to validate the full video pipeline when modesetting. >> >> This call

Re: [PATCH 2/5] drm: Use new mode_valid() helpers in connector probe helper

2017-05-04 Thread Jose Abreu
Hi Ville, On 04-05-2017 15:32, Ville Syrjälä wrote: > On Thu, May 04, 2017 at 03:11:39PM +0100, Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc->mode_valid() helper callbacks to >> validate the mod

Re: [PATCH 2/5] drm: Use new mode_valid() helpers in connector probe helper

2017-05-04 Thread Jose Abreu
Hi Ville, On 04-05-2017 15:32, Ville Syrjälä wrote: > On Thu, May 04, 2017 at 03:11:39PM +0100, Jose Abreu wrote: >> This changes the connector probe helper function to use the new >> encoder->mode_valid() and crtc->mode_valid() helper callbacks to >> validate the mod

[PATCH 4/5] drm: Use mode_valid() in atomic modeset

2017-05-04 Thread Jose Abreu
be accepted in every components. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc:

[PATCH 4/5] drm: Use mode_valid() in atomic modeset

2017-05-04 Thread Jose Abreu
be accepted in every components. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- drivers/gpu/drm/drm_atomic_helper.c | 92 +++-- 1 file changed, 8

[PATCH 2/5] drm: Use new mode_valid() helpers in connector probe helper

2017-05-04 Thread Jose Abreu
gh all the connector's encodersXcrtcs and calls the callback. If at least a valid encoderXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abro

[PATCH 2/5] drm: Use new mode_valid() helpers in connector probe helper

2017-05-04 Thread Jose Abreu
gh all the connector's encodersXcrtcs and calls the callback. If at least a valid encoderXcrtc combination is found which accepts the mode then the function returns MODE_OK. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej

[PATCH 1/5] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-04 Thread Jose Abreu
. Only the callbacks were implemented to simplify review process, following patches will make use of them. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj...@linux.

[PATCH 1/5] drm: Add crtc/encoder/bridge->mode_valid() callbacks

2017-05-04 Thread Jose Abreu
. Only the callbacks were implemented to simplify review process, following patches will make use of them. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- include/drm/drm_bridge.h

[PATCH 0/5] Introduce new mode validation callbacks

2017-05-04 Thread Jose Abreu
e the new crtc->mode_valid() callback in arcpgu and remove the atomic_check() callback. [1] https://patchwork.kernel.org/patch/9702233/ Jose Abreu (5): drm: Add crtc/encoder/bridge->mode_valid() callbacks drm: Use new mode_valid() helpers in connector probe helper drm: Introduce drm_br

[PATCH 0/5] Introduce new mode validation callbacks

2017-05-04 Thread Jose Abreu
e the new crtc->mode_valid() callback in arcpgu and remove the atomic_check() callback. [1] https://patchwork.kernel.org/patch/9702233/ Jose Abreu (5): drm: Add crtc/encoder/bridge->mode_valid() callbacks drm: Use new mode_valid() helpers in connector probe helper drm: Introduce drm_br

[PATCH 3/5] drm: Introduce drm_bridge_mode_valid()

2017-05-04 Thread Jose Abreu
Introduce a new helper function which calls mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu <joab...@synopsys.com> Cc: Carlos Palminha <palmi...@synopsys.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Ville Syrjälä <ville.syrj..

<    3   4   5   6   7   8   9   10   11   12   >