All of lore.kernel.org
 help / color / mirror / Atom feed
From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/9 v2] ARM: at91: usb ohci add dt support
Date: Wed, 7 Mar 2012 11:33:52 +0100	[thread overview]
Message-ID: <20120307103352.GA17745@game.jcrosoft.org> (raw)
In-Reply-To: <20120307093446.GN17370@n2100.arm.linux.org.uk>

On 09:34 Wed 07 Mar     , Russell King - ARM Linux wrote:
> On Wed, Mar 07, 2012 at 10:15:38AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > +static u64 at91_ohci_dma_mask = DMA_BIT_MASK(32);
> > +
> > +static int __devinit ohci_at91_of_init(struct platform_device *pdev)
> > +{
> > +	struct device_node *np = pdev->dev.of_node;
> > +	int i, ret, gpio;
> > +	enum of_gpio_flags flags;
> > +	struct at91_usbh_data	*pdata;
> > +	u32 ports;
> > +
> > +	if (!np)
> > +		return 0;
> > +
> > +	/* Right now device-tree probed devices don't get dma_mask set.
> > +	 * Since shared usb code relies on it, set it here for now.
> > +	 * Once we have dma capability bindings this can go away.
> > +	 */
> > +	if (!pdev->dev.dma_mask)
> > +		pdev->dev.dma_mask = &at91_ohci_dma_mask;
> 
> That sounds like a rather big omission from DT - what's happening to
> address this (before we end up with lots of drivers with this).  Pretty
> much any device which does DMA needs both this and the coherent DMA mask
> set properly.
> 
> We really don't want this kind of workaround scattered in every driver
> which has been converted to DT.
Tegra do it also
I do yet found a good generic way in DT

Today only usb need I do plan to drop it for 3.5 can we have for 3.4

Best Regards,
J.

WARNING: multiple messages have this Message-ID (diff)
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org>
To: Russell King - ARM Linux <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	Nicolas Ferre
	<nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 1/9 v2] ARM: at91: usb ohci add dt support
Date: Wed, 7 Mar 2012 11:33:52 +0100	[thread overview]
Message-ID: <20120307103352.GA17745@game.jcrosoft.org> (raw)
In-Reply-To: <20120307093446.GN17370-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>

On 09:34 Wed 07 Mar     , Russell King - ARM Linux wrote:
> On Wed, Mar 07, 2012 at 10:15:38AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > +static u64 at91_ohci_dma_mask = DMA_BIT_MASK(32);
> > +
> > +static int __devinit ohci_at91_of_init(struct platform_device *pdev)
> > +{
> > +	struct device_node *np = pdev->dev.of_node;
> > +	int i, ret, gpio;
> > +	enum of_gpio_flags flags;
> > +	struct at91_usbh_data	*pdata;
> > +	u32 ports;
> > +
> > +	if (!np)
> > +		return 0;
> > +
> > +	/* Right now device-tree probed devices don't get dma_mask set.
> > +	 * Since shared usb code relies on it, set it here for now.
> > +	 * Once we have dma capability bindings this can go away.
> > +	 */
> > +	if (!pdev->dev.dma_mask)
> > +		pdev->dev.dma_mask = &at91_ohci_dma_mask;
> 
> That sounds like a rather big omission from DT - what's happening to
> address this (before we end up with lots of drivers with this).  Pretty
> much any device which does DMA needs both this and the coherent DMA mask
> set properly.
> 
> We really don't want this kind of workaround scattered in every driver
> which has been converted to DT.
Tegra do it also
I do yet found a good generic way in DT

Today only usb need I do plan to drop it for 3.5 can we have for 3.4

Best Regards,
J.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2012-03-07 10:33 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-07  8:49 [PATCH 0/9 v2] at91 USB DT support Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  8:49 ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 1/9 v2] ARM: at91: usb ohci add dt support Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:34   ` Russell King - ARM Linux
2012-03-07  9:34     ` Russell King - ARM Linux
2012-03-07 10:33     ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2012-03-07 10:33       ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07 16:45       ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07 16:45         ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 2/9 v2] ARM: at91: dt: enable usb ohci for sam9g20, sam9g45 amd sam9x5 Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 3/9 v2] ARM: at91: usb ehci add dt support Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 4/9] ARM: at91: dt: enable usb ehci for sam9g45 and sam9x5 Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 5/9] USB: at91: Device udc add dt support Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-08 20:53   ` Greg KH
2012-03-08 20:53     ` Greg KH
2012-03-07  9:15 ` [PATCH 6/9] ARM: at91: sam9g20 " Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 7/9] usb: add Atmel USBA UDC DT support Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-14  9:27   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-14  9:27     ` Jean-Christophe PLAGNIOL-VILLARD
2012-04-08 15:42     ` Jean-Christophe PLAGNIOL-VILLARD
2012-04-08 15:42       ` Jean-Christophe PLAGNIOL-VILLARD
2012-04-18 13:48       ` Jean-Christophe PLAGNIOL-VILLARD
2012-04-18 13:48         ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 8/9] ARM: at91: sam9g45 add udc " Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15 ` [PATCH 9/9] ARM: at91: sam9x5 " Jean-Christophe PLAGNIOL-VILLARD
2012-03-07  9:15   ` Jean-Christophe PLAGNIOL-VILLARD

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=20120307103352.GA17745@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.com \
    --cc=linux-arm-kernel@lists.infradead.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 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.