Re: [PATCH] agpgart: Reprobe VGA devices when a new GART device is added

2010-03-01 Thread Ben Hutchings
On Sun, 2010-02-28 at 03:30 +, Ben Hutchings wrote: +static void agp_probe_video(struct work_struct *work) +{ + struct pci_dev *pdev = NULL; + + while ((pdev = pci_get_class(0x03, pdev)) != NULL) { Only without the opening brace here... Ben. + if (!pdev

[PATCH] agpgart: Reprobe VGA devices when a new GART device is added

2010-03-01 Thread Ben Hutchings
This addresses http://bugzilla.kernel.org/show_bug.cgi?id=15021. DRM drivers may fail to probe their devices when the associated AGP GART does not yet have a driver, so we reprobe unbound VGA devices when a GART device is added. Signed-off-by: Ben Hutchings b...@decadent.org.uk

Re: [PATCH] agpgart: Reprobe VGA devices when a new GART device is added

2010-03-01 Thread Ben Hutchings
On Mon, Mar 01, 2010 at 08:19:02AM -0800, Eric Anholt wrote: On Mon, 1 Mar 2010 18:33:14 +1000, Dave Airlie airl...@gmail.com wrote: On Sun, Feb 28, 2010 at 1:30 PM, Ben Hutchings b...@decadent.org.uk wrote: This addresses http://bugzilla.kernel.org/show_bug.cgi?id=15021. DRM drivers

[PATCH] r128: Add test for initialisation to all ioctls that require it

2009-08-23 Thread Ben Hutchings
. Signed-off-by: Ben Hutchings b...@decadent.org.uk --- drivers/gpu/drm/r128/r128_cce.c | 18 ++ drivers/gpu/drm/r128/r128_drv.h |8 drivers/gpu/drm/r128/r128_state.c | 36 +++- 3 files changed, 41 insertions(+), 21 deletions(-) diff

[PATCH 2/3] r128: Use request_firmware() to load CCE microcode

2009-08-23 Thread Ben Hutchings
Firmware blob looks like this: __be32 datah __be32 datal Signed-off-by: Ben Hutchings b...@decadent.org.uk --- This has been tested in Debian as a change to 2.6.30. Note that initialisation can fail as a result of missing firmware, and my previous patch adding initialisation tests

Re: [PATCH] radeon_cp: use request_firmware

2009-04-14 Thread Ben Hutchings
Please use b...@decadent.org.uk for all mail related to my work on firmware separation, which has nothing to do with my employment. On Mon, 2009-04-13 at 21:57 +0530, Jaswinder Singh Rajput wrote: Moved datah before datal because datah is required before datal. Firmware blob looks like

Re: [PATCH] radeon_cp: use request_firmware

2009-04-14 Thread Ben Hutchings
On Tue, Apr 14, 2009 at 10:24:41AM +0530, Jaswinder Singh Rajput wrote: On Mon, 2009-04-13 at 19:26 +0100, Ben Hutchings wrote: Please use b...@decadent.org.uk for all mail related to my work on firmware separation, which has nothing to do with my employment. On Mon, 2009-04-13 at 21:57

Re: [PATCH 1/3] mga: Use request_firmware() to load microcode

2009-02-23 Thread Ben Hutchings
On Mon, 2009-02-23 at 13:14 +0200, Ville Syrjälä wrote: On Sun, Feb 22, 2009 at 11:45:21PM +, Ben Hutchings wrote: On Mon, 2009-02-23 at 00:06 +0100, Stephane Marchesin wrote: Hi, This mga patch replaces a firmware that was split in pieces by functionality and that had comments

Re: [PATCH 1/3] mga: Use request_firmware() to load microcode

2009-02-22 Thread Ben Hutchings
On Mon, 2009-02-23 at 00:06 +0100, Stephane Marchesin wrote: Hi, This mga patch replaces a firmware that was split in pieces by functionality and that had comments with a single blob. Each pipe's code was converted to a seperate line of the ihex file. So IMO it's actually decreasing the

Re: [PATCH] radeon_cp: use request_firmware

2008-12-23 Thread Ben Hutchings
people using this patch. Is Debian shipping it? Not at this time. AFAIK Ben Hutchings has tested that patch, although I don't know on what hardware, and if he got other testers. Ben? I had to revise Jaswinder's patch as it allowed a double-free. I'm attaching the version I've tested (which

[PATCH] r128: Use request_firmware() to load CCE microcode

2008-10-19 Thread Ben Hutchings
Firmware blob looks like this: __be32 datah __be32 datal Compile-tested only. Signed-off-by: Ben Hutchings [EMAIL PROTECTED] --- This is pretty similar to Jaswinder Singh's patch for radeon. The licence information is based on conversation with David Airlie. Ben. drivers/gpu/drm