On Fri, Apr 25, 2008 at 09:35:47AM +0200, Jaroslav Kysela wrote: > 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 *'. Need to pass irq data will be fairly common so we should probably have it in its own right and allow additional data be attached via a void *. -- Dmitry -- 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