From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Intel-gfx] [RFC PATCH v2 1/4] drm/i915: add i915_ved.c to setup bridge for VED Date: Wed, 22 Oct 2014 10:43:59 +0200 Message-ID: <20141022084359.GO26941@phenom.ffwll.local> References: <1413873402-4331-1-git-send-email-yao.cheng@intel.com> <1413873402-4331-2-git-send-email-yao.cheng@intel.com> <20141021120908.GA26941@phenom.ffwll.local> <8FF7D634BEE4C2428EFFAB6B7E919E4B017D968D@shsmsx102.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <8FF7D634BEE4C2428EFFAB6B7E919E4B017D968D@shsmsx102.ccr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Cheng, Yao" Cc: "Rao, Ram R" , "intel-gfx@lists.freedesktop.org" , "dri-devel@lists.freedesktop.org" , "Jiang, Fei" , "Abel, Michael J" , "Vetter, Daniel" List-Id: intel-gfx@lists.freedesktop.org On Wed, Oct 22, 2014 at 07:11:21AM +0000, Cheng, Yao wrote: > > -----Original Message----- > > From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel > > Vetter > > Sent: Tuesday, October 21, 2014 8:09 PM > > To: Cheng, Yao > > Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; Kelley, > > Sean V; Vetter, Daniel; Abel, Michael J; Jiang, Fei; Rao, Ram R > > Subject: Re: [Intel-gfx] [RFC PATCH v2 1/4] drm/i915: add i915_ved.c to setup > > bridge for VED > > > > On Tue, Oct 21, 2014 at 02:36:41PM +0800, Yao Cheng wrote: > > > Setup minimum required resources during i915_driver_load: > > > 1. Create a platform device to share MMIO/IRQ resources 2. Make the > > > platform device child of i915 device for runtime PM. > > > 3. Create IRQ chip to forward the VED irqs. > > > VED driver (a standalone drm driver) probes the VED device and creates > > > a new dri card on install. > > > > > > Currently only supports VED on valleyview. > > > Kerneldoc is updated for i915_ved.c. > > > > > > Signed-off-by: Yao Cheng > > > > Please resend with a patch changelog to account for my review comments. > > And Ville's. Plus cc us both. And if there's anything you didn't address, you > > must reply to the review and we need to further discuss this. > > > > Daniel, I see, thanks for the instruction. > Do you mean resending the [RFC PATCH v2] with changelog and cc list? > Or adding changelog/cc when sending [RFC PATCH v3]? I think you could just add the per-patch changelog for both v2 and v3 when sending out v3. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch