Skip to content

Commit 52739f0

Browse files
Simon GeisDominik Brodowski
authored andcommitted
PCMCIA/i82092: delete enter/leave macro
Remove the enter/leave macros and the corresponding calls. These are obsolete since ftrace is available. Co-developed-by: Lukas Panzer <[email protected]> Signed-off-by: Lukas Panzer <[email protected]> Signed-off-by: Simon Geis <[email protected]> Signed-off-by: Dominik Brodowski <[email protected]>
1 parent ac5af87 commit 52739f0

File tree

2 files changed

+6
-60
lines changed

2 files changed

+6
-60
lines changed

drivers/pcmcia/i82092.c

Lines changed: 6 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,6 @@ static int i82092aa_pci_probe(struct pci_dev *dev,
7676
unsigned char configbyte;
7777
int i, ret;
7878

79-
enter("i82092aa_pci_probe");
80-
8179
ret = pci_enable_device(dev);
8280
if (ret)
8381
return ret;
@@ -158,7 +156,6 @@ static int i82092aa_pci_probe(struct pci_dev *dev,
158156
goto err_out_free_sockets;
159157
}
160158

161-
leave("i82092aa_pci_probe");
162159
return 0;
163160

164161
err_out_free_sockets:
@@ -178,14 +175,10 @@ static void i82092aa_pci_remove(struct pci_dev *dev)
178175
{
179176
int i;
180177

181-
enter("i82092aa_pci_remove");
182-
183178
free_irq(dev->irq, i82092aa_interrupt);
184179

185180
for (i = 0; i < socket_count; i++)
186181
pcmcia_unregister_socket(&sockets[i].socket);
187-
188-
leave("i82092aa_pci_remove");
189182
}
190183

191184
static DEFINE_SPINLOCK(port_lock);
@@ -322,8 +315,6 @@ static irqreturn_t i82092aa_interrupt(int irq, void *dev)
322315

323316
unsigned int events, active = 0;
324317

325-
/* enter("i82092aa_interrupt");*/
326-
327318
while (1) {
328319
loopcount++;
329320
if (loopcount > 20) {
@@ -377,7 +368,6 @@ static irqreturn_t i82092aa_interrupt(int irq, void *dev)
377368
break;
378369
}
379370
return IRQ_RETVAL(handled);
380-
/* leave("i82092aa_interrupt");*/
381371
}
382372

383373

@@ -388,32 +378,25 @@ static int card_present(int socketno)
388378
{
389379
unsigned int val;
390380

391-
enter("card_present");
392-
393381
if ((socketno < 0) || (socketno >= MAX_SOCKETS))
394382
return 0;
395383
if (sockets[socketno].io_base == 0)
396384
return 0;
397385

398386

399387
val = indirect_read(socketno, 1); /* Interface status register */
400-
if ((val&12) == 12) {
401-
leave("card_present 1");
388+
if ((val&12) == 12)
402389
return 1;
403-
}
404390

405-
leave("card_present 0");
406391
return 0;
407392
}
408393

409394
static void set_bridge_state(int sock)
410395
{
411-
enter("set_bridge_state");
412396
indirect_write(sock, I365_GBLCTL, 0x00);
413397
indirect_write(sock, I365_GENCTL, 0x00);
414398

415399
indirect_setbit(sock, I365_INTCTL, 0x08);
416-
leave("set_bridge_state");
417400
}
418401

419402

@@ -424,8 +407,6 @@ static int i82092aa_init(struct pcmcia_socket *sock)
424407
pccard_io_map io = { 0, 0, 0, 0, 1 };
425408
pccard_mem_map mem = { .res = &res, };
426409

427-
enter("i82092aa_init");
428-
429410
for (i = 0; i < 2; i++) {
430411
io.map = i;
431412
i82092aa_set_io_map(sock, &io);
@@ -434,8 +415,7 @@ static int i82092aa_init(struct pcmcia_socket *sock)
434415
mem.map = i;
435416
i82092aa_set_mem_map(sock, &mem);
436417
}
437-
438-
leave("i82092aa_init");
418+
439419
return 0;
440420
}
441421

@@ -444,8 +424,6 @@ static int i82092aa_get_status(struct pcmcia_socket *socket, u_int *value)
444424
unsigned int sock = container_of(socket,
445425
struct socket_info, socket)->number;
446426
unsigned int status;
447-
448-
enter("i82092aa_get_status");
449427

450428
/* Interface Status Register */
451429
status = indirect_read(sock, I365_STATUS);
@@ -477,7 +455,6 @@ static int i82092aa_get_status(struct pcmcia_socket *socket, u_int *value)
477455
if (status & I365_CS_POWERON)
478456
(*value) |= SS_POWERON; /* power is applied to the card */
479457

480-
leave("i82092aa_get_status");
481458
return 0;
482459
}
483460

@@ -490,8 +467,6 @@ static int i82092aa_set_socket(struct pcmcia_socket *socket,
490467
unsigned int sock = sock_info->number;
491468
unsigned char reg;
492469

493-
enter("i82092aa_set_socket");
494-
495470
/* First, set the global controller options */
496471

497472
set_bridge_state(sock);
@@ -535,7 +510,6 @@ static int i82092aa_set_socket(struct pcmcia_socket *socket,
535510
dev_err(&sock_info->dev->dev,
536511
"%s called with invalid VCC power value: %i",
537512
__func__, state->Vcc);
538-
leave("i82092aa_set_socket");
539513
return -EINVAL;
540514
}
541515

@@ -557,7 +531,6 @@ static int i82092aa_set_socket(struct pcmcia_socket *socket,
557531
dev_err(&sock_info->dev->dev,
558532
"%s called with invalid VPP power value: %i",
559533
__func__, state->Vcc);
560-
leave("i82092aa_set_socket");
561534
return -EINVAL;
562535
}
563536

@@ -589,7 +562,6 @@ static int i82092aa_set_socket(struct pcmcia_socket *socket,
589562
indirect_write(sock, I365_CSCINT, reg);
590563
(void)indirect_read(sock, I365_CSC);
591564

592-
leave("i82092aa_set_socket");
593565
return 0;
594566
}
595567

@@ -601,20 +573,15 @@ static int i82092aa_set_io_map(struct pcmcia_socket *socket,
601573
unsigned int sock = sock_info->number;
602574
unsigned char map, ioctl;
603575

604-
enter("i82092aa_set_io_map");
605-
606576
map = io->map;
607577

608578
/* Check error conditions */
609-
if (map > 1) {
610-
leave("i82092aa_set_io_map with invalid map");
579+
if (map > 1)
611580
return -EINVAL;
612-
}
581+
613582
if ((io->start > 0xffff) || (io->stop > 0xffff)
614-
|| (io->stop < io->start)) {
615-
leave("i82092aa_set_io_map with invalid io");
583+
|| (io->stop < io->start))
616584
return -EINVAL;
617-
}
618585

619586
/* Turn off the window before changing anything */
620587
if (indirect_read(sock, I365_ADDRWIN) & I365_ENA_IO(map))
@@ -635,7 +602,6 @@ static int i82092aa_set_io_map(struct pcmcia_socket *socket,
635602
if (io->flags & MAP_ACTIVE)
636603
indirect_setbit(sock, I365_ADDRWIN, I365_ENA_IO(map));
637604

638-
leave("i82092aa_set_io_map");
639605
return 0;
640606
}
641607

@@ -649,20 +615,14 @@ static int i82092aa_set_mem_map(struct pcmcia_socket *socket,
649615
unsigned short base, i;
650616
unsigned char map;
651617

652-
enter("i82092aa_set_mem_map");
653-
654618
pcibios_resource_to_bus(sock_info->dev->bus, &region, mem->res);
655619

656620
map = mem->map;
657-
if (map > 4) {
658-
leave("i82092aa_set_mem_map: invalid map");
621+
if (map > 4)
659622
return -EINVAL;
660-
}
661-
662623

663624
if ((mem->card_start > 0x3ffffff) || (region.start > region.end) ||
664625
(mem->speed > 1000)) {
665-
leave("i82092aa_set_mem_map: invalid address / speed");
666626
dev_err(&sock_info->dev->dev,
667627
"invalid mem map for socket %i: %llx to %llx with a start of %x\n",
668628
sock,
@@ -717,7 +677,6 @@ static int i82092aa_set_mem_map(struct pcmcia_socket *socket,
717677
if (mem->flags & MAP_ACTIVE)
718678
indirect_setbit(sock, I365_ADDRWIN, I365_ENA_MEM(map));
719679

720-
leave("i82092aa_set_mem_map");
721680
return 0;
722681
}
723682

@@ -728,11 +687,9 @@ static int i82092aa_module_init(void)
728687

729688
static void i82092aa_module_exit(void)
730689
{
731-
enter("i82092aa_module_exit");
732690
pci_unregister_driver(&i82092aa_pci_driver);
733691
if (sockets[0].io_base > 0)
734692
release_region(sockets[0].io_base, 2);
735-
leave("i82092aa_module_exit");
736693
}
737694

738695
module_init(i82092aa_module_init);

drivers/pcmcia/i82092aa.h

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,6 @@
44

55
#include <linux/interrupt.h>
66

7-
/* Debuging defines */
8-
#ifdef NOTRACE
9-
#define enter(x) printk("Enter: %s, %s line %i\n",x,__FILE__,__LINE__)
10-
#define leave(x) printk("Leave: %s, %s line %i\n",x,__FILE__,__LINE__)
11-
#else
12-
#define enter(x) do {} while (0)
13-
#define leave(x) do {} while (0)
14-
#endif
15-
16-
17-
187
/* prototypes */
198

209
static int i82092aa_pci_probe(struct pci_dev *dev, const struct pci_device_id *id);

0 commit comments

Comments
 (0)