Hi Dave, Can you pick up this, or do I have to reroll the patch with the R-B?
Cheers, Tormod On Sun, Nov 6, 2011 at 4:35 PM, Corbin Simpson <mostawesomedude at gmail.com> wrote: > FWIW, Reviewed-by: Corbin Simpson <MostAwesomeDude at gmail.com> > > Sending from a mobile, pardon my terseness. ~ C. > > On Nov 6, 2011 7:04 AM, "Tormod Volden" <lists.tormod at gmail.com> wrote: >> >> From: Tormod Volden <debian.tormod at gmail.com> >> >> Signed-off-by: Tormod Volden <debian.tormod at gmail.com> >> --- >> ?drivers/char/agp/generic.c | ? ?8 ++++---- >> ?1 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/char/agp/generic.c b/drivers/char/agp/generic.c >> index c5d04e5..2dc2d5b 100644 >> --- a/drivers/char/agp/generic.c >> +++ b/drivers/char/agp/generic.c >> @@ -514,12 +514,12 @@ static void agp_v2_parse_one(u32 *requested_mode, >> u32 *bridge_agpstat, u32 *vga_ >> ? ? ? ?switch (*bridge_agpstat & 7) { >> ? ? ? ?case 4: >> ? ? ? ? ? ? ? ?*bridge_agpstat |= (AGPSTAT2_2X | AGPSTAT2_1X); >> - ? ? ? ? ? ? ? printk(KERN_INFO PFX "BIOS bug. AGP bridge claims to only >> support x4 rate" >> + ? ? ? ? ? ? ? printk(KERN_INFO PFX "BIOS bug. AGP bridge claims to only >> support x4 rate. " >> ? ? ? ? ? ? ? ? ? ? ? ?"Fixing up support for x2 & x1\n"); >> ? ? ? ? ? ? ? ?break; >> ? ? ? ?case 2: >> ? ? ? ? ? ? ? ?*bridge_agpstat |= AGPSTAT2_1X; >> - ? ? ? ? ? ? ? printk(KERN_INFO PFX "BIOS bug. AGP bridge claims to only >> support x2 rate" >> + ? ? ? ? ? ? ? printk(KERN_INFO PFX "BIOS bug. AGP bridge claims to only >> support x2 rate. " >> ? ? ? ? ? ? ? ? ? ? ? ?"Fixing up support for x1\n"); >> ? ? ? ? ? ? ? ?break; >> ? ? ? ?default: >> @@ -711,7 +711,7 @@ static void agp_v3_parse_one(u32 *requested_mode, u32 >> *bridge_agpstat, u32 *vga_ >> ? ? ? ? ? ? ? ? ? ? ? ?*bridge_agpstat &= ~(AGPSTAT3_4X | AGPSTAT3_RSVD); >> ? ? ? ? ? ? ? ? ? ? ? ?*vga_agpstat &= ~(AGPSTAT3_4X | AGPSTAT3_RSVD); >> ? ? ? ? ? ? ? ?} else { >> - ? ? ? ? ? ? ? ? ? ? ? printk(KERN_INFO PFX "Fell back to AGPx4 mode >> because"); >> + ? ? ? ? ? ? ? ? ? ? ? printk(KERN_INFO PFX "Fell back to AGPx4 mode >> because "); >> ? ? ? ? ? ? ? ? ? ? ? ?if (!(*bridge_agpstat & AGPSTAT3_8X)) { >> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?printk(KERN_INFO PFX "bridge couldn't do >> x8. bridge_agpstat:%x (orig=%x)\n", >> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?*bridge_agpstat, origbridge); >> @@ -974,7 +974,7 @@ int agp_generic_create_gatt_table(struct >> agp_bridge_data *bridge) >> ? ? ? ?bridge->driver->cache_flush(); >> ?#ifdef CONFIG_X86 >> ? ? ? ?if (set_memory_uc((unsigned long)table, 1 << page_order)) >> - ? ? ? ? ? ? ? printk(KERN_WARNING "Could not set GATT table memory to >> UC!"); >> + ? ? ? ? ? ? ? printk(KERN_WARNING "Could not set GATT table memory to >> UC!\n"); >> >> ? ? ? ?bridge->gatt_table = (void *)table; >> ?#else >> -- >> 1.7.5.4 >>