Re: [PATCH] dec_esp: Use physical addresses

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Oct 15, 2005 at 02:26:31AM +0100, Ralf Baechle wrote:
> FYI.  This is part of a larger MIPS patch which I'd like to submit
> to akpm as a single patch.

Please don't do that _ever_.  (dito for the other bits sent out)
scsi patches should be submitted to linux-scsi as individual patcvhes.

> Use physical addresses at the interface level, letting drivers remap
> them as appropriate.

> -		esp->eregs = (struct ESP_regs *) (system_base + IOASIC_SCSI);
> +		esp->eregs = (void *)CKSEG1ADDR(dec_kn_slot_base +
> +						IOASIC_SCSI);

Both the old and new code look very odd.  This looks like iomaped-ish
memory and should be using some iomap variant.  The patch is probably
okay for now, but I'd prefer if this used the proper APIs.

-
: 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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux