On Fri, Aug 6, 2010 at 9:37 PM, Alex Deucher <alexdeucher@xxxxxxxxx> 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 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
