lkml.org 
[lkml]   [2009]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [PATCH 02/15] scc_pata: remove DECLARE_SCC_DEV() macro
Date
On Monday 02 February 2009, Daniel K. wrote:
> Bartlomiej Zolnierkiewicz wrote:
> > Index: b/drivers/ide/scc_pata.c
> > ===================================================================
> > --- a/drivers/ide/scc_pata.c
> > +++ b/drivers/ide/scc_pata.c
> > @@ -882,21 +882,16 @@ static const struct ide_dma_ops scc_dma_
> > .dma_sff_read_status = scc_dma_sff_read_status,
> > };
> >
> > -#define DECLARE_SCC_DEV(name_str) \
> > - { \
> > - .name = name_str, \
> > - .init_iops = init_iops_scc, \
> > - .init_dma = scc_init_dma, \
> > - .init_hwif = init_hwif_scc, \
> > - .tp_ops = &scc_tp_ops, \
> > - .port_ops = &scc_port_ops, \
> > - .dma_ops = &scc_dma_ops, \
> > - .host_flags = IDE_HFLAG_SINGLE, \
> > - .pio_mask = ATA_PIO4, \
> > - }
> > -
> > static const struct ide_port_info scc_chipsets[] __devinitdata = {
> > - /* 0 */ DECLARE_SCC_DEV("sccIDE"),
> > + .name = "sccIDE",
> > + .init_iops = init_iops_scc,
> > + .init_dma = scc_init_dma,
> > + .init_hwif = init_hwif_scc,
> > + .tp_ops = &scc_tp_ops,
> > + .port_ops = &scc_port_ops,
> > + .dma_ops = &scc_dma_ops,
> > + .host_flags = IDE_HFLAG_SINGLE,
> > + .pio_mask = ATA_PIO4,
> > };
> >
> > /**
>
> You're a few braces short here, aren't you?

Good catch, thanks!

> The braces in the #define, didn't get carried over in the change.

Actually we may simplify the whole thing while at it.

From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Subject: [PATCH] scc_pata: remove DECLARE_SCC_DEV() macro (v2)

v2:
scc_chipsets[] -> scc_chipset fix (spotted by Daniel K.).

Cc: "Daniel K." <daniel@cluded.net>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
---
drivers/ide/scc_pata.c | 27 +++++++++++----------------
1 file changed, 11 insertions(+), 16 deletions(-)

Index: b/drivers/ide/scc_pata.c
===================================================================
--- a/drivers/ide/scc_pata.c
+++ b/drivers/ide/scc_pata.c
@@ -882,21 +882,16 @@ static const struct ide_dma_ops scc_dma_
.dma_sff_read_status = scc_dma_sff_read_status,
};

-#define DECLARE_SCC_DEV(name_str) \
- { \
- .name = name_str, \
- .init_iops = init_iops_scc, \
- .init_dma = scc_init_dma, \
- .init_hwif = init_hwif_scc, \
- .tp_ops = &scc_tp_ops, \
- .port_ops = &scc_port_ops, \
- .dma_ops = &scc_dma_ops, \
- .host_flags = IDE_HFLAG_SINGLE, \
- .pio_mask = ATA_PIO4, \
- }
-
-static const struct ide_port_info scc_chipsets[] __devinitdata = {
- /* 0 */ DECLARE_SCC_DEV("sccIDE"),
+static const struct ide_port_info scc_chipset __devinitdata = {
+ .name = "sccIDE",
+ .init_iops = init_iops_scc,
+ .init_dma = scc_init_dma,
+ .init_hwif = init_hwif_scc,
+ .tp_ops = &scc_tp_ops,
+ .port_ops = &scc_port_ops,
+ .dma_ops = &scc_dma_ops,
+ .host_flags = IDE_HFLAG_SINGLE,
+ .pio_mask = ATA_PIO4,
};

/**
@@ -910,7 +905,7 @@ static const struct ide_port_info scc_ch

static int __devinit scc_init_one(struct pci_dev *dev, const struct pci_device_id *id)
{
- return init_setup_scc(dev, &scc_chipsets[id->driver_data]);
+ return init_setup_scc(dev, &scc_chipset);
}

/**


\
 
 \ /
  Last update: 2009-02-02 19:39    [W:0.069 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site