Skip to content

[bootloader/storage] new bootloader and fix python issue#205

Merged
Yaya-Cout merged 42 commits intoUpsilonNumworks:upsilon-devfrom
devdl11:upsilon-new-bootloader
May 7, 2022
Merged

[bootloader/storage] new bootloader and fix python issue#205
Yaya-Cout merged 42 commits intoUpsilonNumworks:upsilon-devfrom
devdl11:upsilon-new-bootloader

Conversation

@devdl11
Copy link
Copy Markdown
Member

@devdl11 devdl11 commented Apr 7, 2022

An amazing Upsilon Bootloader !
Thanks @Yaya-Cout for helping me bring this beauty to life ;-)
I also fix by the way the python issue

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

Comment thread bootloader/itoa.cpp Outdated
Comment thread bootloader/itoa.cpp Outdated
Comment thread bootloader/itoa.cpp Outdated
Comment thread bootloader/itoa.cpp Outdated
Comment thread bootloader/itoa.h Outdated
uint64_t timeout = startTime + timeoutDelta;

/* If current time is big enough, currentTime + timeout wraps around the
/* If current time is big enough, currentTime + timeout wraps aroud the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Comment thread ion/src/device/shared/drivers/reset.cpp Outdated
Ion::Device::Cache::disable();

/* Shutdown all clocks and peripherals to mimic a hardware reset. */
/* Shutdown all clocks and periherals to mimic a hardware reset. */
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

? Changed correct spelling to wrong spelling

Comment thread ion/src/device/shared/drivers/wakeup.h Outdated

/* All wakeup functions can be called together without overwriting the same
* register. All together, they will set SYSCFG and EXTi registers as follow:
* register. All togethed, they will set SYSCFG and EXTi registers as follow:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Comment thread ion/src/device/shared/regs/tim.h Outdated

class CCMR : Register64 {
/* We're declaring CCMR as a 64 bits register. CCMR doesn't exist per se,
/* We're declaring CCMR as a 64 bits register. CCMR doesn't exsist per se,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Comment thread ion/src/device/shared/usb/calculator.h Outdated

Descriptor * m_descriptors[8];
/* m_descriptors contains only descriptors that should be returned via the
/* m_descriptors contains only descriptors that sould be returned via the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

devdl11 and others added 6 commits April 8, 2022 13:16
Co-authored-by: Lauryy06 <80424145+Lauryy06@users.noreply.github.com>
Co-authored-by: Lauryy06 <80424145+Lauryy06@users.noreply.github.com>
Co-authored-by: Lauryy06 <80424145+Lauryy06@users.noreply.github.com>
Co-authored-by: Lauryy06 <80424145+Lauryy06@users.noreply.github.com>
Co-authored-by: Lauryy06 <80424145+Lauryy06@users.noreply.github.com>
Co-authored-by: Lauryy06 <80424145+Lauryy06@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

5 similar comments
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2022

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

Comment thread bootloader/messages.h Outdated
Comment thread bootloader/messages.h Outdated
Comment thread bootloader/usb_data.h Outdated
Comment on lines +6 to +7
# Disable Script
sys.exit(0)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but why are you not removing it from the Makefile ?

Comment thread ion/src/device/bootloader/usb/Makefile Outdated
uint32_t * persistence_end_32 = (uint32_t *)&_exam_mode_buffer_end;
assert((persistence_end_32 - persistence_start_32) % 4 == 0);
while (persistence_start_32 < persistence_end_32 && *persistence_start_32 == 0x0) {
uint32_t * persitence_start_32 = (uint32_t *)&_exam_mode_buffer_start;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you edit it ?

Comment thread ion/src/device/shared/usb/dfu.ld Outdated
Comment thread ion/src/device/shared/usb/dfu_interface.h Outdated
Comment thread ion/src/device/shared/usb/dfu_relocated.cpp Outdated
Comment thread ion/src/device/shared/usb/stack/device.cpp Outdated
devdl11 and others added 5 commits April 8, 2022 18:02
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@devdl11 devdl11 marked this pull request as ready for review April 26, 2022 10:46
Copy link
Copy Markdown
Member

@Yaya-Cout Yaya-Cout left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove themes/themes/local/upsilon_light/bootloader/computer.xcf

Comment thread bootloader/boot.h Outdated
Comment thread bootloader/drivers/stm32_drivers.h Outdated
Comment thread bootloader/interface/menus/about.cpp Outdated
Comment thread bootloader/interface/menus/about.h Outdated
Comment thread bootloader/interface/menus/crash.cpp Outdated
Comment thread bootloader/interface/menus/installer.h Outdated
Comment thread bootloader/interface/menus/slot_recovery.cpp Outdated
Comment thread bootloader/interface/menus/slot_recovery.h Outdated
Comment thread bootloader/interface/menus/warning.cpp Outdated
Comment thread bootloader/interface/menus/warning.h Outdated
Co-authored-by: Yaya-Cout <yaya.cout@free.fr>
@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

Comment thread bootloader/interface/static/messages.h Outdated

constexpr static const char * bootloaderVersion = "Version 1.0.0 - FREED0M";

//USB NAMES
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you use capitals letters ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because it's more stylish😎😎

devdl11 and others added 2 commits April 27, 2022 15:44
Co-authored-by: Yaya-Cout <yaya.cout@free.fr>
@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

1 similar comment
@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

Comment thread bootloader/interface/menus/dfu.h Outdated
Co-authored-by: Yaya-Cout <yaya.cout@free.fr>
@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

Comment thread bootloader/boot.cpp Outdated
Comment on lines +40 to +41
// Ion::Device::Flash::DisableInternalProtection();
// Ion::Device::Flash::EraseSector(0);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove comments if unused (below too)

Suggested change
// Ion::Device::Flash::DisableInternalProtection();
// Ion::Device::Flash::EraseSector(0);

Copy link
Copy Markdown
Member Author

@devdl11 devdl11 Apr 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's for the next update x)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But I think i found something out

@github-actions
Copy link
Copy Markdown

.text .rodata .bss .data Total (RAM) Total (ROM)
Base 814864 bytes 439766 bytes 225896 bytes 1776 bytes 227672 bytes 1256406 bytes
Head 813476 bytes 438837 bytes 225896 bytes 1776 bytes 227672 bytes 1254089 bytes
-1388 bytes -929 bytes +0 bytes +0 bytes +0 bytes -2317 bytes
-0.2 % -0.2 % +0.0 % +0.0 % +0.0 % -0.2 %

@Yaya-Cout Yaya-Cout merged commit 7f8e6a6 into UpsilonNumworks:upsilon-dev May 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Impossible to build binpack

4 participants