On Wed, Dec 7, 2016 at 3:06 AM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > On Tue, Dec 06, 2016 at 08:09:30PM -0600, Chris Bostic wrote: >> From: Jeremy Kerr <jk@xxxxxxxxxx> >> >> Signed-off-by: Jeremy Kerr <jk@xxxxxxxxxx> >> Signed-off-by: Chris Bostic <cbostic@xxxxxxxxxx> >> --- >> drivers/fsi/fsi-core.c | 20 ++++++++++++++++++++ >> drivers/fsi/fsi-master.h | 37 +++++++++++++++++++++++++++++++++++++ >> 2 files changed, 57 insertions(+) >> create mode 100644 drivers/fsi/fsi-master.h >> >> diff --git a/drivers/fsi/fsi-core.c b/drivers/fsi/fsi-core.c >> index 3d55bd5..ce9428d 100644 >> --- a/drivers/fsi/fsi-core.c >> +++ b/drivers/fsi/fsi-core.c >> @@ -17,6 +17,26 @@ >> #include <linux/fsi.h> >> #include <linux/module.h> >> >> +#include "fsi-master.h" >> + >> +static atomic_t master_idx = ATOMIC_INIT(-1); > > You don't really want/need an atomic variable, please use the simple ida > interface instead. Greg, Will make the change to simple ida interface. Thanks for your feedback, Chris > > thanks, > > greg k-h -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html