On Fri, 2017-06-09 at 15:07 +0300, Andy Shevchenko wrote: > Use sysfs_match_string() helper instead of open coded variant.
Did I miss maintainer? > > Cc: "Michael S. Tsirkin" <[email protected]> > Cc: Jason Wang <[email protected]> > Signed-off-by: Andy Shevchenko <[email protected]> > --- > drivers/block/virtio_blk.c | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c > index 553cc4c542b4..0e707b8cce9d 100644 > --- a/drivers/block/virtio_blk.c > +++ b/drivers/block/virtio_blk.c > @@ -541,12 +541,9 @@ virtblk_cache_type_store(struct device *dev, > struct device_attribute *attr, > int i; > > BUG_ON(!virtio_has_feature(vblk->vdev, > VIRTIO_BLK_F_CONFIG_WCE)); > - for (i = ARRAY_SIZE(virtblk_cache_types); --i >= 0; ) > - if (sysfs_streq(buf, virtblk_cache_types[i])) > - break; > - > + i = sysfs_match_string(virtblk_cache_types, buf); > if (i < 0) > - return -EINVAL; > + return i; > > virtio_cwrite8(vdev, offsetof(struct virtio_blk_config, wce), > i); > virtblk_update_cache_mode(vdev); -- Andy Shevchenko <[email protected]> Intel Finland Oy _______________________________________________ Virtualization mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/virtualization
