On Fri, 25 Apr 2008, Takashi Iwai wrote: > > Sure. I applied the simple 'void *device_private_data' patch, because > > current usage request is really trivial. We can implement complex code to > > handle data for multiple "extra" devices on AC97 bus later. > > Actually, it's not "used" yet. The ucb1000 reads the data but no one > stores yet. And, if its usage request is trivial, we should use "int Yes, I hope that the appropriate initialization code will be added to SoC drivers, too. > irq" as in the original patch instead of void data and cast. But other SoC (or other) drivers might want to pass to extra devices on AC97 bus something different or more complex. Mark Brown already noted that. I would keep it as 'void *'. Jaroslav ----- Jaroslav Kysela <perex@xxxxxxxx> Linux Kernel Sound Maintainer ALSA Project, Red Hat, Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html