On Wed, Jan 12, 2011 at 11:20:38AM +0000, Jonathan Cameron wrote: > On 01/12/11 09:44, Roland Stigge wrote: > > Hi, > > > > On 01/11/2011 08:34 PM, Jonathan Cameron wrote: > >>> On 01/09/2011 11:31 PM, Jonathan Cameron wrote: > >>>> Pretty clean and nice driver so it was an easy review and should > >>>> be trivial to fix up for a merge. > >>> > >>> Thanks for your review! > >>> > >>> I'm attaching my update. Some notes: > >>> > >>> * I used the attributes out1_* how I found it in the other DAC driver > >>> already available > >> Cool, except for the out12_raw which could easily be the 12th channel. > >> This has to be one of the two options suggested the other day. > >> May require some macro magic. IIRC the relevant macro would be. > >> > >> IIO_DEVICE_ATTR_NAMED(out1and2_raw, out1&2_raw, S_IWUSR, ... > > > > Update attached as suggested. :-) > > > > Thanks! > > > > Roland > > Probably worth sending it to Greg KH in a new thread so that he > picks up on the fact that it is ready to merge. Given the volume > of email he probably gets I doubt he is still reading this thread! > No need to cc that to the list given we have this email. Heh, I'm still here reading this :) Roland sent me an updated version, I'll queue that up after .38-rc1 is out. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html