Re: [PATCH 1/2] sis-agp: Remove SIS 760, handled by amd64-agp

From: Dave Airlie
Date: Thu Mar 25 2010 - 22:31:51 EST


On Fri, Mar 26, 2010 at 12:10 PM, FUJITA Tomonori
<fujita.tomonori@xxxxxxxxxxxxx> wrote:
> On Fri, 26 Mar 2010 11:45:50 +1000
> Dave Airlie <airlied@xxxxxxxxx> wrote:
>
>> On Fri, Mar 26, 2010 at 11:41 AM, FUJITA Tomonori
>> <fujita.tomonori@xxxxxxxxxxxxx> wrote:
>> > On Wed, 24 Mar 2010 03:33:48 +0000
>> > Ben Hutchings <ben@xxxxxxxxxxxxxxx> wrote:
>> >
>> >> SIS 760 is listed in the device tables for both amd64-agp and sis-agp.
>> >> amd64-agp is apparently preferable since it has workarounds for some
>> >> BIOS misconfigurations that sis-agp doesn't handle.
>> >>
>>
>> Will this not cause problems in 32-bit kernels?
>
> I'm not sure. I thought that amd64-agp works on 32-bit kernels since
> fix_northbridge() in amd64-agp.c has the comment that the function is
> used on 64-bit and 32-bit kernels. We need someone who can test the
> patch.
>

Ah yes the amd64-agp bit scared me again, I thought it was a 64-bit only
module, but it seems to get used on 32-bit as well.

Dave.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/