Linux-Rockchip Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Kever Yang <kever.yang@rock-chips.com>
To: Jagan Teki <jagan@amarulasolutions.com>,
	Simon Glass <sjg@chromium.org>,
	Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
Cc: u-boot@lists.denx.de,
	linux-amarula <linux-amarula@amarulasolutions.com>,
	linux-rockchip@lists.infradead.org
Subject: Re: [PATCH 06/11] env: kconfig: Restrict rockchip env for MMC
Date: Mon, 30 Dec 2019 11:05:49 +0800	[thread overview]
Message-ID: <d38ac96c-d4de-8acd-9070-a0847c721bd4@rock-chips.com> (raw)
In-Reply-To: <20191221075440.6944-7-jagan@amarulasolutions.com>


On 2019/12/21 下午3:54, Jagan Teki wrote:
> Rockchip do support SPI flash as well, so there is
> a possibility of using flash environment for those
> use cases.
>
> So, restrict the current env offset, size for MMC.
>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Reviewed-by: Kever Yang <kever.yang@rock-chips.com>

Thanks,
- Kever
> ---
>   env/Kconfig | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/env/Kconfig b/env/Kconfig
> index ed12609f6a..9416a70022 100644
> --- a/env/Kconfig
> +++ b/env/Kconfig
> @@ -488,7 +488,7 @@ config ENV_OFFSET
>   	hex "Environment offset"
>   	depends on ENV_IS_IN_EEPROM || ENV_IS_IN_MMC || ENV_IS_IN_NAND || \
>   		    ENV_IS_IN_SPI_FLASH
> -	default 0x3f8000 if ARCH_ROCKCHIP
> +	default 0x3f8000 if ARCH_ROCKCHIP && ENV_IS_IN_MMC
>   	default 0x88000 if ARCH_SUNXI
>   	default 0xE0000 if ARCH_ZYNQ
>   	default 0x1E00000 if ARCH_ZYNQMP
> @@ -511,7 +511,8 @@ config ENV_SIZE
>   	hex "Environment Size"
>   	default 0x40000 if ENV_IS_IN_SPI_FLASH && ARCH_ZYNQMP
>   	default 0x20000 if ARCH_SUNXI || ARCH_ZYNQ || ARCH_OMAP2PLUS || ARCH_AT91
> -	default 0x8000 if ARCH_ROCKCHIP || ARCH_ZYNQMP || ARCH_VERSAL
> +	default 0x8000 if ARCH_ROCKCHIP && ENV_IS_IN_MMC
> +	default 0x8000 if ARCH_ZYNQMP || ARCH_VERSAL
>   	default 0x4000 if ARC
>   	default 0x1f000
>   	help

  reply	other threads:[~2019-12-30  3:05 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-21  7:54 [PATCH 00/11] rk3399: SPI boot support (fixes, updates) Jagan Teki
     [not found] ` <20191221075440.6944-1-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
2019-12-21  7:54   ` [PATCH 01/11] spi: rk: Limit transfers to (64K - 1) bytes Jagan Teki
     [not found]     ` <20191221075440.6944-2-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
2019-12-23  2:30       ` Kever Yang
2019-12-27  5:55         ` Jagan Teki
2019-12-21  7:54   ` [PATCH 02/11] distro_bootcmd: Add SF support Jagan Teki
2019-12-30  3:07     ` Kever Yang
2020-01-20 17:22     ` Tom Rini
2020-01-20 17:40       ` Alexander Graf
2020-01-23 16:55         ` Jagan Teki
2020-01-23 17:03           ` Tom Rini
2020-01-23 17:11             ` Jagan Teki
2020-01-23 17:15               ` Tom Rini
2020-01-23 17:29                 ` Jagan Teki
2020-01-23 17:59                   ` Tom Rini
2019-12-21  7:54   ` [PATCH 03/11] rockchip: Include SF on distrocmd devices Jagan Teki
     [not found]     ` <20191221075440.6944-4-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
2019-12-30  3:07       ` Kever Yang
2020-01-07  8:44     ` Kever Yang
2019-12-21  7:54   ` [PATCH 04/11] rk3399: Add boot flash script offet, size Jagan Teki
2019-12-30  3:06     ` Kever Yang
2019-12-21  7:54   ` [PATCH 05/11] rk3399: Check MMC env while defining it Jagan Teki
2019-12-30  3:06     ` Kever Yang
2019-12-21  7:54   ` [PATCH 06/11] env: kconfig: Restrict rockchip env for MMC Jagan Teki
2019-12-30  3:05     ` Kever Yang [this message]
2019-12-21  7:54   ` [PATCH 07/11] env: Enable SPI flash env for rockchip Jagan Teki
2019-12-23  2:34     ` Kever Yang
2019-12-27  6:50       ` Jagan Teki
2019-12-27 10:02         ` Kever Yang
2019-12-27 10:10           ` Jagan Teki
2019-12-27 10:30         ` Soeren Moch
2019-12-27 12:04           ` Jagan Teki
2019-12-27 13:19             ` Soeren Moch
     [not found]               ` <30fdf54b-0f71-ffbf-ac57-51050a29b56d-S0/GAf8tV78@public.gmane.org>
2019-12-28 12:08                 ` Jagan Teki
2019-12-30  9:04                   ` Soeren Moch
2019-12-30  2:59     ` Kever Yang
2019-12-21  7:54   ` [PATCH 08/11] rockchip: dts: Sync ROC-RK3399-PC changes from Linux Jagan Teki
     [not found]     ` <20191221075440.6944-9-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
2019-12-30  3:00       ` Kever Yang
2019-12-21  7:54   ` [PATCH 09/11] roc-pc-rk3399: Enable SPI Flash Jagan Teki
     [not found]     ` <20191221075440.6944-10-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
2019-12-30  3:00       ` Kever Yang
2019-12-21  7:54   ` [PATCH 10/11] rockpro-rk3399: " Jagan Teki
2019-12-30  3:00     ` Kever Yang
     [not found]     ` <20191221075440.6944-11-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org>
2019-12-30 11:51       ` Emmanuel Vadot
2020-01-08 12:59         ` Jagan Teki
2019-12-21  7:54   ` [PATCH 11/11] roc-rk3399-pc: Add SPI boot support Jagan Teki
2019-12-30  3:05     ` Kever Yang
2019-12-30  7:52       ` Jagan Teki
     [not found]         ` <CAMty3ZA1u3-21QN4N6W=9Sq3n493BBBB2W_FzAEQC9aze4SoMg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2020-01-07  8:53           ` Kever Yang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d38ac96c-d4de-8acd-9070-a0847c721bd4@rock-chips.com \
    --to=kever.yang@rock-chips.com \
    --cc=jagan@amarulasolutions.com \
    --cc=linux-amarula@amarulasolutions.com \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=philipp.tomsich@theobroma-systems.com \
    --cc=sjg@chromium.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox