Skip to content

Commit

Permalink
fix CONFIG_SATA_SIS=y compile error
Browse files Browse the repository at this point in the history
Static code shouldn't be used from other modules.

drivers/built-in.o: In function `sis_init_one':
sata_sis.c:(.text+0x7634cd): undefined reference to `sis_info133'
sata_sis.c:(.text+0x7634d6): undefined reference to `sis_info133'

While I was at it, I also moved the prototype of this struct to a header
file.

Signed-off-by: Adrian Bunk <[email protected]>
Cc: Jeff Garzik <[email protected]>
Cc: Tejun Heo <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Jeff Garzik <[email protected]>
  • Loading branch information
AdrianBunk authored and Jeff Garzik committed Feb 9, 2007
1 parent 9b14dec commit 77a527e
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
3 changes: 3 additions & 0 deletions drivers/ata/libata.h
Original file line number Diff line number Diff line change
Expand Up @@ -136,4 +136,7 @@ extern void ata_qc_schedule_eh(struct ata_queued_cmd *qc);
/* libata-sff.c */
extern u8 ata_irq_on(struct ata_port *ap);

/* pata_sis.c */
extern struct ata_port_info sis_info133;

#endif /* __LIBATA_H__ */
3 changes: 2 additions & 1 deletion drivers/ata/pata_sis.c
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
#include <scsi/scsi_host.h>
#include <linux/libata.h>
#include <linux/ata.h>
#include "libata.h"

#define DRV_NAME "pata_sis"
#define DRV_VERSION "0.4.5"
Expand Down Expand Up @@ -781,7 +782,7 @@ static struct ata_port_info sis_info100_early = {
.pio_mask = 0x1f, /* pio0-4 */
.port_ops = &sis_66_ops,
};
static struct ata_port_info sis_info133 = {
struct ata_port_info sis_info133 = {
.sht = &sis_sht,
.flags = ATA_FLAG_SLAVE_POSS | ATA_FLAG_SRST,
.pio_mask = 0x1f, /* pio0-4 */
Expand Down
4 changes: 1 addition & 3 deletions drivers/ata/sata_sis.c
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
#include <linux/device.h>
#include <scsi/scsi_host.h>
#include <linux/libata.h>
#include "libata.h"

#define DRV_NAME "sata_sis"
#define DRV_VERSION "0.7"
Expand Down Expand Up @@ -254,9 +255,6 @@ static void sis_scr_write (struct ata_port *ap, unsigned int sc_reg, u32 val)

static int sis_init_one (struct pci_dev *pdev, const struct pci_device_id *ent)
{
/* Provided by the PATA driver */
extern struct ata_port_info sis_info133;

static int printed_version;
struct ata_probe_ent *probe_ent = NULL;
int rc;
Expand Down

0 comments on commit 77a527e

Please sign in to comment.