diff options
| author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-02-21 17:12:17 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-02-21 17:12:17 +0100 |
| commit | 7f2b8af282fb73e808fa0e7743f51b8632130d33 (patch) | |
| tree | d07b705e9228cfdcfbe99cb32c9597709fbd3d39 /drivers/hwtracing/intel_th/pti.c | |
| parent | 37fd0b623023484ef6df79ed46f21f06ecc611ff (diff) | |
| parent | bf7cbaae0831252b416f375ca9b1027ecd4642dd (diff) | |
| download | linux-7f2b8af282fb73e808fa0e7743f51b8632130d33.tar.gz linux-7f2b8af282fb73e808fa0e7743f51b8632130d33.tar.bz2 linux-7f2b8af282fb73e808fa0e7743f51b8632130d33.zip | |
Merge tag 'intel_th-stm-for-greg-20190221' of git://git.kernel.org/pub/scm/linux/kernel/git/ash/stm into char-misc-next
Alexander writes:
stm class/intel_th: Updates for v5.1
These are:
* 2 bugfixes in stm class
* one bugfix in intel_th
* a few minor cleanups
* tag 'intel_th-stm-for-greg-20190221' of git://git.kernel.org/pub/scm/linux/kernel/git/ash/stm:
stm class: Prevent division by zero
stm class: Fix an endless loop in channel allocation
intel_th: Don't reference unassigned outputs
intel_th: pti: Use sysfs_match_string() helper
intel_th: Only create useful device nodes
intel_th: Mark expected switch fall-throughs
intel_th: Update ABI documentation
Diffstat (limited to 'drivers/hwtracing/intel_th/pti.c')
| -rw-r--r-- | drivers/hwtracing/intel_th/pti.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/drivers/hwtracing/intel_th/pti.c b/drivers/hwtracing/intel_th/pti.c index 56694339cb06..0da6b787f553 100644 --- a/drivers/hwtracing/intel_th/pti.c +++ b/drivers/hwtracing/intel_th/pti.c @@ -272,19 +272,17 @@ static ssize_t lpp_dest_store(struct device *dev, struct device_attribute *attr, const char *buf, size_t size) { struct pti_device *pti = dev_get_drvdata(dev); - ssize_t ret = -EINVAL; int i; - for (i = 0; i < ARRAY_SIZE(lpp_dest_str); i++) - if (sysfs_streq(buf, lpp_dest_str[i])) - break; + i = sysfs_match_string(lpp_dest_str, buf); + if (i < 0) + return i; - if (i < ARRAY_SIZE(lpp_dest_str) && pti->lpp_dest_mask & BIT(i)) { - pti->lpp_dest = i; - ret = size; - } + if (!(pti->lpp_dest_mask & BIT(i))) + return -EINVAL; - return ret; + pti->lpp_dest = i; + return size; } static DEVICE_ATTR_RW(lpp_dest); |
