Skip to content

Commit

Permalink
staging: brcm80211: remove unused attributes from struct osl_info
Browse files Browse the repository at this point in the history
Getting rid of the whole osl concept soon this removes most fields
in struct osl_info. Turned out hnddma.c was still using it so this
was fixed and now it gets pointer to bus device from si_info which
it only needs in the attach function for this purpose so reference
to it does not need to be kept. Two unused functions referencing
the removed fields in linux_osl.c also were removed.

Reviewed-by: Brett Rudley <brudley@broadcom.com>
Reviewed-by: Henry Ptasinski <henryp@broadcom.com>
Reviewed-by: Roland Vossen <rvossen@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Arend van Spriel authored and Greg Kroah-Hartman committed Mar 3, 2011
1 parent 8da4a3a commit 4fe9042
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 45 deletions.
3 changes: 0 additions & 3 deletions drivers/staging/brcm80211/include/osl.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,7 @@
/* osl handle type forward declaration */
struct osl_info {
uint pktalloced; /* Number of allocated packet buffers */
bool mmbus; /* Bus supports memory-mapped registers */
uint magic;
void *pdev;
uint bustype;
};

typedef struct osl_dmainfo osldma_t;
Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/brcm80211/include/siutils.h
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ typedef struct si_info {
u32 oob_router; /* oob router registers for axi */
} si_info_t;

#define SI_INFO(sih) (si_info_t *)sih
#define SI_INFO(sih) ((si_info_t *)(sih))

#define GOODCOREADDR(x, b) (((x) >= (b)) && ((x) < ((b) + SI_MAXCORES * SI_CORE_SIZE)) && \
IS_ALIGNED((x), SI_CORE_SIZE))
Expand Down
4 changes: 1 addition & 3 deletions drivers/staging/brcm80211/util/hnddma.c
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,6 @@ typedef struct dma_info {

struct osl_info *osh; /* os handle */
void *pbus; /* bus handle */
si_t *sih; /* sb handle */

bool dma64; /* this dma engine is operating in 64-bit mode */
bool addrext; /* this dma engine supports DmaExtendedAddrChanges */
Expand Down Expand Up @@ -338,8 +337,7 @@ struct hnddma_pub *dma_attach(struct osl_info *osh, char *name, si_t *sih,
di->name[MAXNAMEL - 1] = '\0';

di->osh = osh;
di->sih = sih;
di->pbus = osh->pdev;
di->pbus = ((struct si_info *)sih)->pbus;

/* save tunables */
di->ntxd = (u16) ntxd;
Expand Down
38 changes: 0 additions & 38 deletions drivers/staging/brcm80211/util/linux_osl.c
Original file line number Diff line number Diff line change
Expand Up @@ -43,29 +43,7 @@ struct osl_info *osl_attach(void *pdev, uint bustype)
ASSERT(osh);

memset(osh, 0, sizeof(struct osl_info));

osh->magic = OS_HANDLE_MAGIC;
osh->pdev = pdev;
osh->bustype = bustype;

switch (bustype) {
case PCI_BUS:
case SI_BUS:
case PCMCIA_BUS:
osh->mmbus = true;
break;
case JTAG_BUS:
case SDIO_BUS:
case USB_BUS:
case SPI_BUS:
case RPC_BUS:
osh->mmbus = false;
break;
default:
ASSERT(false);
break;
}

return osh;
}

Expand All @@ -78,22 +56,6 @@ void osl_detach(struct osl_info *osh)
kfree(osh);
}

/* return bus # for the pci device pointed by osh->pdev */
uint osl_pci_bus(struct osl_info *osh)
{
ASSERT(osh && (osh->magic == OS_HANDLE_MAGIC) && osh->pdev);

return ((struct pci_dev *)osh->pdev)->bus->number;
}

/* return slot # for the pci device pointed by osh->pdev */
uint osl_pci_slot(struct osl_info *osh)
{
ASSERT(osh && (osh->magic == OS_HANDLE_MAGIC) && osh->pdev);

return PCI_SLOT(((struct pci_dev *)osh->pdev)->devfn);
}

#if defined(BCMDBG_ASSERT)
void osl_assert(char *exp, char *file, int line)
{
Expand Down

0 comments on commit 4fe9042

Please sign in to comment.