Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove 1s boot delay from parallella-next? #5

Open
olajep opened this issue Aug 2, 2015 · 6 comments
Open

Remove 1s boot delay from parallella-next? #5

olajep opened this issue Aug 2, 2015 · 6 comments
Labels

Comments

@olajep
Copy link
Member

olajep commented Aug 2, 2015

I added a 1 second boot delay to parallella-uboot/parallella-next.
3e9bf5c#diff-dfe3aa552f2a30fd1c63469710ada33fR38

The boot-flash images in https://github.com/parallella/parallella-flash are based on the parallella-next u-boot branch (https://github.com/parallella/parallella-uboot/tree/parallella-next).

However according to this below that was not a good idea:
https://github.com/parallella/parallella-examples/blob/master/rpi-camera/README#L29

    Note that you might need to have a serial cable plugged in or
    have it tied to GND because the u-boot in the parallella-flash
    repo has a 'press any key' to interrupt boot prompt and garbage
    during boot on the serial line can interrupt the start sequence
    if it's floating.

I think we could emulate this by adding:

echo "Press Ctrl-C to abort the boot sequence"
pause 1

to the boot env. Which should be ok since Ctrl-C is unlikely to be in the garbage.

@olajep
Copy link
Member Author

olajep commented Aug 2, 2015

@smunaut
How often do you see this?

@olajep
Copy link
Member Author

olajep commented Aug 2, 2015

@afaerber
I think you've mentioned the same issue to me. But I was under the impression this was with the stock flash image?

Boot delay in default image is already set to 0. Maybe -2 is the correct value then?

#define CONFIG_BOOTDELAY  0 /* -2 to autoboot with no delay and not check for abort */

https://github.com/parallella/parallella-uboot/blob/parallella-gen1/include/configs/parallella.h#L77

@smunaut
Copy link

smunaut commented Aug 2, 2015

How often ? Well everytime pretty much. It just won't boot if my serial cable isn't connected.

@afaerber
Copy link

afaerber commented Aug 2, 2015

@olajep: I set the bootdelay environment variable to something like 3 with the stock firmware. The trick for doing so was powering on without an SD card inserted, then the default boot fails, dropping back to a prompt, where setenv and saveenv can be used to modify the environment.

My assumption was that the RX pin is incompletely configured by U-Boot. It probably needs to be configured to have a safe value by default.

@olajep
Copy link
Member Author

olajep commented Aug 3, 2015

Ok, then it seems CONFIG_BOOTDELAY = 0 will work.
As I mentioned in the first post I think we can have almost the same functionality but the user will have to press Ctrl-C instead of the any-key.

Thanks,
Ola

@olajep olajep added the Blocker label Aug 3, 2015
@afaerber
Copy link

afaerber commented Aug 4, 2015

You really should fix the cause of this, or the next input after U-Boot (e.g., grub2 boot menu with any-key timeout) may be affected just as well.

Ctrl-C does not seem to be a common thing in U-Boot, is there a config to enable such behavior or would you be hacking that in yourself? Deviating from other U-Boot boards/platforms would not be good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants