Hi Hans, On Sat, 6 Jun 2009 15:00:48 +0200, Hans Verkuil wrote: > For video4linux we sometimes need to probe for a single i2c address. > Normally you would do it like this: > > static const unsigned short addrs[] = { > addr, I2C_CLIENT_END > }; > > client = i2c_new_probed_device(adapter, &info, addrs); > > This is a bit awkward and I came up with this macro: > > #define V4L2_I2C_ADDRS(addr, addrs...) \ > ((const unsigned short []){ addr, ## addrs, I2C_CLIENT_END }) > > This can construct a list of one or more i2c addresses on the fly. But this > is something that really belongs in i2c.h, renamed to I2C_ADDRS. > > With this macro we can just do: > > client = i2c_new_probed_device(adapter, &info, I2C_ADDRS(addr)); > > Comments? I'm not a big fan of macros which hide how things work, but if this makes your life easier, why not. Just send a patch and I'll queue it up for 2.6.31. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html