Dear all,

On Thu, 04 Dec 2014 10:27:04 +0900
Inha Song <ideal.s...@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.marc...@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 <suriya...@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.marc...@samsung.com
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot@lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
> >
> 

Best regards,
Hyungwon Hwang
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to