On 08/12/15 17:45, Anshul Garg wrote: > Use list_for_each_entry_safe instead of list_for_each_safe > and list_entry call. > > Signed-off-by: Anshul Garg <aksgarg1989@xxxxxxxxx> Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilder to play with it. Thanks, Jonathan > --- > drivers/iio/inkern.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/iio/inkern.c b/drivers/iio/inkern.c > index c8bad3c..f764eb6 100644 > --- a/drivers/iio/inkern.c > +++ b/drivers/iio/inkern.c > @@ -61,12 +61,10 @@ EXPORT_SYMBOL_GPL(iio_map_array_register); > int iio_map_array_unregister(struct iio_dev *indio_dev) > { > int ret = -ENODEV; > - struct iio_map_internal *mapi; > - struct list_head *pos, *tmp; > + struct iio_map_internal *mapi, *next; > > mutex_lock(&iio_map_list_lock); > - list_for_each_safe(pos, tmp, &iio_map_list) { > - mapi = list_entry(pos, struct iio_map_internal, l); > + list_for_each_entry_safe(mapi, next, &iio_map_list, l) { > if (indio_dev == mapi->indio_dev) { > list_del(&mapi->l); > kfree(mapi); > -- 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