On Tue, May 23, 2017 at 03:40:24PM +0100, Jose Abreu wrote: > 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 crtc->mode_valid(), encoder->mode_valid() and > >>> bridge->mode_valid() callbacks which will be used in followup > >>> patches and also by cleaning the documentation a little bit. This > >>> patch is available at [2] and the series depends on it. > >>> > >>> We proceed by introducing new helpers to call this new callbacks > >>> at 1/10. > >>> > >>> At 2/10 a helper function is introduced that calls all mode_valid() > >>> from a set of bridges. > >>> > >>> Next, at 3/10 we modify the connector probe helper so that only modes > >>> which are supported by a given bridge+encoder+crtc combination are > >>> probbed. > >>> > >>> At 4/10 we call all the mode_valid() callbacks for a given pipeline, > >>> except the connector->mode_valid one, so that the mode is validated. > >>> This is done before calling mode_fixup(). > >>> > >>> Finally, from 5/10 to 10/10 we use the new callbacks in drivers that > >>> can implement it. > >>> > >>> [1] https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.kernel.org_patch_9702233_&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=YohmJizS2YWha2kjTAHGImBe-ghyCyY_4jKYalIhqcU&s=mtXW7k5AwOp9H790Zg-U0ZB_OXCyW-SRQD9H5kTX5Ec&e= > >>> [2] https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop.org_archives_dri-2Ddevel_2017-2DMay_141761.html&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=YohmJizS2YWha2kjTAHGImBe-ghyCyY_4jKYalIhqcU&s=N3-YUnWxQAD9jw0y7xFB0fAuCGa_B6Q6yRsL2OmGWM0&e= > >>> > >>> 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 helper > >>> drm: Use mode_valid() in atomic modeset > >>> drm: arc: Use crtc->mode_valid() callback > >>> drm/bridge: analogix-anx78xx: Use bridge->mode_valid() callback > >>> drm/bridge/synopsys: dw-hdmi: Use bridge->mode_valid() callback > >>> drm/arm: malidp: Use crtc->mode_valid() callback > >>> drm/atmel-hlcdc: Use crtc->mode_valid() callback > >>> drm: vc4: Use crtc->mode_valid() and encoder->mode_valid() callbacks > >> Looks all real nice, I think a bit more time to get acks/reviews/tested-by > >> for the driver conversions and I'll go and vacuum this all up. > > On that: You didn't cc driver maintainers on the driver conversion patches > > (not all are bridge drivers maintainer by Archit&co), without that they > > might miss it. Please remember to do that (you might need to resend to get > > their attention), scripts/get_maintainers.pl helps with that. > > Yeah, I'm really sorry about that. I was in a different time zone > with all my head messed up with jetlag so I missed this and maybe > more :/ Lets wait for some input and I will resend the series if > needed. No worries. btw I've applied the 3 kernel-doc patches now, that should also make it easier to get the driver patches reviewed (since now they apply directly on linux-next). -Daniel > > Thanks! > > Best regards, > Jose Miguel Abreu > > > > > Thanks, Daniel > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel