On Fri, Oct 3, 2014 at 12:49 PM, Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, 3 Oct 2014, Geert Uytterhoeven wrote:
>> On Thu, Oct 2, 2014 at 8:56 AM, Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> wrote:
>> > Allow mac_scsi to be built as a module. Replace the old validation of
>> > __setup options with code that validates both module and __setup options.
>>
>> > --- linux.orig/drivers/scsi/mac_scsi.c 2014-10-02 16:56:17.000000000 +1000
>> > +++ linux/drivers/scsi/mac_scsi.c 2014-10-02 16:56:17.000000000 +1000
>>
>> > @@ -219,15 +179,15 @@ int __init macscsi_detect(struct scsi_ho
>> > return 0;
>> >
>> > if (macintosh_config->ident == MAC_MODEL_IIFX) {
>> > - mac_scsi_regp = via1+0x8000;
>> > - mac_scsi_drq = via1+0xE000;
>> > - mac_scsi_nodrq = via1+0xC000;
>> > + mac_scsi_regp = (unsigned char *) VIA1_BASE + 0x8000;
>> > + mac_scsi_drq = (unsigned char *) VIA1_BASE + 0xE000;
>> > + mac_scsi_nodrq = (unsigned char *) VIA1_BASE + 0xC000;
>> > /* The IIFX should be able to do true DMA, but pseudo-dma doesn't work */
>> > flags = FLAG_NO_PSEUDO_DMA;
>> > } else {
>> > - mac_scsi_regp = via1+0x10000;
>> > - mac_scsi_drq = via1+0x6000;
>> > - mac_scsi_nodrq = via1+0x12000;
>> > + mac_scsi_regp = (unsigned char *) VIA1_BASE + 0x10000;
>> > + mac_scsi_drq = (unsigned char *) VIA1_BASE + 0x6000;
>> > + mac_scsi_nodrq = (unsigned char *) VIA1_BASE + 0x12000;
>> > }
>> >
>> > if (! setup_use_pdma)
>>
>> I don't think the above chunk belongs in this patch.
>
> via1 is not an exported symbol, so this allows mac_scsi to be linked as a
> module.
OK.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
