Re: Adding Nucleo-F030R8 support to Zephyr - runtime error
Bobby
Hello Maciej,
2017-09-14 12:07 GMT+02:00 Maciej Dębski <maciej.debski@...>:
Hello Bobby, hello Neil,thank you for working on this.I did not spend much time on this, I just merged the recent zephyr master into my branch and added these changes:Hello world and blinky are working just fine.I did not merge the Bobby's pull request to my pull request yet:If I understand correctly, the above is about the same issue as your recent update? If you think it is the way you want to resolve the issue with vector table - just update the pull request, Bobby.
Pull request now updated. Please pull and test.
Thank you
Bobby
If I am wrong, then create the 2nd pull request, I will merge them, when you are ready.My pull request to zephyr master for stm32f0 is up to date with all the required changes. I fixed all the issues they asked me to, but no new reviews showed up. I am waiting for now. Here it is:Yours sincerely,Maciej DębskiOn Thu, Sep 14, 2017 at 10:55 AM, Neil Armstrong <narmstrong@...> wrote:Hi Bobby,
I was thinking of something similar, good work !
I'll test it,
@Maciej, would you have time to test it and integrate it to your pull request ?
Thanks,
Neil
On 09/13/2017 09:54 PM, b0661 wrote:
> Hi Neil,
>
> I created another vector table relocation function and adjusted the linker control file.
> Please have a look at https://github.com/b0661/zephyr/commit/6be292fab8580b3afa4a5 0cd59b8c406701d08d3.
>
> I don´t know much about the bootloader mechanisms you mentioned - please review/ comment.
>
> Thanks
> Bobby
>
>
>
> 2017-09-13 14:44 GMT+02:00 Neil Armstrong <narmstrong@... <mailto:narmstrong@...m>>:
>
> On 09/13/2017 02:30 PM, b0661 wrote:
> > Hi Neal,
> >
> > 2017-09-13 14:10 GMT+02:00 Neil Armstrong <narmstrong@... <mailto:narmstrong@...m> <mailto:narmstrong@... m <mailto:narmstrong@... m>>>:
> >
> > On 09/13/2017 02:09 PM, b0661 wrote:
> > > Hi Neal,
> > >
> > > I had a look at the linker script https://github.com/zephyrproject-rtos/zephyr/blob/master/ include/arch/arm/cortex_m/ scripts/linker.ld <https://github.com/zephyrproj ect-rtos/zephyr/blob/master/ include/arch/arm/cortex_m/ scripts/linker.ld> <https://github.com/zephyrproj ect-rtos/zephyr/blob/master/ include/arch/arm/cortex_m/ scripts/linker.ld <https://github.com/zephyrproj ect-rtos/zephyr/blob/master/ include/arch/arm/cortex_m/ scripts/linker.ld>> which is also used for STM32F0.
> > >
> > > Transfer of the vector table to address 0.. of the SRAM will either write to the application .data section or the kernel .bss section. The .bss section will be zeroed just after the relocation of the vector table.
> > >
> > > I think there must be a memory reservation for the vector table at the start of the RAMABLE_REGION.
> >
> > Exact
> >
> > >
> > >
> > > I still do not see why execution in place (XIP) mandates the interrupt vector table to be in SRAM. This should be a different configuration option to be set by e.g. bootloaders. Event bootloaders may avoid that by using trampolines. Based on this option the memory at the start of the SRAM can be reseved, the SRAM mapped to address 0 and the vector table transfered from flash to SRAM.
> >
> > Yes it should, then it should disable all the dynamic update and generation of the vector table.
> >
> >
> > There are use cases where you really do not need the dynamic updates (mine for example - updates are done by the supervising system) but as much RAM as possible. So why not make it a config option?
>
> Yes, I think the SRAM mapped case should be an option, but selected by default. Anyway, something must be pushed along the STM32F0 basic support.
>
> Neil
>
> >
> >
> > >
> > >
> > > Bobby
> > >
> > > 2017-09-11 17:09 GMT+02:00 Neil Armstrong <narmstrong@... <mailto:narmstrong@...m> <mailto:narmstrong@... m <mailto:narmstrong@... m>> <mailto:narmstrong@... m <mailto:narmstrong@... m> <mailto:narmstrong@... m <mailto:narmstrong@... m>>>>:
> > >
> > > Hi,
> > >
> > > Since Cortex-M0 cannot relocate the vector base with a processor register, the vector must be moved.
> > >
> > > Yes you can map the first kbytes of the flash to the first 32k bytes of 0, but you won't tun change the vector or run other applications if you have a bootloader.
> > >
> > > But yes, you may need to add a further Kconfig to completely disable relocation, but also disable vector modification because it can only occur in this SRAM mapped a 0.
> > >
> > > Neil
> > >
> > > On 09/11/2017 04:02 PM, b0661 wrote:
> > > > Hi Neil,
> > > >
> > > > the patch forces the STM32F0 vector table to be in SRAM as CONFIG_FLASH_BASE_ADDRESS and CONFIG_SRAM_BASE_ADDRESS are not 0.
> > > >
> > > > I would appreciate to save precious SRAM and do the remap only if it is needed.
> > > >
> > > > I could not find out how the vector table space is reserved in SRAM. Is this already done or are there more patches to follow to ensure this?
> > > >
> > > > My patch was agnostic of config options by intention. It copies the vector table if the vector table content is not the expected one. All other measures to ensure correct vector table operations are left to the application/soc/board/....
> > > >
> > > > I will test your patch.
> > > >
> > > > Bobby
> > > >
> > > >
> > > > 2017-09-11 14:54 GMT+02:00 Neil Armstrong <narmstrong@... <mailto:narmstrong@...m> <mailto:narmstrong@... m <mailto:narmstrong@... m>> <mailto:narmstrong@... m <mailto:narmstrong@... m> <mailto:narmstrong@... m <mailto:narmstrong@... m>>> <mailto:narmstrong@... m <mailto:narmstrong@... m> <mailto:narmstrong@... m <mailto:narmstrong@... m>> <mailto:narmstrong@... m <mailto:narmstrong@... m> <mailto:narmstrong@... m <mailto:narmstrong@... m>>>>>:
> <https://github.com/b0661/zep> > > >
> > > > Hi Maciej, Bobby,
> > > >
> > > > Could test the following patch ?
> > > >
> > > > If it works, it's the correct way to handle this situation, and should be included in the stm32f0 support patchset.
> > > >
> > > >
> > > > --><------------------------------------------------------- --------------------
> > > >
> > > > diff --git a/arch/arm/core/cortex_m/prep_c.c b/arch/arm/core/cortex_m/prep_ c.c
> > > > index 1382379..57e0e92 100644
> > > > --- a/arch/arm/core/cortex_m/prep_c.c
> > > > +++ b/arch/arm/core/cortex_m/prep_c.c
> > > > @@ -26,12 +26,25 @@
> > > >
> > > > #ifdef CONFIG_ARMV6_M
> > > >
> > > > +/**
> > > > + * @brief On Cortex-M0 platforms, the Vector Base address cannot be changed.
> > > > + * This functions is a hook before the vector memory relocation to
> > > > + * ensure the mapped memory is correct.
> > > > + */
> > > > +void __weak relocate_vector_memory(void)
> > > > +{
> > > > + /* Nothing to do by default */
> > > > +}
> > > > +
> > > > #define VECTOR_ADDRESS 0
> > > > static inline void relocate_vector_table(void)
> > > > {
> > > > #if defined(CONFIG_XIP) && (CONFIG_FLASH_BASE_ADDRESS != 0) || \
> > > > !defined(CONFIG_XIP) && (CONFIG_SRAM_BASE_ADDRESS != 0)
> > > > size_t vector_size = (size_t)_vector_end - (size_t)_vector_start;
> > > > +
> > > > + relocate_vector_memory();
> > > > +
> > > > memcpy(VECTOR_ADDRESS, _vector_start, vector_size);
> > > > #endif
> > > > }
> > > > diff --git a/arch/arm/soc/st_stm32/stm32f0/soc.c b/arch/arm/soc/st_stm32/stm32f 0/soc.c
> > > > index 727c348..2c056c2 100644
> > > > --- a/arch/arm/soc/st_stm32/stm32f0/soc.c
> > > > +++ b/arch/arm/soc/st_stm32/stm32f0/soc.c
> > > > @@ -17,6 +17,16 @@
> > > > #include <cortex_m/exc.h>
> > > >
> > > > /**
> > > > + * @brief On Cortex-M0 platforms, the Vector Base address cannot be changed.
> > > > + * This functions is a hook before the vector memory relocation to
> > > > + * ensure the mapped memory is correct.
> > > > + */
> > > > +void relocate_vector_memory(void)
> > > > +{
> > > > + LL_SYSCFG_SetRemapMemory(LL_SYSCFG_REMAP_SRAM);
> > > > +}
> > > > +
> > > > +/**
> > > > * @brief This function configures the source of stm32cube time base.
> > > > * Cube HAL expects a 1ms tick which matches with k_uptime_get_32.
> > > > * Tick interrupt priority is not used
> > > > --
> > > >
> > > > Thanks,
> > > > Neil
> > > >
> > > >
> > > > On 09/11/2017 02:35 PM, Neil Armstrong wrote:
> > > > > Hi All,
> > > > >
> > > > > On STM32F0 SoCs, the forst 32K cam be remapped to an internal SRAM, but this must be done *before* the relocate_vector_table() call.
> > > > >
> > > > > I will push a patch to add a relocate hook for Cortex-M0 platforms to use the LL LL_SYSCFG_SetRemapMemory() call before the vector table relocating.
> > > > >
> > > > > Neil
> > > > >
> > > > > On 09/11/2017 12:06 PM, Neil Armstrong wrote:
> > > > >> Hi All,
> > > > >>
> > > > >> I have some concern about this commit eb48a0a73c11c2a9cd4c3c91864ca4e0cf52dae8 (arm: armv6-m: Support relocating vector table).
> > > > >>
> > > > >> On STM32, flash is not at address 0, but can be relocated like sram spi-flash, external nand, ..., but should be when booting from flash, but this code is totally erroneous !
> > > > >>
> > > > >> The #if is bogus :
> > > > >> #if defined(CONFIG_XIP) && (CONFIG_FLASH_BASE_ADDRESS != 0) || !defined(CONFIG_XIP) && (CONFIG_SRAM_BASE_ADDRESS != 0)
> > > > >>
> > > > >> And it doesn't take in account what is actually mapped at address 0 !
> > > > >>
> > > > >> On STM32, we can consider if we boot on XIP the flash is mapped on 0, so no need to relocate vectors, but indeed it can be remapped later.
> > > > >>
> > > > >> @Bobby : I think this relocate should be disabled instead if avoiding copy when the values are the same.
> > > > >>
> > > > >> @kumar, erwan : should it be handled with a config disabling the relocation on stm32 ?
> > > > >>
> > > > >> Neil
> > > > >>
> > > > >> On 09/03/2017 07:22 PM, b0661 wrote:
> > > > >>> Hello Maciej,
> > > > >>>
> > > > >>> I assume the problem is - as Yannis pointed out - the code triggers a write to flash which stops in some way execution.
> > > > >>>
> > > > >>> The commit "arch: arm: core: fix vector table relocate write to flash"
> > > > >>> https://github.com/b0661/zephyr/commit/f171c8ca1b2e7398c97d2 423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314
hyr/commit/f171c8ca1b2e7398c97 d2423c146392bc8e3c314>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>>>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>
> > <https://github.com/b0661/zephyr/commit/f171c8ca1b2e7398c97 d2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314> <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314 <https://github.com/b0661/zeph yr/commit/f171c8ca1b2e7398c97d 2423c146392bc8e3c314>>>>>
> > > > >>> works for me on NUCLEO F091RC and I do not see any side effects (but who knows?).
> > > > >>>
> > > > >>> I do not have a board that needs the relocation. So this is untested for such boards.
> > > > >>> Nevertheless the commit passes my local sanity check.
> > > > >>>
> > > > >>> If it works for Nucleo-F030R8 too I can prepare a pull request and see whether it passes CI.
> > > > >>>
> > > > >>> Bobby
> > > > >>>
> > > > >>>
> > > > >>> 2017-08-25 14:24 GMT+02:00 Maciej Dębski <maciej.debski@... <mailto:maciej.debski@...om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om
> <mailto:maciej.debski@rndity.com>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>>>>>:
> > > > >>>
> > > > >>> Gentlemen,
> > > > >>>
> > > > >>> thank you for your quick responses!
> > > > >>>
> > > > >>> As I wanted to provide more info and debug output, I accidentally found the issue.
> > > > >>> This little change in arch/arm/core/cortex_m/prep_c.c caused sys fatal error on my f0 board, even before the stm32f0_init.
> > > > >>>
> > > > >>> Here is the commit:
> > > > >>> https://github.com/zephyrproject-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>>> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>
> <https://github.com/zephyrproject-rtos/zephyr/commit/eb48a0 a73c11c2a9cd4c3c91864ca4e0cf52 dae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>>>> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>>
> > > > >>> On Wed, Aug 23, 2017 at 10:47 AM, Maciej Dębski <maciej.debski@... <mailto:maciej.debski@...> > <https://github.com/zephyrproject-rtos/zephyr/commit/eb48a0 a73c11c2a9cd4c3c91864ca4e0cf52 dae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>>> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>> <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>
> <https://github.com/zephyrproject-rtos/zephyr/commit/eb48a0 a73c11c2a9cd4c3c91864ca4e0cf52 dae8 <https://github.com/zephyrproj ect-rtos/zephyr/commit/eb48a0a 73c11c2a9cd4c3c91864ca4e0cf52d ae8>>>>>
> > > > >>>
> > > > >>> And here are the specific changes causing problem:
> > > > >>>
> > > > >>> diff --git a/arch/arm/core/cortex_m/prep_c.c b/arch/arm/core/cortex_m/prep_ c.c
> > > > >>> index d23dd8b..1382379 100644
> > > > >>> --- a/arch/arm/core/cortex_m/prep_c.c
> > > > >>> +++ b/arch/arm/core/cortex_m/prep_c.c
> > > > >>> @@ -22,9 +22,20 @@
> > > > >>> #include <linker/linker-defs.h>
> > > > >>> #include <nano_internal.h>
> > > > >>> #include <arch/arm/cortex_m/cmsis.h>
> > > > >>> +#include <string.h>
> > > > >>>
> > > > >>> #ifdef CONFIG_ARMV6_M
> > > > >>> -static inline void relocate_vector_table(void) { /* do nothing */ }
> > > > >>> +
> > > > >>> +#define VECTOR_ADDRESS 0
> > > > >>> +static inline void relocate_vector_table(void)
> > > > >>> +{
> > > > >>> +#if defined(CONFIG_XIP) && (CONFIG_FLASH_BASE_ADDRESS != 0) || \
> > > > >>> + !defined(CONFIG_XIP) && (CONFIG_SRAM_BASE_ADDRESS != 0)
> > > > >>> + size_t vector_size = (size_t)_vector_end - (size_t)_vector_start;
> > > > >>> + memcpy(VECTOR_ADDRESS, _vector_start, vector_size);
> > > > >>> +#endif
> > > > >>> +}
> > > > >>> +
> > > > >>> #elif defined(CONFIG_ARMV7_M)
> > > > >>> #ifdef CONFIG_XIP
> > > > >>> #define VECTOR_ADDRESS ((uintptr_t)&_image_rom_start + \
> > > > >>>
> > > > >>>
> > > > >>> When I deleted the new body of the relocate_vector_table function (to do nothing as it was) - blinky and hello world started to work fine again.
> > > > >>> What should I do now? How to report it properly?
> > > > >>>
> > > > >>> Thank you!
> > > > >>>
> > > > >>> Yours faithfully,
> > > > >>> Maciej Dębski
> > > > >>>
> > > > >>>
> > > > >>>
> > > > >>>
om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>> <mailto:maciej.debski@... om <mailto:maciej.debski@... om> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>> <mailto:maciej.debski@... om
> <mailto:maciej.debski@rndity.com> <mailto:maciej.debski@... om <mailto:maciej.debski@... om>>>>>> wrote:
> > > > >>>
> > > > >>> Hello,
> > > > >>>
> > > > >>> I am developing support for nucleo board, with stm32f030r8 MCU. The goal was to compile and run the samples provided with Zephyr, blinky and hello_world.
> > > > >>>
> > > > >>> I managed to finish the job, all was good, so I have done a pull request. Then, one of the reviewers pointed out that new approach to pinctrl nodes and uart pinctrl configuration in stm32 socs DT files was introduced. I was asked to do appropriate changes.
> > > > >>>
> > > > >>> I modified my code to fit the Zephyr master. Sadly, blinky and hello_world have stopped working. The code itself is compiling and flashing fine. Just the board is reporting a fatal error before even my code is executed.
> > > > >>> After checking the code over and over, I think I need help!
> > > > >>>
> > > > >>> I believe most of the values are correct. I just do not fully understand the new dts/arm file structure, which is in Python, maybe I have missed something. Would you be so kind as to look at my code and help me find the issue?
> > > > >>>
> > > > >>> https://github.com/zephyrproject-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>>> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>>>>
> <https://github.com/zephyrproject-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>>> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103> <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103 <https://github.com/zephyrproj ect-rtos/zephyr/pull/1103>>>>>
> > > > >>>
> > > > >>> This is my pull request. I would focus on dts/arm and include/dt-bindings.
> > > > >>>
> > > > >>> Yours faithfully,
> > > > >>> Maciej Dębski
> > > > >>>
> > > > >>>
> > > > >>>
> > > > >>> _______________________________________________
> > > > >>> Zephyr-devel mailing list
> > > > >>> Zephyr-devel@...ect.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>>>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>> <mailto:Zephyr-devel@... hyrproject.org
> <mailto:Zephyr-devel@...phyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>>>>>
> > > > >>> https://lists.zephyrproject.org/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el
> <https://lists.zephyrproject.org/mailman/listinfo/zephyr-de vel>>>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>
> > <https://lists.zephyrproject.org/mailman/listinfo/zephyr-de vel <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>>>>>
> > > > >>>
> > > > >>>
> > > > >>>
> > > > >>>
> > > > >>> _______________________________________________
> > > > >>> Zephyr-devel mailing list
> > > > >>> Zephyr-devel@...ct.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org> <mailto:Zephyr-devel@... hyrproject.org <mailto:Zephyr-devel@... hyrproject.org>>>>
> > > > >>> https://lists.zephyrproject.org/mailman/listinfo/zephyr-deve l <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el>> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el> <https://lists.zephyrproject.o rg/mailman/listinfo/zephyr-dev el
> <https://lists.zephyrproject.org/mailman/listinfo/zephyr-de vel>>>>
> > > > >>>
> > > > >>
> > > > >
> > > >
> > > >
> > >
> > >
> >
> >
>
>