diff options
author | Dominik Brodowski <linux@dominikbrodowski.net> | 2009-10-24 09:45:06 -0400 |
---|---|---|
committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2009-11-08 12:23:15 -0500 |
commit | 9ac3e58ceff0b7b8b981c09c38a28742270eea12 (patch) | |
tree | 04f6137d3128750f5d2ff15c0a52cb17e6c00b35 /drivers | |
parent | 444486a5f9d2737b50e53dc140292899b9497808 (diff) |
pcmcia: deprecate CS_CHECK (bluetooth)
Remove all usages of the CS_CHECK macro and replace them with proper
Linux style calling and return value checking. The extra error reporting may
be dropped, as the PCMCIA core already complains about any (non-driver-author)
errors.
CC: linux-bluetooth@vger.kernel.org
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/bluetooth/bluecard_cs.c | 12 | ||||
-rw-r--r-- | drivers/bluetooth/bt3c_cs.c | 9 | ||||
-rw-r--r-- | drivers/bluetooth/btuart_cs.c | 9 | ||||
-rw-r--r-- | drivers/bluetooth/dtl1_cs.c | 8 |
4 files changed, 9 insertions, 29 deletions
diff --git a/drivers/bluetooth/bluecard_cs.c b/drivers/bluetooth/bluecard_cs.c index b0e569ba730d..1e0c4d822972 100644 --- a/drivers/bluetooth/bluecard_cs.c +++ b/drivers/bluetooth/bluecard_cs.c | |||
@@ -905,22 +905,16 @@ static int bluecard_config(struct pcmcia_device *link) | |||
905 | break; | 905 | break; |
906 | } | 906 | } |
907 | 907 | ||
908 | if (i != 0) { | 908 | if (i != 0) |
909 | cs_error(link, RequestIO, i); | ||
910 | goto failed; | 909 | goto failed; |
911 | } | ||
912 | 910 | ||
913 | i = pcmcia_request_irq(link, &link->irq); | 911 | i = pcmcia_request_irq(link, &link->irq); |
914 | if (i != 0) { | 912 | if (i != 0) |
915 | cs_error(link, RequestIRQ, i); | ||
916 | link->irq.AssignedIRQ = 0; | 913 | link->irq.AssignedIRQ = 0; |
917 | } | ||
918 | 914 | ||
919 | i = pcmcia_request_configuration(link, &link->conf); | 915 | i = pcmcia_request_configuration(link, &link->conf); |
920 | if (i != 0) { | 916 | if (i != 0) |
921 | cs_error(link, RequestConfiguration, i); | ||
922 | goto failed; | 917 | goto failed; |
923 | } | ||
924 | 918 | ||
925 | if (bluecard_open(info) != 0) | 919 | if (bluecard_open(info) != 0) |
926 | goto failed; | 920 | goto failed; |
diff --git a/drivers/bluetooth/bt3c_cs.c b/drivers/bluetooth/bt3c_cs.c index d58e22b9f06a..9787fda45d84 100644 --- a/drivers/bluetooth/bt3c_cs.c +++ b/drivers/bluetooth/bt3c_cs.c | |||
@@ -740,21 +740,16 @@ static int bt3c_config(struct pcmcia_device *link) | |||
740 | goto found_port; | 740 | goto found_port; |
741 | 741 | ||
742 | BT_ERR("No usable port range found"); | 742 | BT_ERR("No usable port range found"); |
743 | cs_error(link, RequestIO, -ENODEV); | ||
744 | goto failed; | 743 | goto failed; |
745 | 744 | ||
746 | found_port: | 745 | found_port: |
747 | i = pcmcia_request_irq(link, &link->irq); | 746 | i = pcmcia_request_irq(link, &link->irq); |
748 | if (i != 0) { | 747 | if (i != 0) |
749 | cs_error(link, RequestIRQ, i); | ||
750 | link->irq.AssignedIRQ = 0; | 748 | link->irq.AssignedIRQ = 0; |
751 | } | ||
752 | 749 | ||
753 | i = pcmcia_request_configuration(link, &link->conf); | 750 | i = pcmcia_request_configuration(link, &link->conf); |
754 | if (i != 0) { | 751 | if (i != 0) |
755 | cs_error(link, RequestConfiguration, i); | ||
756 | goto failed; | 752 | goto failed; |
757 | } | ||
758 | 753 | ||
759 | if (bt3c_open(info) != 0) | 754 | if (bt3c_open(info) != 0) |
760 | goto failed; | 755 | goto failed; |
diff --git a/drivers/bluetooth/btuart_cs.c b/drivers/bluetooth/btuart_cs.c index efd689a062eb..f44d75217b2b 100644 --- a/drivers/bluetooth/btuart_cs.c +++ b/drivers/bluetooth/btuart_cs.c | |||
@@ -669,21 +669,16 @@ static int btuart_config(struct pcmcia_device *link) | |||
669 | goto found_port; | 669 | goto found_port; |
670 | 670 | ||
671 | BT_ERR("No usable port range found"); | 671 | BT_ERR("No usable port range found"); |
672 | cs_error(link, RequestIO, -ENODEV); | ||
673 | goto failed; | 672 | goto failed; |
674 | 673 | ||
675 | found_port: | 674 | found_port: |
676 | i = pcmcia_request_irq(link, &link->irq); | 675 | i = pcmcia_request_irq(link, &link->irq); |
677 | if (i != 0) { | 676 | if (i != 0) |
678 | cs_error(link, RequestIRQ, i); | ||
679 | link->irq.AssignedIRQ = 0; | 677 | link->irq.AssignedIRQ = 0; |
680 | } | ||
681 | 678 | ||
682 | i = pcmcia_request_configuration(link, &link->conf); | 679 | i = pcmcia_request_configuration(link, &link->conf); |
683 | if (i != 0) { | 680 | if (i != 0) |
684 | cs_error(link, RequestConfiguration, i); | ||
685 | goto failed; | 681 | goto failed; |
686 | } | ||
687 | 682 | ||
688 | if (btuart_open(info) != 0) | 683 | if (btuart_open(info) != 0) |
689 | goto failed; | 684 | goto failed; |
diff --git a/drivers/bluetooth/dtl1_cs.c b/drivers/bluetooth/dtl1_cs.c index b881a9cd8741..7cd8614a8ea9 100644 --- a/drivers/bluetooth/dtl1_cs.c +++ b/drivers/bluetooth/dtl1_cs.c | |||
@@ -622,16 +622,12 @@ static int dtl1_config(struct pcmcia_device *link) | |||
622 | goto failed; | 622 | goto failed; |
623 | 623 | ||
624 | i = pcmcia_request_irq(link, &link->irq); | 624 | i = pcmcia_request_irq(link, &link->irq); |
625 | if (i != 0) { | 625 | if (i != 0) |
626 | cs_error(link, RequestIRQ, i); | ||
627 | link->irq.AssignedIRQ = 0; | 626 | link->irq.AssignedIRQ = 0; |
628 | } | ||
629 | 627 | ||
630 | i = pcmcia_request_configuration(link, &link->conf); | 628 | i = pcmcia_request_configuration(link, &link->conf); |
631 | if (i != 0) { | 629 | if (i != 0) |
632 | cs_error(link, RequestConfiguration, i); | ||
633 | goto failed; | 630 | goto failed; |
634 | } | ||
635 | 631 | ||
636 | if (dtl1_open(info) != 0) | 632 | if (dtl1_open(info) != 0) |
637 | goto failed; | 633 | goto failed; |