[PATCH] agp: Fix multi-line warning message whitespace

Tormod Volden lists.tormod at gmail.com
Thu Jan 5 03:03:43 PST 2012


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
>>


More information about the dri-devel mailing list