All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] mrvl_uart.sh and Turris Omnia (was: [EXT] Re: [PATCH v4 0/6] Support for the Turris Omnia router)
Date: Sat, 20 Jan 2018 13:34:42 +0100	[thread overview]
Message-ID: <d8b51bf8-5463-2a65-6ac2-5c8770b46349@suse.de> (raw)
In-Reply-To: <ed8763c18cfa43c097687cfe8f2aefcb@IL-EXCH02.marvell.com>

Hi Kosta,

Am 20.01.2018 um 10:48 schrieb Kostya Porotchkin:
> Which Marvell SoC is used on this platform?

According to Linux DT filename it's an Armada 385.

> The script I added recently covert only the new Armada SoCs like A8K and
> A8K+.
> It will not work with A3K family.

# UART recovery downloader for Armada SoCs

    echo -e "\nMarvell recovery image downloader for Armada SoC family."

    echo -e "- The speed-up option is not awailable in SoC families
prior to A8K+"
    echo -e "- This utility is not compatible with Armada 37xx SoC family\n"

https://patchwork.ozlabs.org/patch/843021/

It doesn't rule out anything other than 37xx, sounding very generic and
making no specific statements about earlier 32-bit models. ;)

If it's known to be 7K/8K only, then you should spell that out please.
Note that I've just sent out some potentially conflicting output
cleanups that I noticed above.

Another comment: The kwboot tool is particularly handy in that -p allows
to send a binary u-boot-spl.kwb (such as from our Open Build Service)
with default of "BOOT_FROM spi" for turris_omnia here, without needing
to rebuild from sources with "BOOT_FROM uart" like I seem to need to for
mrvl_uart.sh. Are you planning to re-implement that for your script?

Regards,
Andreas


> -------- Original message --------
> From: Andreas Färber <afaerber@suse.de>
> Date: 1/20/18 05:34 (GMT+02:00)
> To: Stefan Roese <sr@denx.de>, u-boot at lists.denx.de, Kostya Porotchkin
> <kostap@marvell.com>
> Cc: Marek Behún <marek.behun@nic.cz>, Tomas Hlavacek
> <tomas.hlavacek@nic.cz>
> Subject: [EXT] Re: [U-Boot] [PATCH v4 0/6] Support for the Turris Omnia
> router
> 
> External Email
> 
> ----------------------------------------------------------------------
> Am 20.01.2018 um 02:40 schrieb Andreas Färber:
>> Hi,
>> 
>> Am 18.01.2018 um 18:20 schrieb Stefan Roese:
>>> On 17.01.2018 16:52, Andreas Färber wrote:
>>>> Am 09.06.2017 um 19:28 schrieb Marek Behún:
>>>>> This is the fourth version of patches for adding support for the
>>>>> Turris Omnia board, a router developed by the CZ.NIC.
>>>>
>>>> I'm still facing trouble testing turris_omnia on latest v2018.01.
>>>>
>>>> First, that made me notice there's no README for how to test and deploy.
>>>> I'm aware of temporary:
>>>> sendbeacon /dev/ttyUSBx
>>>
>>> I have to admit, that don't know anything about this "sendbeacon"
>>> tool.
> 
> https://gitlab.labs.nic.cz/turris/misc/tree/master/sendbeacon
> 
>>>> ./tools/kwboot -t -B 115200 /dev/ttyUSBx -b u-boot-spl.kwb -p
>>>
>>> This is what I have used, when I tested / debugged images for
>>> Armada XP / 38x. Please note that the init sequence is somewhat
>>> "fragile" - so I added the -q and -s parameters, to optionally
>>> finetune the startup timings:
>>>
>>> # kwboot
>>> ...
>>>   -q <req-delay>:  use specific request-delay
>>>   -s <resp-timeo>: use specific response-timeout
>>>
>>> You might what to play a bit with these parameters as well.
>> 
>> I saw them but had no idea what to pass as values. ;)
>> I did try -a, but it worked with and without.
>> 
>>> BTW: I don't have access to the Omnia router, so I can't
>>> test anything on this specific platform.
>>>
>>> BTW2: Kosta from Marvell just recently added a new tool / script,
>>> to help debug / boot Marvell MVEBU boards:
>>>
>>> tools/mrvl_uart.sh
>>>
>>> He told me that its better to use than the "old" kwboot tool.
>>> I never found the time to use it up until now, so I have no
>>> personal experience. But I'm pretty sure that Kosta did a
>>> great job here. So please give it a try.
>> 
>> I did not get it to work ... or was not patient enough.
> 
> It seems, just as with kwboot, the tool alone is not enough. If I use
> sendbeacon before and in mrvl_uart.sh just press enter without another
> reset, then the transfer does work for me, and if in the script I add -b
> $default_baudrate to the minicom line, then I also see useful output.
> 
> But it does not affect the reported non-SPL boot breakage, i.e. it still
> reboots from SPI immediately after loading from UART.
> 
> Regards,
> Andreas
> 
> -- 
> SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Felix Imendörffer, Jane Smithard, Graham Norton
> HRB 21284 (AG Nürnberg)


-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

  reply	other threads:[~2018-01-20 12:34 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-09 17:28 [U-Boot] [PATCH v4 0/6] Support for the Turris Omnia router Marek Behún
2017-06-09 17:28 ` [U-Boot] [PATCH v4 1/7] driver/ddr: Add support for setting timing in hws_topology_map Marek Behún
2018-01-10  2:49   ` Chris Packham
2017-06-09 17:28 ` [U-Boot] [PATCH v4 2/7] orion_wdt: Support for the Orion Watchdog Marek Behún
2017-06-09 17:28 ` [U-Boot] [PATCH v4 3/7] arch/arm/dts: Add Turris Omnia device tree Marek Behún
2017-07-07 11:58   ` Stefan Roese
2017-07-07 12:22     ` Marek Behún
2017-07-07 13:36       ` Stefan Roese
2017-06-09 17:28 ` [U-Boot] [PATCH v4 4/7] drivers/i2c/muxes/pca954x: Add pca9547 I2C mux support Marek Behún
2017-09-28 21:27   ` Chris Packham
2017-06-09 17:28 ` [U-Boot] [PATCH v4 5/7] drivers/misc: Add basic support for ATSHA204A Crypto module Marek Behún
2017-06-09 17:28 ` [U-Boot] [PATCH v4 6/7] marvell: armada385: Add the Turris Omnia board Marek Behún
2017-07-12  5:53 ` [U-Boot] [PATCH v4 0/6] Support for the Turris Omnia router Stefan Roese
2018-01-17 15:52 ` Andreas Färber
2018-01-18 17:20   ` Stefan Roese
2018-01-18 17:56     ` Andreas Färber
2018-01-20  1:40     ` Andreas Färber
2018-01-20  2:30       ` Andreas Färber
2018-01-20  9:07         ` Stefan Roese
2018-01-20 14:32           ` Sean Nyekjær
2018-01-21 12:39             ` Andreas Färber
2018-01-21 12:53               ` Sean Nyekjær
2018-01-22  7:57               ` Sean Nyekjær
2018-01-20  3:34       ` Andreas Färber
2018-01-20  9:48         ` [U-Boot] [EXT] " Kostya Porotchkin
2018-01-20 12:34           ` Andreas Färber [this message]
2018-01-21  9:02             ` [U-Boot] mrvl_uart.sh and Turris Omnia (was: [EXT] Re: [PATCH v4 0/6] Support for the Turris Omnia router) Kostya Porotchkin

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=d8b51bf8-5463-2a65-6ac2-5c8770b46349@suse.de \
    --to=afaerber@suse.de \
    --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.