Re: [PATCH v4 00/18] am335x, guardian: update board specific changes

[Lokesh Vutla]

On Fri, 11 Jun 2021 16:13:32 +, gireesh.hirem...@in.bosch.com wrote:
> address the v3 review comments
> 
> >There are some build errors with this series, can you take a look?:
> >https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.denx.de%2Fu-boot%2Fcustodians%2Fu-boot-ti%2F-%2Fjobs%2F275470data=04%7C01%7CGireesh.Hiremath%40in.bosch.com%7Cba3f0660388b4b6a05c808d929a73cca%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0%7C0%7C637586620518929932%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=CM6KhlG0OOJM4iSpXNcqXZXAFI9IzguGEn2%2BaI1fCOA%3Dreserved=0
> >
> >Thanks and regards,
> >Lokesh
> 
> [...]
 
Applied to https://source.denx.de/u-boot/custodians/u-boot-ti.git for-rc, 
thanks!
[01/18] configs: am335x_guardian: Enable clock driver
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/480bb3a3ac
[02/18] am335x, guardian: mem: Add board dependent mem values
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/ebc75a558b
[03/18] configs: am335x_guardian: add ubi fastmap support
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/5e0f3c76ae
[04/18] am335x, guardian: set tftp_load_addr in environment
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/8b37a65af1
[05/18] configs: am335x_guardian: add memtest configs
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/29ddc64f34
[06/18] am335x, guardian: Update pinmux configuration
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/b44a5bdb2f
[07/18] am335x, guardian: set environment variable autoload to no
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/f15c87f911
[08/18] am335x, guardian: code cleanup and boot optimization
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/811ab6d8e4
[09/18] configs: am335x_guardian: set boot delay
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/509f99a012
[10/18] configs: am335x_guardian: disable spl command
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/18fef39c5c
[11/18] am335x, guardian: update swi logic
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/81dedc80ca
[12/18] am335x, guardian: Enable backlight
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/caa92aec8d
[13/18] configs: am335x_guardian: Enable display config
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/df06cac2b6
[14/18] drivers: video: hx8238 fix build bug
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/20a59f0b90
[15/18] am335x, guardian: Enable panel driver Himax HX8238D
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/890cd8cf10
[16/18] am335x, guardian: software update available status is stored in AM3352 
RTC scracth register
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/211000a99c
[17/18] configs: am335x_guardian: Enable bootcount nvmem support
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/4da5f9c3ba
[18/18] configs: am335x_guardian: add register maps support
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/b049ec5f6d
 
--
Thanks and Regards,
Lokesh

[PATCH v4 00/18] am335x, guardian: update board specific changes

[Gireesh.Hiremath]

From: Gireesh Hiremath 

address the v3 review comments

>There are some build errors with this series, can you take a look?:
>https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.denx.de%2Fu-boot%2Fcustodians%2Fu-boot-ti%2F-%2Fjobs%2F275470data=04%7C01%7CGireesh.Hiremath%40in.bosch.com%7Cba3f0660388b4b6a05c808d929a73cca%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0%7C0%7C637586620518929932%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=CM6KhlG0OOJM4iSpXNcqXZXAFI9IzguGEn2%2BaI1fCOA%3Dreserved=0
>
>Thanks and regards,
>Lokesh

Hi Lokesh

I have solved the build errors.

In ptch
v4-0005-configs-am335x_guardian-add-memtest-configs.patch
I moved defined memtest start and end address from guardian
header file to guardian config file

In patch
v4-0011-am335x-guardian-update-swi-logic.patch
added the delay header file

In patch 
v4-0016-am335x-guardian-software-update-available-status-.patch
switches add to select default Magic value for guardian and other
board in Kconfig file

I have tested u-boot on guardian target.
because of CI runner space limitations build done up 95% only
and untill that no error found.

Thanks and Regards,
Gireesh Hiremath

Gireesh Hiremath (15):
  configs: am335x_guardian: Enable clock driver
  configs: am335x_guardian: add ubi fastmap support
  configs: am335x_guardian: add memtest configs
  am335x, guardian: set environment variable autoload to no
  am335x, guardian: code cleanup and boot optimization
  configs: am335x_guardian: set boot delay
  configs: am335x_guardian: disable spl command
  am335x, guardian: update swi logic
  am335x, guardian: Enable backlight
  configs: am335x_guardian: Enable display config
  drivers: video: hx8238 fix build bug
  am335x, guardian: Enable panel driver Himax HX8238D
  am335x, guardian: software update available status is stored in AM3352
RTC scracth register
  configs: am335x_guardian: Enable bootcount nvmem support
  configs: am335x_guardian: add register maps support

Moses Christopher (3):
  am335x, guardian: mem: Add board dependent mem values
  am335x, guardian: set tftp_load_addr in environment
  am335x, guardian: Update pinmux configuration

 arch/arm/dts/am335x-guardian-u-boot.dtsi  |  11 ++
 arch/arm/dts/am335x-guardian.dts  |  14 +-
 .../include/asm/arch-am33xx/mem-guardian.h|  63 
 arch/arm/mach-omap2/am33xx/Kconfig|   2 +
 arch/arm/mach-omap2/am33xx/board.c|   4 +
 arch/arm/mach-omap2/mem-common.c  |   4 +
 board/bosch/guardian/board.c  | 152 +++---
 board/bosch/guardian/mux.c|   3 +-
 configs/am335x_guardian_defconfig |  30 +++-
 drivers/bootcount/Kconfig |  34 +++-
 drivers/bootcount/Makefile|   1 +
 drivers/bootcount/bootcount_nvmem.c   |  57 +++
 drivers/video/Makefile|   2 +-
 drivers/video/hx8238d.c   |   4 +-
 include/configs/am335x_guardian.h |  17 +-
 15 files changed, 356 insertions(+), 42 deletions(-)
 create mode 100644 arch/arm/include/asm/arch-am33xx/mem-guardian.h
 create mode 100644 drivers/bootcount/bootcount_nvmem.c

-- 
2.20.1