From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Peres Subject: Re: pwr/macros: Stop playing Russian roulette on data memory Date: Tue, 05 Aug 2014 01:32:13 +0200 Message-ID: <53E017FD.5000203@free.fr> References: <1406992501-23037-1-git-send-email-rspliet@eclipso.eu> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: nouveau-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "Nouveau" To: Ben Skeggs , Roy Spliet Cc: Nouveau Mailinglist List-Id: nouveau.vger.kernel.org On 05/08/2014 01:26, Ben Skeggs wrote: > On Sun, Aug 3, 2014 at 1:15 AM, Roy Spliet wrote: >> This patch fixes the pwr firmware to play nicely at least on NVA3. Because Martin might send more patches soon, I didn't include a regenerated nvXX.fuc.h. To me it makes more sense if all patches are merged then, and a final patch regenerates the headers in one go. Of course, I did test this patch and found it to work as intended, so feel free to pick up as you please. Roy is right, I should take the time and send you my ppwr patches. If anything, I would like this patch to be merged: http://cgit.freedesktop.org/~mperes/nouveau/commit/?h=ppwr_rework&id=b0bbe7dc6aefb68c7e305ca160f452db053633e7 > Nice catch :) > > Thanks for pointing out that the ucode changes weren't included too, I > amended the commit when I merged it to include them. > > One small note: If patches are against the kernel rather than the > nouveau tree, can you include the "drm/nouveau" prefix too? :) Good to know! > > Thanks, > Ben. > >