From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm: Split out drm_probe_helper.h Date: Thu, 24 Jan 2019 13:39:41 +0100 Message-ID: <20190124123941.GW3271__17591.7867383518$1548333598$gmane$org@phenom.ffwll.local> References: <20190116181018.GA27364@ravnborg.org> <20190117210334.13234-1-daniel.vetter@ffwll.ch> <20190123170015.GA23138@ravnborg.org> <20190124094646.GR3271@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20190124094646.GR3271@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Sam Ravnborg Cc: Neil Armstrong , Daniel Vetter , Liviu Dudau , DRI Development , virtualization@lists.linux-foundation.org, Laurent Pinchart , Benjamin Gaignard , Daniel Vetter , linux-stm32@st-md-mailman.stormreply.com, linux-samsung-soc@vger.kernel.org, Oleksandr Andrushchenko , amd-gfx@lists.freedesktop.org, linux-rockchip@lists.infradead.org, nouveau@lists.freedesktop.org, CK Hu , spice-devel@lists.freedesktop.org, Jani Nikula , linux-arm-msm@vger.kernel.org, intel-gfx@lists.freedesktop.org, etnaviv@lists.freedesktop.org, Jani Nikula , linux-mediatek@lists.infradead.orgRodrigo Vivi List-Id: virtualization@lists.linuxfoundation.org On Thu, Jan 24, 2019 at 10:46:47AM +0100, Daniel Vetter wrote: > On Wed, Jan 23, 2019 at 06:00:15PM +0100, Sam Ravnborg wrote: > > Hi Daniel. > > > > On Thu, Jan 17, 2019 at 10:03:34PM +0100, Daniel Vetter wrote: > > > Having the probe helper stuff (which pretty much everyone needs) in > > > the drm_crtc_helper.h file (which atomic drivers should never need) is > > > confusing. Split them out. > > > > > > To make sure I actually achieved the goal here I went through all > > > drivers. And indeed, all atomic drivers are now free of > > > drm_crtc_helper.h includes. > > > > How are the plans to get this patchset merged? > > There were dependencies on onging drmP.h removal in i915 IIRC? > > I guess my "Minimize drmP.h dependencies" patch-set also have a role in this. > > Working on it, a pull request got lost. I originally wanted to get this > all resolved this week, probably going to be next week now :-( We're lucky, all sorted now & patch merged. I'm also merging your series, correctly interleaved. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch