[Openchrome-devel] pVia->Chipset versus pVia->ChipId
Xavier Bachelot
xavier
Sat Mar 1 04:22:57 PST 2008
Thomas Hellstr?m wrote:
> Xavier Bachelot wrote:
>> Hi all,
>>
>> While looking at via_xvmc.c, I found out that chips are either matched
>> with, for example, (pVia->Chipset == VIA_PM800) or (pVia->ChipId ==
>> PCI_CHIP_VT3259). It looks like the same information to me and I find
>> this to be confusing. Couldn't we choose one and get rid of the other
>> or am I missing something ?
>>
>> Regards,
>> Xavier
>>
> I think it should be sufficient to save keep pVia->ChipId.
> /Thomas
Is there any reason to prefer pVia->ChipId over pVia->Chipset ?
pVia->Chipset is more readable imho. fwiw, in the whole code, there are
864 lines containing pVia->Chipset versus 297 with pVia->ChipId.
Regards,
Xavier
More information about the Openchrome-devel
mailing list