From: Florian Fainelli <f.fainelli@gmail.com>
To: Alan Stern <stern@rowland.harvard.edu>,
Andreas Herrmann <andreas.herrmann@caviumnetworks.com>
Cc: David Daney <david.daney@cavium.com>,
Alex Smith <alex.smith@imgtec.com>,
Linux-MIPS <linux-mips@linux-mips.org>,
linux-usb <linux-usb@vger.kernel.org>
Subject: Re: [PATCH 3/3] USB: host: Introduce flag to enable use of 64-bit dma_mask for ehci-platform
Date: Fri, 14 Nov 2014 09:34:18 -0800 [thread overview]
Message-ID: <54663D1A.8080305@gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1411141020490.1043-100000@iolanthe.rowland.org>
On 11/14/2014 07:23 AM, Alan Stern wrote:
> On Fri, 14 Nov 2014, Andreas Herrmann wrote:
>
>> On Thu, Nov 13, 2014 at 08:44:17PM -0800, Florian Fainelli wrote:
>>> 2014-11-13 13:36 GMT-08:00 Andreas Herrmann
>>> <andreas.herrmann@caviumnetworks.com>:
>>>> ehci-octeon driver used a 64-bit dma_mask. With removal of ehci-octeon
>>>> and usage of ehci-platform ehci dma_mask is now limited to 32 bits
>>>> (coerced in ehci_platform_probe).
>>>>
>>>> Provide a flag in ehci platform data to allow use of 64 bits for
>>>> dma_mask.
>>>
>>> Why not just allow enforcing an arbitrary DMA mask?
>>
>> I thought about that but as it's currently just 32 or 64 bits
>> a flag is sufficient. (At the moment I am not aware that
>> other ehci-platform devices would require something else.)
>>
>> I'll change the flag to a mask if desired.
>> Alan, what's your opinion about this?
>
> I'm not aware of any devices that need a different DMA mask either.
>
> Florian, do you have any reason for thinking such a thing might come
> along? Like Andreas, I don't mind making it more general if there's a
> good reason to do so.
I don't have a specific platform I am thinking about, just that while we
are there allowing a dma_mask to be specified, I would rather pass an
u64 that covers all possible cases.
Thanks!
--
Florian
next prev parent reply other threads:[~2014-11-14 17:34 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-13 21:36 [PATCH 0/3] USB: host: Misc patches to remove hard-coded octeon platform information Andreas Herrmann
2014-11-13 21:36 ` [PATCH 1/3] USB: host: Remove ehci-octeon and ohci-octeon drivers Andreas Herrmann
2014-11-14 9:31 ` Ralf Baechle
2014-11-14 9:58 ` Andreas Herrmann
2014-11-25 1:21 ` Greg KH
2014-11-25 10:23 ` Andreas Herrmann
2014-11-25 11:28 ` [PATCH 1/3 v2] " Andreas Herrmann
2014-11-25 17:19 ` Greg KH
2014-11-13 21:36 ` [PATCH 2/3] USB: host: Remove hard-coded octeon platform information for ehci/ohci Andreas Herrmann
2014-11-14 9:33 ` Ralf Baechle
2014-11-13 21:36 ` [PATCH 3/3] USB: host: Introduce flag to enable use of 64-bit dma_mask for ehci-platform Andreas Herrmann
2014-11-14 4:44 ` Florian Fainelli
2014-11-14 8:51 ` Andreas Herrmann
2014-11-14 15:23 ` Alan Stern
2014-11-14 17:34 ` Florian Fainelli [this message]
2014-11-13 22:13 ` [PATCH 0/3] USB: host: Misc patches to remove hard-coded octeon platform information Alan Stern
2014-11-14 11:47 ` Andreas Herrmann
2014-11-14 20:32 ` Aaro Koskinen
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=54663D1A.8080305@gmail.com \
--to=f.fainelli@gmail.com \
--cc=alex.smith@imgtec.com \
--cc=andreas.herrmann@caviumnetworks.com \
--cc=david.daney@cavium.com \
--cc=linux-mips@linux-mips.org \
--cc=linux-usb@vger.kernel.org \
--cc=stern@rowland.harvard.edu \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.