diff options
| author | Stephen Boyd <sboyd@codeaurora.org> | 2015-06-01 18:47:54 -0700 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2015-06-03 13:19:55 +0100 |
| commit | ff268b56ce8c86be920e1e6a15d68b168bed66c9 (patch) | |
| tree | 2556c2994985836f26b3fccbbc063052a6481c71 | |
| parent | b787f68c36d49bb1d9236f403813641efa74a031 (diff) | |
| download | lwn-ff268b56ce8c86be920e1e6a15d68b168bed66c9.tar.gz lwn-ff268b56ce8c86be920e1e6a15d68b168bed66c9.zip | |
regulator: core: Don't spew backtraces on duplicate sysfs
We don't consider a failure to add the sysfs node as a problem,
so use sysfs_create_link_nowarn() so that we don't print a
backtrace when duplicated files exist. Also, downgrade the printk
message to a debug statement so that we're quiet here. This
allows multiple drivers to request a CPU's regulator so that
CPUfreq and AVSish drivers can coexist.
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 443eaab933fc..f6989485c382 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -1192,10 +1192,10 @@ static struct regulator *create_regulator(struct regulator_dev *rdev, if (regulator->supply_name == NULL) goto overflow_err; - err = sysfs_create_link(&rdev->dev.kobj, &dev->kobj, + err = sysfs_create_link_nowarn(&rdev->dev.kobj, &dev->kobj, buf); if (err) { - rdev_warn(rdev, "could not add device link %s err %d\n", + rdev_dbg(rdev, "could not add device link %s err %d\n", dev->kobj.name, err); /* non-fatal */ } |