On 28 November 2014 at 08:10, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: > Michal Kazior <michal.kazior@xxxxxxxxx> writes: > >> Hmm.. I have a couple of ideas: > > [...] > >> 2. Have a dedicatd pci-specific structure: >> >> struct ath10k_pci_supported_chip { >> u16 dev_id; >> u32 chip_id; >> }; >> >> struct ath10k_pci_supported_chip ath10k_pci_supported_chips[] = { >> { QCA988X_2_0_DEVICE_ID, QCA988X_HW_2_0_CHIP_ID_REV }, >> // ... >> }; >> >> Probably the simplest and has least impact. > > Yeah, this seems to be the best. Can you send v2? Ok. I'll go with this approach and post a v2 when I'm done. Michał -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html