From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PULL] drm-misc-next Date: Thu, 8 Aug 2019 17:29:38 +0200 Message-ID: <20190808152938.vxdykmx7nio3xdmz@flea> References: <20190803104719.gwb6hmreeaqyye6n@flea> <20190806160146.sj7l5wrrjtt5nhbz@flea> <20190806161132.GC7444@phenom.ffwll.local> <20190807120246.tl4hhztr3qna3qf6@flea> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2076128058==" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: "DRM maintainer tools announcements, discussion, and development" , Intel Graphics Development , dri-devel , Rodrigo Vivi , Sean Paul List-Id: dri-devel@lists.freedesktop.org --===============2076128058== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="tpm5r3tjneldi45c" Content-Disposition: inline --tpm5r3tjneldi45c Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi, On Wed, Aug 07, 2019 at 02:30:25PM +0200, Daniel Vetter wrote: > On Wed, Aug 7, 2019 at 2:02 PM Maxime Ripard wrote: > > On Tue, Aug 06, 2019 at 06:11:32PM +0200, Daniel Vetter wrote: > > > On Tue, Aug 06, 2019 at 06:01:46PM +0200, Maxime Ripard wrote: > > > > On Tue, Aug 06, 2019 at 10:33:53AM +1000, Dave Airlie wrote: > > > > > On Sat, 3 Aug 2019 at 20:47, Maxime Ripard wrote: > > > > > > > > > > > > Hi Daniel, Dave, > > > > > > > > > > > > Here is the first (and pretty late) drm-misc-next PR. > > > > > > > > > > > > It's pretty big due to the lateness, but there's nothing really major > > > > > > showing up. It's pretty much the usual bunch of reworks, fixes, and > > > > > > new helpers being introduced. > > > > > > > > > > dim: 415d2e9e0757 ("Revert "drm/gem: Rename drm_gem_dumb_map_offset() > > > > > to drm_gem_map_offset()""): mandatory review missing. > > > > > dim: be855382bacb ("Revert "drm/panfrost: Use drm_gem_map_offset()""): > > > > > mandatory review missing. > > > > > dim: e4eee93d2577 ("drm/vgem: drop DRM_AUTH usage from the driver"): > > > > > mandatory review missing. > > > > > dim: 88209d2c5035 ("drm/msm: drop DRM_AUTH usage from the driver"): > > > > > mandatory review missing. > > > > > dim: ccdae4257569 ("drm/nouveau: remove open-coded drm_invalid_op()"): > > > > > mandatory review missing. > > > > > > > > > > Pretty sure review in drm-misc-next is a rule. I don't even see acks > > > > > on most of these. > > > > > > > > Ugh, sorry for that. I guess I'm still pretty new to the > > > > maintainer-side of dim, which commands did you use to check that? > > > > > > dim apply-pull does this. If all committers use the tooling as they should > > > they shouldn't be able to push patches which violate anything here, that's > > > why dim request-pull doesn't reject. > > > > Yeah, sure, I meant to ask if there was anyway to check this before > > sending the PR on our end. > > I think we'd need to create new command. Or maybe we should integrate > it as part of the pull request generation, as an information/warning > that there might be problems you need to explain. Want to write a dim > patch? I just sent something that should catch that, but considering my dim-fu, you probably want to review it :) Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --tpm5r3tjneldi45c Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXUw/4gAKCRDj7w1vZxhR xQhZAQDIFh4FiAC2OiEynoM7OgTcLbBRVCAtxnX0wjbRXAmUqQD/VHWeXxKlsWKq HA8j4Z84fwIgTw1b3CpyUilEOC13Bwo= =2oc6 -----END PGP SIGNATURE----- --tpm5r3tjneldi45c-- --===============2076128058== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs --===============2076128058==--