Skip to content

bsp: u-boot-fio: imx6ullevk: increase CONFIG_ENV_SIZE#1118

Merged
ricardosalveti merged 3 commits intofoundriesio:mainfrom
igoropaniuk:imx6ullevk_env
Apr 21, 2023
Merged

bsp: u-boot-fio: imx6ullevk: increase CONFIG_ENV_SIZE#1118
ricardosalveti merged 3 commits intofoundriesio:mainfrom
igoropaniuk:imx6ullevk_env

Conversation

@igoropaniuk
Copy link
Copy Markdown
Contributor

Set CONFIG_ENV_SIZE=0x4000 in lmp.cfg.

This fixes env size exceed error:
Saving Environment to FAT... Env export buffer too small: 8188, but need 8228

@quaresmajose
Copy link
Copy Markdown
Member

quaresmajose commented Apr 21, 2023

Maybe this is also needed in lmp-base?

@ricardosalveti
Copy link
Copy Markdown
Member

rsalveti@evatuf:~/build/lmp/layers/meta-lmp$ cat meta-lmp-bsp/recipes-bsp/u-boot/u-boot-fio/imx6ullevk/fw_env.config
/mnt/boot/uboot.env     0x0000          0x2000

Also needs alignment.

We also have other boards that might have similar issue:

meta-lmp-bsp/recipes-bsp/u-boot/u-boot-fio/apalis-imx6/fw_env.config:/mnt/boot/uboot.env     0x0000          0x2000
meta-lmp-bsp/recipes-bsp/u-boot/u-boot-fio/imx6ulevk/fw_env.config:/mnt/boot/uboot.env     0x0000          0x2000

Set CONFIG_ENV_SIZE=0x4000 in lmp.cfg.

This fixes env size exceed error:
Saving Environment to FAT... Env export buffer too small: 8188, but need 8228

Signed-off-by: Igor Opaniuk <[email protected]>
Set CONFIG_ENV_SIZE=0x4000 in lmp.cfg.

This fixes env size exceed error:
Saving Environment to FAT... Env export buffer too small: 8188, but need 8228

Signed-off-by: Igor Opaniuk <[email protected]>
Set CONFIG_ENV_SIZE=0x4000 in lmp.cfg.

This fixes env size exceed error:
Saving Environment to FAT... Env export buffer too small: 8188, but need 8228

Signed-off-by: Igor Opaniuk <[email protected]>
@igoropaniuk
Copy link
Copy Markdown
Contributor Author

@quaresmajose in lmp-base we don't use so big amount of env variables in boot.cmd script

Copy link
Copy Markdown
Member

@quaresmajose quaresmajose left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardosalveti ricardosalveti merged commit 0b44f15 into foundriesio:main Apr 21, 2023
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.

4 participants