diff options
author | Kevin Hilman <khilman@deeprootsystems.com> | 2010-02-24 14:05:45 -0500 |
---|---|---|
committer | Paul Walmsley <paul@pwsan.com> | 2010-02-24 14:05:45 -0500 |
commit | 24d82e3421a48a0db68026275dca64537291cf8f (patch) | |
tree | 52361cd142ac654bd3c2316fefc97238ce322193 /arch | |
parent | 0007122ad85cc36b1c18c0b59344093ca210d206 (diff) |
OMAP: omap_device: when 'called from invalid state', print state
The omap_device_[enable|idle|shutdown] functions print a warning
when called from an invalid state. Print the invalid state in
the warning messages. This also uses __func__ to get the function
name.
Also, move the entire print string onto a single line to facilitate
grepping or error messages. Recent discussions on LKML show
strong preference for grep-able code vs. strict 80 column limit.
Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
Signed-off-by: Paul Walmsley <paul@pwsan.com>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/arm/plat-omap/omap_device.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/arm/plat-omap/omap_device.c b/arch/arm/plat-omap/omap_device.c index 40289e095fc6..c739a046cc02 100644 --- a/arch/arm/plat-omap/omap_device.c +++ b/arch/arm/plat-omap/omap_device.c | |||
@@ -466,8 +466,8 @@ int omap_device_enable(struct platform_device *pdev) | |||
466 | od = _find_by_pdev(pdev); | 466 | od = _find_by_pdev(pdev); |
467 | 467 | ||
468 | if (od->_state == OMAP_DEVICE_STATE_ENABLED) { | 468 | if (od->_state == OMAP_DEVICE_STATE_ENABLED) { |
469 | WARN(1, "omap_device: %s.%d: omap_device_enable() called from " | 469 | WARN(1, "omap_device: %s.%d: %s() called from invalid state %d\n", |
470 | "invalid state\n", od->pdev.name, od->pdev.id); | 470 | od->pdev.name, od->pdev.id, __func__, od->_state); |
471 | return -EINVAL; | 471 | return -EINVAL; |
472 | } | 472 | } |
473 | 473 | ||
@@ -505,8 +505,8 @@ int omap_device_idle(struct platform_device *pdev) | |||
505 | od = _find_by_pdev(pdev); | 505 | od = _find_by_pdev(pdev); |
506 | 506 | ||
507 | if (od->_state != OMAP_DEVICE_STATE_ENABLED) { | 507 | if (od->_state != OMAP_DEVICE_STATE_ENABLED) { |
508 | WARN(1, "omap_device: %s.%d: omap_device_idle() called from " | 508 | WARN(1, "omap_device: %s.%d: %s() called from invalid state %d\n", |
509 | "invalid state\n", od->pdev.name, od->pdev.id); | 509 | od->pdev.name, od->pdev.id, __func__, od->_state); |
510 | return -EINVAL; | 510 | return -EINVAL; |
511 | } | 511 | } |
512 | 512 | ||
@@ -538,8 +538,8 @@ int omap_device_shutdown(struct platform_device *pdev) | |||
538 | 538 | ||
539 | if (od->_state != OMAP_DEVICE_STATE_ENABLED && | 539 | if (od->_state != OMAP_DEVICE_STATE_ENABLED && |
540 | od->_state != OMAP_DEVICE_STATE_IDLE) { | 540 | od->_state != OMAP_DEVICE_STATE_IDLE) { |
541 | WARN(1, "omap_device: %s.%d: omap_device_shutdown() called " | 541 | WARN(1, "omap_device: %s.%d: %s() called from invalid state %d\n", |
542 | "from invalid state\n", od->pdev.name, od->pdev.id); | 542 | od->pdev.name, od->pdev.id, __func__, od->_state); |
543 | return -EINVAL; | 543 | return -EINVAL; |
544 | } | 544 | } |
545 | 545 | ||