lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 19 Jan 2015 02:53:49 +0000 From: 敬锐 <micky_ching@...lsil.com.cn> To: Lee Jones <lee.jones@...aro.org> CC: "sameo@...ux.intel.com" <sameo@...ux.intel.com>, "devel@...uxdriverproject.org" <devel@...uxdriverproject.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>, "rogerable@...ltek.com" <rogerable@...ltek.com>, 王炜 <wei_wang@...lsil.com.cn> Subject: Re: [PATCH 09/10] mfd: rtsx: add support for rts525A On 01/18/2015 07:13 PM, Lee Jones wrote: >> @@ -97,7 +97,7 @@ static void rts5249_force_power_down(struct rtsx_pcr *pcr, u8 pm_state) >> > rtsx_pci_write_register(pcr, AUTOLOAD_CFG_BASE + 3, 0x01, 0); >> > >> > if (pm_state == HOST_ENTER_S3) { >> >- if (PCI_PID(pcr) == 0x524A) >> >+ if (PCI_PID(pcr) == 0x524A || PCI_PID(pcr) == 0x525A) > Shouldn't these be defined somewhere? > > I have a particular distaste for magic numbers. > This is the chip ID number, no need define, just using it. if we define, it will like RTS525A_PCI_ID, so bad.
Powered by blists - more mailing lists