On Fri, Aug 6, 2010 at 9:37 PM, Alex Deucher <alexdeucher at gmail.com> wrote: > +++ b/drivers/gpu/drm/radeon/r300.c > @@ -1332,6 +1332,12 @@ static int r300_startup(struct radeon_device *rdev) > ? ? ? ? ? ? ? ?if (r) > ? ? ? ? ? ? ? ? ? ? ? ?return r; > ? ? ? ?} > + > + ? ? ? /* allocate wb buffer */ > + ? ? ? radeon_wb_init(rdev); > + ? ? ? if (r) > + ? ? ? ? ? ? ? return r; > + > ? ? ? ?/* Enable IRQ */ > ? ? ? ?r100_irq_set(rdev); > ? ? ? ?rdev->config.r300.hdp_cntl = RREG32(RADEON_HOST_PATH_CNTL);
Hello, Should this be "r = radeon_wb_init(rdev);" ? It looks like all calls of this function in this patch (and the r6xx/r7xx/evergreen one) could use this addition. Thanks, Josh
