From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Thu, 30 Jul 2015 16:20:15 +0200
Subject: [U-Boot] [PATCH v2 4/4] usb: lpc32xx: add host USB driver
In-Reply-To: <4F172219764C784B84C2C1FF44E7DFB103004473@003FCH1MPN2-041.003f.mgd2.msft.net>
References: <1438186450-4076-5-git-send-email-slemieux.tyco@gmail.com>
 <201507300133.25140.marex@denx.de>
 <4F172219764C784B84C2C1FF44E7DFB103004473@003FCH1MPN2-041.003f.mgd2.msft.net>
Message-ID: <201507301620.15693.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thursday, July 30, 2015 at 04:13:14 PM, LEMIEUX, SYLVAIN wrote:
> Hi Marek,

Hi!

> Thanks for the feedback. I will do the change and submit a new revision of
> the patch.
> 
> For your question regarding reference to UART 5 inside the USB driver;
> * It is possible to also  route the UART5 Tx/Rx pin to the USB D+ and D-
> pins; the driver ensure this is not selected.
>    For details, refer to UM10326, "LPC32x0 and LPC32x0/01 User manual -
> Rev. 3" table 459 and paragraph 17.2.3.2.6.
> * I will add comments in the code.

Well in that case, you should pull out a separate pinmux code and do the
pinmuxing in the board setup code, definitelly not in the USB driver.

Best regards,
Marek Vasut