aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@stericsson.com>2010-10-21 15:06:44 -0400
committerGrant Likely <grant.likely@secretlab.ca>2010-10-22 11:51:05 -0400
commiteb288a1f45e2aa903ac8edf67dc6d59df0369fe1 (patch)
treef43886e9b374065dd29d930116f58d5ef47524fc
parent73e1ac16258fe7e55cce4691e32c0b44d4579e08 (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.c10
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 }