6 Jun
2023
6 Jun
'23
6:12 p.m.
On Tue, 06 Jun 2023 18:10:13 +0200, Mark Brown wrote:
On Tue, Jun 06, 2023 at 06:08:51PM +0200, Takashi Iwai wrote:
Shenghao Ding wrote:
- //Lenovo devices
- if ((subid == 0x387d) || (subid == 0x387e) || (subid == 0x3881)
|| (subid == 0x3884) || (subid == 0x3886) || (subid == 0x38a7)|| (subid == 0x38a8) || (subid == 0x38ba) || (subid == 0x38bb)|| (subid == 0x38be) || (subid == 0x38bf) || (subid == 0x38c3)|| (subid == 0x38cb) || (subid == 0x38cd))tas_priv->catlog_id = LENOVO;- else
tas_priv->catlog_id = OTHERS;Hmm, I don't like checking subid here, but we can live with it for now...
Might be more idiomatically/neatly written as a switch statement though.
Yes, it'll make it easier to add a new entry, too.
thanks,
Takashi