Hi Caleb, On Tue, 19 Dec 2023 at 21:34, Caleb Connolly <[email protected]> wrote: > > This board is entirely supported by the generic arch code and DTS. The > board code used to handle turning on the vbus regulator, however this is > now handled via DT. > > With this, the board specific defconfig is also no longer needed, so > drop it as well.
I would like to understand how the board specific bits are to be handled going forward. One of the examples for QCS404 derived board uses I2C eeprom to save the ethernet MAC address. How should one configure that via this generic target? Would you expect arch/arm/mach-snapdragon/board.c to hold that code? Or we should have a separate defconfig and board support code for that? BTW, qcs404-evb does provide networking support but at the time when I added its support the network port was broken. It may very well be the case that someone tries to enable networking on qcs404-evb in the future. The same holds true for how we should add support for fastboot mode entry on a button press when U-boot starts as a first stage bootloader. It's always good to see generalization coming in but that should come with alternate methods to support existing board needs. -Sumit > > Signed-off-by: Caleb Connolly <[email protected]> > --- > board/qualcomm/qcs404-evb/Makefile | 6 ---- > board/qualcomm/qcs404-evb/qcs404-evb.c | 49 ----------------------------- > configs/qcs404evb_defconfig | 56 > ---------------------------------- > 3 files changed, 111 deletions(-) > > diff --git a/board/qualcomm/qcs404-evb/Makefile > b/board/qualcomm/qcs404-evb/Makefile > deleted file mode 100644 > index 4665827e0859..000000000000 > --- a/board/qualcomm/qcs404-evb/Makefile > +++ /dev/null > @@ -1,6 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0+ > -# > -# (C) Copyright 2022 Sumit Garg <[email protected]> > -# > - > -obj-y += qcs404-evb.o > diff --git a/board/qualcomm/qcs404-evb/qcs404-evb.c > b/board/qualcomm/qcs404-evb/qcs404-evb.c > deleted file mode 100644 > index 1a4b1f97a3ae..000000000000 > --- a/board/qualcomm/qcs404-evb/qcs404-evb.c > +++ /dev/null > @@ -1,49 +0,0 @@ > -// SPDX-License-Identifier: GPL-2.0+ > -/* > - * Board init file for QCS404-EVB > - * > - * (C) Copyright 2022 Sumit Garg <[email protected]> > - */ > - > -#include <common.h> > -#include <cpu_func.h> > -#include <dm.h> > -#include <env.h> > -#include <init.h> > -#include <asm/cache.h> > -#include <asm/gpio.h> > -#include <asm/global_data.h> > -#include <fdt_support.h> > - > -DECLARE_GLOBAL_DATA_PTR; > - > -void qcom_board_init(void) > -{ > - struct udevice *pmic_gpio; > - struct gpio_desc usb_vbus_boost_pin; > - int ret, node; > - > - ret = uclass_get_device_by_name(UCLASS_GPIO, > - "pms405_gpios@c000", > - &pmic_gpio); > - if (ret < 0) { > - printf("Failed to find pms405_gpios@c000 node.\n"); > - return; > - } > - > - node = fdt_subnode_offset(gd->fdt_blob, dev_of_offset(pmic_gpio), > - "usb_vbus_boost_pin"); > - if (node < 0) { > - printf("Failed to find usb_hub_reset_pm dt node.\n"); > - return; > - } > - ret = gpio_request_by_name_nodev(offset_to_ofnode(node), "gpios", 0, > - &usb_vbus_boost_pin, 0); > - if (ret < 0) { > - printf("Failed to request usb_hub_reset_pm gpio.\n"); > - return; > - } > - > - dm_gpio_set_dir_flags(&usb_vbus_boost_pin, > - GPIOD_IS_OUT | GPIOD_IS_OUT_ACTIVE); > -} > diff --git a/configs/qcs404evb_defconfig b/configs/qcs404evb_defconfig > deleted file mode 100644 > index 7daaa449ffbd..000000000000 > --- a/configs/qcs404evb_defconfig > +++ /dev/null > @@ -1,56 +0,0 @@ > -CONFIG_ARM=y > -CONFIG_SYS_BOARD="qcs404-evb" > -CONFIG_SKIP_LOWLEVEL_INIT=y > -CONFIG_COUNTER_FREQUENCY=19000000 > -CONFIG_POSITION_INDEPENDENT=y > -CONFIG_ARCH_SNAPDRAGON=y > -CONFIG_DEFAULT_DEVICE_TREE="qcs404-evb" > -CONFIG_IDENT_STRING="\nQualcomm QCS404-EVB" > -CONFIG_SYS_LOAD_ADDR=0x80000000 > -CONFIG_FIT=y > -CONFIG_FIT_VERBOSE=y > -CONFIG_BOOTDELAY=5 > -CONFIG_USE_BOOTARGS=y > -CONFIG_BOOTARGS="earlycon ignore_loglevel root= clk_ignore_unused" > -CONFIG_SAVE_PREV_BL_FDT_ADDR=y > -CONFIG_SAVE_PREV_BL_INITRAMFS_START_ADDR=y > -# CONFIG_DISPLAY_CPUINFO is not set > -CONFIG_HUSH_PARSER=y > -CONFIG_SYS_MAXARGS=64 > -CONFIG_SYS_CBSIZE=512 > -CONFIG_CMD_GPIO=y > -CONFIG_CMD_GPT=y > -CONFIG_CMD_MMC=y > -CONFIG_CMD_PART=y > -CONFIG_CMD_USB=y > -CONFIG_CMD_EXT2=y > -CONFIG_CMD_EXT4=y > -CONFIG_CMD_EXT4_WRITE=y > -CONFIG_CMD_FAT=y > -CONFIG_CMD_FS_GENERIC=y > -# CONFIG_NET is not set > -CONFIG_CLK=y > -CONFIG_CLK_QCOM_QCS404=y > -CONFIG_MSM_GPIO=y > -CONFIG_QCOM_PMIC_GPIO=y > -CONFIG_MISC=y > -CONFIG_MMC_HS400_SUPPORT=y > -CONFIG_MMC_SDHCI=y > -CONFIG_MMC_SDHCI_ADMA=y > -CONFIG_MMC_SDHCI_MSM=y > -CONFIG_PHY=y > -CONFIG_PHY_QCOM_USB_HS_28NM=y > -CONFIG_PHY_QCOM_USB_SS=y > -CONFIG_PINCTRL=y > -CONFIG_PINCTRL_QCOM_QCS404=y > -CONFIG_DM_PMIC=y > -CONFIG_PMIC_QCOM=y > -CONFIG_MSM_SERIAL=y > -CONFIG_SPMI_MSM=y > -CONFIG_USB=y > -CONFIG_USB_XHCI_HCD=y > -CONFIG_USB_XHCI_DWC3=y > -CONFIG_USB_DWC3=y > -CONFIG_USB_DWC3_GENERIC=y > -CONFIG_USB_STORAGE=y > -CONFIG_LMB_MAX_REGIONS=64 > > -- > 2.42.1 >
