Okay, thanks for your patience with my lack of understanding. I'll work with Hariprasad tomorrow to get a revised patch out with the "const" change and eliminate the redundant check of CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN to guard the file contents. Thanks for caching these improvements!
And given that it's pretty empty at work here on this holiday and how beautiful it is outside, I think it's time to head home to enjoy lunch outside! Casey ________________________________________ From: Joe Perches [j...@perches.com] Sent: Monday, February 16, 2015 11:18 AM To: Casey Leedom Cc: Hariprasad S; James E.J. Bottomley; net...@vger.kernel.org; linux-kernel@vger.kernel.org; linux-scsi; David Miller Subject: Re: chelsio: Use a more common const struct pci_device_id foo[] style On Mon, 2015-02-16 at 19:07 +0000, Casey Leedom wrote: > I understand that OS-independence issues aren't something which are > normally accommodated, but as long as definitions don't introduce > unnecessary "foreign intrusion" I would hope that it would be okay. > As I noted, our t4_regs.h file is also OS-independent and used by six > other OS device drivers. Putting Linux definitions into > t4_pci_id_tbl.h would be somewhat akin to injecting Linux dependencies > into t4_regs.h. > > But, if the change must be made, then we'll just maintain a > translation between our Common Code and the kernel.org code. If > that's the case, probably the best documentation for the proposed > CH_PCI_ID_TABLE_ENTRY_DATA might be something like: > > * CH_PCI_ID_TABLE_ENTRY_DATA > * -- Used for the individual PCI Device ID entries for the > PCI_VDEVICE() "dev" > * -- parameter. > > So it sounds like Chelsio would be required to make this change > then? I'm still unclear on the likes of responsibility/authority > here. We're being told that we must do this but we have to be the > ones requesting it? Sorry for my confusion. It's just a suggested patch. It's your code, you don't _have_ to do anything. > (Which is doubly apparent since I came into work this morning only to > realize that it's a company holiday. Color me a moron today.) Yay for holidays. It can be personally beneficial to take the day off. It could also be very productive to work with no distractions. As always, your choice... cheers, Joe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/