diff options
author | Linus Walleij <linus.walleij@stericsson.com> | 2010-10-21 15:06:44 -0400 |
---|---|---|
committer | Grant Likely <grant.likely@secretlab.ca> | 2010-10-22 11:51:05 -0400 |
commit | eb288a1f45e2aa903ac8edf67dc6d59df0369fe1 (patch) | |
tree | f43886e9b374065dd29d930116f58d5ef47524fc | |
parent | 73e1ac16258fe7e55cce4691e32c0b44d4579e08 (diff) |
spi: fixed odd static string conventions in core code
This patch removes convention of passing a static string as a
parameter to another static string. The convention is intended to
reduce text usage by sharing the common bits of the string, but the
implementation is inherently fragile (a change to one format string
but not the other will nullify any possible advantage), it isn't
necessarily a net win depending on what this compiler does, and it
it reduces code readability.
Signed-off-by: Linus Walleij <linus.walleij@stericsson.com>
[grant.likely@secretlab.ca: removed dev_dbg->dev_err hunk]
Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
-rw-r--r-- | drivers/spi/spi.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index 0845cd4c5155..709c836607de 100644 --- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c | |||
@@ -300,16 +300,16 @@ int spi_add_device(struct spi_device *spi) | |||
300 | */ | 300 | */ |
301 | status = spi_setup(spi); | 301 | status = spi_setup(spi); |
302 | if (status < 0) { | 302 | if (status < 0) { |
303 | dev_err(dev, "can't %s %s, status %d\n", | 303 | dev_err(dev, "can't setup %s, status %d\n", |
304 | "setup", dev_name(&spi->dev), status); | 304 | dev_name(&spi->dev), status); |
305 | goto done; | 305 | goto done; |
306 | } | 306 | } |
307 | 307 | ||
308 | /* Device may be bound to an active driver when this returns */ | 308 | /* Device may be bound to an active driver when this returns */ |
309 | status = device_add(&spi->dev); | 309 | status = device_add(&spi->dev); |
310 | if (status < 0) | 310 | if (status < 0) |
311 | dev_err(dev, "can't %s %s, status %d\n", | 311 | dev_err(dev, "can't add %s, status %d\n", |
312 | "add", dev_name(&spi->dev), status); | 312 | dev_name(&spi->dev), status); |
313 | else | 313 | else |
314 | dev_dbg(dev, "registered child %s\n", dev_name(&spi->dev)); | 314 | dev_dbg(dev, "registered child %s\n", dev_name(&spi->dev)); |
315 | 315 | ||
@@ -658,7 +658,7 @@ int spi_setup(struct spi_device *spi) | |||
658 | */ | 658 | */ |
659 | bad_bits = spi->mode & ~spi->master->mode_bits; | 659 | bad_bits = spi->mode & ~spi->master->mode_bits; |
660 | if (bad_bits) { | 660 | if (bad_bits) { |
661 | dev_dbg(&spi->dev, "setup: unsupported mode bits %x\n", | 661 | dev_err(&spi->dev, "setup: unsupported mode bits %x\n", |
662 | bad_bits); | 662 | bad_bits); |
663 | return -EINVAL; | 663 | return -EINVAL; |
664 | } | 664 | } |