Hi Jean > This file is part of the Linux kernel, not the lm_sensors project. We do > not modify it. The correct thing to do is to define the value in the > driver file itself, near the top. If you look at other drivers such as > i2c-i801, that's what they do. > > So you'd simply add something like: > > #ifndef PCI_DEVICE_ID_AL_M1563 > #define PCI_DEVICE_ID_AL_M1563 0x1563 > #endif > > near the top of i2c-ali1563.c, and leave pci_ids.h unchanged. Yes, you are right, thanks Best Regards Chunhao ===========================================================================================The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original author of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such person, please kindly reply the sender indicating accordingly and delete all copies of it from your computer and network server immediately. We thank you for your cooperation. It is advisable that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email that does not relate to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.===========================================================================================If your computer is unable to decode Chinese font, please ignore the following message. They essentially repea! t the English statement above.???H???????t?????q?l???]???????K?????T, ?????v???o?H?H???w?????H?H???\????. ?????z???D?Q???w?????H?H???]???????]?b???g???v?????????U???????H??, ???z?i?????o?H?H?????Y?N?H???q?q???P???????A???????H????. ?????z???X?@, ?????????P??. ?S??????, ???????g???v?????????????q?l?????K???T???????O?Q?Y???T????. ?H???P?????q?l???~?L???????e,???o?????????q?l?????????N??.