diff options
author | Dominik Brodowski <linux@dominikbrodowski.net> | 2008-08-03 05:58:53 -0400 |
---|---|---|
committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2008-08-22 20:29:55 -0400 |
commit | 610e23749e87920136e1a221266a43cb7e3823f1 (patch) | |
tree | 5696af5b6e7be04e17a852e6c810798930a9b6c4 /drivers/pcmcia/cs.c | |
parent | 635d19bea0e91df473a81391ec8f3db2d049a218 (diff) |
pcmcia: deprecate CS_BAD_ATTRIBUTE, CS_BAD_TYPE and CS_BAD_PAGE
CS_BAD_TYPE was only used in cs.c and already properly annotated by error
messages. CS_BAD_ATTRIBUTE and CS_BAD_PAGE mean a badly written driver, so
ds_dbg() output and -EINVAL seems to be enough.
(includes bugfix from and
Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
)
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia/cs.c')
-rw-r--r-- | drivers/pcmcia/cs.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/pcmcia/cs.c b/drivers/pcmcia/cs.c index 1c6c8c8f57c9..48386f31e610 100644 --- a/drivers/pcmcia/cs.c +++ b/drivers/pcmcia/cs.c | |||
@@ -451,7 +451,7 @@ static int socket_setup(struct pcmcia_socket *skt, int initial_delay) | |||
451 | if (status & SS_CARDBUS) { | 451 | if (status & SS_CARDBUS) { |
452 | if (!(skt->features & SS_CAP_CARDBUS)) { | 452 | if (!(skt->features & SS_CAP_CARDBUS)) { |
453 | cs_err(skt, "cardbus cards are not supported.\n"); | 453 | cs_err(skt, "cardbus cards are not supported.\n"); |
454 | return CS_BAD_TYPE; | 454 | return -EINVAL; |
455 | } | 455 | } |
456 | skt->state |= SOCKET_CARDBUS; | 456 | skt->state |= SOCKET_CARDBUS; |
457 | } | 457 | } |
@@ -465,7 +465,7 @@ static int socket_setup(struct pcmcia_socket *skt, int initial_delay) | |||
465 | skt->socket.Vcc = skt->socket.Vpp = 50; | 465 | skt->socket.Vcc = skt->socket.Vpp = 50; |
466 | else { | 466 | else { |
467 | cs_err(skt, "unsupported voltage key.\n"); | 467 | cs_err(skt, "unsupported voltage key.\n"); |
468 | return CS_BAD_TYPE; | 468 | return -EIO; |
469 | } | 469 | } |
470 | 470 | ||
471 | if (skt->power_hook) | 471 | if (skt->power_hook) |
@@ -482,7 +482,7 @@ static int socket_setup(struct pcmcia_socket *skt, int initial_delay) | |||
482 | skt->ops->get_status(skt, &status); | 482 | skt->ops->get_status(skt, &status); |
483 | if (!(status & SS_POWERON)) { | 483 | if (!(status & SS_POWERON)) { |
484 | cs_err(skt, "unable to apply power.\n"); | 484 | cs_err(skt, "unable to apply power.\n"); |
485 | return CS_BAD_TYPE; | 485 | return -EIO; |
486 | } | 486 | } |
487 | 487 | ||
488 | status = socket_reset(skt); | 488 | status = socket_reset(skt); |