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? The braces in the #define, didn't get carried over in the change. Daniel K. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html