Message ID | 1417504034-25522-1-git-send-email-human.hwang@samsung.com |
---|---|
State | Not Applicable |
Delegated to: | Minkyu Kang |
Headers | show |
Hello all, On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: > This is v10 of the patchset adding support Odroud XU3 board. > > link to the previous version: > v2: https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html > v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html > v4: https://patchwork.ozlabs.org/patch/407411/ > v5: https://patchwork.ozlabs.org/patch/407941/ > v6: https://patchwork.ozlabs.org/patch/408647/ > v7: https://patchwork.ozlabs.org/patch/410204/ > v8: https://patchwork.ozlabs.org/patch/410709/ > v9: https://patchwork.ozlabs.org/patch/415514/ > > The patchset by Akshay Saraswat is already merged into u-boot-samsung. > Please ignore the instruction about the additional patchset in v8. > > How to test this patch: > 1. git clone http://git.denx.de/u-boot-samsung.git > 2. Apply this patchset > 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config > 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 > 5. Now you can use u-boot-dtb.bin for your downloading > > WARNING: (update in V10) > It was impossible to make the u-boot image smaller than 335872 bytes, and also > I did not want to waste my time to make it fit for that restriction, because > the discussion about this issue is in progress in malining list. I used BL2 by > Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is removed for > testing. If you need the BL2 for testing, please contact me or Suriyan. > > Note: If you use micro SD card for your test you have to apply the below > patch additionally. This patch is needed, because micro sd card is > recognized as MMC1 instead of MMC0. Additional work is needed to make it > work regardless of device id. > > Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also made > real patch for me. I squashed some part of that patchset, and included the > remainings into my patchset. > > diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-common.h > index ba591e7..437eaae 100644 > --- a/include/configs/exynos5-common.h > +++ b/include/configs/exynos5-common.h > @@ -109,7 +109,7 @@ > > #define CONFIG_SYS_MONITOR_BASE 0x00000000 > > -#define CONFIG_SYS_MMC_ENV_DEV 0 > +#define CONFIG_SYS_MMC_ENV_DEV 1 > > #define CONFIG_SECURE_BL1_ONLY > > Changes for v2: > - Add a patch to add new common setup header file for Odroid X2/U3 > and Odroid XU3 > > Changes for v3: > - Remove the patch which adds new common setup header file from v2 > - Remove the wrong patch to fix GPIO information of Exynos 5800 > - Remove unnecessary node from DT file > - Remove unnecessary features from config file > - Fix some trivial typos in comments > > Changes for v4: > - Add MMC FIFO buffer's configuration to DT file > - Make CONFIG_OF_CONTROL be set by the target information > - Add basic document to doc/README.odroid-xu3 > - Add CONFIG_CMD_EXT4 to config file > - Add environment size and offset to config file > - Add extra default environment to make bootable without modification > - Remove unnecessary features from config file > > Changes for v5: > - Add the specific build instruction > - Update the information of patchset by Akshay Saraswat > - Convert /include/ to #include in DT file > > Changes for v6: > - Separate out the documentation to new commit > - Remove unnecessary header file inclusuib from the board-specific setup file > - Make the function board_clock_init be declared, only when > CONFIG_BOARD_EARLY_INIT_F is defined > > Changes for v7: > - Fix several errata in the documentation > - Remove OF_CONTROL dependency from !SPL_BUILD > > Changes for v8: > - Remove unnecessary properties in DT mmc node > > Changes for v9: > - Remove the first patch which change GPIO entries order. It became useless, > after DT support is added > - Remove useless variables in the default environment > - Add the new contents to the documentation of Odroid X2/U2, instead of > making new document for Odorid XU3 > - Remove the detailed information in the header file, and leave the > reference to the documentation > > Changes for v10: > - Move the non common configs in Exynos5420 config file to each board file > - Remove the config unsets which are added to make the result image small > This is needless now, because the image is not small enough even though these > unsets are added. > - Remove redundant DT node and properties > - Remove the odroid-xu3 board file and make odroid-xu3 a variant of smdk5420 > - Fix some errata in documentation > - Include the patch by Sjoerd Simons as 4th patch > > Hyungwon Hwang (3): > config: exynos5420: move non common configs to specific board files > Odroid-XU3: Add support for Odroid-XU3 > Odroid-XU3: Add documentation for Odroid-XU3 > > Sjoerd Simons (1): > Odroid-XU3: Add entry for DTS EHCI GPIO > > arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ > arch/arm/dts/Makefile | 3 +- > arch/arm/dts/exynos5422-odroidxu3.dts | 49 +++++++++++++++++++++++++++++++ > board/samsung/smdk5420/Kconfig | 13 +++++++++ > configs/odroid-xu3_defconfig | 4 +++ > doc/README.odroid | 46 ++++++++++++++++++----------- > include/configs/exynos5420-common.h | 18 ------------ > include/configs/odroid_xu3.h | 54 +++++++++++++++++++++++++++++++++++ > include/configs/peach-pi.h | 6 ++++ > include/configs/peach-pit.h | 6 ++++ > include/configs/smdk5420.h | 11 +++++++ > 11 files changed, 179 insertions(+), 35 deletions(-) > create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts > create mode 100644 configs/odroid-xu3_defconfig > create mode 100644 include/configs/odroid_xu3.h > I talked to man from the hardkernel, and he said that there is no restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline U-Boot. So I think, in this case it should be provided also with those patches. Then we have a complete board setup. To Hyungwon, There is a size limit for each patch (100KiB), so if you would like to add such binary blob, you should add it using this site: http://www.denx.de/wiki/attach/U-Boot/TooBigPatches after a registration. Best regards
Hello all, On Wed, Dec 3, 2014 at 4:46 AM, Przemyslaw Marczak <p.marczak@samsung.com> wrote: > Hello all, > > > On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: >> >> This is v10 of the patchset adding support Odroud XU3 board. >> >> link to the previous version: >> v2: https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html >> v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html >> v4: https://patchwork.ozlabs.org/patch/407411/ >> v5: https://patchwork.ozlabs.org/patch/407941/ >> v6: https://patchwork.ozlabs.org/patch/408647/ >> v7: https://patchwork.ozlabs.org/patch/410204/ >> v8: https://patchwork.ozlabs.org/patch/410709/ >> v9: https://patchwork.ozlabs.org/patch/415514/ >> >> The patchset by Akshay Saraswat is already merged into u-boot-samsung. >> Please ignore the instruction about the additional patchset in v8. >> >> How to test this patch: >> 1. git clone http://git.denx.de/u-boot-samsung.git >> 2. Apply this patchset >> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config >> 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 >> 5. Now you can use u-boot-dtb.bin for your downloading >> >> WARNING: (update in V10) >> It was impossible to make the u-boot image smaller than 335872 bytes, and >> also >> I did not want to waste my time to make it fit for that restriction, >> because >> the discussion about this issue is in progress in malining list. I used >> BL2 by >> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is removed >> for >> testing. If you need the BL2 for testing, please contact me or Suriyan. >> >> Note: If you use micro SD card for your test you have to apply the below >> patch additionally. This patch is needed, because micro sd card is >> recognized as MMC1 instead of MMC0. Additional work is needed to make it >> work regardless of device id. >> >> Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also >> made >> real patch for me. I squashed some part of that patchset, and included the >> remainings into my patchset. >> >> diff --git a/include/configs/exynos5-common.h >> b/include/configs/exynos5-common.h >> index ba591e7..437eaae 100644 >> --- a/include/configs/exynos5-common.h >> +++ b/include/configs/exynos5-common.h >> @@ -109,7 +109,7 @@ >> >> #define CONFIG_SYS_MONITOR_BASE 0x00000000 >> >> -#define CONFIG_SYS_MMC_ENV_DEV 0 >> +#define CONFIG_SYS_MMC_ENV_DEV 1 >> >> #define CONFIG_SECURE_BL1_ONLY >> >> Changes for v2: >> - Add a patch to add new common setup header file for Odroid X2/U3 >> and Odroid XU3 >> >> Changes for v3: >> - Remove the patch which adds new common setup header file from v2 >> - Remove the wrong patch to fix GPIO information of Exynos 5800 >> - Remove unnecessary node from DT file >> - Remove unnecessary features from config file >> - Fix some trivial typos in comments >> >> Changes for v4: >> - Add MMC FIFO buffer's configuration to DT file >> - Make CONFIG_OF_CONTROL be set by the target information >> - Add basic document to doc/README.odroid-xu3 >> - Add CONFIG_CMD_EXT4 to config file >> - Add environment size and offset to config file >> - Add extra default environment to make bootable without modification >> - Remove unnecessary features from config file >> >> Changes for v5: >> - Add the specific build instruction >> - Update the information of patchset by Akshay Saraswat >> - Convert /include/ to #include in DT file >> >> Changes for v6: >> - Separate out the documentation to new commit >> - Remove unnecessary header file inclusuib from the board-specific setup >> file >> - Make the function board_clock_init be declared, only when >> CONFIG_BOARD_EARLY_INIT_F is defined >> >> Changes for v7: >> - Fix several errata in the documentation >> - Remove OF_CONTROL dependency from !SPL_BUILD >> >> Changes for v8: >> - Remove unnecessary properties in DT mmc node >> >> Changes for v9: >> - Remove the first patch which change GPIO entries order. It became >> useless, >> after DT support is added >> - Remove useless variables in the default environment >> - Add the new contents to the documentation of Odroid X2/U2, instead of >> making new document for Odorid XU3 >> - Remove the detailed information in the header file, and leave the >> reference to the documentation >> >> Changes for v10: >> - Move the non common configs in Exynos5420 config file to each board file >> - Remove the config unsets which are added to make the result image small >> This is needless now, because the image is not small enough even though >> these >> unsets are added. >> - Remove redundant DT node and properties >> - Remove the odroid-xu3 board file and make odroid-xu3 a variant of >> smdk5420 >> - Fix some errata in documentation >> - Include the patch by Sjoerd Simons as 4th patch >> >> Hyungwon Hwang (3): >> config: exynos5420: move non common configs to specific board files >> Odroid-XU3: Add support for Odroid-XU3 >> Odroid-XU3: Add documentation for Odroid-XU3 >> >> Sjoerd Simons (1): >> Odroid-XU3: Add entry for DTS EHCI GPIO >> >> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ >> arch/arm/dts/Makefile | 3 +- >> arch/arm/dts/exynos5422-odroidxu3.dts | 49 >> +++++++++++++++++++++++++++++++ >> board/samsung/smdk5420/Kconfig | 13 +++++++++ >> configs/odroid-xu3_defconfig | 4 +++ >> doc/README.odroid | 46 ++++++++++++++++++----------- >> include/configs/exynos5420-common.h | 18 ------------ >> include/configs/odroid_xu3.h | 54 >> +++++++++++++++++++++++++++++++++++ >> include/configs/peach-pi.h | 6 ++++ >> include/configs/peach-pit.h | 6 ++++ >> include/configs/smdk5420.h | 11 +++++++ >> 11 files changed, 179 insertions(+), 35 deletions(-) >> create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts >> create mode 100644 configs/odroid-xu3_defconfig >> create mode 100644 include/configs/odroid_xu3.h >> > > I talked to man from the hardkernel, and he said that there is no > restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline > U-Boot. So I think, in this case it should be provided also with those > patches. Then we have a complete board setup. > > To Hyungwon, > There is a size limit for each patch (100KiB), so if you would like to add > such binary blob, you should add it using this site: > http://www.denx.de/wiki/attach/U-Boot/TooBigPatches > after a registration. > I do have a signed BL2 (1MB U-Boot image - same as for the Odroid U3) at http://forum.odroid.com/viewtopic.php?f=98&t=7388 It has the full package - BL1/BL2/TZ and the sd_fusing script which works for both SD as well as eMMC. If we go with this BL2, then CONFIG_ENV_OFFSET (currently set at 1280 * 1024) will clash with the TZ area in SD card. ( I am assuming its OK for eMMC as the ENV resides in the user partition and not in boot0) This offset is OK for the Odroid-U3 as the TZ is smaller (156 K). For the Odroid U3: free area is after 2111 + (156 * 2) = 2423. The ENV starts at 1280K = offset block 2560. For the Odroid-XU3 the TZ is 256K. Free area after the blobs starts from 2111 + (256 * 2) = 2623. Hence, we will have to shift CONFIG_ENV_OFFSET to anything greater than 2623 (512 byte) blocks. Anything with CONFIG_ENV_OFFSET (SZ_1K * 1312) should work. Thanks and Regards - Suriyan > Best regards > -- > Przemyslaw Marczak > Samsung R&D Institute Poland > Samsung Electronics > p.marczak@samsung.com
Hi, On 3 December 2014 at 05:46, Przemyslaw Marczak <p.marczak@samsung.com> wrote: > Hello all, > > > On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: >> >> This is v10 of the patchset adding support Odroud XU3 board. >> >> link to the previous version: >> v2: https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html >> v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html >> v4: https://patchwork.ozlabs.org/patch/407411/ >> v5: https://patchwork.ozlabs.org/patch/407941/ >> v6: https://patchwork.ozlabs.org/patch/408647/ >> v7: https://patchwork.ozlabs.org/patch/410204/ >> v8: https://patchwork.ozlabs.org/patch/410709/ >> v9: https://patchwork.ozlabs.org/patch/415514/ >> >> The patchset by Akshay Saraswat is already merged into u-boot-samsung. >> Please ignore the instruction about the additional patchset in v8. >> >> How to test this patch: >> 1. git clone http://git.denx.de/u-boot-samsung.git >> 2. Apply this patchset >> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config >> 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 >> 5. Now you can use u-boot-dtb.bin for your downloading >> >> WARNING: (update in V10) >> It was impossible to make the u-boot image smaller than 335872 bytes, and >> also >> I did not want to waste my time to make it fit for that restriction, >> because >> the discussion about this issue is in progress in malining list. I used >> BL2 by >> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is removed >> for >> testing. If you need the BL2 for testing, please contact me or Suriyan. >> >> Note: If you use micro SD card for your test you have to apply the below >> patch additionally. This patch is needed, because micro sd card is >> recognized as MMC1 instead of MMC0. Additional work is needed to make it >> work regardless of device id. >> >> Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also >> made >> real patch for me. I squashed some part of that patchset, and included the >> remainings into my patchset. >> >> diff --git a/include/configs/exynos5-common.h >> b/include/configs/exynos5-common.h >> index ba591e7..437eaae 100644 >> --- a/include/configs/exynos5-common.h >> +++ b/include/configs/exynos5-common.h >> @@ -109,7 +109,7 @@ >> >> #define CONFIG_SYS_MONITOR_BASE 0x00000000 >> >> -#define CONFIG_SYS_MMC_ENV_DEV 0 >> +#define CONFIG_SYS_MMC_ENV_DEV 1 >> >> #define CONFIG_SECURE_BL1_ONLY >> >> Changes for v2: >> - Add a patch to add new common setup header file for Odroid X2/U3 >> and Odroid XU3 >> >> Changes for v3: >> - Remove the patch which adds new common setup header file from v2 >> - Remove the wrong patch to fix GPIO information of Exynos 5800 >> - Remove unnecessary node from DT file >> - Remove unnecessary features from config file >> - Fix some trivial typos in comments >> >> Changes for v4: >> - Add MMC FIFO buffer's configuration to DT file >> - Make CONFIG_OF_CONTROL be set by the target information >> - Add basic document to doc/README.odroid-xu3 >> - Add CONFIG_CMD_EXT4 to config file >> - Add environment size and offset to config file >> - Add extra default environment to make bootable without modification >> - Remove unnecessary features from config file >> >> Changes for v5: >> - Add the specific build instruction >> - Update the information of patchset by Akshay Saraswat >> - Convert /include/ to #include in DT file >> >> Changes for v6: >> - Separate out the documentation to new commit >> - Remove unnecessary header file inclusuib from the board-specific setup >> file >> - Make the function board_clock_init be declared, only when >> CONFIG_BOARD_EARLY_INIT_F is defined >> >> Changes for v7: >> - Fix several errata in the documentation >> - Remove OF_CONTROL dependency from !SPL_BUILD >> >> Changes for v8: >> - Remove unnecessary properties in DT mmc node >> >> Changes for v9: >> - Remove the first patch which change GPIO entries order. It became >> useless, >> after DT support is added >> - Remove useless variables in the default environment >> - Add the new contents to the documentation of Odroid X2/U2, instead of >> making new document for Odorid XU3 >> - Remove the detailed information in the header file, and leave the >> reference to the documentation >> >> Changes for v10: >> - Move the non common configs in Exynos5420 config file to each board file >> - Remove the config unsets which are added to make the result image small >> This is needless now, because the image is not small enough even though >> these >> unsets are added. >> - Remove redundant DT node and properties >> - Remove the odroid-xu3 board file and make odroid-xu3 a variant of >> smdk5420 >> - Fix some errata in documentation >> - Include the patch by Sjoerd Simons as 4th patch >> >> Hyungwon Hwang (3): >> config: exynos5420: move non common configs to specific board files >> Odroid-XU3: Add support for Odroid-XU3 >> Odroid-XU3: Add documentation for Odroid-XU3 >> >> Sjoerd Simons (1): >> Odroid-XU3: Add entry for DTS EHCI GPIO >> >> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ >> arch/arm/dts/Makefile | 3 +- >> arch/arm/dts/exynos5422-odroidxu3.dts | 49 >> +++++++++++++++++++++++++++++++ >> board/samsung/smdk5420/Kconfig | 13 +++++++++ >> configs/odroid-xu3_defconfig | 4 +++ >> doc/README.odroid | 46 ++++++++++++++++++----------- >> include/configs/exynos5420-common.h | 18 ------------ >> include/configs/odroid_xu3.h | 54 >> +++++++++++++++++++++++++++++++++++ >> include/configs/peach-pi.h | 6 ++++ >> include/configs/peach-pit.h | 6 ++++ >> include/configs/smdk5420.h | 11 +++++++ >> 11 files changed, 179 insertions(+), 35 deletions(-) >> create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts >> create mode 100644 configs/odroid-xu3_defconfig >> create mode 100644 include/configs/odroid_xu3.h >> > > I talked to man from the hardkernel, and he said that there is no > restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline > U-Boot. So I think, in this case it should be provided also with those > patches. Then we have a complete board setup. > > To Hyungwon, > There is a size limit for each patch (100KiB), so if you would like to add > such binary blob, you should add it using this site: > http://www.denx.de/wiki/attach/U-Boot/TooBigPatches > after a registration. If the intent is to upload a hexdump of a few of the files, that does sound like a convenient option. Very unfortunate though. At least for the trustzone file it is mostly zeroes, and seems to consist of 3 separate very small images. So perhaps we should have a tool that pieces them together? Regards, Simon
Hello, On 12/03/2014 05:28 PM, Simon Glass wrote: > Hi, > > On 3 December 2014 at 05:46, Przemyslaw Marczak <p.marczak@samsung.com> wrote: >> Hello all, >> >> >> On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: >>> >>> This is v10 of the patchset adding support Odroud XU3 board. >>> >>> link to the previous version: >>> v2: https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html >>> v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html >>> v4: https://patchwork.ozlabs.org/patch/407411/ >>> v5: https://patchwork.ozlabs.org/patch/407941/ >>> v6: https://patchwork.ozlabs.org/patch/408647/ >>> v7: https://patchwork.ozlabs.org/patch/410204/ >>> v8: https://patchwork.ozlabs.org/patch/410709/ >>> v9: https://patchwork.ozlabs.org/patch/415514/ >>> >>> The patchset by Akshay Saraswat is already merged into u-boot-samsung. >>> Please ignore the instruction about the additional patchset in v8. >>> >>> How to test this patch: >>> 1. git clone http://git.denx.de/u-boot-samsung.git >>> 2. Apply this patchset >>> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config >>> 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 >>> 5. Now you can use u-boot-dtb.bin for your downloading >>> >>> WARNING: (update in V10) >>> It was impossible to make the u-boot image smaller than 335872 bytes, and >>> also >>> I did not want to waste my time to make it fit for that restriction, >>> because >>> the discussion about this issue is in progress in malining list. I used >>> BL2 by >>> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is removed >>> for >>> testing. If you need the BL2 for testing, please contact me or Suriyan. >>> >>> Note: If you use micro SD card for your test you have to apply the below >>> patch additionally. This patch is needed, because micro sd card is >>> recognized as MMC1 instead of MMC0. Additional work is needed to make it >>> work regardless of device id. >>> >>> Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also >>> made >>> real patch for me. I squashed some part of that patchset, and included the >>> remainings into my patchset. >>> >>> diff --git a/include/configs/exynos5-common.h >>> b/include/configs/exynos5-common.h >>> index ba591e7..437eaae 100644 >>> --- a/include/configs/exynos5-common.h >>> +++ b/include/configs/exynos5-common.h >>> @@ -109,7 +109,7 @@ >>> >>> #define CONFIG_SYS_MONITOR_BASE 0x00000000 >>> >>> -#define CONFIG_SYS_MMC_ENV_DEV 0 >>> +#define CONFIG_SYS_MMC_ENV_DEV 1 >>> >>> #define CONFIG_SECURE_BL1_ONLY >>> >>> Changes for v2: >>> - Add a patch to add new common setup header file for Odroid X2/U3 >>> and Odroid XU3 >>> >>> Changes for v3: >>> - Remove the patch which adds new common setup header file from v2 >>> - Remove the wrong patch to fix GPIO information of Exynos 5800 >>> - Remove unnecessary node from DT file >>> - Remove unnecessary features from config file >>> - Fix some trivial typos in comments >>> >>> Changes for v4: >>> - Add MMC FIFO buffer's configuration to DT file >>> - Make CONFIG_OF_CONTROL be set by the target information >>> - Add basic document to doc/README.odroid-xu3 >>> - Add CONFIG_CMD_EXT4 to config file >>> - Add environment size and offset to config file >>> - Add extra default environment to make bootable without modification >>> - Remove unnecessary features from config file >>> >>> Changes for v5: >>> - Add the specific build instruction >>> - Update the information of patchset by Akshay Saraswat >>> - Convert /include/ to #include in DT file >>> >>> Changes for v6: >>> - Separate out the documentation to new commit >>> - Remove unnecessary header file inclusuib from the board-specific setup >>> file >>> - Make the function board_clock_init be declared, only when >>> CONFIG_BOARD_EARLY_INIT_F is defined >>> >>> Changes for v7: >>> - Fix several errata in the documentation >>> - Remove OF_CONTROL dependency from !SPL_BUILD >>> >>> Changes for v8: >>> - Remove unnecessary properties in DT mmc node >>> >>> Changes for v9: >>> - Remove the first patch which change GPIO entries order. It became >>> useless, >>> after DT support is added >>> - Remove useless variables in the default environment >>> - Add the new contents to the documentation of Odroid X2/U2, instead of >>> making new document for Odorid XU3 >>> - Remove the detailed information in the header file, and leave the >>> reference to the documentation >>> >>> Changes for v10: >>> - Move the non common configs in Exynos5420 config file to each board file >>> - Remove the config unsets which are added to make the result image small >>> This is needless now, because the image is not small enough even though >>> these >>> unsets are added. >>> - Remove redundant DT node and properties >>> - Remove the odroid-xu3 board file and make odroid-xu3 a variant of >>> smdk5420 >>> - Fix some errata in documentation >>> - Include the patch by Sjoerd Simons as 4th patch >>> >>> Hyungwon Hwang (3): >>> config: exynos5420: move non common configs to specific board files >>> Odroid-XU3: Add support for Odroid-XU3 >>> Odroid-XU3: Add documentation for Odroid-XU3 >>> >>> Sjoerd Simons (1): >>> Odroid-XU3: Add entry for DTS EHCI GPIO >>> >>> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ >>> arch/arm/dts/Makefile | 3 +- >>> arch/arm/dts/exynos5422-odroidxu3.dts | 49 >>> +++++++++++++++++++++++++++++++ >>> board/samsung/smdk5420/Kconfig | 13 +++++++++ >>> configs/odroid-xu3_defconfig | 4 +++ >>> doc/README.odroid | 46 ++++++++++++++++++----------- >>> include/configs/exynos5420-common.h | 18 ------------ >>> include/configs/odroid_xu3.h | 54 >>> +++++++++++++++++++++++++++++++++++ >>> include/configs/peach-pi.h | 6 ++++ >>> include/configs/peach-pit.h | 6 ++++ >>> include/configs/smdk5420.h | 11 +++++++ >>> 11 files changed, 179 insertions(+), 35 deletions(-) >>> create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts >>> create mode 100644 configs/odroid-xu3_defconfig >>> create mode 100644 include/configs/odroid_xu3.h >>> >> >> I talked to man from the hardkernel, and he said that there is no >> restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline >> U-Boot. So I think, in this case it should be provided also with those >> patches. Then we have a complete board setup. >> >> To Hyungwon, >> There is a size limit for each patch (100KiB), so if you would like to add >> such binary blob, you should add it using this site: >> http://www.denx.de/wiki/attach/U-Boot/TooBigPatches >> after a registration. > > If the intent is to upload a hexdump of a few of the files, that does > sound like a convenient option. Very unfortunate though. At least for > the trustzone file it is mostly zeroes, and seems to consist of 3 > separate very small images. So perhaps we should have a tool that > pieces them together? > > Regards, > Simon > Yes, this makes sense. And I think that we can modify the layout to have the order: BL1|BL2|TZSW|U-BOOT, then we can make one or two binnaries: signed blob and U-Boot. Best regards,
On 3 December 2014 at 09:08, Przemyslaw Marczak <p.marczak@samsung.com> wrote: > Hello, > > > On 12/03/2014 05:28 PM, Simon Glass wrote: >> >> Hi, >> >> On 3 December 2014 at 05:46, Przemyslaw Marczak <p.marczak@samsung.com> >> wrote: >>> >>> Hello all, >>> >>> >>> On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: >>>> >>>> >>>> This is v10 of the patchset adding support Odroud XU3 board. >>>> >>>> link to the previous version: >>>> v2: https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html >>>> v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html >>>> v4: https://patchwork.ozlabs.org/patch/407411/ >>>> v5: https://patchwork.ozlabs.org/patch/407941/ >>>> v6: https://patchwork.ozlabs.org/patch/408647/ >>>> v7: https://patchwork.ozlabs.org/patch/410204/ >>>> v8: https://patchwork.ozlabs.org/patch/410709/ >>>> v9: https://patchwork.ozlabs.org/patch/415514/ >>>> >>>> The patchset by Akshay Saraswat is already merged into u-boot-samsung. >>>> Please ignore the instruction about the additional patchset in v8. >>>> >>>> How to test this patch: >>>> 1. git clone http://git.denx.de/u-boot-samsung.git >>>> 2. Apply this patchset >>>> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config >>>> 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 >>>> 5. Now you can use u-boot-dtb.bin for your downloading >>>> >>>> WARNING: (update in V10) >>>> It was impossible to make the u-boot image smaller than 335872 bytes, >>>> and >>>> also >>>> I did not want to waste my time to make it fit for that restriction, >>>> because >>>> the discussion about this issue is in progress in malining list. I used >>>> BL2 by >>>> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is removed >>>> for >>>> testing. If you need the BL2 for testing, please contact me or Suriyan. >>>> >>>> Note: If you use micro SD card for your test you have to apply the below >>>> patch additionally. This patch is needed, because micro sd card is >>>> recognized as MMC1 instead of MMC0. Additional work is needed to make it >>>> work regardless of device id. >>>> >>>> Thanks for Sjoerd Simons. Not only did he comment my patchset, but he >>>> also >>>> made >>>> real patch for me. I squashed some part of that patchset, and included >>>> the >>>> remainings into my patchset. >>>> >>>> diff --git a/include/configs/exynos5-common.h >>>> b/include/configs/exynos5-common.h >>>> index ba591e7..437eaae 100644 >>>> --- a/include/configs/exynos5-common.h >>>> +++ b/include/configs/exynos5-common.h >>>> @@ -109,7 +109,7 @@ >>>> >>>> #define CONFIG_SYS_MONITOR_BASE 0x00000000 >>>> >>>> -#define CONFIG_SYS_MMC_ENV_DEV 0 >>>> +#define CONFIG_SYS_MMC_ENV_DEV 1 >>>> >>>> #define CONFIG_SECURE_BL1_ONLY >>>> >>>> Changes for v2: >>>> - Add a patch to add new common setup header file for Odroid X2/U3 >>>> and Odroid XU3 >>>> >>>> Changes for v3: >>>> - Remove the patch which adds new common setup header file from v2 >>>> - Remove the wrong patch to fix GPIO information of Exynos 5800 >>>> - Remove unnecessary node from DT file >>>> - Remove unnecessary features from config file >>>> - Fix some trivial typos in comments >>>> >>>> Changes for v4: >>>> - Add MMC FIFO buffer's configuration to DT file >>>> - Make CONFIG_OF_CONTROL be set by the target information >>>> - Add basic document to doc/README.odroid-xu3 >>>> - Add CONFIG_CMD_EXT4 to config file >>>> - Add environment size and offset to config file >>>> - Add extra default environment to make bootable without modification >>>> - Remove unnecessary features from config file >>>> >>>> Changes for v5: >>>> - Add the specific build instruction >>>> - Update the information of patchset by Akshay Saraswat >>>> - Convert /include/ to #include in DT file >>>> >>>> Changes for v6: >>>> - Separate out the documentation to new commit >>>> - Remove unnecessary header file inclusuib from the board-specific setup >>>> file >>>> - Make the function board_clock_init be declared, only when >>>> CONFIG_BOARD_EARLY_INIT_F is defined >>>> >>>> Changes for v7: >>>> - Fix several errata in the documentation >>>> - Remove OF_CONTROL dependency from !SPL_BUILD >>>> >>>> Changes for v8: >>>> - Remove unnecessary properties in DT mmc node >>>> >>>> Changes for v9: >>>> - Remove the first patch which change GPIO entries order. It became >>>> useless, >>>> after DT support is added >>>> - Remove useless variables in the default environment >>>> - Add the new contents to the documentation of Odroid X2/U2, instead of >>>> making new document for Odorid XU3 >>>> - Remove the detailed information in the header file, and leave the >>>> reference to the documentation >>>> >>>> Changes for v10: >>>> - Move the non common configs in Exynos5420 config file to each board >>>> file >>>> - Remove the config unsets which are added to make the result image >>>> small >>>> This is needless now, because the image is not small enough even >>>> though >>>> these >>>> unsets are added. >>>> - Remove redundant DT node and properties >>>> - Remove the odroid-xu3 board file and make odroid-xu3 a variant of >>>> smdk5420 >>>> - Fix some errata in documentation >>>> - Include the patch by Sjoerd Simons as 4th patch >>>> >>>> Hyungwon Hwang (3): >>>> config: exynos5420: move non common configs to specific board files >>>> Odroid-XU3: Add support for Odroid-XU3 >>>> Odroid-XU3: Add documentation for Odroid-XU3 >>>> >>>> Sjoerd Simons (1): >>>> Odroid-XU3: Add entry for DTS EHCI GPIO >>>> >>>> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ >>>> arch/arm/dts/Makefile | 3 +- >>>> arch/arm/dts/exynos5422-odroidxu3.dts | 49 >>>> +++++++++++++++++++++++++++++++ >>>> board/samsung/smdk5420/Kconfig | 13 +++++++++ >>>> configs/odroid-xu3_defconfig | 4 +++ >>>> doc/README.odroid | 46 >>>> ++++++++++++++++++----------- >>>> include/configs/exynos5420-common.h | 18 ------------ >>>> include/configs/odroid_xu3.h | 54 >>>> +++++++++++++++++++++++++++++++++++ >>>> include/configs/peach-pi.h | 6 ++++ >>>> include/configs/peach-pit.h | 6 ++++ >>>> include/configs/smdk5420.h | 11 +++++++ >>>> 11 files changed, 179 insertions(+), 35 deletions(-) >>>> create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts >>>> create mode 100644 configs/odroid-xu3_defconfig >>>> create mode 100644 include/configs/odroid_xu3.h >>>> >>> >>> I talked to man from the hardkernel, and he said that there is no >>> restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline >>> U-Boot. So I think, in this case it should be provided also with those >>> patches. Then we have a complete board setup. >>> >>> To Hyungwon, >>> There is a size limit for each patch (100KiB), so if you would like to >>> add >>> such binary blob, you should add it using this site: >>> http://www.denx.de/wiki/attach/U-Boot/TooBigPatches >>> after a registration. >> >> >> If the intent is to upload a hexdump of a few of the files, that does >> sound like a convenient option. Very unfortunate though. At least for >> the trustzone file it is mostly zeroes, and seems to consist of 3 >> separate very small images. So perhaps we should have a tool that >> pieces them together? >> >> Regards, >> Simon >> > Yes, this makes sense. > And I think that we can modify the layout to have the order: > BL1|BL2|TZSW|U-BOOT, Yes! That is much more sane. > then we can make one or two binnaries: signed blob and U-Boot. Regards, Simon
Hi All, In Tizen mainline u-boot, CONFIG_ENV_OFFSET have already been fixed to SZ_1K * 3136 from SZ_1K * 1280. It looks good to change CONFIG_ENV_OFFSET also in u-boot mainline to use the same offset with Tizen u-boot. How about your opinion? Best Regards, Inha Song. 2014년 12월 04일 01:15, Suriyan Ramasami 쓴 글: > Hello all, > > On Wed, Dec 3, 2014 at 4:46 AM, Przemyslaw Marczak > <p.marczak@samsung.com> wrote: >> Hello all, >> >> >> On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: >>> This is v10 of the patchset adding support Odroud XU3 board. >>> >>> link to the previous version: >>> v2: https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html >>> v3: https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html >>> v4: https://patchwork.ozlabs.org/patch/407411/ >>> v5: https://patchwork.ozlabs.org/patch/407941/ >>> v6: https://patchwork.ozlabs.org/patch/408647/ >>> v7: https://patchwork.ozlabs.org/patch/410204/ >>> v8: https://patchwork.ozlabs.org/patch/410709/ >>> v9: https://patchwork.ozlabs.org/patch/415514/ >>> >>> The patchset by Akshay Saraswat is already merged into u-boot-samsung. >>> Please ignore the instruction about the additional patchset in v8. >>> >>> How to test this patch: >>> 1. git clone http://git.denx.de/u-boot-samsung.git >>> 2. Apply this patchset >>> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- odroid-xu3_config >>> 4. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- -j8 >>> 5. Now you can use u-boot-dtb.bin for your downloading >>> >>> WARNING: (update in V10) >>> It was impossible to make the u-boot image smaller than 335872 bytes, and >>> also >>> I did not want to waste my time to make it fit for that restriction, >>> because >>> the discussion about this issue is in progress in malining list. I used >>> BL2 by >>> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is removed >>> for >>> testing. If you need the BL2 for testing, please contact me or Suriyan. >>> >>> Note: If you use micro SD card for your test you have to apply the below >>> patch additionally. This patch is needed, because micro sd card is >>> recognized as MMC1 instead of MMC0. Additional work is needed to make it >>> work regardless of device id. >>> >>> Thanks for Sjoerd Simons. Not only did he comment my patchset, but he also >>> made >>> real patch for me. I squashed some part of that patchset, and included the >>> remainings into my patchset. >>> >>> diff --git a/include/configs/exynos5-common.h >>> b/include/configs/exynos5-common.h >>> index ba591e7..437eaae 100644 >>> --- a/include/configs/exynos5-common.h >>> +++ b/include/configs/exynos5-common.h >>> @@ -109,7 +109,7 @@ >>> >>> #define CONFIG_SYS_MONITOR_BASE 0x00000000 >>> >>> -#define CONFIG_SYS_MMC_ENV_DEV 0 >>> +#define CONFIG_SYS_MMC_ENV_DEV 1 >>> >>> #define CONFIG_SECURE_BL1_ONLY >>> >>> Changes for v2: >>> - Add a patch to add new common setup header file for Odroid X2/U3 >>> and Odroid XU3 >>> >>> Changes for v3: >>> - Remove the patch which adds new common setup header file from v2 >>> - Remove the wrong patch to fix GPIO information of Exynos 5800 >>> - Remove unnecessary node from DT file >>> - Remove unnecessary features from config file >>> - Fix some trivial typos in comments >>> >>> Changes for v4: >>> - Add MMC FIFO buffer's configuration to DT file >>> - Make CONFIG_OF_CONTROL be set by the target information >>> - Add basic document to doc/README.odroid-xu3 >>> - Add CONFIG_CMD_EXT4 to config file >>> - Add environment size and offset to config file >>> - Add extra default environment to make bootable without modification >>> - Remove unnecessary features from config file >>> >>> Changes for v5: >>> - Add the specific build instruction >>> - Update the information of patchset by Akshay Saraswat >>> - Convert /include/ to #include in DT file >>> >>> Changes for v6: >>> - Separate out the documentation to new commit >>> - Remove unnecessary header file inclusuib from the board-specific setup >>> file >>> - Make the function board_clock_init be declared, only when >>> CONFIG_BOARD_EARLY_INIT_F is defined >>> >>> Changes for v7: >>> - Fix several errata in the documentation >>> - Remove OF_CONTROL dependency from !SPL_BUILD >>> >>> Changes for v8: >>> - Remove unnecessary properties in DT mmc node >>> >>> Changes for v9: >>> - Remove the first patch which change GPIO entries order. It became >>> useless, >>> after DT support is added >>> - Remove useless variables in the default environment >>> - Add the new contents to the documentation of Odroid X2/U2, instead of >>> making new document for Odorid XU3 >>> - Remove the detailed information in the header file, and leave the >>> reference to the documentation >>> >>> Changes for v10: >>> - Move the non common configs in Exynos5420 config file to each board file >>> - Remove the config unsets which are added to make the result image small >>> This is needless now, because the image is not small enough even though >>> these >>> unsets are added. >>> - Remove redundant DT node and properties >>> - Remove the odroid-xu3 board file and make odroid-xu3 a variant of >>> smdk5420 >>> - Fix some errata in documentation >>> - Include the patch by Sjoerd Simons as 4th patch >>> >>> Hyungwon Hwang (3): >>> config: exynos5420: move non common configs to specific board files >>> Odroid-XU3: Add support for Odroid-XU3 >>> Odroid-XU3: Add documentation for Odroid-XU3 >>> >>> Sjoerd Simons (1): >>> Odroid-XU3: Add entry for DTS EHCI GPIO >>> >>> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ >>> arch/arm/dts/Makefile | 3 +- >>> arch/arm/dts/exynos5422-odroidxu3.dts | 49 >>> +++++++++++++++++++++++++++++++ >>> board/samsung/smdk5420/Kconfig | 13 +++++++++ >>> configs/odroid-xu3_defconfig | 4 +++ >>> doc/README.odroid | 46 ++++++++++++++++++----------- >>> include/configs/exynos5420-common.h | 18 ------------ >>> include/configs/odroid_xu3.h | 54 >>> +++++++++++++++++++++++++++++++++++ >>> include/configs/peach-pi.h | 6 ++++ >>> include/configs/peach-pit.h | 6 ++++ >>> include/configs/smdk5420.h | 11 +++++++ >>> 11 files changed, 179 insertions(+), 35 deletions(-) >>> create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts >>> create mode 100644 configs/odroid-xu3_defconfig >>> create mode 100644 include/configs/odroid_xu3.h >>> >> I talked to man from the hardkernel, and he said that there is no >> restrictions about adding BL1/BL2/TZSW as a binary blob to the Mainline >> U-Boot. So I think, in this case it should be provided also with those >> patches. Then we have a complete board setup. >> >> To Hyungwon, >> There is a size limit for each patch (100KiB), so if you would like to add >> such binary blob, you should add it using this site: >> http://www.denx.de/wiki/attach/U-Boot/TooBigPatches >> after a registration. >> > I do have a signed BL2 (1MB U-Boot image - same as for the Odroid U3) > at http://forum.odroid.com/viewtopic.php?f=98&t=7388 > It has the full package - BL1/BL2/TZ and the sd_fusing script which > works for both SD as well as eMMC. > If we go with this BL2, then CONFIG_ENV_OFFSET (currently set at 1280 > * 1024) will clash with the TZ area in SD card. ( I am assuming its OK > for eMMC as the ENV resides in the user partition and not in boot0) > This offset is OK for the Odroid-U3 as the TZ is smaller (156 K). For > the Odroid U3: free area is after 2111 + (156 * 2) = 2423. The ENV > starts at 1280K = offset block 2560. > For the Odroid-XU3 the TZ is 256K. Free area after the blobs starts > from 2111 + (256 * 2) = 2623. > Hence, we will have to shift CONFIG_ENV_OFFSET to anything greater > than 2623 (512 byte) blocks. Anything with CONFIG_ENV_OFFSET (SZ_1K * > 1312) should work. > > Thanks and Regards > - Suriyan > >> Best regards >> -- >> Przemyslaw Marczak >> Samsung R&D Institute Poland >> Samsung Electronics >> p.marczak@samsung.com > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot >
Dear all, On Wed, 03 Dec 2014 09:31:44 -0800 Simon Glass <sjg@chromium.org> wrote: > On 3 December 2014 at 09:08, Przemyslaw Marczak > <p.marczak@samsung.com> wrote: > > Hello, > > > > > > On 12/03/2014 05:28 PM, Simon Glass wrote: > >> > >> Hi, > >> > >> On 3 December 2014 at 05:46, Przemyslaw Marczak > >> <p.marczak@samsung.com> wrote: > >>> > >>> Hello all, > >>> > >>> > >>> On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: > >>>> > >>>> > >>>> This is v10 of the patchset adding support Odroud XU3 board. > >>>> > >>>> link to the previous version: > >>>> v2: > >>>> https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html > >>>> v3: > >>>> https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html > >>>> v4: https://patchwork.ozlabs.org/patch/407411/ v5: > >>>> https://patchwork.ozlabs.org/patch/407941/ v6: > >>>> https://patchwork.ozlabs.org/patch/408647/ v7: > >>>> https://patchwork.ozlabs.org/patch/410204/ v8: > >>>> https://patchwork.ozlabs.org/patch/410709/ v9: > >>>> https://patchwork.ozlabs.org/patch/415514/ > >>>> > >>>> The patchset by Akshay Saraswat is already merged into > >>>> u-boot-samsung. Please ignore the instruction about the > >>>> additional patchset in v8. > >>>> > >>>> How to test this patch: > >>>> 1. git clone http://git.denx.de/u-boot-samsung.git > >>>> 2. Apply this patchset > >>>> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- > >>>> odroid-xu3_config 4. make ARCH=arm > >>>> CROSS_COMPILE=arm-linux-gnueabihf- -j8 5. Now you can use > >>>> u-boot-dtb.bin for your downloading > >>>> > >>>> WARNING: (update in V10) > >>>> It was impossible to make the u-boot image smaller than 335872 > >>>> bytes, and > >>>> also > >>>> I did not want to waste my time to make it fit for that > >>>> restriction, because > >>>> the discussion about this issue is in progress in malining list. > >>>> I used BL2 by > >>>> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is > >>>> removed for > >>>> testing. If you need the BL2 for testing, please contact me or > >>>> Suriyan. > >>>> > >>>> Note: If you use micro SD card for your test you have to apply > >>>> the below patch additionally. This patch is needed, because > >>>> micro sd card is recognized as MMC1 instead of MMC0. Additional > >>>> work is needed to make it work regardless of device id. > >>>> > >>>> Thanks for Sjoerd Simons. Not only did he comment my patchset, > >>>> but he also > >>>> made > >>>> real patch for me. I squashed some part of that patchset, and > >>>> included the > >>>> remainings into my patchset. > >>>> > >>>> diff --git a/include/configs/exynos5-common.h > >>>> b/include/configs/exynos5-common.h > >>>> index ba591e7..437eaae 100644 > >>>> --- a/include/configs/exynos5-common.h > >>>> +++ b/include/configs/exynos5-common.h > >>>> @@ -109,7 +109,7 @@ > >>>> > >>>> #define CONFIG_SYS_MONITOR_BASE 0x00000000 > >>>> > >>>> -#define CONFIG_SYS_MMC_ENV_DEV 0 > >>>> +#define CONFIG_SYS_MMC_ENV_DEV 1 > >>>> > >>>> #define CONFIG_SECURE_BL1_ONLY > >>>> > >>>> Changes for v2: > >>>> - Add a patch to add new common setup header file for Odroid > >>>> X2/U3 and Odroid XU3 > >>>> > >>>> Changes for v3: > >>>> - Remove the patch which adds new common setup header file from > >>>> v2 > >>>> - Remove the wrong patch to fix GPIO information of Exynos 5800 > >>>> - Remove unnecessary node from DT file > >>>> - Remove unnecessary features from config file > >>>> - Fix some trivial typos in comments > >>>> > >>>> Changes for v4: > >>>> - Add MMC FIFO buffer's configuration to DT file > >>>> - Make CONFIG_OF_CONTROL be set by the target information > >>>> - Add basic document to doc/README.odroid-xu3 > >>>> - Add CONFIG_CMD_EXT4 to config file > >>>> - Add environment size and offset to config file > >>>> - Add extra default environment to make bootable without > >>>> modification > >>>> - Remove unnecessary features from config file > >>>> > >>>> Changes for v5: > >>>> - Add the specific build instruction > >>>> - Update the information of patchset by Akshay Saraswat > >>>> - Convert /include/ to #include in DT file > >>>> > >>>> Changes for v6: > >>>> - Separate out the documentation to new commit > >>>> - Remove unnecessary header file inclusuib from the > >>>> board-specific setup file > >>>> - Make the function board_clock_init be declared, only when > >>>> CONFIG_BOARD_EARLY_INIT_F is defined > >>>> > >>>> Changes for v7: > >>>> - Fix several errata in the documentation > >>>> - Remove OF_CONTROL dependency from !SPL_BUILD > >>>> > >>>> Changes for v8: > >>>> - Remove unnecessary properties in DT mmc node > >>>> > >>>> Changes for v9: > >>>> - Remove the first patch which change GPIO entries order. It > >>>> became useless, > >>>> after DT support is added > >>>> - Remove useless variables in the default environment > >>>> - Add the new contents to the documentation of Odroid X2/U2, > >>>> instead of making new document for Odorid XU3 > >>>> - Remove the detailed information in the header file, and leave > >>>> the reference to the documentation > >>>> > >>>> Changes for v10: > >>>> - Move the non common configs in Exynos5420 config file to each > >>>> board file > >>>> - Remove the config unsets which are added to make the result > >>>> image small > >>>> This is needless now, because the image is not small enough > >>>> even though > >>>> these > >>>> unsets are added. > >>>> - Remove redundant DT node and properties > >>>> - Remove the odroid-xu3 board file and make odroid-xu3 a variant > >>>> of smdk5420 > >>>> - Fix some errata in documentation > >>>> - Include the patch by Sjoerd Simons as 4th patch > >>>> > >>>> Hyungwon Hwang (3): > >>>> config: exynos5420: move non common configs to specific > >>>> board files Odroid-XU3: Add support for Odroid-XU3 > >>>> Odroid-XU3: Add documentation for Odroid-XU3 > >>>> > >>>> Sjoerd Simons (1): > >>>> Odroid-XU3: Add entry for DTS EHCI GPIO > >>>> > >>>> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ > >>>> arch/arm/dts/Makefile | 3 +- > >>>> arch/arm/dts/exynos5422-odroidxu3.dts | 49 > >>>> +++++++++++++++++++++++++++++++ > >>>> board/samsung/smdk5420/Kconfig | 13 +++++++++ > >>>> configs/odroid-xu3_defconfig | 4 +++ > >>>> doc/README.odroid | 46 > >>>> ++++++++++++++++++----------- > >>>> include/configs/exynos5420-common.h | 18 ------------ > >>>> include/configs/odroid_xu3.h | 54 > >>>> +++++++++++++++++++++++++++++++++++ > >>>> include/configs/peach-pi.h | 6 ++++ > >>>> include/configs/peach-pit.h | 6 ++++ > >>>> include/configs/smdk5420.h | 11 +++++++ > >>>> 11 files changed, 179 insertions(+), 35 deletions(-) > >>>> create mode 100644 arch/arm/dts/exynos5422-odroidxu3.dts > >>>> create mode 100644 configs/odroid-xu3_defconfig > >>>> create mode 100644 include/configs/odroid_xu3.h > >>>> > >>> > >>> I talked to man from the hardkernel, and he said that there is no > >>> restrictions about adding BL1/BL2/TZSW as a binary blob to the > >>> Mainline U-Boot. So I think, in this case it should be provided > >>> also with those patches. Then we have a complete board setup. > >>> > >>> To Hyungwon, > >>> There is a size limit for each patch (100KiB), so if you would > >>> like to add > >>> such binary blob, you should add it using this site: > >>> http://www.denx.de/wiki/attach/U-Boot/TooBigPatches > >>> after a registration. > >> > >> > >> If the intent is to upload a hexdump of a few of the files, that > >> does sound like a convenient option. Very unfortunate though. At > >> least for the trustzone file it is mostly zeroes, and seems to > >> consist of 3 separate very small images. So perhaps we should have > >> a tool that pieces them together? > >> > >> Regards, > >> Simon > >> > > Yes, this makes sense. > > And I think that we can modify the layout to have the order: > > BL1|BL2|TZSW|U-BOOT, > > Yes! That is much more sane. > > > then we can make one or two binnaries: signed blob and U-Boot. > Hmm. It looks good and simple. But when using SD card for boot media, the developer must check the size of u-boot image whether the layout changes or not. As you know, the environment is saved in somewhere in the SD card (mostly fixed to some point of the front part of it). In this situation, is it worth to change the layout like that? > Regards, > Simon Best regards, Hyungwon Hwang
Dear all, On Thu, 04 Dec 2014 10:27:04 +0900 Inha Song <ideal.song@samsung.com> wrote: > Hi All, > > In Tizen mainline u-boot, CONFIG_ENV_OFFSET have already been fixed > to SZ_1K * 3136 from SZ_1K * 1280. > It looks good to change CONFIG_ENV_OFFSET also in u-boot mainline to > use the same offset with Tizen u-boot. > > How about your opinion? As I see the other boards' config files, there is no one principle to designate the environment offset, right? Then blk# 3136 seems reasonable for me. As Suriyan said, it must be bigger than blk# 2623. Can it be 256.5 Kbyte will be wasted, if blk# 3136 will be used for environment offset. Is it serious? If it is, I think blk# 2623 can be a good option for this. > > Best Regards, > Inha Song. > > 2014년 12월 04일 01:15, Suriyan Ramasami 쓴 글: > > Hello all, > > > > On Wed, Dec 3, 2014 at 4:46 AM, Przemyslaw Marczak > > <p.marczak@samsung.com> wrote: > >> Hello all, > >> > >> > >> On 12/02/2014 08:07 AM, Hyungwon Hwang wrote: > >>> This is v10 of the patchset adding support Odroud XU3 board. > >>> > >>> link to the previous version: > >>> v2: > >>> https://www.mail-archive.com/u-boot@lists.denx.de/msg152275.html > >>> v3: > >>> https://www.mail-archive.com/u-boot%40lists.denx.de/msg152677.html > >>> v4: https://patchwork.ozlabs.org/patch/407411/ v5: > >>> https://patchwork.ozlabs.org/patch/407941/ v6: > >>> https://patchwork.ozlabs.org/patch/408647/ v7: > >>> https://patchwork.ozlabs.org/patch/410204/ v8: > >>> https://patchwork.ozlabs.org/patch/410709/ v9: > >>> https://patchwork.ozlabs.org/patch/415514/ > >>> > >>> The patchset by Akshay Saraswat is already merged into > >>> u-boot-samsung. Please ignore the instruction about the > >>> additional patchset in v8. > >>> > >>> How to test this patch: > >>> 1. git clone http://git.denx.de/u-boot-samsung.git > >>> 2. Apply this patchset > >>> 3. make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- > >>> odroid-xu3_config 4. make ARCH=arm > >>> CROSS_COMPILE=arm-linux-gnueabihf- -j8 5. Now you can use > >>> u-boot-dtb.bin for your downloading > >>> > >>> WARNING: (update in V10) > >>> It was impossible to make the u-boot image smaller than 335872 > >>> bytes, and also > >>> I did not want to waste my time to make it fit for that > >>> restriction, because > >>> the discussion about this issue is in progress in malining list. > >>> I used BL2 by > >>> Suriyan Ramasami <suriyan.r@gmail.com> which the restriction is > >>> removed for > >>> testing. If you need the BL2 for testing, please contact me or > >>> Suriyan. > >>> > >>> Note: If you use micro SD card for your test you have to apply > >>> the below patch additionally. This patch is needed, because micro > >>> sd card is recognized as MMC1 instead of MMC0. Additional work is > >>> needed to make it work regardless of device id. > >>> > >>> Thanks for Sjoerd Simons. Not only did he comment my patchset, > >>> but he also made > >>> real patch for me. I squashed some part of that patchset, and > >>> included the remainings into my patchset. > >>> > >>> diff --git a/include/configs/exynos5-common.h > >>> b/include/configs/exynos5-common.h > >>> index ba591e7..437eaae 100644 > >>> --- a/include/configs/exynos5-common.h > >>> +++ b/include/configs/exynos5-common.h > >>> @@ -109,7 +109,7 @@ > >>> > >>> #define CONFIG_SYS_MONITOR_BASE 0x00000000 > >>> > >>> -#define CONFIG_SYS_MMC_ENV_DEV 0 > >>> +#define CONFIG_SYS_MMC_ENV_DEV 1 > >>> > >>> #define CONFIG_SECURE_BL1_ONLY > >>> > >>> Changes for v2: > >>> - Add a patch to add new common setup header file for Odroid X2/U3 > >>> and Odroid XU3 > >>> > >>> Changes for v3: > >>> - Remove the patch which adds new common setup header file from v2 > >>> - Remove the wrong patch to fix GPIO information of Exynos 5800 > >>> - Remove unnecessary node from DT file > >>> - Remove unnecessary features from config file > >>> - Fix some trivial typos in comments > >>> > >>> Changes for v4: > >>> - Add MMC FIFO buffer's configuration to DT file > >>> - Make CONFIG_OF_CONTROL be set by the target information > >>> - Add basic document to doc/README.odroid-xu3 > >>> - Add CONFIG_CMD_EXT4 to config file > >>> - Add environment size and offset to config file > >>> - Add extra default environment to make bootable without > >>> modification > >>> - Remove unnecessary features from config file > >>> > >>> Changes for v5: > >>> - Add the specific build instruction > >>> - Update the information of patchset by Akshay Saraswat > >>> - Convert /include/ to #include in DT file > >>> > >>> Changes for v6: > >>> - Separate out the documentation to new commit > >>> - Remove unnecessary header file inclusuib from the > >>> board-specific setup file > >>> - Make the function board_clock_init be declared, only when > >>> CONFIG_BOARD_EARLY_INIT_F is defined > >>> > >>> Changes for v7: > >>> - Fix several errata in the documentation > >>> - Remove OF_CONTROL dependency from !SPL_BUILD > >>> > >>> Changes for v8: > >>> - Remove unnecessary properties in DT mmc node > >>> > >>> Changes for v9: > >>> - Remove the first patch which change GPIO entries order. It > >>> became useless, > >>> after DT support is added > >>> - Remove useless variables in the default environment > >>> - Add the new contents to the documentation of Odroid X2/U2, > >>> instead of making new document for Odorid XU3 > >>> - Remove the detailed information in the header file, and leave > >>> the reference to the documentation > >>> > >>> Changes for v10: > >>> - Move the non common configs in Exynos5420 config file to each > >>> board file > >>> - Remove the config unsets which are added to make the result > >>> image small This is needless now, because the image is not small > >>> enough even though these > >>> unsets are added. > >>> - Remove redundant DT node and properties > >>> - Remove the odroid-xu3 board file and make odroid-xu3 a variant > >>> of smdk5420 > >>> - Fix some errata in documentation > >>> - Include the patch by Sjoerd Simons as 4th patch > >>> > >>> Hyungwon Hwang (3): > >>> config: exynos5420: move non common configs to specific board > >>> files Odroid-XU3: Add support for Odroid-XU3 > >>> Odroid-XU3: Add documentation for Odroid-XU3 > >>> > >>> Sjoerd Simons (1): > >>> Odroid-XU3: Add entry for DTS EHCI GPIO > >>> > >>> arch/arm/cpu/armv7/exynos/Kconfig | 4 +++ > >>> arch/arm/dts/Makefile | 3 +- > >>> arch/arm/dts/exynos5422-odroidxu3.dts | 49 > >>> +++++++++++++++++++++++++++++++ > >>> board/samsung/smdk5420/Kconfig | 13 +++++++++ > >>> configs/odroid-xu3_defconfig | 4 +++ > >>> doc/README.odroid | 46 > >>> ++++++++++++++++++----------- > >>> include/configs/exynos5420-common.h | 18 ------------ > >>> include/configs/odroid_xu3.h | 54 > >>> +++++++++++++++++++++++++++++++++++ > >>> include/configs/peach-pi.h | 6 ++++ > >>> include/configs/peach-pit.h | 6 ++++ > >>> include/configs/smdk5420.h | 11 +++++++ 11 files > >>> changed, 179 insertions(+), 35 deletions(-) create mode 100644 > >>> arch/arm/dts/exynos5422-odroidxu3.dts create mode 100644 > >>> configs/odroid-xu3_defconfig create mode 100644 > >>> include/configs/odroid_xu3.h > >>> > >> I talked to man from the hardkernel, and he said that there is no > >> restrictions about adding BL1/BL2/TZSW as a binary blob to the > >> Mainline U-Boot. So I think, in this case it should be provided > >> also with those patches. Then we have a complete board setup. > >> > >> To Hyungwon, > >> There is a size limit for each patch (100KiB), so if you would > >> like to add such binary blob, you should add it using this site: > >> http://www.denx.de/wiki/attach/U-Boot/TooBigPatches > >> after a registration. > >> > > I do have a signed BL2 (1MB U-Boot image - same as for the Odroid > > U3) at http://forum.odroid.com/viewtopic.php?f=98&t=7388 > > It has the full package - BL1/BL2/TZ and the sd_fusing script which > > works for both SD as well as eMMC. > > If we go with this BL2, then CONFIG_ENV_OFFSET (currently set at > > 1280 > > * 1024) will clash with the TZ area in SD card. ( I am assuming its > > OK for eMMC as the ENV resides in the user partition and not in > > boot0) This offset is OK for the Odroid-U3 as the TZ is smaller > > (156 K). For the Odroid U3: free area is after 2111 + (156 * 2) = > > 2423. The ENV starts at 1280K = offset block 2560. > > For the Odroid-XU3 the TZ is 256K. Free area after the blobs starts > > from 2111 + (256 * 2) = 2623. > > Hence, we will have to shift CONFIG_ENV_OFFSET to anything greater > > than 2623 (512 byte) blocks. Anything with CONFIG_ENV_OFFSET (SZ_1K > > * 1312) should work. > > > > Thanks and Regards > > - Suriyan > > > >> Best regards > >> -- > >> Przemyslaw Marczak > >> Samsung R&D Institute Poland > >> Samsung Electronics > >> p.marczak@samsung.com > > _______________________________________________ > > U-Boot mailing list > > U-Boot@lists.denx.de > > http://lists.denx.de/mailman/listinfo/u-boot > > > Best regards, Hyungwon Hwang
diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-common.h index ba591e7..437eaae 100644 --- a/include/configs/exynos5-common.h +++ b/include/configs/exynos5-common.h @@ -109,7 +109,7 @@ #define CONFIG_SYS_MONITOR_BASE 0x00000000 -#define CONFIG_SYS_MMC_ENV_DEV 0 +#define CONFIG_SYS_MMC_ENV_DEV 1 #define CONFIG_SECURE_BL1_ONLY