From: Randy Dunlap <rdunlap@xenotime.net>
To: Kees Bakker <kees.bakker@xs4all.nl>
Cc: linux-kernel@vger.kernel.org, Keith Packard <keithp@keithp.com>
Subject: Re: Filename case conflict Documentation/io-mapping.txt and Documentation/IO-mapping.txt
Date: Wed, 30 Jun 2010 15:18:06 -0700 [thread overview]
Message-ID: <20100630151806.08a1823c.rdunlap@xenotime.net> (raw)
In-Reply-To: <97A9C0E5-DA16-4C78-89A3-BBC08EF16518@xs4all.nl>
On Wed, 30 Jun 2010 22:58:21 +0200 Kees Bakker wrote:
> Hi,
>
> Do we really want these two filenames as they are right now?
> We have:
> Documentation/IO-mapping.txt
> Documentation/io-mapping.txt
>
> kees@plankie:~/src/linux-2.6$ git blame Documentation/IO-mapping.txt|head
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 1) [ NOTE: The virt_to_bus() and bus_to_virt() functions have been
> 5872fb94 (Randy Dunlap 2009-01-29 16:28:02 -0800 2) superseded by the functionality provided by the PCI DMA interface
> 5872fb94 (Randy Dunlap 2009-01-29 16:28:02 -0800 3) (see Documentation/PCI/PCI-DMA-mapping.txt). They continue
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 4) to be documented below for historical purposes, but new code
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 5) must not use them. --davidm 00/12/12 ]
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 6)
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 7) [ This is a mail message in response to a query on IO mapping, thus the
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 8) strange format for a "document" ]
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 9)
> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 10) The AHA-1542 is a bus-master device, and your patch makes the driver give the
>
> kees@plankie:~/src/linux-2.6$ git blame Documentation/io-mapping.txt|head
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 1) The io_mapping functions in linux/io-mapping.h provide an abstraction for
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 2) efficiently mapping small regions of an I/O device to the CPU. The initial
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 3) usage is to support the large graphics aperture on 32-bit processors where
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 4) ioremap_wc cannot be used to statically map the entire aperture to the CPU
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 5) as it would consume too much of the kernel address space.
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 6)
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 7) A mapping object is created during driver initialization using
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 8)
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 9) struct io_mapping *io_mapping_create_wc(unsigned long base,
> 9663f2e6 (Keith Packard 2008-10-30 19:38:18 -0700 10) unsigned long size)
>
> So the IO-mapping.txt was there first. Keith, can't we rename your
> file? For example,
>
> Documentation/io-mapping.txt => Documentation/io-mapping.h.txt
> --
Alternatively:
$ git mv Documentation/IO-mapping.txt Documentation/bus-virt-phys-mapping.txt
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
next prev parent reply other threads:[~2010-06-30 22:18 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-30 20:58 Kees Bakker
2010-06-30 22:18 ` Randy Dunlap [this message]
2010-07-01 7:58 ` Kees Bakker
2010-07-02 15:49 ` Keith Packard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100630151806.08a1823c.rdunlap@xenotime.net \
--to=rdunlap@xenotime.net \
--cc=kees.bakker@xs4all.nl \
--cc=keithp@keithp.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®