On Thu, 17 Aug 2017 07:55:45 -0400 Harinath Nampally <harinath922@xxxxxxxxx> wrote: > > This patch fixes by detaching the event related information from > > chip_info struct, > >>> and based on channel type and event direction the corresponding > > event configuration registers > >>> are picked dynamically. Hence multiple events can be handled in > > read/write callbacks. > >> which chip can have which event(s)? > > I am planning to add 'supported events' field in One small point. Don't put the word bugfix in the title (and fix spelling of enable!). I know this is obviously a false restriction on the driver, but it doesn't not work, it is just limited in features without this. This issue is that this is not really material that should be going into stable kernels. It's an improvement though so good to have it! Jonathan > > > > struct mma_chip_info which indicates which chip can have which events. > > During initialization in 'mma_chip_info_table' would set this > > 'supported events' field for each chip. > > But I wonder should I add those changes as part of this patch? > > is it necessary or can it be documentation? > I think its not necessary as we only have Freefall and Transient events > for now. > Ok I will just update the documentation. > > > > And this patch should have been called "v2". please include a persistent version history to v3 of this patch. > Sure I will send v3 patch, should I use '--in-reply-to' option of git > send-email to send v3 patch as reply to > original thread? > > On 08/17/2017 07:24 AM, Martin Kepplinger wrote: > >>>> This patch fixes by detaching the event related information from > >> chip_info struct, > >>>> and based on channel type and event direction the corresponding > >> event configuration registers > >>>> are picked dynamically. Hence multiple events can be handled in > >> read/write callbacks. > >>> which chip can have which event(s)? > >> I am planning to add 'supported events' field in > >> > >> struct mma_chip_info which indicates which chip can have which events. > >> During initialization in 'mma_chip_info_table' would set this > >> 'supported events' field for each chip. > >> But I wonder should I add those changes as part of this patch? > > is it necessary or can it be documentation? > > > > And this patch should have been called "v2". please include a persistent version history to v3 of this patch. > -- 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
