On Mon, Jun 26, 2017 at 6:55 AM, Ram Chandra Jangir <rjan...@codeaurora.org> wrote: > IPQ806x AP148 and DB149 boards didn't have the UCI ubootenv > section initialized, so the usage of fw_printenv required manual > configuration. With this change, the "fw_printenv" and "fw_setenv" > command will automatically work on NOR and NAND based platforms. > > Signed-off-by: Ram Chandra Jangir <rjan...@codeaurora.org> > --- > package/boot/uboot-envtools/files/ipq | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/package/boot/uboot-envtools/files/ipq > b/package/boot/uboot-envtools/files/ipq > index 16c7ba6..01ae220 100644 > --- a/package/boot/uboot-envtools/files/ipq > +++ b/package/boot/uboot-envtools/files/ipq > @@ -13,7 +13,29 @@ touch /etc/config/ubootenv > > board=$(ipq806x_board_name) > > +default_uboot_env () {
Is there a better name for this? It's really just for boards that can boot NOR or NAND to decide where the env is? > + UBOOTENV_PART=$(cat /proc/mtd | grep APPSBLENV) > + mtd_dev=$(echo $UBOOTENV_PART | awk '{print $1}' | sed 's/:$//') > + mtd_size=0x$(echo $UBOOTENV_PART | awk '{print $2}') > + mtd_erase=0x$(echo $UBOOTENV_PART | awk '{print $3}') > + nor_flash=`find /sys/bus/spi/devices/*/mtd -name ${mtd_dev}` Can you stick with one syntax here? $(...) > + > + if [ -n "$nor_flash" ]; then > + uboot_env_size=$mtd_size > + else > + # size is fixed to 0x40000 in u-boot > + uboot_env_size=0x40000 > + fi > + > + sectors=$(( $uboot_env_size / $mtd_erase )) > + echo /dev/$mtd_dev 0x0 $uboot_env_size $mtd_erase $sectors > +} > + > case "$board" in > +ap148 | db149) > + env=$(default_uboot_env) > + [ -n "${env}" ] && ubootenv_add_uci_config $env When is it ever not defined? > + ;; > ea8500) > ubootenv_add_uci_config "/dev/mtd10" "0x0" "0x20000" "0x20000" > ;; > -- > 2.7.2 > > > _______________________________________________ > Lede-dev mailing list > Lede-dev@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/lede-dev _______________________________________________ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev