[Openchrome-devel] pVia->Chipset versus pVia->ChipId

[Xavier Bachelot]

Xavier Bachelot wrote:
> 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.
> 
oops, I got the numbers completely wrong (including the .svn files).
The real numbers are 94 for pVia->Chipset and 39 for pVia->ChipId.

X.