nmhw21: sync ui leds with onboard leds.

BugzID: 59055

Signed-off-by: Lucien Mueller <lucien.mueller@netmodule.com>

arch/arm/Kconfig

@@ -414,14 +414,6 @@ config TARGET_AM335X_NRHW24
 	select DM_SERIAL
 	select DM_GPIO
 
-config TARGET_AM335X_HW25
-	bool "Support am335x_hw25"
-	select CPU_V7
-	select SUPPORT_SPL
-	select DM
-	select DM_SERIAL
-	select DM_GPIO
-
 config TARGET_AM335X_SL50
 	bool "Support am335x_sl50"
 	select CPU_V7
@@ -923,7 +915,6 @@ source "board/nm/netbird_v2/Kconfig"
 source "board/nm/nrhw20/Kconfig"
 source "board/nm/nmhw21/Kconfig"
 source "board/nm/nrhw24/Kconfig"
-source "board/nm/hw25/Kconfig"
 source "board/olimex/mx23_olinuxino/Kconfig"
 source "board/phytec/pcm051/Kconfig"
 source "board/phytec/pcm052/Kconfig"

board/nm/common/bdparser.h

@@ -30,8 +30,6 @@
  *                         Fixed wrong sizeof type in GetPartition()
  *                         Changed 64 bit type to "long long" from struct
  *                         Added BD_VerifySha1Hmac() function
- *  20200615  rs           Added BD_Hw_Type tag
- * 
  *****************************************************************************/
 
 /**
@@ -162,8 +160,7 @@ typedef enum _BD_Tags
 
   BD_BOM_Patch          =    33,   /**<  "UInt8"      -> Hardware BOM patch of the equipment (BOM changes, same PCB, not SW relevant) */
   BD_Prod_Variant_Name  =    34,   /**<  "String"     -> Product variant */
-  BD_Hw_Type            =    35,   /*<<  "UInt16"     -> Hardware Type, e.g. 24 for HW24, 26 for HW26 */
-  
+
   BD_Ui_Adapter_Type    =  4096,   /**<  "UInt16"     -> IV OG2 UI adapterboard type (0 = not present) */
   BD_Voltage            =  4098,   /**<  "UInt8"      -> Primary Voltage (1=12-60V, 2=40-160V, 3=24-60V, 4=?, 5=12-24V) */
   
@@ -184,10 +181,10 @@ typedef enum _BD_Tags
   BD_Patch              =  4126,   /**<  "UInt8"      -> Board patch level (after production) */
   BD_Pd_Phy2            =  4127,   /**<  "String"     -> */
   PD_SerDes             =  4128,   /**<  "UInt16"     -> SERDES Configuration (e.g. NB1800) */
-  PD_Shield             =  4129,   /**<  "UInt16"     -> Assembled Shield (0=COM/IO, 1=DualCAN), 2=CAN/GNSS, 3=DualCAN Passive */
+  PD_Shield             =  4129,   /**<  "UInt16"     -> Assembled Shield (0=COM/IO, 1=DualCAN) */
 
   /* project specific tags */
-  BD_BootPart           = 32768,   /**<  "UInt8" */
+  BD_BootPart			= 32768,   /**<  "UInt8" */
 
   BD_None_Type          = 65535,   /**<  "Void"       -> None */
 }

board/nm/common/board_descriptor.c

@@ -17,7 +17,6 @@
  *                         code cleanup (tabs/indentation)
  *  20110211  rs           partition table handling
  *  20190330  rs           cleanup after years of chaotic development
- *  20200615  rs           added bd_get_hw_type()
  *
  *****************************************************************************/
 #include <common.h>
@@ -218,16 +217,6 @@ int bd_get_variantname(char *variantname, size_t len)
 	return 0;
 }
 
-void bd_get_hw_type(int* type)
-{
-	uint16_t hwtype = 0;
-
-	if ( !_get_uint16( BD_Hw_Type, 0, &hwtype) )
-		debug("%s() no Hw Type found\n", __func__);
-
-	*type = hwtype;
-}
-
 void bd_get_hw_version(int* ver, int* rev)
 {
 	uint8_t hwver = 0;
@@ -279,26 +268,6 @@ uint32_t bd_get_fpgainfo(void)
 	return fpgainfo;
 }
 
-int bd_get_pd_dio(char *config, size_t len)
-{
-	if ( !_get_string(BD_Pd_DIO, 0, config, len) ) {
-		debug("%s() no DIO info\n", __func__);
-		return -1;
-	}
-
-	return 0;
-}
-
-int bd_get_pd_serial(char *config, size_t len)
-{
-	if ( !_get_string(BD_Pd_Serial, 0, config, len) ) {
-		debug("%s() no serial port info\n", __func__);
-		return -1;
-	}
-
-	return 0;
-}
-
 int bd_get_pd_module(uint32_t slot, char *config, size_t len)
 {
 	if ( !_get_string(BD_Pd_Module0 + slot, 0, config, len) ) {

board/nm/common/board_descriptor.h

@@ -17,13 +17,10 @@ int bd_get_context(BD_Context *bdctx, uint32_t i2caddress, uint32_t offset);
 
 int bd_get_prodname(char *prodname, size_t len);
 int bd_get_variantname(char *variantname, size_t len);
-void bd_get_hw_type(int* type);
 void bd_get_hw_version(int* ver, int* rev);
 void bd_get_hw_patch(int* patch);
 int bd_get_mac(int index, uint8_t *macaddr, size_t len);
 uint32_t bd_get_fpgainfo(void);
-int bd_get_pd_dio(char *config, size_t len);
-int bd_get_pd_serial(char *config, size_t len);
 int bd_get_pd_module(uint32_t slot, char *config, size_t len);
 int bd_get_sim_config(char* simconfig, size_t len);
 int bd_get_devicetree(char* devicetreename, size_t len);

board/nm/common/da9063.h

@@ -11,34 +11,20 @@
 #ifndef DA9063_H
 #define DA9063_H
 
+/*
+TODO: Add include for uint32_t (stdtypes.h)
+#include <types.h>
+*/
 
 #define CONFIG_PMIC_I2C_BUS	0
 #define CONFIG_PMIC_I2C_ADDR	0x58	/* Pages 0 and 1, Pages 2 and 3 -> 0x59 */
 
 #define PMIC_REG_STATUS_A		0x01	/* Status of ON_KEY, WAKE, COMP1V2, DVC */
-#define PMIC_REG_STATUS_A_COMP1V2_MASK	0x08
-
 #define PMIC_REG_FAULT_LOG		0x05	/* PMIC fault log register, holding reset reason */
 #define PMIC_FAULT_TWD_ERROR_MASK	0x01	/* Watchdog timeout detected */
-#define PMIC_FAULT_POR_MASK		0x02	/* Startup from No-Power/RTC/Delivery mode */
 
-#define PMIC_REG_EVENT_A		0x06
-#define PMIC_REG_EVENT_ONKEY_MASK	0x01
-#define PMIC_REG_EVENT_RTC_ALARM_MASK	0x02
-#define PMIC_REG_EVENT_RTC_TICK_MASK	0x04
-#define PMIC_REG_EVENT_EVENTS_B_MASK	0x20
-
-#define PMIC_REG_EVENT_B		0x07
-#define PMIC_REG_EVENT_COMP1V2_MASK	0x04
-
-#define PMIC_REG_IRQ_MASK_A	0x0A
-#define PMIC_REG_IRQ_MASK_B	0x0B
-#define PMIC_REG_IRQ_MASK_C	0x0C
-#define PMIC_REG_IRQ_MASK_D	0x0D
-
-#define PMIC_REG_CONTROL_A      0x0E    /* Control register for power states */
-#define PMIC_REG_CONTROL_D	0x11    /* Control register for blink/watchdog */
-#define PMIC_REG_GPIO14_15	0x1C    /* Configuration of GPIO14/15 (mode, wake) */
+#define PMIC_REG_CONTROL_D      0x11    /* Control register for blink/watchdog */
+#define PMIC_REG_GPIO14_15      0x1C    /* Configuration of GPIO14/15 (mode, wake) */
 #define PMIC_REG_GPIO_MODE0_7	0x1D	/* Control register for GPIOs 0..7 */
 #define PMIC_REG_GPIO_MODE8_15	0x1E	/* Control register for GPIOs 8..15 */
 
@@ -57,7 +43,6 @@
 #define PMIC_LDOx_CONF_MASK	0x80
 
 #define PMIC_REG_ID_4_3		0x84
-#define PMIC_REG_ID_6_5		0x85
 
 #define PMIC_REG_BUCK_ILIM_A	0x9A
 #define PMIC_REG_BUCK_ILIM_B	0x9B
@@ -83,9 +68,6 @@
 
 #define PMIC_REG_TRIM_CLDR	0x120	/* Calendar Trim register, 2's complement, 1.9ppm per bit */
 
-#define PMIC_GP_ID_0		0x121	/* General purpose ID 0 (R/W) */
-#define PMIC_GP_ID_1		0x122	/* General purpose ID 1 (R/W) */
-
 #define PMIC_REG_CONFIG_ID	0x184	/* OTP Config ID <ver.rev> */
 
 

board/nm/common/ether_crc.c

@@ -1,36 +0,0 @@
-/*
- * ether_crc.c
- *
- * Ethernet CRC computation
- *
- * Copyright (C) 2018-2020 NetModule AG - http://www.netmodule.com/
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-#include <common.h>
-
-#include "ether_crc.h"
-
-
-uint32_t ether_crc(size_t len, uint8_t const *p)
-{
-	uint32_t crc;
-	unsigned i;
-
-	crc = ~0;
-	while (len--) {
-		crc ^= *p++;
-		for (i = 0; i < 8; i++)
-			crc = (crc >> 1) ^ ((crc & 1) ? 0xedb88320 : 0);
-	}
-
-	/* an reverse the bits, cuz of way they arrive -- last-first */
-	crc = (crc >> 16) | (crc << 16);
-	crc = (crc >> 8 & 0x00ff00ff) | (crc << 8 & 0xff00ff00);
-	crc = (crc >> 4 & 0x0f0f0f0f) | (crc << 4 & 0xf0f0f0f0);
-	crc = (crc >> 2 & 0x33333333) | (crc << 2 & 0xcccccccc);
-	crc = (crc >> 1 & 0x55555555) | (crc << 1 & 0xaaaaaaaa);
-	
-	return crc;
-}

board/nm/common/ether_crc.h

@@ -1,18 +0,0 @@
-/*
- * ether_crc.h
- *
- * Ethernet CRC computation
- *
- * Copyright (C) 2018-2020 NetModule AG - http://www.netmodule.com/
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-#ifndef ETHER_CRC_H
-#define ETHER_CRC_H
-
-
-extern uint32_t ether_crc(size_t len, uint8_t const *p);
-
-
-#endif /* ETHER_CRC_H */

board/nm/hw25/Kconfig

@@ -1,26 +0,0 @@
-if TARGET_AM335X_HW25
-
-config SYS_BOARD
-        default "hw25"
-
-config SYS_VENDOR
-        default "nm"
-
-config SYS_SOC
-        default "am33xx"
-
-config SYS_CONFIG_NAME
-        default "am335x_hw25"
-
-config CONS_INDEX
-        int "UART used for console"
-        range 1 6
-        default 1
-        help
-          The AM335x SoC has a total of 6 UARTs (UART0 to UART5 as referenced
-          in documentation, etc) available to it.  Depending on your specific
-          board you may want something other than UART0 as for example the IDK
-          uses UART3 so enter 4 here.
-
-endif
-

board/nm/hw25/Makefile

@@ -1,13 +0,0 @@
-#
-# Makefile
-#
-# Copyright (C) 2011 Texas Instruments Incorporated - http://www.ti.com/
-#
-# SPDX-License-Identifier:	GPL-2.0+
-#
-
-ifeq ($(CONFIG_SKIP_LOWLEVEL_INIT),)
-obj-y	:= mux.o
-endif
-
-obj-y	+= board.o ../common/bdparser.o ../common/board_descriptor.o ../common/da9063.o ../common/ether_crc.o fileaccess.o

board/nm/hw25/board.h

@@ -1,26 +0,0 @@
-/*
- * board.h
- *
- * TI AM335x boards information header
- *
- * Copyright (C) 2011, Texas Instruments, Incorporated - http://www.ti.com/
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-#ifndef _BOARD_H_
-#define _BOARD_H_
-
-/*
- * We have three pin mux functions that must exist.  We must be able to enable
- * uart0, for initial output and i2c2 to read the main EEPROM.  We then have a
- * main pinmux function that can be overridden to enable all other pinmux that
- * is required on the board.
- */
-void enable_uart0_pin_mux(void);
-
-void enable_board_pin_mux(void);
-
-#define GPIO_TO_PIN(bank, gpio)		(32 * (bank) + (gpio))
-
-#endif

board/nm/hw25/fileaccess.c

@@ -1,40 +0,0 @@
-#include <common.h>
-#include <fs.h>
-
-#define BLOCK_DEVICE "mmc"
-#define OVERLAY_PART "1:3"
-
-int read_file(const char* filename, char *buf, int size)
-{
-	loff_t filesize = 0;
-	loff_t len;
-	int ret;
-
-	if (fs_set_blk_dev(BLOCK_DEVICE, OVERLAY_PART, FS_TYPE_EXT) != 0) {
-		puts("Error, can not set blk device\n");
-		return -1;
-	}
-
-	/* Read at most file size bytes */
-	if (fs_size(filename, &filesize)) {
-		return -1;
-	}
-
-	if (filesize < size)
-		size = filesize;
-
-	/* For very unclear reasons the block device needs to be set again after the call to fs_size() */
-	if (fs_set_blk_dev(BLOCK_DEVICE, OVERLAY_PART, FS_TYPE_EXT) != 0) {
-		puts("Error, can not set blk device\n");
-		return -1;
-	}
-
-	if ((ret = fs_read(filename, (ulong)buf, 0, size, &len))) {
-		printf("Can't read file %s (size %d, len %lld, ret %d)\n", filename, size, len, ret);
-		return -1;
-	}
-
-	buf[len] = 0;
-
-	return len;
-}

board/nm/hw25/fileaccess.h

@@ -1,14 +0,0 @@
-/**@file	/home/eichenberger/projects/nbhw16/u-boot/board/nm/netbird_v2/fileaccess.h
- * @author	eichenberger
- * @version	704
- * @date
- * 	Created:	Tue 06 Jun 2017 02:02:33 PM CEST \n
- * 	Last Update:	Tue 06 Jun 2017 02:02:33 PM CEST
- */
-#ifndef FILEACCESS_H
-#define FILEACCESS_H
-
-void fs_set_console(void);
-int read_file(const char* filename, char *buf, int size);
-
-#endif // FILEACCESS_H

board/nm/hw25/mux.c

@@ -1,223 +0,0 @@
-/*
- * mux.c
- *
- * Copyright (C) 2018-2019 NetModule AG - http://www.netmodule.com/
- * Copyright (C) 2011 Texas Instruments Incorporated - http://www.ti.com/
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation version 2.
- *
- * This program is distributed "as is" WITHOUT ANY WARRANTY of any
- * kind, whether express or implied; without even the implied warranty
- * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- */
-
-#include <common.h>
-#include <asm/arch/sys_proto.h>
-#include <asm/arch/hardware.h>
-#include <asm/arch/mux.h>
-#include <asm/io.h>
-#include "board.h"
-
-static struct module_pin_mux gpio_pin_mux[] = {
-	/*
-	* CPU GPIOs
-	*
-	* (J18) GPIO0_16: RST_PHY~
-	* (U10) GPIO0_22: SEL_RS232/RS485~
-	* (T10) GPIO0_23: EN_RS485_TERM~
-	* (T11) GPIO0_26: IO_OUT1
-	* (U12) GPIO0_27: IO_OUT2
-	*
-	* (T12) GPIO1_12: IO_IN0
-	* (T13) GPIO1_13: IO_IN1
-	* (T14) GPIO1_14: IO_IN2
-	* (T15) GPIO1_15: IO_IN3
-	*
-	* (T13) GPIO2_0: RST_SDCARD~
-	* (L17) GPIO2_18: GSM_PWR_EN
-	* (L16) GPIO2_19: RST_GSM
-	*
-	* (K18) GPIO3_9: WLAN_IRQ
-	* (L18) GPIO3_10: WLAN_EN
-	* (C12) GPIO3_17: SIM_SEL
-	*/
-
-	/* Bank 0 */
-	{OFFSET(mii1_txd3), (MODE(7) | PULLUDDIS)},				/* (J18) gpio0[16] */  /* RST_PHY~ */
-	{OFFSET(gpmc_ad8), (MODE(7) | PULLUDDIS)},				/* (U10) gpio0[22] */  /* SEL_RS232/RS485~ */
-	{OFFSET(gpmc_ad9), (MODE(7) | PULLUDDIS)},				/* (T10) gpio0[23] */  /* EN_RS485_TERM~ */
-	{OFFSET(gpmc_ad10), (MODE(7) | PULLUDDIS)},				/* (T11) gpio0[26] */  /* IO_OUT1 */
-	{OFFSET(gpmc_ad11), (MODE(7) | PULLUDDIS)},				/* (U12) gpio0[27] */  /* IO_OUT2 */
-
-	/* Bank 1 */
-	{OFFSET(gpmc_ad12), (MODE(7) | PULLUDDIS | RXACTIVE)},			/* (T12) gpio1[12] */  /* IO_IN0 */
-	{OFFSET(gpmc_ad13), (MODE(7) | PULLUDDIS | RXACTIVE)},			/* (R12) gpio1[13] */  /* IO_IN1 */
-	{OFFSET(gpmc_ad14), (MODE(7) | PULLUDDIS | RXACTIVE)},			/* (V13) gpio1[14] */  /* IO_IN2 */
-	{OFFSET(gpmc_ad15), (MODE(7) | PULLUDDIS | RXACTIVE)},			/* (U13) gpio1[15] */  /* IO_IN3 */
-
-	/* TODO: What about all the unused GPMC pins ? */
-
-	/* Bank 2 */
-	{OFFSET(gpmc_be1n), (MODE(7) | PULLUDDIS)},				/* (T13) gpio2[0] */  /* RST_SDCARD~ */
-	{OFFSET(mii1_rxd3), (MODE(7) | PULLUDDIS)},				/* (L17) gpio2[18] */  /* GSM_PWR_EN */
-	{OFFSET(mii1_rxd2),  (MODE(7) | PULLUDDIS)},				/* (L16) gpio2[19] */  /* RST_GSM */
-
-
-#if 0
-	/* TODO: What is this meant for? */
-	{OFFSET(lcd_data3), (MODE(7) | PULLUDEN | PULLUP_EN)},	/* (R4) gpio2[9] */  /* SYSBOOT_3 */
-	{OFFSET(lcd_data4), (MODE(7) | PULLUDEN | PULLUP_EN)},	/* (T1) gpio2[10] */  /* SYSBOOT_4 */
-
-	/* TODO: Check other unued pins from sysboot block */
-	/* Ensure PU/PD does not work against external signal */
-	/*
-	 * SYSBOOT 0,1,5,12,13 = Low
-	 * SYSBOOT 2 = High
-	 */
-#endif
-
-	/* Bank 3 */
-	{OFFSET(mii1_txclk),  (MODE(7) | PULLUDDIS | RXACTIVE)},		/* (K18) gpio3[9] */  /* WLAN_IRQ */
-	{OFFSET(mii1_rxclk), (MODE(7) | PULLUDDIS)},				/* (L18) gpio3[10] */  /* WLAN_EN */
-	{OFFSET(mcasp0_ahclkr), (MODE(7) | PULLUDEN | PULLDOWN_EN)},		/* (C12) gpio3[17] */  /* SIM_SEL */
-	{-1}
-};
-
-/* I2C0 PMIC */
-static struct module_pin_mux i2c0_pin_mux[] = {
-	{OFFSET(i2c0_sda), (MODE(0) | RXACTIVE | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (C17) I2C0_SDA */
-	{OFFSET(i2c0_scl), (MODE(0) | RXACTIVE | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (C16) I2C0_SCL */
-	{-1}
-};
-
-/* I2C2 System */
-static struct module_pin_mux i2c2_pin_mux[] = {
-	{OFFSET(uart1_rtsn), (MODE(3) | RXACTIVE | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (D17) I2C2_SCL */
-	{OFFSET(uart1_ctsn), (MODE(3) | RXACTIVE | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (D18) I2C2_SDA */
-	{-1},
-};
-
-/* RMII1: Ethernet */
-static struct module_pin_mux rmii1_pin_mux[] = {
-	/* RMII */
-	{OFFSET(mii1_crs), MODE(1) | PULLUDDIS | RXACTIVE},			/* (H17) rmii1_crs */
-	{OFFSET(mii1_rxerr), MODE(7) | PULLUDEN | PULLDOWN_EN | RXACTIVE},	/* (J15) gpio (rxerr) */
-	{OFFSET(mii1_rxd0), MODE(1) | PULLUDDIS | RXACTIVE},			/* (M16) rmii1_rxd0 */
-	{OFFSET(mii1_rxd1), MODE(1) | PULLUDDIS | RXACTIVE},			/* (L15) rmii1_rxd1 */
-	{OFFSET(mii1_txen), MODE(1) | PULLUDDIS},				/* (J16) rmii1_txen */
-	{OFFSET(mii1_txd0), MODE(1) | PULLUDDIS},				/* (K17) rmii1_txd0 */
-	{OFFSET(mii1_txd1), MODE(1) | PULLUDDIS},				/* (K16) rmii1_txd1 */
-	{OFFSET(rmii1_refclk), MODE(0) | PULLUDDIS | RXACTIVE},			/* (H18) rmii1_refclk */
-
-	/* SMI */
-	{OFFSET(mdio_clk), MODE(0) | PULLUDDIS},				/* (M18) mdio_clk */
-	{OFFSET(mdio_data), MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE},		/* (M17) mdio_data */
-
-	/* 25MHz Clock Output */
-	{OFFSET(xdma_event_intr0), MODE(3)},					/* (A15) clkout1 (25 MHz clk for PHY) */
-	{-1}
-};
-
-/* RMII2: Ethernet */
-static struct module_pin_mux rmii2_pin_mux[] = {
-	/* RMII */
-	{OFFSET(gpmc_a9), MODE(3) | PULLUDDIS | RXACTIVE},			/* (U16) rmii2_crs */
-	{OFFSET(gpmc_wpn), MODE(7) | PULLUDEN | PULLDOWN_EN | RXACTIVE},	/* (U17) gpio (rxerr) */
-	{OFFSET(gpmc_a11), MODE(3) | PULLUDDIS | RXACTIVE},			/* (V17) rmii2_rxd0 */
-	{OFFSET(gpmc_a10), MODE(3) | PULLUDDIS | RXACTIVE},			/* (T16) rmii2_rxd1 */
-	{OFFSET(gpmc_a0), MODE(3) | PULLUDDIS},					/* (R13) rmii2_txen */
-	{OFFSET(gpmc_a5), MODE(3) | PULLUDDIS},					/* (V15) rmii2_txd0 */
-	{OFFSET(gpmc_a4), MODE(3) | PULLUDDIS},					/* (R14) rmii2_txd1 */
-	{OFFSET(mii1_col), MODE(1) | PULLUDDIS | RXACTIVE},			/* (H16) rmii2_refclk */
-	{-1},
-};
-
-/* MMC0: WiFi */
-static struct module_pin_mux mmc0_sdio_pin_mux[] = {
-	{OFFSET(mmc0_clk), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (G17) MMC0_CLK */
-	{OFFSET(mmc0_cmd), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (G18) MMC0_CMD */
-	{OFFSET(mmc0_dat0), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (G16) MMC0_DAT0 */
-	{OFFSET(mmc0_dat1), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (G15) MMC0_DAT1 */
-	{OFFSET(mmc0_dat2), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (F18) MMC0_DAT2 */
-	{OFFSET(mmc0_dat3), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (F17) MMC0_DAT3 */
-	{-1}
-};
-
-/* MMC1: eMMC */
-static struct module_pin_mux mmc1_emmc_pin_mux[] = {
-	{OFFSET(gpmc_csn1), (MODE(2) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (U9) MMC1_CLK */
-	{OFFSET(gpmc_csn2), (MODE(2) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (V9) MMC1_CMD */
-	{OFFSET(gpmc_ad0), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (U7) MMC1_DAT0 */
-	{OFFSET(gpmc_ad1), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (V7) MMC1_DAT1 */
-	{OFFSET(gpmc_ad2), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (R8) MMC1_DAT2 */
-	{OFFSET(gpmc_ad3), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (T8) MMC1_DAT3 */
-	{OFFSET(gpmc_ad4), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (U8) MMC1_DAT4 */
-	{OFFSET(gpmc_ad5), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (V8) MMC1_DAT5 */
-	{OFFSET(gpmc_ad6), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (R9) MMC1_DAT6 */
-	{OFFSET(gpmc_ad7), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (T9) MMC1_DAT7 */
-	{-1}
-};
-
-/* USB_DRVBUS not used -> configure as GPIO */
-static struct module_pin_mux usb_pin_mux[] = {
-	{OFFSET(usb0_drvvbus), (MODE(7) | PULLUDDIS)},				/* (F16) USB0_DRVVBUS */
-	{OFFSET(usb1_drvvbus), (MODE(7) | PULLUDDIS)},				/* (F15) USB1_DRVVBUS */
-	{-1}
-};
-
-/* UART0: User (Debug/Console) */
-static struct module_pin_mux uart0_pin_mux[] = {
-	{OFFSET(uart0_rxd), (MODE(0) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (E15) UART0_RXD */
-	{OFFSET(uart0_txd), (MODE(0) | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (E16) UART0_TXD */
-	{-1},
-};
-
-/* UART5: RS232/RS485 */
-/* CTS is unused - set to GPIO mode */
-static struct module_pin_mux uart5_pin_mux[] = {
-	{OFFSET(lcd_data9), (MODE(4) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (U2) UART5_RXD */
-	{OFFSET(lcd_data8), (MODE(4) | PULLUDEN | PULLUP_EN)},			/* (U1) UART5_TXD */
-	{OFFSET(lcd_data14), (MODE(7) | PULLUDEN | PULLDOWN_EN)},			/* (V4) uart5_ctsn */
-	{OFFSET(lcd_data15), (MODE(6) | PULLUDEN | PULLUP_EN)},			/* (T5) uart5_rtsn */
-	{-1}
-};
-
-static struct module_pin_mux unused_pin_mux[] = {
-	/* SYSBOOT6, 7, 10, 11: Not used pulldown active, receiver disabled */
-	{OFFSET(lcd_data6), (MODE(7) | PULLUDEN | PULLDOWN_EN)},
-	{OFFSET(lcd_data7), (MODE(7) | PULLUDEN | PULLDOWN_EN)},
-	{OFFSET(lcd_data10), (MODE(7) | PULLUDEN | PULLDOWN_EN)},
-	{OFFSET(lcd_data11), (MODE(7) | PULLUDEN | PULLDOWN_EN)},
-
-	/* TODO: GPMCA1..3, A6..8 */
-
-	{-1}
-};
-
-
-void enable_board_pin_mux(void)
-{
-	configure_module_pin_mux(gpio_pin_mux);
-
-	configure_module_pin_mux(rmii1_pin_mux);
-	configure_module_pin_mux(rmii2_pin_mux);
-	configure_module_pin_mux(mmc0_sdio_pin_mux);
-	configure_module_pin_mux(mmc1_emmc_pin_mux);
-	configure_module_pin_mux(usb_pin_mux);
-
-	configure_module_pin_mux(i2c0_pin_mux);
-	configure_module_pin_mux(i2c2_pin_mux);
-
-	configure_module_pin_mux(uart5_pin_mux);
-
-	configure_module_pin_mux(unused_pin_mux);
-}
-
-void enable_uart0_pin_mux(void)
-{
-	configure_module_pin_mux(uart0_pin_mux);
-}
-

board/nm/hw25/u-boot.lds

@@ -1,158 +0,0 @@
-/*
- * Copyright (c) 2004-2008 Texas Instruments
- *
- * (C) Copyright 2002
- * Gary Jennejohn, DENX Software Engineering, <garyj@denx.de>
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-OUTPUT_FORMAT("elf32-littlearm", "elf32-littlearm", "elf32-littlearm")
-OUTPUT_ARCH(arm)
-ENTRY(_start)
-SECTIONS
-{
-	. = 0x00000000;
-
-	. = ALIGN(4);
-	.text :
-	{
-		*(.__image_copy_start)
-		*(.vectors)
-		CPUDIR/start.o (.text*)
-                board/nm/hw25/built-in.o (.text*)
-		*(.text*)
-	}
-
-	. = ALIGN(4);
-	.rodata : { *(SORT_BY_ALIGNMENT(SORT_BY_NAME(.rodata*))) }
-
-	. = ALIGN(4);
-	.data : {
-		*(.data*)
-	}
-
-	. = ALIGN(4);
-
-	. = .;
-
-	. = ALIGN(4);
-	.u_boot_list : {
-		KEEP(*(SORT(.u_boot_list*)));
-	}
-
-	. = ALIGN(4);
-
-	.__efi_runtime_start : {
-		*(.__efi_runtime_start)
-	}
-
-	.efi_runtime : {
-		*(efi_runtime_text)
-		*(efi_runtime_data)
-	}
-
-	.__efi_runtime_stop : {
-		*(.__efi_runtime_stop)
-	}
-
-	.efi_runtime_rel_start :
-	{
-		*(.__efi_runtime_rel_start)
-	}
-
-	.efi_runtime_rel : {
-		*(.relefi_runtime_text)
-		*(.relefi_runtime_data)
-	}
-
-	.efi_runtime_rel_stop :
-	{
-		*(.__efi_runtime_rel_stop)
-	}
-
-	. = ALIGN(4);
-
-	.image_copy_end :
-	{
-		*(.__image_copy_end)
-	}
-
-	.rel_dyn_start :
-	{
-		*(.__rel_dyn_start)
-	}
-
-	.rel.dyn : {
-		*(.rel*)
-	}
-
-	.rel_dyn_end :
-	{
-		*(.__rel_dyn_end)
-	}
-
-	.hash : { *(.hash*) }
-
-	.end :
-	{
-		*(.__end)
-	}
-
-	_image_binary_end = .;
-
-	/*
-	 * Deprecated: this MMU section is used by pxa at present but
-	 * should not be used by new boards/CPUs.
-	 */
-	. = ALIGN(4096);
-	.mmutable : {
-		*(.mmutable)
-	}
-
-/*
- * Compiler-generated __bss_start and __bss_end, see arch/arm/lib/bss.c
- * __bss_base and __bss_limit are for linker only (overlay ordering)
- */
-
-	.bss_start __rel_dyn_start (OVERLAY) : {
-		KEEP(*(.__bss_start));
-		__bss_base = .;
-	}
-
-	.bss __bss_base (OVERLAY) : {
-		*(.bss*)
-		 . = ALIGN(4);
-		 __bss_limit = .;
-	}
-
-	.bss_end __bss_limit (OVERLAY) : {
-		KEEP(*(.__bss_end));
-	}
-
-	.dynsym _image_binary_end : { *(.dynsym) }
-	.dynbss : { *(.dynbss) }
-	.dynstr : { *(.dynstr*) }
-	.dynamic : { *(.dynamic*) }
-	.gnu.hash : { *(.gnu.hash) }
-	.plt : { *(.plt*) }
-	.interp : { *(.interp*) }
-	.gnu : { *(.gnu*) }
-	.ARM.exidx : { *(.ARM.exidx*) }
-}

board/nm/netbird_v2/board.c

@@ -80,6 +80,8 @@ static struct ctrl_dev *cdev = (struct ctrl_dev *)CTRL_DEVICE_BASE;
 
 static BD_Context   bdctx[3];		/* The descriptor context */
 
+#define SHIELD_COM_IO	0
+#define SHIELD_DUALCAN	1
 
 static int _bd_init(void)
 {
@@ -589,16 +591,6 @@ static struct shield_command known_shield_commands[] = {
 		"fdt set $can1 status okay;",
 		can_shield_init
 	},
-	{
-		SHIELD_DUALCAN_PASSIVE,
-		"dualcan",
-		"shield dualcan termination off off",
-		"fdt get value can0 /aliases d_can0;" \
-		"fdt get value can1 /aliases d_can1;" \
-		"fdt set $can0 status okay;" \
-		"fdt set $can1 status okay;",
-		can_shield_init
-	}
 };
 
 static const struct shield_command* get_shield_command(int shield_id)

board/nm/netbird_v2/shield.h

@@ -8,11 +8,6 @@
 #ifndef SHIELD_H
 #define SHIELD_H
 
-#define SHIELD_COM_IO		0
-#define SHIELD_DUALCAN		1
-#define SHIELD_CAN_GNSS		2
-#define SHIELD_DUALCAN_PASSIVE	3
-
 struct shield_t{
     char name[64];
     int (*setmode)(char * const argv[], int argc);

board/nm/netbird_v2/shield_can.c

@@ -9,8 +9,8 @@
 
 #define NETBIRD_GPIO_RST_SHIELD_N GPIO_TO_PIN(0, 27)
 #define NETBIRD_GPIO_LATCH GPIO_TO_PIN(0, 7)
-#define NETBIRD_GPIO_MODE_0 GPIO_TO_PIN(1, 10)
-#define NETBIRD_GPIO_MODE_1 GPIO_TO_PIN(1, 8)
+#define NETBIRD_GPIO_MODE_0 GPIO_TO_PIN(1, 8)
+#define NETBIRD_GPIO_MODE_1 GPIO_TO_PIN(1, 10)
 
 
 static int shield_slot_initialized = 0;

board/nm/nmhw21/Kconfig

@@ -15,7 +15,7 @@ config SYS_CONFIG_NAME
 config CONS_INDEX
         int "UART used for console"
         range 1 6
-        default 3
+	default 3
         help
           The AM335x SoC has a total of 6 UARTs (UART0 to UART5 as referenced
           in documentation, etc) available to it.  Depending on your specific

board/nm/nmhw21/Makefile

@@ -10,4 +10,4 @@ ifeq ($(CONFIG_SKIP_LOWLEVEL_INIT),)
 obj-y	:= mux.o
 endif
 
-obj-y	+= board.o ../common/bdparser.o ../common/board_descriptor.o ../common/da9063.o ../common/ether_crc.o fileaccess.o sja1105.o ui.o um.o reset_reason.o
+obj-y	+= board.o ../common/bdparser.o ../common/board_descriptor.o ../common/da9063.o fileaccess.o sja1105.o ui.o um.o

board/nm/nmhw21/board.h

@@ -13,10 +13,8 @@
 
 void enable_uart0_pin_mux(void);
 void enable_uart2_pin_mux(void);
-void enable_uart4_pin_mux(void);
 void enable_spi1_mux(void);
-void enable_led_mux(void);
-void enable_led_mux_v32(void);
+
 void enable_board_pin_mux(void);
 
 #define GPIO_TO_PIN(bank, gpio)		(32 * (bank) + (gpio))

board/nm/nmhw21/mux.c

@@ -23,22 +23,24 @@
 
 static struct module_pin_mux gpio_pin_mux[] = {
 	/*
-	* (V2)  GPIO0_8: RS232_485n_SEL (V3.2)
-	* (V3)  GPIO0_9: RS485_DE (V3.2)
 	* (J18) GPIO0_16: ETH_SW_RST~ (V2.0)
 	* (K15) GPIO0_17: CTRL.INT~
 	* (T10) GPIO0_23: CAN_TERM1~ (V1.0)
+	* (T17) GPIO0_30: LED0.GN
 	*
 	* (T12) GPIO1_12: SIM_SW
 	* (V13) GPIO1_14: GNSS_RST~
 	* (U13) GPIO1_15: CAN_TERM0~ (V1.0)
 	* (R14) GPIO1_20: BT_EN
 	* (V15) GPIO1_21: GSM_PWR_EN
+	* (U15) GPIO1_22: LED1.RD
+	* (V16) GPIO1_24: LED1.GN
 	* (U16) GPIO1_25: RST_GSM
 	* (T16) GPIO1_26: WLAN_EN
 	* (V17) GPIO1_27: WLAN_IRQ
+	* (U18) GPIO1_28: LED0.RD
 	*
-	* (U3) GPIO2_16: TIMEPULSE (HW26), see note [1]
+	* (U3) GPIO2_16: SIM_PRES~ (V2.0)
 	* (R6) GPIO2_25: RST_ETH~
 	*
 	* (J17) GPIO3_4: GNSS_EXTINT
@@ -46,31 +48,47 @@ static struct module_pin_mux gpio_pin_mux[] = {
 	* (L18) GPIO3_10: CTRL.RST
 	* (C12) GPIO3_17: UI_RST~
 	* (A14) GPIO3_21: RST_HUB~ (USB)
-	*
-	* [1] No PU/PD allowed as TIMEPULSE is internally connected with SAFEBOOT_N.
-	*     SAFEBOOT_N must be left open/floating.
 	*/
 
 	/* Bank 0 */
 	{OFFSET(mii1_txd3), (MODE(7) | PULLUDDIS)},				/* (J18) GPIO0_16: ETH_SW_RST~ (V2.0) */
 	{OFFSET(mii1_txd2), (MODE(7) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (K15) GPIO0_17: CTRL.INT~ */
 	{OFFSET(gpmc_ad9), (MODE(7) | PULLUDDIS)},				/* (T10) GPIO0_23: CAN_TERM1~ */
+	{OFFSET(gpmc_wait0), (MODE(7) | PULLUDDIS)},				/* (T17) GPIO0_30: LED0.GN */
 
 	/* Bank 1 */
-	{OFFSET(gpmc_ad12), (MODE(7) | PULLUDEN | PULLUP_EN)},			/* (T12) GPIO1_12: SIM_SW */
+	{OFFSET(gpmc_ad12), (MODE(7) | PULLUDDIS)},				/* (T12) GPIO1_12: SIM_SW */
 	{OFFSET(gpmc_ad14), (MODE(7) | PULLUDDIS)},				/* (V13) GPIO1_14: GNSS_RST~ */
 	{OFFSET(gpmc_ad15), (MODE(7) | PULLUDDIS)},				/* (U13) GPIO1_15: CAN_TERM0~ */
 
 	{OFFSET(gpmc_a4), (MODE(7) | PULLUDDIS)},				/* (R14) gpio1_20: BT_EN */
 	{OFFSET(gpmc_a5), (MODE(7) | PULLUDDIS)},				/* (V15) gpio1_21: GSM_PWR_EN */
+	{OFFSET(gpmc_a6), (MODE(7) | PULLUDDIS)},				/* (U15) GPIO1_22: LED1.RD */
+	{OFFSET(gpmc_a8), (MODE(7) | PULLUDDIS)},				/* (V16) GPIO1_24: LED1.GN */
 	{OFFSET(gpmc_a9), (MODE(7) | PULLUDDIS)},				/* (U16) gpio1_25: RST_GSM */
 	{OFFSET(gpmc_a10), (MODE(7) | PULLUDDIS)},				/* (T16) gpio1_26: WLAN_EN */
 	{OFFSET(gpmc_a11), (MODE(7) | PULLUDDIS | RXACTIVE)},			/* (V17) gpio1_27: WLAN_IRQ */
+	{OFFSET(gpmc_be1n), (MODE(7) | PULLUDDIS)},				/* (U18) GPIO1_28: LED0.RD */
+
+	/* TODO: What about all the unused GPMC pins ? */
 
 	/* Bank 2 */
-	{OFFSET(lcd_data10), (MODE(7) | PULLUDDIS | RXACTIVE)},			/* (U3) GPIO2_16: TIMEPULSE */
+	{OFFSET(lcd_data10), (MODE(7) | PULLUDDIS)},				/* (U3) GPIO2_16: SIM_PRES~ */
 	{OFFSET(lcd_ac_bias_en), (MODE(7) | PULLUDDIS)},			/* (R6) GPIO2_25: RST_ETH~ */
 
+#if 0
+	/* TODO: What is this meant for? */
+	{OFFSET(lcd_data3), (MODE(7) | PULLUDEN | PULLUP_EN)},	/* (R4) gpio2[9] */  /* SYSBOOT_3 */
+	{OFFSET(lcd_data4), (MODE(7) | PULLUDEN | PULLUP_EN)},	/* (T1) gpio2[10] */  /* SYSBOOT_4 */
+
+	/* TODO: Check other unued pins from sysboot block */
+	/* Ensure PU/PD does not work against external signal */
+	/*
+	 * SYSBOOT 0,1,5,12,13 = Low
+	 * SYSBOOT 2 = High
+	 */
+#endif
+
 	/* Bank 3 */
 	{OFFSET(mii1_rxdv), (MODE(7) | PULLUDDIS)},			/* (J17) GPIO3_4: GNSS_EXTINT */
 	{OFFSET(mii1_txclk), (MODE(7) | PULLUDDIS)},			/* (K18) GPIO3_9: CTRL.W_DIS */
@@ -80,39 +98,6 @@ static struct module_pin_mux gpio_pin_mux[] = {
 	{-1}
 };
 
-static struct module_pin_mux led_pin_mux[] = {
-	/*
-	* (T17) GPIO0_30: LED0.GN
-	* (U15) GPIO1_22: LED1.RD
-	* (V16) GPIO1_24: LED1.GN
-	* (U18) GPIO1_28: LED0.RD
-	*/
-
-	{OFFSET(gpmc_wait0), (MODE(7) | PULLUDDIS)},				/* (T17) GPIO0_30: LED0.GN */
-	{OFFSET(gpmc_a6), (MODE(7) | PULLUDDIS)},				/* (U15) GPIO1_22: LED1.RD */
-	{OFFSET(gpmc_a8), (MODE(7) | PULLUDDIS)},				/* (V16) GPIO1_24: LED1.GN */
-	{OFFSET(gpmc_be1n), (MODE(7) | PULLUDDIS)},				/* (U18) GPIO1_28: LED0.RD */
-	{-1}
-};
-
-static struct module_pin_mux led_pin_mux_v32[] = {
-	/*
-	* (C15) GPIO0_6: MB_LED_PWM
-	* (U15) GPIO1_22: LED1.RD
-	* (T15) GPIO1_23: LED0.GN (formerly: (T17) GPIO0_30)
-	* (V16) GPIO1_24: LED1.GN
-	* (U18) GPIO1_28: LED0.RD
-	*/
-
-	{OFFSET(spi0_cs1), (MODE(7) | PULLUDDIS)},				/* (C15) GPIO0_6: MB_LED_PWM */
-	{OFFSET(gpmc_a6), (MODE(7) | PULLUDDIS)},				/* (U15) GPIO1_22: LED1.RD */
-	{OFFSET(gpmc_a7), (MODE(7) | PULLUDDIS)},				/* (T15) GPIO1_23: LED0.GN */
-	{OFFSET(gpmc_a8), (MODE(7) | PULLUDDIS)},				/* (V16) GPIO1_24: LED1.GN */
-	{OFFSET(gpmc_be1n), (MODE(7) | PULLUDDIS)},				/* (U18) GPIO1_28: LED0.RD */
-	{-1}
-};
-
-
 /* I2C0 PMIC */
 static struct module_pin_mux i2c0_pin_mux[] = {
 	{OFFSET(i2c0_sda), (MODE(0) | RXACTIVE | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (C17) I2C0_SDA */
@@ -218,22 +203,8 @@ static struct module_pin_mux uart2_pin_mux[] = {
 
 /* UART3: GNSS */
 static struct module_pin_mux uart3_pin_mux[] = {
-	{OFFSET(mii1_rxd3), (MODE(1) | PULLUDDIS | RXACTIVE)},			/* (L17) UART3_RXD */
-	{OFFSET(mii1_rxd2), (MODE(1) | PULLUDDIS | SLEWCTRL)},			/* (L16) UART3_TXD */
-	{-1}
-};
-
-/* UART4: User RS232/485 (V3.2 only) */
-static struct module_pin_mux uart4_pin_mux[] = {
-	/*
-	 * CTSn = SEL_RS232/RS485~: Default = Low -> RS485 mode
-	 * RTSn = RS485_DE: Default = Low -> RS485 transmitter disabled
-	 *        Configure as GPIO in U-Boot to keep disabled, Linux will change to RTSn
-	 */
-	{OFFSET(gpmc_wait0), (MODE(6) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (T17) UART4_RXD */
-	{OFFSET(gpmc_wpn), (MODE(6) | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (U17) UART4_TXD */
-	{OFFSET(lcd_data12), (MODE(7) | PULLUDEN | PULLDOWN_EN)},		/* (V2) uart4_ctsn */
-	{OFFSET(lcd_data13), (MODE(7) | PULLUDEN | PULLDOWN_EN)},		/* (V3) uart4_rtsn */
+	{OFFSET(mii1_rxd3), (MODE(1) | PULLUDEN | PULLUP_EN | RXACTIVE)},	/* (L17) UART3_RXD */
+	{OFFSET(mii1_rxd2), (MODE(1) | PULLUDEN | PULLUP_EN | SLEWCTRL)},	/* (L16) UART3_TXD */
 	{-1}
 };
 
@@ -297,22 +268,7 @@ void enable_uart2_pin_mux(void)
 	configure_module_pin_mux(uart2_pin_mux);
 }
 
-void enable_uart4_pin_mux(void)
-{
-	configure_module_pin_mux(uart4_pin_mux);
-}
-
 void enable_spi1_mux(void)
 {
 	configure_module_pin_mux(spi1_pin_mux);
 }
-
-void enable_led_mux(void)
-{
-	configure_module_pin_mux(led_pin_mux);
-}
-
-void enable_led_mux_v32(void)
-{
-	configure_module_pin_mux(led_pin_mux_v32);
-}

board/nm/nmhw21/reset_reason.c

@@ -1,81 +0,0 @@
-/*
- * reset_reason.c
- *
- * Reset/start reason handling
- *
- * Copyright (C) 2021 NetModule AG - https://www.netmodule.com/
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-#include <common.h>
-#include "../common/ether_crc.h"
-#include "reset_reason.h"
-
-
-void rr_set_reset_reason(volatile struct reset_registers* reset_regs, uint32_t reason)
-{
-	reset_regs->rr_value = reason;
-	reset_regs->rr_value_crc = ether_crc(sizeof(reset_regs->rr_value),
-					     (const uint8_t*)&(reset_regs->rr_value));
-}
-
-bool rr_is_reset_reason_valid(volatile const struct reset_registers* reset_regs)
-{
-	const uint32_t crc = ether_crc(sizeof(reset_regs->rr_value), 
-				       (const uint8_t*)&(reset_regs->rr_value));
-	return crc == reset_regs->rr_value_crc;
-}
-
-void rr_set_start_reason(volatile struct reset_registers* reset_regs, uint32_t event)
-{
-	/* Store start events in shared memory region for OS */
-	reset_regs->sr_magic = SR_MAGIC;
-	reset_regs->sr_events = event;
-	reset_regs->sr_checksum = ether_crc(sizeof(reset_regs->sr_events),
-					    (const uint8_t*)&(reset_regs->sr_events));	
-}
-
-bool rr_is_start_reason_valid(volatile const struct reset_registers* reset_regs)
-{
-	if (reset_regs->sr_magic == SR_MAGIC) {
-		const uint32_t crc = ether_crc(sizeof(reset_regs->sr_events), 
-					(const uint8_t*)&(reset_regs->sr_events));
-		if (crc == reset_regs->sr_checksum) {
-			return true;
-		}
-	}
-	
-	return false;
-}
-
-void rr_start_reason_to_str(uint32_t events, char* buffer, size_t bufsize)
-{
-	if (events == 0) {
-		strncpy(buffer, "-\n", bufsize);
-	}
-	else {
-		buffer[0] = 0;
-		if (events & SR_POR)
-			strncat(buffer, "PowerOn, ", bufsize);
-		if (events & SR_WATCHDOG)
-			strncat(buffer, "Watchdog, ", bufsize);
-		if (events & SR_REBOOT)
-			strncat(buffer, "Reboot, ", bufsize);
-		if (events & SR_WAKEUP)
-			strncat(buffer, "Wakeup, ", bufsize);
-
-		if (events & SR_EVT_IGNITION)
-			strncat(buffer, "Ignition, ", bufsize);
-		if (events & SR_EVT_RTC_ALARM)
-			strncat(buffer, "RTC, ", bufsize);
-		if (events & SR_EVT_RTC_TICK)
-			strncat(buffer, "Tick, ", bufsize);
-		if (events & SR_EVT_GPI)
-			strncat(buffer, "GPI, ", bufsize);
-		if (events & SR_EVT_BUTTON)
-			strncat(buffer, "Button, ", bufsize);
-
-		/* Trim last comma, no 0 len check required, at least one entry is present */
-		buffer[strlen(buffer)-2] = 0;
-	}
-}

board/nm/nmhw21/reset_reason.h

@@ -1,60 +0,0 @@
-/*
- * reset_reason.h
- *
- * Reset/start reason handling
- *
- * Copyright (C) 2021 NetModule AG - https://www.netmodule.com/
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-#ifndef RESET_REASON_H
-#define RESET_REASON_H
-
-struct reset_registers {
-	/* Reboot Reasons, set by OS, expect watchdog set by bootloader */
-	uint32_t rr_value;
-	uint32_t rr_value_crc;
-
-	/* Start Reasons as determined by hardware */
-	uint32_t sr_magic;	/* Token to check presence of following fields */
-	uint32_t sr_events;	/* Events bitmask, see SE_... defines */
-	uint32_t sr_checksum;	/* Checksum over se_events */
-};
-
-/* Watchdog reboot reason event */
-#define RR_POWEROFF_PATTERN		0x00000000
-#define RR_EXTERNAL_WATCHDOG_PATTERN	0x781f9ce2
-#define RR_BOOT_PATTERN			0x424f4f54	/* ‘BOOT’, 0xb9808470 */
-#define RR_REBOOT_PATTERN		0x5245424f	/* ‘REBO’, 0x7d5d9d66 */
-#define RR_OOPS_PATTERN			0x4F4F5053	/* ‘OOPS’, 0x2b85bc5f */
-#define RR_WAKE_PATTERN			0x57414B45	/* 'WAKE', 0x7b0acb48 */
-
-/* Start reason token 'SRTE' */
-#define SR_MAGIC			0x53525445
-
-/* Possible start events (see sr_events) */
-#define SR_POR  			0x00000001
-#define SR_WATCHDOG  			0x00000010
-#define SR_REBOOT  			0x00000020
-#define SR_WAKEUP  			0x00000080	/* See SR_EVT_xx bits */
-
-/* In case of wake-up, these are the events that caused the start */
-#define SR_EVT_IGNITION  		0x00000100
-#define SR_EVT_RTC_ALARM  		0x00000200	/* RTC date/time alarm */
-#define SR_EVT_RTC_TICK  		0x00000400	/* RTC tick based alarm */
-#define SR_EVT_GPI			0x00000800	/* General purpose input(s) */
-#define SR_EVT_BUTTON			0x00001000	
-#define SR_EVT_WAKE_MASK		0x00001F00
-
-
-
-extern void rr_set_reset_reason(volatile struct reset_registers* reset_regs, uint32_t reason);
-extern bool rr_is_reset_reason_valid(volatile const struct reset_registers* reset_regs);
-
-extern void rr_set_start_reason(volatile struct reset_registers* reset_regs, uint32_t event);
-extern bool rr_is_start_reason_valid(volatile const struct reset_registers* reset_regs);
-extern void rr_start_reason_to_str(uint32_t events, char* buffer, size_t bufsize);
-
-
-#endif /* RESET_REASON_H */

board/nm/nmhw21/sja1105.c

@@ -90,10 +90,10 @@ void sja1105_write_reg(uint32_t address, uint32_t data)
 
 void sja1105_configure_firmware(void)
 {
-    static const uint8_t config_data_0[] = { 0x80, 0x20, 0x00, 0x00, 0x9E, 0x00, 0x03, 0x0E, 0x06, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x50, 0x21, 0x6F, 0x25, 0x6B, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF };
-    static const uint8_t config_data_1[] = { 0x80, 0x20, 0x04, 0x00, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF9, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0x03, 0x68, 0x8B, 0xEA, 0x07, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x02, 0x7D, 0x0B, 0xCB, 0xF2, 0xF8, 0x00, 0x00, 0x00, 0x00, 0x31, 0x80, 0x7F, 0x09, 0x52, 0x68, 0x0D, 0x08, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x1A, 0x6A, 0xF6, 0x23, 0x53, 0x10, 0x00, 0x00, 0x00, 0xF7, 0xBD, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0xEF, 0x7B, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0xDE, 0xF7, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0xBD, 0xEF, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0x7B, 0xDF, 0xF5, 0x8D, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xC0, 0x04, 0xA6, 0x06, 0x09, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x23, 0xDA, 0xB5, 0xBD, 0xC8, 0x00, 0x3F, 0xFC, 0x08, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 };
-    static const uint8_t config_data_2[] = { 0x80, 0x20, 0x08, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x08, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x08, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x0E, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x04, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x0E, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x04, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x83, 0x75, 0xF9, 0x8D, 0x0D, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x25, 0x0E, 0x7C, 0xBD, 0x00, 0x01, 0x25, 0xC0, 0x70, 0x94, 0x84, 0x50, 0x0E, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x03, 0xC8, 0xA7, 0xCE, 0xE6, 0x00, 0x71, 0xC0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xC3, 0xF7, 0x04, 0xB9, 0x11, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x0A, 0x57, 0x1F, 0x81, 0x3F, 0x06, 0x44, 0x00, 0x00, 0x00, 0x00, 0x04, 0x08, 0x00, 0x00, 0x00, 0x00, 0xFF, 0x0D, 0xB0, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0x00, 0xFF, 0xFF, 0xFF, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x58, 0x00, 0x00, 0x00, 0x5A, 0x9C, 0x37, 0x20, 0x4E, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x3A, 0x5D, 0x5E, 0x24, 0xA4, 0x9A, 0x00, 0x00, 0x7B, 0x51, 0xDA, 0x7D, 0x00, 0x00, 0x00, 0x00 };
-    static const uint8_t config_data_3[] = { 0x80, 0x20, 0x0C, 0x00, 0x00, 0x00, 0x00, 0x00, 0x37, 0x6E, 0x02, 0x8B };
+	static const uint8_t config_data_0[] = { 0x80, 0x20, 0x00, 0x00, 0x9E, 0x00, 0x03, 0x0E, 0x06, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x50, 0x21, 0x6F, 0x25, 0x6B, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x03, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x07, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0B, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF };
+	static const uint8_t config_data_1[] = { 0x80, 0x20, 0x04, 0x00, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x0F, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFE, 0xF7, 0x00, 0x00, 0x13, 0xFF, 0xFF, 0xFF, 0xFA, 0x2E, 0x19, 0xF8, 0x07, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x02, 0x7D, 0x0B, 0xCB, 0xF2, 0xF8, 0x00, 0x00, 0x00, 0x00, 0x31, 0x80, 0x7F, 0x09, 0x52, 0x68, 0x0D, 0x08, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x1A, 0x6A, 0xF6, 0x23, 0x53, 0x10, 0x00, 0x00, 0x00, 0xF7, 0xBD, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0xEF, 0x7B, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0xDE, 0xF7, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0xBD, 0xEF, 0xF5, 0x8D, 0x10, 0x00, 0x00, 0x00, 0x7B, 0xDF, 0xF5, 0x8D, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xC0, 0x04, 0xA6, 0x06, 0x09, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x23, 0xDA, 0xB5, 0xBD, 0xC8, 0x00, 0x3F, 0xFC, 0x08, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 };
+	static const uint8_t config_data_2[] = { 0x80, 0x20, 0x08, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x08, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x08, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x0E, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x04, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x3F, 0xFC, 0x0E, 0x00, 0x00, 0x00, 0x00, 0x07, 0xFC, 0x01, 0x04, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x83, 0x75, 0xF9, 0x8D, 0x0D, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x25, 0x0E, 0x7C, 0xBD, 0x00, 0x01, 0x25, 0xC0, 0x70, 0x94, 0x84, 0x50, 0x0E, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x03, 0xC8, 0xA7, 0xCE, 0xE6, 0x00, 0x71, 0xC0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xC3, 0xF7, 0x04, 0xB9, 0x11, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x0A, 0x57, 0x1F, 0x81, 0x3F, 0x06, 0x44, 0x00, 0x00, 0x00, 0x00, 0x04, 0x08, 0x00, 0x00, 0x00, 0x00, 0xFF, 0x0D, 0xB0, 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0x00, 0xFF, 0xFF, 0xFF, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x58, 0x00, 0x00, 0x00, 0x5A, 0x9C, 0x37, 0x20, 0x4E, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x3A, 0x5D, 0x5E, 0x24, 0xA4, 0x9A, 0x00, 0x00, 0x7B, 0x51, 0xDA, 0x7D, 0x00, 0x00, 0x00, 0x00 };
+	static const uint8_t config_data_3[] = { 0x80, 0x20, 0x0C, 0x00, 0x00, 0x00, 0x00, 0x00, 0x37, 0x6E, 0x02, 0x8B };
 
 	uint32_t val;
 	int rc;

board/nm/nrhw20/board.c

@@ -3,7 +3,7 @@
  *
  * Board functions for Netmodule NRHW 20, based on AM335x EVB
  *
- * Copyright (C) 2018-2020 NetModule AG - http://www.netmodule.com/
+ * Copyright (C) 2018-2019 NetModule AG - http://www.netmodule.com/
  * Copyright (C) 2011, Texas Instruments, Incorporated - http://www.ti.com/
  *
  * SPDX-License-Identifier:	GPL-2.0+
@@ -43,7 +43,6 @@
 
 /* TODO: place in proper header file */
 extern void serial_set_console_index(int index);
-extern int console_init_f(void);
 
 
 DECLARE_GLOBAL_DATA_PTR;
@@ -138,8 +137,6 @@ DECLARE_GLOBAL_DATA_PTR;
 #define IOEXT_LED4_GREEN_MASK		(1U << 9)
 #define IOEXT_LEDS_ALL_MASK		(0x03FF)
 
-#define NUM_INDICATOR_LEDS		5
-
 
 #define DDR3_CLOCK_FREQUENCY		(400)
 
@@ -162,6 +159,10 @@ static struct ctrl_dev *cdev = (struct ctrl_dev *)CTRL_DEVICE_BASE;
 #define PD_ADDRESS			(0x0200)  /* Product descriptor */
 #define PARTITION_ADDRESS		(0x0600)  /* Partition Table */
 
+#define SHIELD_COM_IO	0
+#define SHIELD_DUALCAN	1
+
+
 static BD_Context   bdctx[3];		/* The descriptor contexts */
 
 
@@ -221,7 +222,7 @@ static void set_status_led(int red, int green)
 	da9063_release_i2c_bus(bus);
 }
 
-static void set_indicator_led(unsigned led, int red, int green)
+static void set_indicator(unsigned led, int red, int green)
 {
 	int old_bus;
 	uint led_red_mask = 0x1U << (2*led);
@@ -324,13 +325,8 @@ struct serial_device *default_serial_console(void)
 		/* TODO: Sorry, what are we doing here.... */
 		/* This is at least dangerous if not completely wrong */
 		/* TODO: Using UART1 for moment until it is clear what to do */
-#if 0
-		enable_uart5_pin_mux();
-		return &eserial6_device;
-#else
 		enable_uart1_pin_mux();
 		return &eserial2_device;
-#endif
 	}
 }
 
@@ -394,10 +390,9 @@ static void init_pmic_spl(void)
 	(void)da9063_set_reg(PMIC_REG_BUCK_ILIM_B, 0x50);
 	(void)da9063_set_reg(PMIC_REG_BUCK_ILIM_C, 0xFF);
 
-	/*
-	 * FB 57727 Use synchronous mode for buck converters
-	 * This solves the occasional rail lock up problem
-	 */
+	/* TODO: C comments, not C++ */
+	// FB 57727 Use synchronous mode for buck converters
+	// This solves the occasional rail lock up problem
 	(void)da9063_set_reg(PMIC_REG_BCORE1_CONF, 0x81);
 	(void)da9063_set_reg(PMIC_REG_BCORE2_CONF, 0x81);
 	(void)da9063_set_reg(PMIC_REG_BIO_CONF, 0x81);
@@ -405,7 +400,8 @@ static void init_pmic_spl(void)
 	(void)da9063_set_reg(PMIC_REG_BPERI_CONF, 0x81);
 
 	/* Enable charging of RTC backup capacitor (1mA, 3.1V) */
-	(void)da9063_set_reg(PMIC_REG_BBAT_CONT, 0xAF);
+	(void)da9063_set_reg(PMIC_REG_BBAT_CONT, 0xCF);
+	/* TODO: Check documentation 1 mA correct ? */
 
 	da9063_release_i2c_bus(bus);
 }
@@ -415,7 +411,7 @@ struct reset_registers {
 	uint32_t value_crc;
 };
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 
 /* TODO: Move ethernet crc to dedicated file */
 static uint32_t ether_crc(size_t len, uint8_t const *p)
@@ -481,17 +477,16 @@ void am33xx_spl_board_init(void)
 	init_pmic_spl();
 
 	init_leds();
-#ifdef CONFIG_NRSW_BUILD
-	set_indicator_led(0, 1, 1); /* Orange */
-#else
+#if 0	/* TODO: missing ? */
 	set_status_led(1, 0); /* Red */
-	set_indicator_led(0, 1, 0); /* Red */
 #endif
+	set_indicator(0, 1, 1); /* Orange */
 
 	/* Set MPU Frequency to what we detected now that voltages are set */
 	do_setup_dpll(&dpll_mpu_regs, &dpll_mpu_opp100);
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
+	/* TODO: Move to board_late_init? It is not urgent to have this in SPL. */
 	check_pmic_reset_reason(RESET_REASON_SHM_LOCATION);
 #endif
 
@@ -627,32 +622,18 @@ static void init_pcie_slot(void)
 
 static void init_gsm(void)
 {
+	/* TODO: Check and update for Toby-L2 */
 	/*
-	 * Perform power up sequence for TOBY-L2 modem.
-	 * 
-	 * TOBY-L2 series can be switched on in one of the following ways:
-	 * - Rising edge on the VCC pin to a valid voltage for module supply, 
-	 *   i.e. applying module supply
-	 * - Low level on the PWR_ON pin, which is normally set high by an 
-	 *   internal pull-up, for a valid time period when the applied VCC 
-	 *   voltage is within the valid operating range (see section 4.2.8). 
-	 * - Low level on the RESET_N pin, which is normally set high by an 
-	 *   internal pull-up, for a valid time period when the applied VCC
-	 *   voltage is within the valid operating range (see section 4.2.9). 
-	 * 
-	 * PWR_ON low time: 5 ms - Low time to switch-on the module
-	 * RESET_N low time: 18..800 ms - Low time to switch-on the module
-	 *                   2.1..15 s - Low time to reset the module
-	 *                   16 s - Low time to switch-off the module
-	 * 
+	 * Perform power up sequence for Huawei ME909s.
+	 *
+	 * Modem is initially disabled and does not start automatically.
+	 * Enable power and start modem by "pressing" POWER_ON_OFF input for 1.0s.
+	 *
 	 * References:
-	 *  - uBlox TOBY-L2 Datasheet UBX-13004573 - R24
-	 *    2.3.1 Module power-on
-	 *    4.2.8 PWR_ON pin
-	 *    4.2.9 RESET_N pin
-	 * 
+	 *  - HUAWEI ME909s Series LTE LGA Module Hardware Guide, Issue 02, Date 2016-02-20
+	 *    3.3.2 Power Supply VBAT Interface
+	 *    3.4.2 Power-on/off Pin, Figure 3.7
 	 */
-
 	int bus;
 
 	puts("GSM:   ");
@@ -714,12 +695,9 @@ int board_init(void)
 
 	/* Let user know we're starting */
 	init_leds();
-#ifdef CONFIG_NRSW_BUILD
 	set_status_led(1, 1);	/* Orange */
-#else
-	set_status_led(1, 1);		/* Orange */
-	set_indicator_led(0, 0, 0);	/* Off */
-#endif
+	/* TODO: Status LED correct? */
+
 	printf("OSC:   %lu MHz\n", get_osclk()/1000000);
 
 	return 0;
@@ -733,7 +711,6 @@ int board_init(void)
  * - Available tty interfaces
  *   - ttyS1: standard console (default)
  *   - ttyS0: COM/IO shield
- *   - ttyNull0: Dummy device if no real UART is available
  */
 void set_console(void)
 {
@@ -759,21 +736,7 @@ void set_console(void)
 			setenv("defaultconsole", buf);
 		}
 	}
-
-	/*
-	 * If ttyS0 (COM_IO shield) is selected, make sure we really have
-	 * a COM_IO shield. Otherwise use the dummy tty. Actually this should
-	 * not happen. So it is just a sanity check to prevent the kernel from
-	 * crashing due to missing console, if something is configured wrong.
-	 * (Obviously thats just another hack, but there are no other options on
-	 * this platform because of a lot of conflicts between pins and uarts.)
-	 */
-	if (strstr(defaultconsole, "ttyS0")) {
-		if (bd_get_shield(0) != SHIELD_COM_IO) {
-			setenv("defaultconsole", "ttyNull");
-		}
-	}
-
+	
 #if defined(CONFIG_PRE_CONSOLE_BUFFER)
 	defaultconsole = getenv("defaultconsole");
 
@@ -831,7 +794,7 @@ static void get_variant_name(void)
 
 static void get_hw_version(void)
 {
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 	char hw_versions[16];
 	char new_env[256];	/* current bootargs = 84 bytes */
 #endif
@@ -841,7 +804,8 @@ static void get_hw_version(void)
 
 	printf("HW20:  V%d.%d\n", hw_ver, hw_rev);
 
-#ifdef CONFIG_NRSW_BUILD
+	/* TODO: Check if this can be removed eventually */
+#ifdef CONFIG_NRSW
 	/* add hardware versions to environment */
 	snprintf(hw_versions, sizeof(hw_versions), "CP=%d.%d", hw_ver, hw_rev);
 	snprintf(new_env, sizeof(new_env), "setenv bootargs $bootargs %s", hw_versions);
@@ -876,10 +840,7 @@ static void get_pmic_version(void)
 static void check_jtag_boot(void)
 {
 	if (is_jtag_boot(CONFIG_JTAG_MARKER_UBOOT)) {
-		char *bootcmd = getenv("bootcmd");
 		setenv ("bootcmd", "");
-		/* Save original bootcmd in "bootcmd_orig" to allow manual boot */
-		setenv ("bootcmd_orig", bootcmd);
 		puts("Detected JTAG boot. Waiting on command line\n");
 	}
 }
@@ -927,12 +888,6 @@ static struct shield_command known_shield_commands[] = {
 		"shield dualcan termination off off",
 		can_shield_init
 	},
-	{
-		SHIELD_DUALCAN_PASSIVE,
-		"dualcan-passive",
-		"shield dualcan-passive",
-		can_shield_passive_init
-	}
 };
 
 static const struct shield_command* get_shield_command(int shield_id)
@@ -1010,24 +965,25 @@ static bool get_button_state(void)
 	return pressed;
 }
 
-static void blink_led(int red, int green)
+static void blink_led(int pulses)
 {
 	const int pulse_width = 400*1000; /* 400ms */
-	int i;
 
-	/* Assumes status led is on (green), indicator off */
-	/* udelay(pulse_width); */
-	set_status_led(red, green);
-	for (i=0; i<NUM_INDICATOR_LEDS; i++) {
-		set_indicator_led(i, red, green);
+	/* Assumes status led on, indicator off */
+	set_status_led(0, 0);
+
+	while (pulses) {
+		udelay(pulse_width);
+		set_status_led(1, 1);
+
+		udelay(pulse_width);
+		set_status_led(0, 0);
+
+		pulses--;
 	}
 
 	udelay(pulse_width);
-
-	set_status_led(0, 1);   /* Green */
-	for (i=0; i<NUM_INDICATOR_LEDS; i++) {
-		set_indicator_led(i, 0, 0);
-	}
+	set_status_led(0, 1);
 }
 
 static void check_reset_button(void)
@@ -1044,11 +1000,11 @@ static void check_reset_button(void)
 
 		if (counter == 2000) {
 			/* Indicate factory reset threshold */
-			blink_led(0, 1);	/* Green */
+			blink_led(1);
 		}
 		else if (counter == 12000) {
 			/* Indicate recovery boot threshold */
-			blink_led(1, 0);	/* Red */
+			blink_led(2);
 		}
 	} while (counter < 12000);
 
@@ -1120,8 +1076,8 @@ int board_late_init(void)
 	set_console();
 	shield_init();
 
-	set_status_led(0, 0);	/* WAN led off */
-	set_indicator_led(0, 0, 1); /* Green */
+	set_status_led(0, 1);	/* Green */
+	set_indicator(0, 0, 1); /* Green */
 
 	check_fct();
 	check_jtag_boot();
@@ -1267,98 +1223,46 @@ static void ft_enable_node(void* blob, const char* name)
 	}
 }
 
-/*
- * Modify the name of a gpio in a gpio-line-names string list.
- */
-static void ft_set_gpio_name(void *blob, const char* gpio, int pin, const char* name)
+static void ft_dio(void *blob, int shield_type)
 {
-	int node_ofs = fdt_path_offset(blob, gpio);
-	int gpios = -1;
-	const char* text;
-	int pos = 0;
-	int i;
-	char buffer[512];
-
-	if (node_ofs == -1) {
-		printf("Can't find node %s\n", gpio);
-		goto end;
-	}
-
-	/* get number of IOs in node */
-	gpios = fdt_getprop_u32_default_node(blob, node_ofs, 0, "ngpios", -1);
-	if (gpios == -1 || gpios > 64) {
-		printf("Illegal number of gpios %d\n", gpios);
-		goto end;
-	}
-
-	/* get string array with names */
-	const struct fdt_property* prop = fdt_get_property(blob, node_ofs, "gpio-line-names", NULL);
-	if (prop == NULL) {
-		goto end;
-	}
-
-	/* modify given name */
-	for (i=0; i<gpios; i++) {
-		if (i == pin) {
-			/* Take provided name if GPIO pin is matched */
-			text = name;
-		}
-		else {
-			/* Take existing name from string list */
-			(void)fdt_get_string_index(blob, node_ofs, "gpio-line-names", i, &text);
-		}
-
-		/* Add name to new string list */
-		if ((pos + strlen(text) + 1) < sizeof(buffer)) {
-			strncpy(buffer+pos, text, sizeof(buffer)-pos);
-			pos += strlen(text) + 1;
-		}
-		else {
-			printf("ft_set_gpio_name() Buffer too small\n");
-			goto end;
-		}
-	}
-
-	(void)fdt_setprop(blob, node_ofs, "gpio-line-names", buffer, pos);
-
-end: ;
-}
-
-/*
- * Enable digital IOs provided by COM/IO shield in gpio nodes
- */
-static void ft_comio_gpios(void *blob)
-{
-	/* gpio0_7: COM/IO relay output */
-	ft_set_gpio_name(blob, "gpio0", 7, "COMIO_OUT0");
-
-	/* gpio1_8: COM/IO digital input */
-	ft_set_gpio_name(blob, "gpio1", 8, "COMIO_IN0");
-}
-
-static void ft_shields(void* blob)
-{
-	int shield_type = -1;
-
-	shield_type = bd_get_shield(0);
 	switch (shield_type) {
+		/* If COM/IO shield is present enable its I/Os */
 		case SHIELD_COM_IO:
-			ft_comio_gpios(blob);
 			ft_enable_node(blob, "/netbox_dio_comio");
-			ft_enable_node(blob, "serial0");
-			break;
-
-		case SHIELD_DUALCAN:
-		case SHIELD_DUALCAN_PASSIVE:
-			ft_enable_node(blob, "d-can0");
-			ft_enable_node(blob, "d-can1");
 			break;
 
 		default:
 			ft_enable_node(blob, "/netbox_dio_default");
 			break;
-	};
+	}
+}
 
+static void ft_serial(void *blob, int shield_type)
+{
+	switch (shield_type) {
+		/* Enable uart1 (ttyS0) always as kernel needs it as fallback
+			console, if (ttyS1) is not available as console. */
+
+		default:
+			/* TODO: Should use alias serial0 */
+			ft_enable_node(blob, "/ocp/serial@44e09000");
+			break;
+	}
+}
+
+static void ft_dcan(void *blob, int shield_type)
+{
+	switch (shield_type) {
+		/* If Dual CAN shield is present enable dcan0, dcan1N1 */
+		case SHIELD_DUALCAN:
+		/* TODO: Should use alias d_can0, d_can1 */
+			ft_enable_node(blob, "/ocp/can@481cc000");
+			ft_enable_node(blob, "/ocp/can@481d0000");
+			break;
+
+		default:
+			break;
+	}
 }
 
 static void ft_bootloader_version(void *blob)
@@ -1388,10 +1292,18 @@ static void ft_hw_info(void *blob)
 
 int ft_board_setup(void *blob, bd_t *bd)
 {
+	/* TODO: Rework to behave like HW24, once this one is finalized */
+	/* ft_shields() function with switch case for shields */
+	int shield_type = -1;
+
 	ft_bootloader_version(blob);
 	ft_hw_info(blob);
 
-	ft_shields(blob);
+	shield_type = bd_get_shield(0);
+
+	ft_dio(blob, shield_type);
+	ft_serial(blob, shield_type);
+	ft_dcan(blob, shield_type);
 
 	return 0;
 }

board/nm/nrhw20/board.h

@@ -21,6 +21,9 @@ void enable_uart0_pin_mux(void);
 void disable_uart0_pin_mux(void);
 void enable_uart1_pin_mux(void);
 
+/* TODO: Remove */
+/*void enable_uart5_pin_mux(void);*/
+
 void enable_board_pin_mux(void);
 
 #define GPIO_TO_PIN(bank, gpio)		(32 * (bank) + (gpio))

board/nm/nrhw20/mux.c

@@ -250,3 +250,11 @@ void enable_uart1_pin_mux(void)
 {
 	configure_module_pin_mux(uart1_pin_mux);
 }
+
+/* TODO: Remove */
+/*
+void enable_uart5_pin_mux(void)
+{
+	configure_module_pin_mux(uart5_pin_mux);
+}
+*/

board/nm/nrhw20/shield.c

@@ -68,7 +68,6 @@ U_BOOT_CMD(
 	shield,	6,	1,	do_shieldmode,
 	"Set the shield mode",
 	"dualcan termination [on|off] [on|off]\n"
-	"shield dualcan-passive\n"
 	"shield comio mode [rs232|rs485] termination [on|off]\n"
 );
 

board/nm/nrhw20/shield.h

@@ -8,11 +8,6 @@
 #ifndef SHIELD_H
 #define SHIELD_H
 
-#define SHIELD_COM_IO		0
-#define SHIELD_DUALCAN		1
-#define SHIELD_CAN_GNSS		2
-#define SHIELD_DUALCAN_PASSIVE	3
-
 struct shield_t{
     char name[64];
     int (*setmode)(char * const argv[], int argc);

board/nm/nrhw20/shield_can.c

@@ -11,8 +11,8 @@
 
 #define NETBIRD_GPIO_RST_SHIELD_N	GPIO_TO_PIN(0, 27)
 #define NETBIRD_GPIO_LATCH		GPIO_TO_PIN(0, 7)
-#define NETBIRD_GPIO_MODE_0		GPIO_TO_PIN(1, 10)
-#define NETBIRD_GPIO_MODE_1		GPIO_TO_PIN(1, 8)
+#define NETBIRD_GPIO_MODE_0		GPIO_TO_PIN(1, 8)
+#define NETBIRD_GPIO_MODE_1		GPIO_TO_PIN(1, 10)
 
 
 static int shield_slot_initialized = 0;
@@ -184,18 +184,8 @@ static int set_shieldmode(char * const argv[], int argc)
 	return configure_shieldmode(get_mode_from_args(argv, argc));
 }
 
-static int no_options(char * const argv[], int argc)
-{
-	if (argc != 0) {
-		debug("Too many arguments\n");
-		return -1;
-	}
-
-	return 0;
-}
-
-
-static struct shield_t can_shield = {
+/* TODO: Static ? */
+struct shield_t can_shield = {
 	"dualcan", set_shieldmode
 };
 
@@ -203,12 +193,3 @@ void can_shield_init(void)
 {
 	shield_register(&can_shield);
 }
-
-static struct shield_t can_shield_passive = {
-	"dualcan-passive", no_options
-};
-
-void can_shield_passive_init(void)
-{
-	shield_register(&can_shield_passive);
-}

board/nm/nrhw20/shield_can.h

@@ -5,6 +5,5 @@ int shield_can_init(void);
 int shield_can_setmode(int mode);
 
 void can_shield_init(void);
-void can_shield_passive_init(void);
 
 #endif // SHIELD_CAN_H

board/nm/nrhw24/board.c

@@ -3,7 +3,7 @@
  *
  * Board functions for Netmodule NRHW 24, based on AM335x EVB
  *
- * Copyright (C) 2018-2020 NetModule AG - http://www.netmodule.com/
+ * Copyright (C) 2018-2019 NetModule AG - http://www.netmodule.com/
  * Copyright (C) 2011, Texas Instruments, Incorporated - http://www.ti.com/
  *
  * SPDX-License-Identifier:	GPL-2.0+
@@ -43,7 +43,6 @@
 
 /* TODO: place in proper header file */
 extern void serial_set_console_index(int index);
-extern int console_init_f(void);
 
 
 DECLARE_GLOBAL_DATA_PTR;
@@ -53,7 +52,6 @@ DECLARE_GLOBAL_DATA_PTR;
  *
  * (A17) GPIO0_2: RST_GNSS~
  * (A16) GPIO0_5: EXTINT_GNSS
- * (C15) GPIO0_6: TIMEPULSE_GNSS
  *
  * (J18) GPIO0_16: RST_PHY~
  * (U12) GPIO0_27: RST_SHIELD~
@@ -116,6 +114,10 @@ static struct ctrl_dev *cdev = (struct ctrl_dev *)CTRL_DEVICE_BASE;
 #define PD_ADDRESS			(0x0200)  /* Product descriptor */
 #define PARTITION_ADDRESS		(0x0600)  /* Partition Table */
 
+#define SHIELD_COM_IO	0
+#define SHIELD_DUALCAN	1
+
+
 static BD_Context   bdctx[3];		/* The descriptor contexts */
 
 
@@ -150,7 +152,6 @@ err_free_gpio:
 
 static void init_leds(void)
 {
-	/* No init code required */
 }
 
 static void set_status_led(int red, int green)
@@ -262,8 +263,10 @@ struct serial_device *default_serial_console(void)
 
 #ifndef CONFIG_SKIP_LOWLEVEL_INIT
 
+// TODO: Double check !
 static const struct ddr_data ddr3_data = {
 	/* Ratios were optimized by DDR3 training software from TI */
+	/* TODO: Evaluate new values and update */
 	.datardsratio0 = 0x39, /* 0x39 */
 	.datawdsratio0 = 0x3f, /* 0x40 */ /* 3f */
 	.datafwsratio0 = 0x98, /* 0x96 */ /* 98 */
@@ -345,7 +348,7 @@ struct reset_registers {
 	uint32_t value_crc;
 };
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 
 /* TODO: Move ethernet crc to dedicated file */
 static uint32_t ether_crc(size_t len, uint8_t const *p)
@@ -411,15 +414,14 @@ void am33xx_spl_board_init(void)
 	init_pmic_spl();
 
 	init_leds();
-#ifndef CONFIG_NRSW_BUILD
 	set_status_led(1, 0); /* Red */
 	set_indicator_led(1, 0); /* Red */
-#endif
 
 	/* Set MPU Frequency to what we detected now that voltages are set */
 	do_setup_dpll(&dpll_mpu_regs, &dpll_mpu_opp100);
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
+	/* TODO: Move to board_late_init? It is not urgent to have this in SPL. */
 	check_pmic_reset_reason(RESET_REASON_SHM_LOCATION);
 #endif
 
@@ -549,50 +551,20 @@ static void init_gsm(void)
 	 *    4.2.8 PWR_ON pin
 	 *    4.2.9 RESET_N pin
 	 * 
-	 * Functionality Yocto:
+	 * Functionality:
 	 *  - Leave GSM power enable as is (default at power up = off)
 	 *  - Set reset line inactive (note: inverter logic in HW present)
 	 *  - Leave button unpressed (note: inverter logic in HW present)
 	 *  - Modem shall be enabled by Linux system by enabling GSM power 
 	 *    supply
 	 */
-#ifdef CONFIG_NRSW_BUILD
-	int bus;
 
 	puts("GSM:   ");
 
-	bus = da9063_claim_i2c_bus();
-
-	/* TODO: Keep Power-On and use GSM Modem Reset Signal to restart */
-
-	REQUEST_AND_SET_GPIO(GPIO_RST_GSM);		/* Assert reset (active high) */
-	REQUEST_AND_CLEAR_GPIO(GPIO_PWR_GSM);		/* Keep power switch inactive (released) */
-
-	da9063_set_gpio(PMIC_GSM_SUPPLY_EN_IO, 0);	/* Switch GSM Supply off */
-	mdelay(30+100);					/* Give time to discharge supply */
-							/* Keep of for 100ms, #3.3.2 */
-
-	da9063_set_gpio(PMIC_GSM_SUPPLY_EN_IO, 1);	/* Enable GSM supply */
-	mdelay(10);
-
-	gpio_set_value(GPIO_RST_GSM, 0);		/* Take modem out of reset */
-	mdelay(300);					/* Wait for power to stabilizy, #3.4.2 */
-
-	gpio_set_value(GPIO_PWR_GSM, 1);		/* Generate power on event, #3.4.2 */
-	mdelay(1200);
-	gpio_set_value(GPIO_PWR_GSM, 0);
-
-	da9063_release_i2c_bus(bus);
-
-	puts("ready\n");
-#else
-	puts("GSM:   ");
-
 	REQUEST_AND_CLEAR_GPIO(GPIO_RST_GSM);		/* Set reset inactive (active high) */
 	REQUEST_AND_CLEAR_GPIO(GPIO_PWR_GSM);		/* Set power switch inactive/released (active high) */
 
 	puts("init\n");
-#endif
 }
 
 static void init_gnss(void)
@@ -688,7 +660,7 @@ void set_console(void)
 		 */
 		setenv("defaultconsole", "ttyS0");
 
-		if (read_file("/root/boot/consoledev", buf, sizeof(buf)) > 3) {
+		if (read_file("/root/boot/consoledev",buf, sizeof(buf)) > 3) {
 			if (strstr(buf, "tty") == buf) {
 				int i;
 				/* TODO: What is this code doing? */
@@ -743,7 +715,7 @@ static void get_variant_name(void)
 
 static void get_hw_version(void)
 {
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 	char hw_versions[16];
 	char new_env[256];	/* current bootargs = 84 bytes */
 #endif
@@ -753,7 +725,7 @@ static void get_hw_version(void)
 
 	printf("HW24:  V%d.%d\n", hw_ver, hw_rev);
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 	/* add hardware versions to environment */
 	snprintf(hw_versions, sizeof(hw_versions), "CP=%d.%d", hw_ver, hw_rev);
 	snprintf(new_env, sizeof(new_env), "setenv bootargs $bootargs %s", hw_versions);
@@ -788,10 +760,7 @@ static void get_pmic_version(void)
 static void check_jtag_boot(void)
 {
 	if (is_jtag_boot(CONFIG_JTAG_MARKER_UBOOT)) {
-		char *bootcmd = getenv("bootcmd");
 		setenv ("bootcmd", "");
-		/* Save original bootcmd in "bootcmd_orig" to allow manual boot */
-		setenv ("bootcmd_orig", bootcmd);
 		puts("Detected JTAG boot. Waiting on command line\n");
 	}
 }
@@ -839,12 +808,6 @@ static struct shield_command known_shield_commands[] = {
 		"shield dualcan termination off off",
 		can_shield_init
 	},
-	{
-		SHIELD_DUALCAN_PASSIVE,
-		"dualcan-passive",
-		"shield dualcan-passive",
-		can_shield_passive_init
-	}
 };
 
 static const struct shield_command* get_shield_command(int shield_id)
@@ -1185,55 +1148,33 @@ static void ft_enable_node(void* blob, const char* name)
 static void ft_set_gpio_name(void *blob, const char* gpio, int pin, const char* name)
 {
 	int node_ofs = fdt_path_offset(blob, gpio);
-	int gpios = -1;
-	const char* text;
-	int pos = 0;
-	int i;
-	char buffer[512];
+	if (node_ofs != -1) {
+		const struct fdt_property* prop = fdt_get_property(blob, node_ofs, "gpio-line-names", NULL);
+		if (prop != NULL) {
+			const char* text;
+			int pos = 0;
+			int i;
+			char buffer[512];
 
-	if (node_ofs == -1) {
-		printf("Can't find node %s\n", gpio);
-		goto end;
-	}
+			/* TODO: Determine number of entries from ngpios field */
 
-	/* get number of IOs in node */
-	gpios = fdt_getprop_u32_default_node(blob, node_ofs, 0, "ngpios", -1);
-	if (gpios == -1 || gpios > 64) {
-		printf("Illegal number of gpios %d\n", gpios);
-		goto end;
-	}
+			for (i=0; i<32; i++) {
+				if (i == pin) {
+					/* Take provided name if GPIO pin is matched */
+					text = name;
+				}
+				else {
+					/* Take existing name from strin list */
+					(void)fdt_get_string_index(blob, node_ofs, "gpio-line-names", i, &text);
+				}
+				/* Add name to new string list */
+				strncpy(buffer+pos, text, sizeof(buffer)-pos);
+				pos += strlen(text) + 1;
+			}
 
-	/* get string array with names */
-	const struct fdt_property* prop = fdt_get_property(blob, node_ofs, "gpio-line-names", NULL);
-	if (prop == NULL) {
-		goto end;
-	}
-
-	/* modify given name */
-	for (i=0; i<gpios; i++) {
-		if (i == pin) {
-			/* Take provided name if GPIO pin is matched */
-			text = name;
-		}
-		else {
-			/* Take existing name from string list */
-			(void)fdt_get_string_index(blob, node_ofs, "gpio-line-names", i, &text);
-		}
-
-		/* Add name to new string list */
-		if ((pos + strlen(text) + 1) < sizeof(buffer)) {
-			strncpy(buffer+pos, text, sizeof(buffer)-pos);
-			pos += strlen(text) + 1;
-		}
-		else {
-			printf("ft_set_gpio_name() Buffer too small\n");
-			goto end;
+			(void)fdt_setprop(blob, node_ofs, "gpio-line-names", buffer, pos);
 		}
 	}
-
-	(void)fdt_setprop(blob, node_ofs, "gpio-line-names", buffer, pos);
-
-end: ;
 }
 
 /*
@@ -1242,10 +1183,10 @@ end: ;
 static void ft_comio_gpios(void *blob)
 {
 	/* gpio0_7: COM/IO relay output */
-	ft_set_gpio_name(blob, "gpio0", 7, "COMIO_OUT0");
+	ft_set_gpio_name(blob, "/ocp/gpio@44e07000", 7, "COMIO_OUT0");
 
 	/* gpio1_8: COM/IO digital input */
-	ft_set_gpio_name(blob, "gpio1", 8, "COMIO_IN0");
+	ft_set_gpio_name(blob, "/ocp/gpio@4804c000", 8, "COMIO_IN0");
 }
 
 static void ft_shields(void* blob)
@@ -1257,21 +1198,18 @@ static void ft_shields(void* blob)
 		case SHIELD_COM_IO:
 			ft_comio_gpios(blob);
 			ft_enable_node(blob, "/netbox_dio_comio");
-			ft_enable_node(blob, "serial0");
+			/* TODO: Should use alias serial0 */
+			ft_enable_node(blob, "/ocp/serial@44e09000");
 			break;
 
 		case SHIELD_DUALCAN:
-		case SHIELD_DUALCAN_PASSIVE:
-			ft_enable_node(blob, "d-can0");
-			ft_enable_node(blob, "d-can1");
+			/* TODO: Should use alias d_can0, d_can1 */
+			ft_enable_node(blob, "/ocp/can@481cc000");
+			ft_enable_node(blob, "/ocp/can@481d0000");
 			break;
 
 		default:
-			/* 
-			 * Enable uart1 (ttyS0) always as kernel needs it as fallback console, 
-			 * if (ttyS1) is not available as console.
-			 */
-			ft_enable_node(blob, "serial0");
+			ft_enable_node(blob, "/netbox_dio_default");
 			break;
 	};
 
@@ -1302,36 +1240,6 @@ static void ft_hw_info(void *blob)
 	}
 }
 
-static void ft_override_thermal(void *blob)
-{
-	const char* temp_alert0 = getenv("temp_alert0");
-	if (temp_alert0 != NULL) {
-		int node_ofs = -1;
-		int temp_in_degs = 0;
-
-		temp_in_degs = simple_strtoul(temp_alert0, NULL, 10);
-		if (temp_in_degs == 0) {
-			temp_in_degs = 95;
-		} else if (temp_in_degs < 20) {
-			temp_in_degs = 20;
-		} else if (temp_in_degs > 120) {
-			temp_in_degs = 120;
-		}
-
-		printf("WARNING: Overriding CPU thermal alert to %d°C, critical to 125°C\n", temp_in_degs);
-
-		node_ofs = fdt_path_offset(blob, "/thermal-zones/cpu-thermal/trips/cpu-alert0");
-		if (node_ofs >= 0) {
-			fdt_setprop_inplace_u32(blob, node_ofs, "temperature", temp_in_degs*1000);
-		}
-
-		node_ofs = fdt_path_offset(blob, "/thermal-zones/cpu-thermal/trips/cpu-crit");
-		if (node_ofs >= 0) {
-			fdt_setprop_inplace_u32(blob, node_ofs, "temperature", 125*1000);
-		}
-	}
-}
-
 int ft_board_setup(void *blob, bd_t *bd)
 {
 	ft_bootloader_version(blob);
@@ -1339,8 +1247,14 @@ int ft_board_setup(void *blob, bd_t *bd)
 
 	ft_shields(blob);
 
-
-	ft_override_thermal(blob);
+	/* 
+	 * Enable uart1 (ttyS0) always as kernel needs it as fallback console, 
+	 * if (ttyS1) is not available as console.
+	 */
+	/* TODO: This does not work, as pins are then blocked for CAN
+	 * Double check with NRSW
+	 */
+	/* ft_enable_node(blob, "/ocp/serial@44e09000"); */
 
 	return 0;
 }

board/nm/nrhw24/shield.c

@@ -68,7 +68,6 @@ U_BOOT_CMD(
 	shield,	6,	1,	do_shieldmode,
 	"Set the shield mode",
 	"dualcan termination [on|off] [on|off]\n"
-	"shield dualcan-passive\n"
 	"shield comio mode [rs232|rs485] termination [on|off]\n"
 );
 

board/nm/nrhw24/shield.h

@@ -8,11 +8,6 @@
 #ifndef SHIELD_H
 #define SHIELD_H
 
-#define SHIELD_COM_IO		0
-#define SHIELD_DUALCAN		1
-#define SHIELD_CAN_GNSS		2
-#define SHIELD_DUALCAN_PASSIVE	3
-
 struct shield_t{
     char name[64];
     int (*setmode)(char * const argv[], int argc);

board/nm/nrhw24/shield_can.c

@@ -11,8 +11,8 @@
 
 #define NETBIRD_GPIO_RST_SHIELD_N	GPIO_TO_PIN(0, 27)
 #define NETBIRD_GPIO_LATCH		GPIO_TO_PIN(0, 7)
-#define NETBIRD_GPIO_MODE_0		GPIO_TO_PIN(1, 10)
-#define NETBIRD_GPIO_MODE_1		GPIO_TO_PIN(1, 8)
+#define NETBIRD_GPIO_MODE_0		GPIO_TO_PIN(1, 8)
+#define NETBIRD_GPIO_MODE_1		GPIO_TO_PIN(1, 10)
 
 
 static int shield_slot_initialized = 0;
@@ -184,18 +184,8 @@ static int set_shieldmode(char * const argv[], int argc)
 	return configure_shieldmode(get_mode_from_args(argv, argc));
 }
 
-static int no_options(char * const argv[], int argc)
-{
-	if (argc != 0) {
-		debug("Too many arguments\n");
-		return -1;
-	}
-
-	return 0;
-}
-
-
-static struct shield_t can_shield = {
+/* TODO: Static ? */
+struct shield_t can_shield = {
 	"dualcan", set_shieldmode
 };
 
@@ -203,12 +193,3 @@ void can_shield_init(void)
 {
 	shield_register(&can_shield);
 }
-
-static struct shield_t can_shield_passive = {
-	"dualcan-passive", no_options
-};
-
-void can_shield_passive_init(void)
-{
-	shield_register(&can_shield_passive);
-}

board/nm/nrhw24/shield_can.h

@@ -5,6 +5,5 @@ int shield_can_init(void);
 int shield_can_setmode(int mode);
 
 void can_shield_init(void);
-void can_shield_passive_init(void);
 
 #endif // SHIELD_CAN_H

common/main.c

@@ -113,12 +113,19 @@ int login (void)
 				}
 			}
 			else {
-				/* SHA256 */
+				/* SHA1 */
 				char *cp = sha_crypt(password, stored_pw_hash);	/* TODO: Salt = PW? */
 				res = memcmp(cp, stored_pw_hash, actread);
 				free(cp);
 				if (res == 0)
 					break;
+/*
+				if (memcmp(cp, stored_pw_hash, actread) == 0) {
+					free(cp);
+					break;
+				}
+				free(cp);
+*/
 			}
 
 			/* TODO: exponentional delay */

common/spl/spl.c

@@ -458,11 +458,9 @@ void preloader_console_init(void)
 	gd->have_console = 1;
 
 #ifdef CONFIG_NRSW_BUILD
-	puts("\n" SPL_VERSION "\n");
-#else
-	puts("\n");
+	puts("\n" SPL_VERSION);
 #endif
-	puts("U-Boot SPL " PLAIN_VERSION " (" U_BOOT_DATE " - " \
+	puts("\nU-Boot SPL " PLAIN_VERSION " (" U_BOOT_DATE " - " \
 			U_BOOT_TIME ")\n");
 #ifdef CONFIG_SPL_DISPLAY_PRINT
 	spl_display_print();

configs/am335x_hw25_defconfig

@@ -1,49 +0,0 @@
-CONFIG_ARM=y
-CONFIG_TARGET_AM335X_HW25=y
-CONFIG_SPL_STACK_R_ADDR=0x82000000
-CONFIG_SPL=y
-CONFIG_SPL_STACK_R=y
-CONFIG_FIT=y
-CONFIG_SYS_EXTRA_OPTIONS="EMMC_BOOT"
-CONFIG_HUSH_PARSER=y
-CONFIG_AUTOBOOT_KEYED=y
-CONFIG_AUTOBOOT_PROMPT="Press s to abort autoboot in %d seconds\n"
-CONFIG_AUTOBOOT_STOP_STR="s"
-CONFIG_CMD_BOOTZ=y
-# CONFIG_CMD_IMLS is not set
-CONFIG_CMD_ASKENV=y
-# CONFIG_CMD_FLASH is not set
-CONFIG_CMD_MMC=y
-CONFIG_CMD_I2C=y
-CONFIG_CMD_USB=y
-# CONFIG_CMD_SETEXPR is not set
-CONFIG_CMD_DHCP=y
-CONFIG_CMD_MII=y
-CONFIG_CMD_PING=y
-CONFIG_CMD_EXT2=y
-CONFIG_CMD_EXT4=y
-CONFIG_CMD_EXT4_WRITE=y
-CONFIG_CMD_FAT=y
-CONFIG_CMD_FS_GENERIC=y
-CONFIG_DFU_TFTP=y
-CONFIG_SYS_NS16550=y
-CONFIG_USB=y
-CONFIG_USB_MUSB_HOST=y
-CONFIG_USB_MUSB_GADGET=y
-CONFIG_USB_GADGET=y
-CONFIG_USB_GADGET_DOWNLOAD=y
-CONFIG_G_DNL_MANUFACTURER="Texas Instruments"
-CONFIG_G_DNL_VENDOR_NUM=0x0451
-CONFIG_G_DNL_PRODUCT_NUM=0xd022
-CONFIG_OF_LIBFDT=y
-# CONFIG_BOOTP_PXE_CLIENTARCH is not set
-# CONFIG_CMD_PXE is not set
-# CONFIG_CMD_BOOTEFI is not set
-# CONFIG_CMD_XIMG is not set
-# CONFIG_CMD_ELF is not set
-# CONFIG_FPGA is not set
-# CONFIG_CMD_FPGA is not set
-# CONFIG_CMD_PMIC is not set
-# CONFIG_EFI_LOADER is not set
-# CONFIG_CMD_LOADB is not set
-# CONFIG_CMD_LOADS is not set

drivers/i2c/omap24xx_i2c.c

@@ -420,10 +420,8 @@ static int omap24_i2c_read(struct i2c_adapter *adap, uchar chip, uint addr,
 			}
 			if (status == 0 || (status & I2C_STAT_NACK)) {
 				i2c_error = 1;
-#ifdef CONFIG_NRSW_BUILD
-				printf("i2c_read: error waiting for addr ACK (status=0x%x)\n",
-				       status);
-#endif
+				/*printf("i2c_read: error waiting for addr ACK (status=0x%x)\n",
+				       status);*/
 				goto rd_exit;
 			}
 			if (alen) {
@@ -738,7 +736,7 @@ U_BOOT_I2C_ADAP_COMPLETE(omap24_1, omap24_i2c_init, omap24_i2c_probe,
 #endif
 
 U_BOOT_I2C_ADAP_COMPLETE(omap24_2, omap24_i2c_init, omap24_i2c_probe,
-			 omap24_i2c_read, omap24_i2c_write, omap24_i2c_setspeed,
+			 omap24_i2c_read, omap24_i2c_write, NULL,
 			 CONFIG_SYS_OMAP24_I2C_SPEED2,
 			 CONFIG_SYS_OMAP24_I2C_SLAVE2,
 			 2)
@@ -751,7 +749,7 @@ U_BOOT_I2C_ADAP_COMPLETE(omap24_2, omap24_i2c_init, omap24_i2c_probe,
 #endif
 
 U_BOOT_I2C_ADAP_COMPLETE(omap24_3, omap24_i2c_init, omap24_i2c_probe,
-			 omap24_i2c_read, omap24_i2c_write, omap24_i2c_setspeed,
+			 omap24_i2c_read, omap24_i2c_write, NULL,
 			 CONFIG_SYS_OMAP24_I2C_SPEED3,
 			 CONFIG_SYS_OMAP24_I2C_SLAVE3,
 			 3)
@@ -764,7 +762,7 @@ U_BOOT_I2C_ADAP_COMPLETE(omap24_3, omap24_i2c_init, omap24_i2c_probe,
 #endif
 
 U_BOOT_I2C_ADAP_COMPLETE(omap24_4, omap24_i2c_init, omap24_i2c_probe,
-			 omap24_i2c_read, omap24_i2c_write, omap24_i2c_setspeed,
+			 omap24_i2c_read, omap24_i2c_write, NULL,
 			 CONFIG_SYS_OMAP24_I2C_SPEED4,
 			 CONFIG_SYS_OMAP24_I2C_SLAVE4,
 			 4)

drivers/serial/serial-uclass.c

@@ -33,6 +33,9 @@ static int console_index = CONFIG_CONS_INDEX - 1;
 static int console_index = 0;
 #endif
 
+/* TODO: Rename CONSOLE_INDEX to console_index, to make clear it
+ * is a variable now
+ */
 void serial_set_console_index(int index)
 {
 	console_index = index;

include/configs/am335x_hw25.h

@@ -1,320 +0,0 @@
-/*
- * am335x_hw25.h
- *
- * Copyright (C) 2018-2019 NetModule AG - http://www.netmodule.com/
- * Copyright (C) 2011 Texas Instruments Incorporated - http://www.ti.com/
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation version 2.
- *
- * This program is distributed "as is" WITHOUT ANY WARRANTY of any
- * kind, whether express or implied; without even the implied warranty
- * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#ifndef __CONFIG_AM335X_HW25_H
-#define __CONFIG_AM335X_HW25_H
-
-#include <configs/ti_am335x_common.h>
-
-/* Disable U-Boot load from filesystems, to save around 10 kB SPL image size */
-#ifdef CONFIG_SYS_MMCSD_FS_BOOT_PARTITION
-# undef CONFIG_SYS_MMCSD_FS_BOOT_PARTITION
-#endif
-
-#undef CONFIG_SPL_AM33XX_ENABLE_RTC32K_OSC
-
-#undef CONFIG_HW_WATCHDOG
-#undef CONFIG_OMPAP_WATCHDOG
-#undef CONFIG_SPL_WATCHDOG_SUPPORT
-
-#ifndef CONFIG_SPL_BUILD
-# define CONFIG_TIMESTAMP
-# define CONFIG_LZO
-#endif
-
-#define CONFIG_SYS_BOOTM_LEN		(16 << 20)
-
-#define MACH_TYPE_TIAM335EVM		3589	/* Until the next sync */
-#define CONFIG_MACH_TYPE		MACH_TYPE_TIAM335EVM
-#define CONFIG_BOARD_LATE_INIT
-
-/* TODO: It could be preconsole buffer is not properly working in SPL
- * Observed lock ups when printing too much text.
-#define CONFIG_PRE_CONSOLE_BUFFER
-#define CONFIG_PRE_CON_BUF_ADDR   	0x80000000
-#define CONFIG_PRE_CON_BUF_SZ     	64*1024
-*/
-
-/* Clock Defines */
-#define V_OSCK				0 /* 0 means detect from sysboot1 config */
-#define V_SCLK				(V_OSCK)
-
-#include <config_distro_bootcmd.h>
-
-/* Dynamic override for PHY_ANEG_TIMEOUT value */
-#ifndef CONFIG_SPL_BUILD
-# ifndef __ASSEMBLER__
-int eth_phy_timeout(void);
-# endif
-#endif
-#define PHY_ANEG_TIMEOUT		eth_phy_timeout()
-#define PHY_ANEG_DEFAULT_TIMEOUT	5000
-
-#define CONFIG_ARP_TIMEOUT		200
-#undef CONFIG_NET_RETRY_COUNT
-#define CONFIG_NET_RETRY_COUNT		5
-#define CONFIG_BOOTP_MAY_FAIL
-
-#ifndef CONFIG_SPL_BUILD
-
-/*
- * Memory map for booting Linux
- *
- * 0x80000000   32MB	KERNEL_ADDR (kernel_addr), kernel execution address
- * 0x82000000  190MB    KERNEL_ADDR_R (kernel_addr_r), FIT image/kernel loading address
- * 			 kernel will be relocated kernel_addr
- *                       for FIT images, ramdisc and dtb will be relocated to 
- *                       top of bootmemory (0x8e000000 downwards)
- * 0x8BE00000    1MB	FDT_ADDR_R (fdt_addr_r), device tree if separate from kernel/FIT
- * 0x8BF00000	 1MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
- * 0x8C000000   32MB	LOAD_ADDR (load_addr), loading address for generic files
- * <end of boot memory>
- * 0x8E000000    4B     NRSW reset reason
- *              32MB    <>, Free space
- * 0x90000000  256MB	<>, Free space, 512MB systems
- * 0xA0000000  512MB	<>, Free space, 1GB systems only
- * 0xC0000000		End of RAM
- */
-
-#define KERNEL_ADDR	"0x80000000"
-#define KERNEL_ADDR_R	"0x82000000"
-#define FDT_ADDR_R	"0x8BE00000"
-#define PXE_ADDR	"0x8BF00000"
-#define LOAD_ADDR	"0x8C000000"
-
-/*
- * Limit boot memory to 256 MBytes to comply with kernel initial memory layout
- * This is the official way to restrict image load addresses. 
- * Don't use xx_high_addr variables.
- */
-#define BOOTM_SIZE      "0x0E000000"	
-
-/* Set boot command depending of software environment */
-#ifndef CONFIG_NRSW_BUILD
-/* Yocto/OSTree boot command */
-#define MAIN_BOOTCMD "run boot_ostree"
-#else
-/* NRSW boot command */
-#define MAIN_BOOTCMD "run sdboot"
-#endif
-
-/* TODO: Might need to check ti_cpsw.rx_packet_max when running on top of AM335x switch */
-
-#define CONFIG_EXTRA_ENV_SETTINGS \
-	/* Memory Adresses */ \
-	"fdt_addr_r=" FDT_ADDR_R "\0" \
-	"kernel_addr=" KERNEL_ADDR "\0" /* NRSW only */ \
-	"kernel_addr_r=" KERNEL_ADDR_R  "\0" \
-	"load_addr=" LOAD_ADDR "\0" \
-	"pxefile_addr_r=" PXE_ADDR "\0" \
-	"bootm_size=" BOOTM_SIZE "\0" \
-	\
-	/* Misc */ \
-	"defaultconsole=ttyS0\0" \
-	"fdt_skip_update=yes\0" \
-	"bootdelay=0\0" \
-	\
-	/* Networking */ \
-	"ethprime=cpsw\0" \
-	"ethopts=ti_cpsw.rx_packet_max=1526\0" \
-	"ipaddr=192.168.1.1\0" \
-	"serverip=192.168.1.254\0" \
-	"tftptimeout=2000\0" \
-	"tftptimeoutcountmax=5\0" \
-	"bootpretryperiod=10000\0" \
-	"autoload=false\0" \
-	\
-	/* OSTree boot */ \
-	"bootcmd_otenv=ext4load mmc 1:1 $load_addr /boot/loader/uEnv.txt; " \
-			"setenv bootargs_prev $bootargs; " \
-			"env import -t $load_addr $filesize; setenv bootargs $bootargs_prev $bootargs root=/dev/ram0 console=$defaultconsole,115200 " \
-			"$ethopts rw ostree_root=/dev/mmcblk1p1\0" \
-	"bootcmd_rd_in_mmc=ext4load mmc 1:1 $kernel_addr_r /boot$kernel_image; " \
-			"bootm $kernel_addr_r\0" \
-	"boot_ostree=run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
-	\
-	/* NRSW boot */ \
-	"root_part=1\0" /* from NRSW, required here? set from board.c */ \
-	"kernel_image=kernel.bin\0" \
-	"fdt_image=am335x-hw25-prod1.dtb\0" \
-	"add_sd_bootargs=setenv bootargs $bootargs root=/dev/${mmc_dev}p$root_part rootfstype=ext4 " \
-			"console=$defaultconsole,115200 rootwait loglevel=4 ti_cpsw.rx_packet_max=1526\0" \
-	"add_version_bootargs=setenv bootargs $bootargs\0" \
-	"sdbringup=echo Try bringup boot && ext4load mmc 1:$root_part $kernel_addr /boot/zImage && " \
-			"ext4load mmc 1:$root_part $fdt_addr /boot/$fdt_image && setenv bootargs $bootargs rw;\0" \
-	"sdprod=ext4load mmc 1:$root_part $kernel_addr /boot/$kernel_image && " \
-			"ext4load mmc 1:$root_part $fdt_addr /boot/$fdt_image && setenv bootargs $bootargs ro;\0" \
-	"sdboot=env set fdt_addr " FDT_ADDR_R "; "\
-			"if mmc dev 1; then echo Copying Linux from SD to RAM...; "\
-			"if test -e mmc 1:$root_part /boot/$kernel_image; then run sdprod; " \
-			"else run sdbringup; fi; " \
-			/* For v4.19 kernel $mmc_dev should be "mmcblk1" (read from DT), for v3.18 kernel: "mmcblk0" */ \
-			"fdt addr $fdt_addr;if fdt get value mmc_dev /nm_env nm,mmc-dev;then;else setenv mmc_dev mmcblk0;fi;" \
-			"run add_sd_bootargs; run add_version_bootargs; " \
-			"bootz $kernel_addr - $fdt_addr; fi\0" \
-	\
-	/* Boot command */ \
-	"bootcmd=" MAIN_BOOTCMD "\0" \
-	\
-	/* Recovery boot (same for OSTree and NRSW) */ \
-	"recovery=run pxe_recovery || setenv ipaddr $ipaddr; setenv serverip $serverip; run tftp_recovery\0" \
-	/* setenv ipaddr and serverip is necessary, because dhclient destroys the IPs internally */ \
-	"pxe_recovery=mdio up $ethprime && dhcp && pxe get && pxe boot\0" \
-	"tftp_recovery=tftpboot $kernel_addr_r recovery-image; tftpboot $fdt_addr_r recovery-dtb; " \
-			"setenv bootargs rdinit=/etc/preinit console=$defaultconsole,115200 " \
-			"debug $ethopts; " \
-			"bootz $kernel_addr_r - $fdt_addr_r\0" /* kernel_addr_r */
-
-#endif
-
-
-/* TODO: Check if ok for NRSW? */
-#define CONFIG_ZERO_BOOTDELAY_CHECK
-
-/* UART Configuration */
-#define CONFIG_SYS_NS16550_COM1			0x44e09000	/* UART0: XModem Boot, Debug UART */
-
-/* TODO: Preparation in case UART5 shall be used later */
-#if 0
-#define CONFIG_SYS_NS16550_COM2			0x48022000	/* UART1: Unused, see note below */
-#define CONFIG_SYS_NS16550_COM3			0x48024000	/* UART2: Unused, see note below */
-/* NOTE: NS16550 definitions are cumulative, need to set COM2 to have COM3 */
-#define CONFIG_SYS_NS16550_COM4			0x481A6000	/* UART3: - */
-#define CONFIG_SYS_NS16550_COM5			0x481A8000	/* UART4: - */
-#define CONFIG_SYS_NS16550_COM6			0x481AA000	/* UART5: User UART */
-#endif
-
-#define CONFIG_I2C
-#define CONFIG_I2C_MULTI_BUS
-
-#define CONFIG_CMD_EEPROM
-#define CONFIG_SYS_I2C_EEPROM_ADDR              0x50	/* Main EEPROM */
-#define CONFIG_SYS_I2C_EEPROM_ADDR_LEN          2
-#define CONFIG_SYS_I2C_SPEED                    100000
-#define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS       4
-#define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS   50	/* TODO: Can this be reduced to 20ms */
-
-/* Put Environment in eMMC */
-#define CONFIG_ENV_OFFSET			(512 * 128)	/* @ 512*256 SPL starts */
-#define CONFIG_ENV_SIZE				(4 * 1024)
-#define CONFIG_ENV_IS_IN_MMC
-#define CONFIG_SYS_MMC_ENV_DEV			1
-
-#undef CONFIG_SPL_ENV_SUPPORT
-#undef CONFIG_SPL_NAND_SUPPORT
-#undef CONFIG_SPL_ONENAND_SUPPORT
-
-/* We need to disable SPI to not confuse the eeprom env driver */
-#undef CONFIG_SPI
-#undef CONFIG_SPI_BOOT
-#undef CONFIG_SPL_OS_BOOT
-
-#define CONFIG_SPL_YMODEM_SUPPORT
-
-#define CONFIG_SPL_LDSCRIPT		"$(CPUDIR)/am33xx/u-boot-spl.lds"
-
-#define CONFIG_SUPPORT_EMMC_BOOT
-
-/*
- * USB configuration.  We enable MUSB support, both for host and for
- * gadget.  We set USB0 as peripheral and USB1 as host, based on the
- * board schematic and physical port wired to each.  Then for host we
- * add mass storage support and for gadget we add both RNDIS ethernet
- * and DFU.
- */
-#define CONFIG_USB_MUSB_DSPS
-#define CONFIG_ARCH_MISC_INIT
-#define CONFIG_USB_MUSB_PIO_ONLY
-#define CONFIG_USB_MUSB_DISABLE_BULK_COMBINE_SPLIT
-#define CONFIG_AM335X_USB0
-#define CONFIG_AM335X_USB0_MODE		MUSB_HOST
-
-/* To support eMMC booting */
-#define CONFIG_STORAGE_EMMC
-#define CONFIG_FASTBOOT_FLASH_MMC_DEV   1
-
-#ifdef CONFIG_USB_MUSB_HOST
-#define CONFIG_USB_STORAGE
-#endif
-
-#ifdef CONFIG_USB_MUSB_GADGET
-/* Removing USB gadget and can be enabled adter adding support usb DM */
-#ifndef CONFIG_DM_ETH
-#define CONFIG_USB_ETHER
-#define CONFIG_USB_ETH_RNDIS
-#define CONFIG_USBNET_HOST_ADDR	"de:ad:be:af:00:00"
-#endif /* CONFIG_DM_ETH */
-#endif /* CONFIG_USB_MUSB_GADGET */
-
-
-/*
- * Disable MMC DM for SPL build and can be re-enabled after adding
- * DM support in SPL
- */
-#ifdef CONFIG_SPL_BUILD
-#undef CONFIG_DM_MMC
-#undef CONFIG_TIMER
-#endif
-
-#if defined(CONFIG_SPL_BUILD)
-/* Remove other SPL modes. */
-#undef CONFIG_SPL_NAND_SUPPORT
-#define CONFIG_ENV_IS_NOWHERE
-#undef CONFIG_PARTITION_UUIDS
-#undef CONFIG_EFI_PARTITION
-#endif
-
-/* Network. */
-#define CONFIG_IP_DEFRAG		/* so we can use large tftp blocks */
-#define CONFIG_TFTP_TSIZE		/* tftp transfer size, progress bar */
-
-#define CONFIG_PHYLIB
-#define CONFIG_PHY_SMSC
-
-#define CONFIG_CMD_MEMTEST
-#define CONFIG_SYS_MEMTEST_START    0x84000000
-#define CONFIG_SYS_MEMTEST_END      0x87900000
-
-
-#ifdef CONFIG_NRSW_BUILD
-/* support for NM packed bootloader */
-#define CONFIG_NM_BOOTLOADER_FORMAT
-
-/* password protected login */
-#define CONFIG_CRYPT
-#define CONFIG_NM_LOGIN
-#define CONFIG_NM_LOGIN_PART	"1:3"	/* TODO: Define location of file for OSTree/Yocto */
-#define CONFIG_NM_LOGIN_PASSWD	"/root/boot/bootpass"
-#endif
-
-#define CONFIG_CMD_PXE
-
-#define CONFIG_OF_BOARD_SETUP
-
-#define CONFIG_JTAG_MARKER_SPL		0x402FFF00
-#define CONFIG_JTAG_MARKER_UBOOT	0x807FFF00
-
-/* NRSW PMIC Reset Reason */
-#define RESET_REASON_SHM_LOCATION	0x8e000000
-
-
-/* SPL command is not needed */
-#undef CONFIG_CMD_SPL
-
-/* Never enable ISO it is broken and can lead to a crash */
-#undef CONFIG_ISO_PARTITION
-
-#endif	/* ! __CONFIG_AM335X_HW25_H */

include/configs/am335x_nmhw21.h

@@ -63,112 +63,70 @@ int eth_phy_timeout(void);
 
 #ifndef CONFIG_SPL_BUILD
 
+
 /*
  * Memory map for booting Linux
  *
- * 0x80000000   32MB	KERNEL_ADDR (kernel_addr), kernel execution address
- * 0x82000000  190MB    KERNEL_ADDR_R (kernel_addr_r), FIT image/kernel loading address
- * 			 kernel will be relocated kernel_addr
- *                       for FIT images, ramdisc and dtb will be relocated to 
- *                       top of bootmemory (0x8e000000 downwards)
- * 0x8BE00000    1MB	FDT_ADDR_R (fdt_addr_r), device tree if separate from kernel/FIT
- * 0x8BF00000	 1MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
- * 0x8C000000   32MB	LOAD_ADDR (load_addr), loading address for generic files
- * <end of boot memory>
- * 0x8E000000    4B     NRSW reset reason
- *              32MB    <>, Free space
- * 0x90000000  256MB	<>, Free space, 512MB systems
+ * 0x80000000   63MB	KERNEL_ADDR (kernel_addr), kernel execution address
+ * 0x83F00000    1MB	FDT_ADDR (fdt_addr_r), device tree loading address if not included in kernel
+ * 0x84000000  126MB	RD_ADDR (ramdisk_addr_r), ramdisc loading address
+ * 0x8BE00000    2MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
+ * 0x8C000000    1MB	LOAD_ADDR (load_addr), loading address for generic files
+ * 0x8C100000   63MB	KERNEL_ADDR_R (kernel_addr_r), kernel loading address (will be relocated to kernel_addr)
+ * 0x90000000  256MB	<>, Free space 512MB systems
  * 0xA0000000  512MB	<>, Free space, 1GB systems only
  * 0xC0000000		End of RAM
  */
 
 #define KERNEL_ADDR	"0x80000000"
-#define KERNEL_ADDR_R	"0x82000000"
-#define FDT_ADDR_R	"0x8BE00000"
-#define PXE_ADDR	"0x8BF00000"
+#define FDT_ADDR	"0x83F00000"
+#define RD_ADDR		"0x84000000"
+#define PXE_ADDR	"0x8BE00000"
 #define LOAD_ADDR	"0x8C000000"
+#define KERNEL_ADDR_R	"0x8C100000"
 
 /*
- * Limit boot memory to 256 MBytes to comply with kernel initial memory layout
- * This is the official way to restrict image load addresses. 
- * Don't use xx_high_addr variables.
+ * Avoid copying ramdisc and dtb above 512MB, as it breaks Linux boot.
+ * -1 means "do not copy" to high address, use in place.
  */
-#define BOOTM_SIZE      "0x0E000000"	
-
-/* Set boot command depending of software environment */
-#ifndef CONFIG_NRSW_BUILD
-/* Yocto/OSTree boot command */
-#define MAIN_BOOTCMD "run boot_ostree"
-#else
-/* NRSW boot command */
-#define MAIN_BOOTCMD "run sdboot"
-#endif
+#define INITRD_HIGH_ADDR "0x84000000"
+#define FDT_HIGH_ADDR    "0xffffffff"
 
 #define CONFIG_EXTRA_ENV_SETTINGS \
-	/* Memory Adresses */ \
-	"fdt_addr_r=" FDT_ADDR_R "\0" \
-	"kernel_addr=" KERNEL_ADDR "\0" /* NRSW only */ \
+	"fdt_image=am335x-nmhw21-prod1.dtb\0" \
+	"fdt_addr_r=" FDT_ADDR "\0" \
+	"fdt_high=" FDT_HIGH_ADDR "\0" \
+	"initrd_high=" INITRD_HIGH_ADDR "\0" \
+	"kernel_addr=" KERNEL_ADDR "\0" \
 	"kernel_addr_r=" KERNEL_ADDR_R  "\0" \
 	"load_addr=" LOAD_ADDR "\0" \
 	"pxefile_addr_r=" PXE_ADDR "\0" \
-	"bootm_size=" BOOTM_SIZE "\0" \
-	\
-	/* Misc */ \
+	"ramdisk_addr_r=" RD_ADDR "\0" \
 	"defaultconsole=ttyS2\0" \
 	"fdt_skip_update=yes\0" \
-	"bootdelay=0\0" \
-	\
-	/* Networking */ \
 	"ethprime=cpsw\0" \
 	"ethopts=ti_cpsw.rx_packet_max=1526\0" \
-	"ipaddr=192.168.1.1\0" \
-	"serverip=192.168.1.254\0" \
-	"tftptimeout=2000\0" \
-	"tftptimeoutcountmax=5\0" \
-	"bootpretryperiod=10000\0" /* 2000 */ \
-	"autoload=false\0" \
-	\
-	/* OSTree boot */ \
 	"bootcmd_otenv=ext4load mmc 1:1 $load_addr /boot/loader/uEnv.txt; " \
 			"setenv bootargs_prev $bootargs; " \
 			"env import -t $load_addr $filesize; setenv bootargs $bootargs_prev $bootargs root=/dev/ram0 console=$defaultconsole,115200 " \
 			"$ethopts rw ostree_root=/dev/mmcblk1p1\0" \
 	"bootcmd_rd_in_mmc=ext4load mmc 1:1 $kernel_addr_r /boot$kernel_image; " \
 			"bootm $kernel_addr_r\0" \
-	"boot_ostree=run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
-	\
-	/* NRSW boot */ \
-	"root_part=1\0" /* from NRSW, required here? set from board.c */ \
-	"kernel_image=kernel.bin\0" \
-	"fdt_image=am335x-nmhw21-prod1.dtb\0" \
-	"add_sd_bootargs=setenv bootargs $bootargs root=/dev/${mmc_dev}p$root_part rootfstype=ext4 " \
-			"console=$defaultconsole,115200 rootwait loglevel=4 ti_cpsw.rx_packet_max=1526\0" \
-	"add_version_bootargs=setenv bootargs $bootargs\0" \
-	"sdbringup=echo Try bringup boot && ext4load mmc 1:$root_part $kernel_addr /boot/zImage && " \
-			"ext4load mmc 1:$root_part $fdt_addr /boot/$fdt_image && setenv bootargs $bootargs rw;\0" \
-	"sdprod=ext4load mmc 1:$root_part $kernel_addr /boot/$kernel_image && " \
-			"ext4load mmc 1:$root_part $fdt_addr /boot/$fdt_image && setenv bootargs $bootargs ro;\0" \
-	"sdboot=env set fdt_addr " FDT_ADDR_R "; "\
-			"if mmc dev 1; then echo Copying Linux from SD to RAM...; "\
-			"if test -e mmc 1:$root_part /boot/$kernel_image; then run sdprod; " \
-			"else run sdbringup; fi; " \
-			/* For v4.19 kernel $mmc_dev should be "mmcblk1" (read from DT), for v3.18 kernel: "mmcblk0" */ \
-			"fdt addr $fdt_addr;if fdt get value mmc_dev /nm_env nm,mmc-dev;then;else setenv mmc_dev mmcblk0;fi;" \
-			"run add_sd_bootargs; run add_version_bootargs; " \
-			"bootz $kernel_addr - $fdt_addr; fi\0" \
-	\
-	/* Boot command */ \
-	"bootcmd=" MAIN_BOOTCMD "\0" \
-	\
-	/* Recovery boot (same for OSTree and NRSW) */ \
-	"recovery=run pxe_recovery || setenv ipaddr $ipaddr; setenv serverip $serverip; run tftp_recovery\0" \
-	/* setenv ipaddr and serverip is necessary, because dhclient destroys the IPs internally */ \
-	"pxe_recovery=mdio up $ethprime && dhcp && pxe get && pxe boot\0" \
-	"tftp_recovery=tftpboot $kernel_addr_r recovery-image; tftpboot $fdt_addr_r recovery-dtb; " \
+	"bootcmd=run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
+	"bootdelay=0\0" \
+	"ipaddr=192.168.1.1\0" \
+	"serverip=192.168.1.254\0" \
+	"tftptimeout=2000\0" \
+	"tftptimeoutcountmax=5\0" \
+	"bootpretryperiod=10000\0" \
+	"autoload=false\0" \
+	"tftp_recovery=tftpboot $kernel_addr recovery-image; tftpboot $fdt_addr_r recovery-dtb; " \
 			"setenv bootargs rdinit=/etc/preinit console=$defaultconsole,115200 " \
-			"debug $ethopts; " \
-			"bootz $kernel_addr_r - $fdt_addr_r\0" /* kernel_addr_r */
-
+			"debug ethopts; " \
+			"bootz $kernel_addr - $fdt_addr_r\0" \
+	"pxe_recovery=mdio up $ethprime && dhcp && pxe get && pxe boot\0" \
+	"recovery=run pxe_recovery || setenv ipaddr $ipaddr; setenv serverip $serverip; run tftp_recovery\0" \
+			/* setenv ipaddr and serverip is necessary, because dhclient can destroy the IPs internally */
 #endif
 
 #define CONFIG_ZERO_BOOTDELAY_CHECK
@@ -269,9 +227,6 @@ int eth_phy_timeout(void);
 #endif
 
 /* Network. */
-#define CONFIG_IP_DEFRAG		/* so we can use large tftp blocks */
-#define CONFIG_TFTP_TSIZE		/* tftp transfer size, progress bar */
-
 #define CONFIG_PHYLIB
 #define CONFIG_PHY_SMSC
 
@@ -279,18 +234,6 @@ int eth_phy_timeout(void);
 #define CONFIG_SYS_MEMTEST_START    0x84000000
 #define CONFIG_SYS_MEMTEST_END      0x87900000
 
-
-#ifdef CONFIG_NRSW_BUILD
-/* support for NM packed bootloader */
-#define CONFIG_NM_BOOTLOADER_FORMAT
-
-/* password protected login */
-#define CONFIG_CRYPT
-#define CONFIG_NM_LOGIN
-#define CONFIG_NM_LOGIN_PART	"1:3"	/* TODO: Define location of file for OSTree/Yocto */
-#define CONFIG_NM_LOGIN_PASSWD	"/root/boot/bootpass"
-#endif
-
 #define CONFIG_CMD_PXE
 
 #define CONFIG_OF_BOARD_SETUP
@@ -298,10 +241,6 @@ int eth_phy_timeout(void);
 #define CONFIG_JTAG_MARKER_SPL		0x402FFF00
 #define CONFIG_JTAG_MARKER_UBOOT	0x807FFF00
 
-/* Reset and Start Reason */
-#define RESET_REASON_SHM_LOCATION	0x8e000000
-
-
 /* SPL command is not needed */
 #undef CONFIG_CMD_SPL
 

include/configs/am335x_nrhw20.h

@@ -19,6 +19,11 @@
 
 #include <configs/ti_am335x_common.h>
 
+/* TODO: Inject via build system */
+#define CONFIG_NRSW
+
+
+
 /* Disable U-Boot load from filesystems, to save around 10 kB SPL image size */
 #ifdef CONFIG_SYS_MMCSD_FS_BOOT_PARTITION
 # undef CONFIG_SYS_MMCSD_FS_BOOT_PARTITION
@@ -72,45 +77,48 @@ int eth_phy_timeout(void);
 
 #ifndef CONFIG_SPL_BUILD
 
+
 /*
  * Memory map for booting Linux
  *
- * 0x80000000   32MB	KERNEL_ADDR (kernel_addr), kernel execution address
- * 0x82000000  190MB    KERNEL_ADDR_R (kernel_addr_r), FIT image/kernel loading address
- * 			 kernel will be relocated kernel_addr
- *                       for FIT images, ramdisc and dtb will be relocated to 
- *                       top of bootmemory (0x8e000000 downwards)
- * 0x8BE00000    1MB	FDT_ADDR_R (fdt_addr_r), device tree if separate from kernel/FIT
- * 0x8BF00000	 1MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
- * 0x8C000000   32MB	LOAD_ADDR (load_addr), loading address for generic files
- * <end of boot memory>
+ * 0x80000000   63MB	KERNEL_ADDR (kernel_addr), kernel execution address
+ * 0x83F00000    1MB	FDT_ADDR_R (fdt_addr_r), device tree loading address if not included in kernel
+ *             126MB	INIT_RD_HIGH (initrd_high), ramdisc top address for relocation
+ * 0x8BE00000	 2MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
+ * 0x8C000000    1MB	LOAD_ADDR (load_addr), loading address for generic files
+ * 0x8C100000   31MB	KERNEL_ADDR_R (kernel_addr_r), kernel loading address (will be relocated to kernel_addr)
  * 0x8E000000    4B     NRSW reset reason
- *              32MB    <>, Free space
- * 0x90000000  256MB	<>, Free space, 512MB systems
+ * 0x90000000  256MB	<>, Free space 512MB systems
  * 0xA0000000  512MB	<>, Free space, 1GB systems only
  * 0xC0000000		End of RAM
+ * 
+ * ((0x84000000  126MB	RD_ADDR (ramdisk_addr_r), ramdisc loading address))
  */
 
 #define KERNEL_ADDR	"0x80000000"
-#define KERNEL_ADDR_R	"0x82000000"
-#define FDT_ADDR_R	"0x8BE00000"
-#define PXE_ADDR	"0x8BF00000"
+/*#define FDT_ADDR	"0x82000000"	*/ /* NRSW, trying to use FDT_ADDR_R = 0x83F00000 instead */
+#define FDT_ADDR_R	"0x83F00000"
+#define PXE_ADDR	"0x8BE00000"
 #define LOAD_ADDR	"0x8C000000"
+#define KERNEL_ADDR_R	"0x8C100000"
 
+/* TODO: Check this with 1 GByte system */
+/* Most likely ramdisk and FDT will be loaded to too high adresses and boot will fail */
+#if 0
 /*
- * Limit boot memory to 256 MBytes to comply with kernel initial memory layout
- * This is the official way to restrict image load addresses. 
- * Don't use xx_high_addr variables.
+ * Avoid copying ramdisc and dtb above 512MB, as it breaks Linux boot.
+ * -1 means "do not copy" to high address, use in place.
  */
-#define BOOTM_SIZE      "0x0E000000"	
+#define INITRD_HIGH_ADDR "0x8BE0000"
+#define RD_ADDR		"0x84000000"
+#define FDT_HIGH_ADDR	"0x87000000"
+#define FDT_HIGH_ADDR	"0xffffffff"
+
+	"fdt_high=" FDT_HIGH_ADDR "\0" /* Breaks NRSW, required by Yocto ! */ \
+	"fdt_addr=" FDT_ADDR "\0" /* NRSW only, breaks yocto, can we move that to fdt_addr_r ? */ \
+	"initrd_high=" INITRD_HIGH_ADDR "\0" /* (0x84000000) ->  INIT_RD_ADDR (0x88000000) */ \
+	"ramdisk_addr_r=" RD_ADDR "\0" \
 
-/* Set boot command depending of software environment */
-#ifndef CONFIG_NRSW_BUILD
-/* Yocto/OSTree boot command */
-#define MAIN_BOOTCMD "run boot_ostree"
-#else
-/* NRSW boot command */
-#define MAIN_BOOTCMD "run sdboot"
 #endif
 
 #define CONFIG_EXTRA_ENV_SETTINGS \
@@ -120,7 +128,6 @@ int eth_phy_timeout(void);
 	"kernel_addr_r=" KERNEL_ADDR_R  "\0" \
 	"load_addr=" LOAD_ADDR "\0" \
 	"pxefile_addr_r=" PXE_ADDR "\0" \
-	"bootm_size=" BOOTM_SIZE "\0" \
 	\
 	/* Misc */ \
 	"defaultconsole=ttyS1\0" \
@@ -140,11 +147,12 @@ int eth_phy_timeout(void);
 	/* OSTree boot */ \
 	"bootcmd_otenv=ext4load mmc 1:1 $load_addr /boot/loader/uEnv.txt; " \
 			"setenv bootargs_prev $bootargs; " \
-			"env import -t $load_addr $filesize; setenv bootargs $bootargs_prev $bootargs root=/dev/ram0 console=$defaultconsole,115200 " \
+			"env import -t $load_addr $filesize; " \
+			"setenv bootargs $bootargs_prev $bootargs root=/dev/ram0 console=$defaultconsole,115200 " \
 			"$ethopts rw ostree_root=/dev/mmcblk1p1\0" \
 	"bootcmd_rd_in_mmc=ext4load mmc 1:1 $kernel_addr_r /boot$kernel_image; " \
 			"bootm $kernel_addr_r\0" \
-	"boot_ostree=run shieldcmd; run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
+	"bootcmd=run shieldcmd; run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
 	\
 	/* NRSW boot */ \
 	"root_part=1\0" /* from NRSW, required here? set from board.c */ \
@@ -165,21 +173,85 @@ int eth_phy_timeout(void);
 			"fdt addr $fdt_addr;if fdt get value mmc_dev /nm_env nm,mmc-dev;then;else setenv mmc_dev mmcblk0;fi;" \
 			"run add_sd_bootargs; run add_version_bootargs; run shieldcmd; " \
 			"bootz $kernel_addr - $fdt_addr; fi\0" \
+	/* "bootcmd=run sdboot\0" */ \
 	\
-	/* Boot command */ \
-	"bootcmd=" MAIN_BOOTCMD "\0" \
-	\
-	/* Recovery boot (same for OSTree and NRSW) */ \
+	/* Recovery boot */ \
 	"recovery=run pxe_recovery || setenv ipaddr $ipaddr; setenv serverip $serverip; run tftp_recovery\0" \
-	/* setenv ipaddr and serverip is necessary, because dhclient destroys the IPs internally */ \
+	/* setenv ipaddr and serverip is necessary, because dhclient can destroy the IPs internally */ \
 	"pxe_recovery=mdio up $ethprime && dhcp && pxe get && pxe boot\0" \
-	"tftp_recovery=tftpboot $kernel_addr_r recovery-image; tftpboot $fdt_addr_r recovery-dtb; " \
+	"tftp_recovery=tftpboot $kernel_addr_r recovery-image; tftpboot $fdt_addr_r recovery-dtb; " /* kernel_addr_r ? */ \
 			"setenv bootargs rdinit=/etc/preinit console=$defaultconsole,115200 " \
 			"debug $ethopts; " \
 			"bootz $kernel_addr_r - $fdt_addr_r\0" /* kernel_addr_r */
 
 #endif
 
+#if 0
+
+#ifndef CONFIG_SPL_BUILD
+#define KERNEL_ADDR			"0x80000000"
+#define LOAD_ADDR			"0x83000000"
+#define FDT_ADDR			"0x82000000"
+#define PXE_ADDR			"0x82800000"
+#define FDT_HIGH_ADDR			"0x87000000"
+#define INIT_RD_ADDR			"0x88000000"
+
+#define CONFIG_EXTRA_ENV_SETTINGS \
+	"kernel_image=kernel.bin\0"	\
+	"fdt_image=openwrt-nrhw20-nb1601.dtb\0"\
+	"modeboot=sdboot\0" \
+	"fdt_addr=" FDT_ADDR "\0" \
+	"kernel_addr=" KERNEL_ADDR "\0" \
+	"load_addr=" LOAD_ADDR "\0" \
+	"root_part=1\0" /* Default root partition, overwritten in board file */ \
+	"defaultconsole=ttyS1\0" /* Default output console */ \
+	"add_sd_bootargs=setenv bootargs $bootargs root=/dev/mmcblk1p$root_part rootfstype=ext4 " \
+			"console=$defaultconsole,115200 rootwait loglevel=4 ti_cpsw.rx_packet_max=1526\0" \
+	"add_version_bootargs=setenv bootargs $bootargs\0" \
+	"fdt_skip_update=yes\0" \
+	"ethprime=cpsw\0" \
+	"sdbringup=echo Try bringup boot && ext4load mmc 1:$root_part $kernel_addr /boot/zImage && " \
+			"ext4load mmc 1:$root_part $fdt_addr /boot/$fdt_image && setenv bootargs $bootargs rw;\0" \
+	"sdprod=ext4load mmc 1:$root_part $kernel_addr /boot/$kernel_image && " \
+			"ext4load mmc 1:$root_part $fdt_addr /boot/$fdt_image && setenv bootargs $bootargs ro;\0" \
+	"sdboot=if mmc dev 1; then echo Copying Linux from SD to RAM...; "\
+			"if test -e mmc 1:$root_part /boot/$kernel_image; then run sdprod; " \
+			"else run sdbringup; fi; " \
+			"run add_sd_bootargs; run add_version_bootargs; run shieldcmd; " \
+			"bootz $kernel_addr - $fdt_addr; fi\0" \
+	"bootcmd_otenv=ext4load mmc 1:1 $load_addr /boot/loader/uEnv.txt; " \
+			"setenv bootargs_prev $bootargs; " \
+			"env import -t $load_addr $filesize; " \
+			"setenv bootargs $bootargs_prev $bootargs root=/dev/ram0 console=$defaultconsole,115200 " \
+			"$ethopts rw ostree_root=/dev/mmcblk1p1\0" \
+	"bootcmd_rd_in_mmc=ext4load mmc 1:1 $kernel_addr_r /boot$kernel_image; " \
+			"bootm $kernel_addr_r\0" \
+	"bootcmd=run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
+	"ipaddr=192.168.1.1\0" \
+	"serverip=192.168.1.254\0" \
+	"pxefile_addr_r=" PXE_ADDR "\0" \
+	"fdt_addr_r=" FDT_ADDR "\0" \
+	"fdt_high=" FDT_HIGH_ADDR "\0" \
+	"kernel_addr_r=" KERNEL_ADDR "\0" \
+	"ramdisk_addr_r=" LOAD_ADDR "\0" \
+	"initrd_high=" INIT_RD_ADDR "\0" \
+	"bootpretryperiod=1000\0" \
+	"tftptimeout=2000\0" \
+	"tftptimeoutcountmax=5\0" \
+	"bootpretryperiod=2000\0" \
+	"autoload=false\0" \
+	"shieldcmd=\0" \
+	"tftp_recovery=tftpboot $kernel_addr recovery-image; tftpboot $fdt_addr recovery-dtb; " \
+			"setenv bootargs rdinit=/etc/preinit console=$defaultconsole,115200 " \
+			"debug ti_cpsw.rx_packet_max=1526; run shieldcmd; " \
+			"bootz $kernel_addr - $fdt_addr\0" \
+	"pxe_recovery=sleep 3 && dhcp && pxe get && pxe boot\0" \
+	"recovery=run pxe_recovery || setenv ipaddr $ipaddr; setenv serverip $serverip; run tftp_recovery\0" \
+			/* setenv ipaddr and serverip is necessary, because dhclient can destroy the IPs inernally */
+#endif
+
+#endif
+
 
 /* TODO: Check if ok for NRSW? */
 #define CONFIG_ZERO_BOOTDELAY_CHECK
@@ -282,9 +354,6 @@ int eth_phy_timeout(void);
 #endif
 
 /* Network. */
-#define CONFIG_IP_DEFRAG		/* so we can use large tftp blocks */
-#define CONFIG_TFTP_TSIZE		/* tftp transfer size, progress bar */
-
 #define CONFIG_PHYLIB
 
 /* Ethernet Switch Support */
@@ -298,7 +367,7 @@ int eth_phy_timeout(void);
 #define CONFIG_SYS_MEMTEST_END      0x87900000
 
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 /* support for NM packed bootloader */
 #define CONFIG_NM_BOOTLOADER_FORMAT
 
@@ -317,7 +386,7 @@ int eth_phy_timeout(void);
 #define CONFIG_JTAG_MARKER_UBOOT	0x807FFF00
 
 /* NRSW PMIC Reset Reason */
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 #define RESET_REASON_SHM_LOCATION	0x8e000000
 #define EXTERNAL_WATCHDOG_PATTERN	0x781f9ce2
 #endif

include/configs/am335x_nrhw24.h

@@ -19,6 +19,11 @@
 
 #include <configs/ti_am335x_common.h>
 
+/* TODO: Inject via build system */
+#define CONFIG_NRSW
+
+
+
 /* Disable U-Boot load from filesystems, to save around 10 kB SPL image size */
 #ifdef CONFIG_SYS_MMCSD_FS_BOOT_PARTITION
 # undef CONFIG_SYS_MMCSD_FS_BOOT_PARTITION
@@ -70,45 +75,48 @@ int eth_phy_timeout(void);
 
 #ifndef CONFIG_SPL_BUILD
 
+
 /*
  * Memory map for booting Linux
  *
- * 0x80000000   32MB	KERNEL_ADDR (kernel_addr), kernel execution address
- * 0x82000000  190MB    KERNEL_ADDR_R (kernel_addr_r), FIT image/kernel loading address
- * 			 kernel will be relocated kernel_addr
- *                       for FIT images, ramdisc and dtb will be relocated to 
- *                       top of bootmemory (0x8e000000 downwards)
- * 0x8BE00000    1MB	FDT_ADDR_R (fdt_addr_r), device tree if separate from kernel/FIT
- * 0x8BF00000	 1MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
- * 0x8C000000   32MB	LOAD_ADDR (load_addr), loading address for generic files
- * <end of boot memory>
+ * 0x80000000   63MB	KERNEL_ADDR (kernel_addr), kernel execution address
+ * 0x83F00000    1MB	FDT_ADDR_R (fdt_addr_r), device tree loading address if not included in kernel
+ *             126MB	INIT_RD_HIGH (initrd_high), ramdisc top address for relocation
+ * 0x8BE00000	 2MB	PXE_ADDR (pxefile_addr_r), pxe configuration file (pxe get command)
+ * 0x8C000000    1MB	LOAD_ADDR (load_addr), loading address for generic files
+ * 0x8C100000   31MB	KERNEL_ADDR_R (kernel_addr_r), kernel loading address (will be relocated to kernel_addr)
  * 0x8E000000    4B     NRSW reset reason
- *              32MB    <>, Free space
- * 0x90000000  256MB	<>, Free space, 512MB systems
+ * 0x90000000  256MB	<>, Free space 512MB systems
  * 0xA0000000  512MB	<>, Free space, 1GB systems only
  * 0xC0000000		End of RAM
+ * 
+ * ((0x84000000  126MB	RD_ADDR (ramdisk_addr_r), ramdisc loading address))
  */
 
 #define KERNEL_ADDR	"0x80000000"
-#define KERNEL_ADDR_R	"0x82000000"
-#define FDT_ADDR_R	"0x8BE00000"
-#define PXE_ADDR	"0x8BF00000"
+/*#define FDT_ADDR	"0x82000000"	*/ /* NRSW, trying to use FDT_ADDR_R = 0x83F00000 instead */
+#define FDT_ADDR_R	"0x83F00000"
+#define PXE_ADDR	"0x8BE00000"
 #define LOAD_ADDR	"0x8C000000"
+#define KERNEL_ADDR_R	"0x8C100000"
 
+/* TODO: Check this with 1 GByte system */
+/* Most likely ramdisk and FDT will be loaded to too high adresses and boot will fail */
+#if 0
 /*
- * Limit boot memory to 256 MBytes to comply with kernel initial memory layout
- * This is the official way to restrict image load addresses. 
- * Don't use xx_high_addr variables.
+ * Avoid copying ramdisc and dtb above 512MB, as it breaks Linux boot.
+ * -1 means "do not copy" to high address, use in place.
  */
-#define BOOTM_SIZE      "0x0E000000"	
+#define INITRD_HIGH_ADDR "0x8BE0000"
+#define RD_ADDR		"0x84000000"
+#define FDT_HIGH_ADDR	"0x87000000"
+#define FDT_HIGH_ADDR	"0xffffffff"
+
+	"fdt_high=" FDT_HIGH_ADDR "\0" /* Breaks NRSW, required by Yocto ! */ \
+	"fdt_addr=" FDT_ADDR "\0" /* NRSW only, breaks yocto, can we move that to fdt_addr_r ? */ \
+	"initrd_high=" INITRD_HIGH_ADDR "\0" /* (0x84000000) ->  INIT_RD_ADDR (0x88000000) */ \
+	"ramdisk_addr_r=" RD_ADDR "\0" \
 
-/* Set boot command depending of software environment */
-#ifndef CONFIG_NRSW_BUILD
-/* Yocto/OSTree boot command */
-#define MAIN_BOOTCMD "run boot_ostree"
-#else
-/* NRSW boot command */
-#define MAIN_BOOTCMD "run sdboot"
 #endif
 
 #define CONFIG_EXTRA_ENV_SETTINGS \
@@ -118,7 +126,6 @@ int eth_phy_timeout(void);
 	"kernel_addr_r=" KERNEL_ADDR_R  "\0" \
 	"load_addr=" LOAD_ADDR "\0" \
 	"pxefile_addr_r=" PXE_ADDR "\0" \
-	"bootm_size=" BOOTM_SIZE "\0" \
 	\
 	/* Misc */ \
 	"defaultconsole=ttyS1\0" \
@@ -142,7 +149,7 @@ int eth_phy_timeout(void);
 			"$ethopts rw ostree_root=/dev/mmcblk1p1\0" \
 	"bootcmd_rd_in_mmc=ext4load mmc 1:1 $kernel_addr_r /boot$kernel_image; " \
 			"bootm $kernel_addr_r\0" \
-	"boot_ostree=run shieldcmd; run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
+	"bootcmd=run shieldcmd; run bootcmd_otenv; run bootcmd_rd_in_mmc\0" \
 	\
 	/* NRSW boot */ \
 	"root_part=1\0" /* from NRSW, required here? set from board.c */ \
@@ -163,18 +170,15 @@ int eth_phy_timeout(void);
 			"fdt addr $fdt_addr;if fdt get value mmc_dev /nm_env nm,mmc-dev;then;else setenv mmc_dev mmcblk0;fi;" \
 			"run add_sd_bootargs; run add_version_bootargs; run shieldcmd; " \
 			"bootz $kernel_addr - $fdt_addr; fi\0" \
+	/* "bootcmd=run sdboot\0" */ \
 	\
-	/* Boot command */ \
-	"bootcmd=" MAIN_BOOTCMD "\0" \
-	\
-	/* Recovery boot (same for OSTree and NRSW) */ \
+	/* Recovery boot */ \
 	"recovery=run pxe_recovery || setenv ipaddr $ipaddr; setenv serverip $serverip; run tftp_recovery\0" \
-	/* setenv ipaddr and serverip is necessary, because dhclient destroys the IPs internally */ \
+	/* setenv ipaddr and serverip is necessary, because dhclient can destroy the IPs internally */ \
 	"pxe_recovery=mdio up $ethprime && dhcp && pxe get && pxe boot\0" \
-	"tftp_recovery=tftpboot $kernel_addr_r recovery-image; tftpboot $fdt_addr_r recovery-dtb; " \
+	"tftp_recovery=tftpboot $kernel_addr_r recovery-image; tftpboot $fdt_addr_r recovery-dtb; " /* kernel_addr_r ? */ \
 			"setenv bootargs rdinit=/etc/preinit console=$defaultconsole,115200 " \
 			"debug $ethopts; " \
-			"run shieldcmd; " \
 			"bootz $kernel_addr_r - $fdt_addr_r\0" /* kernel_addr_r */
 
 #endif
@@ -184,8 +188,8 @@ int eth_phy_timeout(void);
 #define CONFIG_ZERO_BOOTDELAY_CHECK
 
 /* UART Configuration */
-#define CONFIG_SYS_NS16550_COM1			0x44e09000	/* UART0: XModem Boot, Shield */
-#define CONFIG_SYS_NS16550_COM2			0x48022000	/* UART1: Debug UART (Internal) */
+#define CONFIG_SYS_NS16550_COM1			0x44e09000	/* UART0: XModem Boot */
+#define CONFIG_SYS_NS16550_COM2			0x48022000	/* UART1: eMMC Boot, User UART */
 
 #define CONFIG_I2C
 #define CONFIG_I2C_MULTI_BUS
@@ -268,9 +272,6 @@ int eth_phy_timeout(void);
 #endif
 
 /* Network. */
-#define CONFIG_IP_DEFRAG		/* so we can use large tftp blocks */
-#define CONFIG_TFTP_TSIZE		/* tftp transfer size, progress bar */
-
 #define CONFIG_PHYLIB
 #define CONFIG_PHY_SMSC
 
@@ -279,7 +280,7 @@ int eth_phy_timeout(void);
 #define CONFIG_SYS_MEMTEST_END      0x87900000
 
 
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 /* support for NM packed bootloader */
 #define CONFIG_NM_BOOTLOADER_FORMAT
 
@@ -298,7 +299,7 @@ int eth_phy_timeout(void);
 #define CONFIG_JTAG_MARKER_UBOOT	0x807FFF00
 
 /* NRSW PMIC Reset Reason */
-#ifdef CONFIG_NRSW_BUILD
+#ifdef CONFIG_NRSW
 #define RESET_REASON_SHM_LOCATION	0x8e000000
 #define EXTERNAL_WATCHDOG_PATTERN	0x781f9ce2
 #endif

lib/display_options.c

@@ -13,25 +13,6 @@
 #include <linux/ctype.h>
 #include <asm/io.h>
 
-#ifdef CONFIG_NRSW_BUILD
-#include <version_autogenerated.h>
-#endif
-
-#ifdef CONFIG_NRSW_BUILD
-
-int display_options (void)
-{
-	printf ("\n\n%s\n", UBOOT_VERSION);
-#if defined(BUILD_TAG)
-	printf ("%s, Build: %s\n\n", version_string, BUILD_TAG);
-#else
-	printf ("%s\n\n", version_string);
-#endif
-	return 0;
-}
-
-#else /* CONFIG_NRSW_BUILD */
-
 int display_options (void)
 {
 #if defined(BUILD_TAG)
@@ -42,8 +23,6 @@ int display_options (void)
 	return 0;
 }
 
-#endif /* CONFIG_NRSW_BUILD */
-
 void print_freq(uint64_t freq, const char *s)
 {
 	unsigned long m = 0;