* Gadiyar, Anand <gadiyar@xxxxxx> [071209 20:16]: > Hi all, > > I'm resending this patch with the #else removed. I'm also sending a few > other ways in which we could do this. > > Option 1: > Same patch as the last time without the #else > > This contains lots of #ifdef statements but there are no #elses Yeah, not good.. > Option 2: > No #ifdefs. > > Using OMAP242X_* for 2420 specific names > OMAP243X_* for 2430 specific names/names present from 243X onwards > OMAP24XX_* for names common to all 24XX, 34XX > OMAP34XX_* for 34XX specific names > > (or something similar could be worked out) This is better on short term. It needs an associated patch to change drivers to use new names. > Option 3: > > Name all possible names OMAP_* (or OMAP2_*) and use a runtime check to return > the correct request line number. > > I'm sending sample patches for all three options. Let me know what you think. Hey, this looks like a good solution in the long run. Except, let's just load the table for each processor that is compiled in to save memory. You could load the associated processor dma request configuration from devices_2420.c, devices_2430.c and devices_3430.c. Regards, Tony - To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
