Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-19 Thread Igor Grinberg
Hi Nikolay, On 06/17/14 20:49, Nikolay Dimitrov wrote: Hi Igor, On 6/17/2014 9:26 AM, Igor Grinberg wrote: That is exactly what we do already (code is on the way) and IMO what we should aim for. I really didn't knew this in the beginning before seeing your answers, this would be

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-17 Thread Igor Grinberg
Hi Nikolay, On 06/16/14 17:33, Nikolay Dimitrov wrote: Hi Igor, My personal opinion is that unless you intend to run the binary on multiple IMX6 variants, That is exactly what we do already (code is on the way) and IMO what we should aim for. there's no need to do expensive checks in

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-17 Thread Stefano Babic
Hi Otavio, On 16/06/2014 02:46, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by: Otavio Salvador ota...@ossystems.com.br --- drivers/usb/host/ehci-mx6.c |

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-17 Thread Otavio Salvador
On Tue, Jun 17, 2014 at 11:56 AM, Stefano Babic sba...@denx.de wrote: Hi Otavio, On 16/06/2014 02:46, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by: Otavio

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-17 Thread Nikolay Dimitrov
Hi Marek, On 6/16/2014 5:53 PM, Marek Vasut wrote: Please do not top-post. Please excuse me - I was unable to find posting guidelines for the mailing list, so just posted by habit (last time I had to post to another list, they wanted me to top-post). With the upcoming DM implementation,

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-17 Thread Marek Vasut
On Tuesday, June 17, 2014 at 06:21:05 PM, Nikolay Dimitrov wrote: Hi Marek, On 6/16/2014 5:53 PM, Marek Vasut wrote: Please do not top-post. Please excuse me - I was unable to find posting guidelines for the mailing list, so just posted by habit (last time I had to post to another list,

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-17 Thread Nikolay Dimitrov
Hi Igor, On 6/17/2014 9:26 AM, Igor Grinberg wrote: That is exactly what we do already (code is on the way) and IMO what we should aim for. I really didn't knew this in the beginning before seeing your answers, this would be definitely easier to support. For me it is just an artificial

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-16 Thread Igor Grinberg
Hi Otavio, On 06/16/14 03:46, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by: Otavio Salvador ota...@ossystems.com.br --- drivers/usb/host/ehci-mx6.c |

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-16 Thread Otavio Salvador
On Mon, Jun 16, 2014 at 4:05 AM, Igor Grinberg grinb...@compulab.co.il wrote: Hi Otavio, On 06/16/14 03:46, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by:

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-16 Thread Nikolay Dimitrov
: Message: 30 Date: Mon, 16 Jun 2014 10:05:08 +0300 From: Igor Grinberg grinb...@compulab.co.il Subject: Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL To: Otavio Salvador ota...@ossystems.com.br, U-Boot Mailing List u-boot@lists.denx.de Cc: Marek Vasut ma...@denx.de, Fabio Estevam

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-16 Thread Marek Vasut
On 6/16/2014 1:00 PM, u-boot-requ...@lists.denx.de wrote: Message: 30 Date: Mon, 16 Jun 2014 10:05:08 +0300 From: Igor Grinberg grinb...@compulab.co.il Subject: Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL To: Otavio Salvador ota...@ossystems.com.br, U-Boot Mailing List

[U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-15 Thread Otavio Salvador
The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by: Otavio Salvador ota...@ossystems.com.br --- drivers/usb/host/ehci-mx6.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-)

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-15 Thread Marek Vasut
On Monday, June 16, 2014 at 02:46:48 AM, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by: Otavio Salvador ota...@ossystems.com.br ---

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-15 Thread Otavio Salvador
On Sun, Jun 15, 2014 at 9:49 PM, Marek Vasut ma...@denx.de wrote: On Monday, June 16, 2014 at 02:46:48 AM, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to support the different base address for this case. Signed-off-by:

Re: [U-Boot] [PATCH 1/6] usb: ehci: mx6: Add support for i.MX6SL

2014-06-15 Thread Marek Vasut
On Monday, June 16, 2014 at 03:11:03 AM, Otavio Salvador wrote: On Sun, Jun 15, 2014 at 9:49 PM, Marek Vasut ma...@denx.de wrote: On Monday, June 16, 2014 at 02:46:48 AM, Otavio Salvador wrote: The i.MX6SL has a different base address for the controller. This patch adapts the driver to