[Nouveau] [PATCH 01/10] drm/nv50: decode PGRAPH status registers on TLB flush fail

2012-08-19 Thread Marcin Slusarz
Now it outputs: nouveau E[ PGRAPH][:02:00.0] PGRAPH TLB flush idle timeout fail nouveau E[ PGRAPH][:02:00.0] PGRAPH_STATUS: BUSY DISPATCH VFETCH CCACHE_UNK4 STRMOUT_GSCHED_UNK5 UNK14XX UNK1CXX CLIPID ZCULL ENG2D UNK34XX TPRAST TPROP ROP (0x011fde03) nouveau E[ PGRAPH][:02:00.0]

[Nouveau] [PATCH 03/10] drm/nouveau: guard nv_error printk pairs with nv_printk_enabled

2012-08-19 Thread Marcin Slusarz
With CONFIG_NOUVEAU_DEBUG set to 0 or subdev debug flags set to 0 code paths which use nv_error and printk would print awkwardly formatted data. Guard those code paths with nv_printk_enabled. Additionally, fix reporting levels of: nv20_graph_intr, nv40_graph_intr and nv50_graph_intr.

[Nouveau] [PATCH 04/10] drm/nouveau: constify instances of nouveau_bitfield and nouveau_enum structs

2012-08-19 Thread Marcin Slusarz
Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/core/engine/copy/nva3.c|2 +- drivers/gpu/drm/nouveau/core/engine/copy/nvc0.c|2 +- drivers/gpu/drm/nouveau/core/engine/crypt/nv84.c |2 +- drivers/gpu/drm/nouveau/core/engine/crypt/nv98.c |

[Nouveau] [PATCH 05/10] drm/nouveau: quiet some static-related sparse noise

2012-08-19 Thread Marcin Slusarz
Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/core/core/gpuobj.c |2 +- drivers/gpu/drm/nouveau/core/core/object.c |8 drivers/gpu/drm/nouveau/core/core/option.c |2 +-

[Nouveau] [PATCH 06/10] drm/nouveau: sparse fixes

2012-08-19 Thread Marcin Slusarz
drivers/gpu/drm/nouveau/core/core/object.c:452:22: warning: non-ANSI function declaration of function 'nouveau_object_debug' drivers/gpu/drm/nouveau/core/subdev/device/base.c:64:10: warning: Initializer entry defined twice drivers/gpu/drm/nouveau/core/subdev/device/base.c:73:10: also defined

[Nouveau] [PATCH 07/10] drm/nouveau: add sparse annotations to nv04_fifo_pause/nv04_fifo_start

2012-08-19 Thread Marcin Slusarz
Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/core/engine/fifo/nv04.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/nouveau/core/engine/fifo/nv04.c b/drivers/gpu/drm/nouveau/core/engine/fifo/nv04.c index

[Nouveau] [PATCH 08/10] drm/nouveau: use ioread32_native/iowrite32_native for fifo control registers

2012-08-19 Thread Marcin Slusarz
Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- Do I miss something important why was it written this way? --- drivers/gpu/drm/nouveau/core/engine/fifo/base.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/core/engine/fifo/base.c

[Nouveau] [PATCH 09/10] drm/nv50/evo: store iomem pointer in properly typed field

2012-08-19 Thread Marcin Slusarz
Abuse of function pointer was noticed by sparse. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nv50_evo.c | 18 -- 1 files changed, 12 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nv50_evo.c

[Nouveau] [PATCH 10/10] drm/nouveau: fix off-by-one bugs related to command submission in IB mode

2012-08-19 Thread Marcin Slusarz
Commit drm/nouveau: port all engines to new engine module format changed IB size calculation to be less wasteful, but didn't take into account already existing off-by-one bugs :). So: - ib_max is the last entry, so we need to +1 when calculating number of free entries - nv50_dma_wait already

Re: [Nouveau] [PATCH 10/10] drm/nouveau: fix off-by-one bugs related to command submission in IB mode

2012-08-19 Thread Maarten Maathuis
On Sun, Aug 19, 2012 at 11:02 PM, Marcin Slusarz marcin.slus...@gmail.com wrote: Commit drm/nouveau: port all engines to new engine module format changed IB size calculation to be less wasteful, but didn't take into account already existing off-by-one bugs :). So: - ib_max is the last entry,

Re: [Nouveau] nouveau Macbook EDID fix upstreaming...

2012-08-19 Thread Ben Skeggs
On Fri, Aug 17, 2012 at 04:33:25PM +0800, Daniel J Blueman wrote: It looks like Ryan's Nouveau patch [1,2] to get the I2C working for EDID (thus modelines) is crucial for avoiding the Nvidia binary drivers on MacbookPro 2012s. Any plans/chance for it to be upstreamed to eg 3.6-rc3? Not at