All of lore.kernel.org
 help / color / mirror / Atom feed
From: Przemyslaw Marczak <p.marczak@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Testing u-boot-dm/next
Date: Mon, 13 Apr 2015 17:26:19 +0200	[thread overview]
Message-ID: <552BE01B.4060909@samsung.com> (raw)
In-Reply-To: <CAPnjgZ2B6sYjTxTLwqjKj3WyazBUdX0tNr8nHnOq0CQB8Yt-NA@mail.gmail.com>

Hello Simon,

On 04/13/2015 04:31 PM, Simon Glass wrote:
> Hi Prazemyslaw,
>
> On 10 April 2015 at 12:08, Przemyslaw Marczak <p.marczak@samsung.com> wrote:
>> Hello,
>>
>>
>> On 04/10/2015 07:57 PM, Simon Glass wrote:
>>>
>>> Hi Przemyslaw,
>>>
>>> On 10 April 2015 at 11:50, Przemyslaw Marczak <p.marczak@samsung.com>
>>> wrote:
>>>>
>>>> Hello Simon,
>>>>
>>>>
>>>> On 04/10/2015 07:40 PM, Simon Glass wrote:
>>>>>
>>>>>
>>>>> Hi Przemyslaw,
>>>>>
>>>>> On 9 April 2015 at 01:55, Przemyslaw Marczak <p.marczak@samsung.com>
>>>>> wrote:
>>>>>>
>>>>>>
>>>>>> Hello Simon,
>>>>>>
>>>>>>
>>>>>> On 04/09/2015 05:07 AM, Simon Glass wrote:
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> I have quite a few patches queued up in the next branch of u-boot-dm,
>>>>>>> ready
>>>>>>> for when the merge window options.
>>>>>>>
>>>>>>> If anyone has time and can give it a spin on their board, it would be
>>>>>>> much
>>>>>>> appreciated!
>>>>>>>
>>>>>>> Regards,
>>>>>>> Simon
>>>>>>>
>>>>>>
>>>>>> I tested this on Odroid U3.
>>>>>> It looks that everything works fine. I see that ehci_hcd_init() is not
>>>>>> used,
>>>>>> then where can we call the board_usb_init()?
>>>>>
>>>>>
>>>>>
>>>>> Thanks for testing.
>>>>>
>>>>> Can board_usb_init() be called from within the board_init() code
>>>>> instead? Or are you trying to avoid doing this init unless USB is
>>>>> actually used? If so, then I'll need to have a think about it.
>>>>>
>>>>> Regards,
>>>>> Simon
>>>>>
>>>>
>>>> I noticed, that after changes the USB command doesn't recognize the mass
>>>> storage device. I haven't too much time for checking it deeper.
>>>> I will check it on Monday and give you a response.
>>>
>>>
>>> OK thanks. You can also try removing CONFIG_DM_USB as a comparison.
>>> I'm happy to switch this back to not use driver model until any
>>> problems are resolved.
>>>
>>> Regards,
>>> Simon
>>>
>>
>> Ok, I will also check this. Or if find the time, I will fix the storage
>> issue for dm-usb.
>
> Thanks. Just a note on my testing. I used a power hub with 3 USB flash
> sticks and a USB Ethernet dongle. Also I plugged a unpowered hub into
> the powered hub, with its own USB stick. I tried this on snow/pit both
> with and without DM_USB.
>
> For mass storage the detection method is different (you now register
> drivers which can handle the various interface classes, etc.) but the
> effect is intended to be the same. But there may be an error
> somewhere.
>
> Regards,
> Simon
>

Sorry, I haven't time for this today.
I hope to check this tomorrow.

Best regards,
-- 
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com

      reply	other threads:[~2015-04-13 15:26 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-09  3:07 [U-Boot] Testing u-boot-dm/next Simon Glass
2015-04-09  3:11 ` Simon Glass
2015-04-09 12:52   ` Bin Meng
2015-04-10 17:42     ` Simon Glass
2015-04-11 13:22       ` Bin Meng
2015-04-11 13:48         ` Bin Meng
2015-04-11 14:00           ` Simon Glass
2015-04-13 14:27           ` Simon Glass
2015-04-13 15:05             ` Bin Meng
2015-04-13 15:20               ` Simon Glass
2015-04-11 13:57         ` Simon Glass
2015-04-13 16:27   ` Stephen Warren
2015-04-13 17:04     ` Stephen Warren
2015-04-13 17:29       ` Simon Glass
2015-04-13 17:36         ` Stephen Warren
2015-04-13 17:46           ` Simon Glass
2015-04-13 17:52             ` Simon Glass
2015-04-13 19:03               ` Stephen Warren
2015-04-13 20:17                 ` Simon Glass
2015-04-14  3:25                   ` Simon Glass
2015-04-14  3:42                     ` Simon Glass
2015-04-25  4:46   ` Stephen Warren
2015-04-25 14:49     ` Simon Glass
2015-04-09  7:55 ` Przemyslaw Marczak
2015-04-10 17:40   ` Simon Glass
2015-04-10 17:50     ` Przemyslaw Marczak
2015-04-10 17:57       ` Simon Glass
2015-04-10 18:08         ` Przemyslaw Marczak
2015-04-13 14:31           ` Simon Glass
2015-04-13 15:26             ` Przemyslaw Marczak [this message]

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=552BE01B.4060909@samsung.com \
    --to=p.marczak@samsung.com \
    --cc=u-boot@lists.denx.de \
    /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.