July 2022 - U-Boot

[PATCH v4] patman: By default don't pass "--no-tree" to checkpatch for linux
by Douglas Anderson 26 Jul '22

26 Jul '22

When you pass "--no-tree" to checkpatch it disables some extra checks that are important for Linux. Specifically I want checks like: warning: DT compatible string "boogie,woogie" appears un-documented check ./Documentation/devicetree/bindings/ Let's make the default for Linux to _not_ pass --no-tree. We'll have a config option and command line flag to override. Signed-off-by: Douglas Anderson <dianders(a)chromium.org> --- Without BooleanOptionalAction (python 3.9), it's a little odd to make the help text understandable while having the help text account for the project settings when printing the default (which it can only print as "True" or "False". It feels like no matter how you swing it you've got to use a double-negative or use weird syntax like I did. I removed previous review/testing tags since at least parts of this patch changed in non-trivial ways. This patch is now a singleton since the other patches in the series landed or became irrelevant. Changes in v4: - Don't use BooleanOptionalAction tools/patman/checkpatch.py | 11 +++++++---- tools/patman/control.py | 7 ++++--- tools/patman/main.py | 6 ++++++ tools/patman/settings.py | 3 ++- 4 files changed, 19 insertions(+), 8 deletions(-) diff --git a/tools/patman/checkpatch.py b/tools/patman/checkpatch.py index 70ba561c2686..d1b902dd9627 100644 --- a/tools/patman/checkpatch.py +++ b/tools/patman/checkpatch.py @@ -186,7 +186,7 @@ def check_patch_parse(checkpatch_output, verbose=False): return result -def check_patch(fname, verbose=False, show_types=False): +def check_patch(fname, verbose=False, show_types=False, use_tree=False): """Run checkpatch.pl on a file and parse the results. Args: @@ -194,6 +194,7 @@ def check_patch(fname, verbose=False, show_types=False): verbose: True to print out every line of the checkpatch output as it is parsed show_types: Tell checkpatch to show the type (number) of each message + use_tree (bool): If False we'll pass '--no-tree' to checkpatch. Returns: namedtuple containing: @@ -210,7 +211,9 @@ def check_patch(fname, verbose=False, show_types=False): stdout: Full output of checkpatch """ chk = find_check_patch() - args = [chk, '--no-tree'] + args = [chk] + if not use_tree: + args.append('--no-tree') if show_types: args.append('--show-types') output = command.output(*args, fname, raise_on_error=False) @@ -236,13 +239,13 @@ def get_warning_msg(col, msg_type, fname, line, msg): line_str = '' if line is None else '%d' % line return '%s:%s: %s: %s\n' % (fname, line_str, msg_type, msg) -def check_patches(verbose, args): +def check_patches(verbose, args, use_tree): '''Run the checkpatch.pl script on each patch''' error_count, warning_count, check_count = 0, 0, 0 col = terminal.Color() for fname in args: - result = check_patch(fname, verbose) + result = check_patch(fname, verbose, use_tree=use_tree) if not result.ok: error_count += result.errors warning_count += result.warnings diff --git a/tools/patman/control.py b/tools/patman/control.py index b40382388e07..bf426cf7bcf4 100644 --- a/tools/patman/control.py +++ b/tools/patman/control.py @@ -64,7 +64,7 @@ def prepare_patches(col, branch, count, start, end, ignore_binary, signoff): patchstream.insert_cover_letter(cover_fname, series, to_do) return series, cover_fname, patch_files -def check_patches(series, patch_files, run_checkpatch, verbose): +def check_patches(series, patch_files, run_checkpatch, verbose, use_tree): """Run some checks on a set of patches This santiy-checks the patman tags like Series-version and runs the patches @@ -77,6 +77,7 @@ def check_patches(series, patch_files, run_checkpatch, verbose): run_checkpatch (bool): True to run checkpatch.pl verbose (bool): True to print out every line of the checkpatch output as it is parsed + use_tree (bool): If False we'll pass '--no-tree' to checkpatch. Returns: bool: True if the patches had no errors, False if they did @@ -86,7 +87,7 @@ def check_patches(series, patch_files, run_checkpatch, verbose): # Check the patches, and run them through 'git am' just to be sure if run_checkpatch: - ok = checkpatch.check_patches(verbose, patch_files) + ok = checkpatch.check_patches(verbose, patch_files, use_tree) else: ok = True return ok @@ -165,7 +166,7 @@ def send(args): col, args.branch, args.count, args.start, args.end, args.ignore_binary, args.add_signoff) ok = check_patches(series, patch_files, args.check_patch, - args.verbose) + args.verbose, args.check_patch_use_tree) ok = ok and gitutil.check_suppress_cc_config() diff --git a/tools/patman/main.py b/tools/patman/main.py index d0c1a16b2b91..5fb78dc428ca 100755 --- a/tools/patman/main.py +++ b/tools/patman/main.py @@ -81,6 +81,12 @@ send.add_argument('--no-binary', action='store_true', dest='ignore_binary', send.add_argument('--no-check', action='store_false', dest='check_patch', default=True, help="Don't check for patch compliance") +send.add_argument('--tree', dest='check_patch_use_tree', default=False, + action='store_true', + help=("Set `tree` to True. If `tree` is False then we'll " + "pass '--no-tree' to checkpatch (default: tree=%(default)s)")) +send.add_argument('--no-tree', dest='check_patch_use_tree', + action='store_false', help="Set `tree` to False") send.add_argument('--no-tags', action='store_false', dest='process_tags', default=True, help="Don't process subject tags as aliases") send.add_argument('--no-signoff', action='store_false', dest='add_signoff', diff --git a/tools/patman/settings.py b/tools/patman/settings.py index 6c5f6c8ed992..cee18bf01a65 100644 --- a/tools/patman/settings.py +++ b/tools/patman/settings.py @@ -23,6 +23,7 @@ _default_settings = { "u-boot": {}, "linux": { "process_tags": "False", + "check_patch_use_tree": "True", }, "gcc": { "process_tags": "False", @@ -71,7 +72,7 @@ class _ProjectConfigParser(ConfigParser.SafeConfigParser): >>> config = _ProjectConfigParser("linux") >>> config.readfp(StringIO(sample_config)) >>> sorted((str(a), str(b)) for (a, b) in config.items("settings")) - [('am_hero', 'True'), ('process_tags', 'False')] + [('am_hero', 'True'), ('check_patch_use_tree', 'True'), ('process_tags', 'False')] # Check to make sure that settings works with unknown project. >>> config = _ProjectConfigParser("unknown") -- 2.37.0.170.g444d1eabd0-goog

2 2

[PATCH 0/9] Add DM support for atmel NAND driver
by Balamanikandan Gunasundar 26 Jul '22

26 Jul '22

This patch series adds support for NAND flash. The series adds DM support and replaces the existing NAND driver NAND_ATMEL. The drivers are ported from Linux based on the work done by Boris brezillon Balamanikandan Gunasundar (9): nand: atmel: Add DM based NAND driver nand: atmel: Add pmecc driver mfd: syscon: Add atmel-matrix registers definition memory: atmel-ebi: add Atmel EBI (External Bus Interface) driver mfd: syscon: atmel-smc: Add driver for atmel SMC configs: at91: sam9x60ek: Enable DM based nand driver ARM: dts: at91: sam9x60: Add nodes for EBI and NAND ARM: dts: at91: sam9x60ek: Enable NAND support board: sam9x60ek: remove nand init from board file MAINTAINERS | 1 + arch/arm/dts/sam9x60.dtsi | 42 + arch/arm/dts/sam9x60ek.dts | 103 + board/atmel/sam9x60ek/sam9x60ek.c | 59 - configs/sam9x60ek_mmc_defconfig | 9 +- configs/sam9x60ek_nandflash_defconfig | 9 +- configs/sam9x60ek_qspiflash_defconfig | 8 +- drivers/Kconfig | 2 + drivers/Makefile | 1 + drivers/memory/Kconfig | 7 + drivers/memory/Makefile | 1 + drivers/memory/atmel_ebi.c | 37 + drivers/mfd/Kconfig | 4 + drivers/mfd/Makefile | 1 + drivers/mfd/atmel-smc.c | 364 +++ drivers/mtd/nand/raw/Kconfig | 8 + drivers/mtd/nand/raw/Makefile | 1 + drivers/mtd/nand/raw/atmel/Makefile | 5 + drivers/mtd/nand/raw/atmel/nand-controller.c | 2300 ++++++++++++++++++ drivers/mtd/nand/raw/atmel/pmecc.c | 969 ++++++++ drivers/mtd/nand/raw/atmel/pmecc.h | 94 + include/linux/mfd/syscon/atmel-matrix.h | 112 + include/linux/mfd/syscon/atmel-smc.h | 119 + 23 files changed, 4188 insertions(+), 68 deletions(-) create mode 100644 drivers/memory/atmel_ebi.c create mode 100644 drivers/mfd/Kconfig create mode 100644 drivers/mfd/Makefile create mode 100644 drivers/mfd/atmel-smc.c create mode 100644 drivers/mtd/nand/raw/atmel/Makefile create mode 100644 drivers/mtd/nand/raw/atmel/nand-controller.c create mode 100644 drivers/mtd/nand/raw/atmel/pmecc.c create mode 100644 drivers/mtd/nand/raw/atmel/pmecc.h create mode 100644 include/linux/mfd/syscon/atmel-matrix.h create mode 100644 include/linux/mfd/syscon/atmel-smc.h -- 2.25.1

3 13

[PATCH] fastboot: release usb_gadget on reboot commands
by Mattijs Korpershoek 26 Jul '22

26 Jul '22

When host issues "fastboot reboot fastboot", it's expected that the board drops the USB connection before resetting. On some boards, such as Khadas VIM3L and SEI610, this is not the case. We observe the following error: $ fastboot reboot fastboot Rebooting into fastboot OKAY [ 0.004s] fastboot: error: Failed to boot into userspace fastboot; one or more components might be unbootable. This does not happen when we use the RST button on the board, nor in linux. We might hit a undefined hardware behavior, where D+ and D- are in an unknown state. Change the logic in cmd/fastboot.c to do a "reset request" instead of actually calling reset. This allows us to get out of the infinite loop in fastboot.c, de-initialize the usb stack and do a clean reset. Signed-off-by: Mattijs Korpershoek <mkorpershoek(a)baylibre.com> --- cmd/fastboot.c | 3 +++ drivers/usb/gadget/f_fastboot.c | 3 ++- drivers/usb/gadget/g_dnl.c | 17 +++++++++++++++++ include/g_dnl.h | 5 +++++ 4 files changed, 27 insertions(+), 1 deletion(-) diff --git a/cmd/fastboot.c b/cmd/fastboot.c index 033a2c95e8f0..97ce311e6d9b 100644 --- a/cmd/fastboot.c +++ b/cmd/fastboot.c @@ -87,6 +87,9 @@ exit: g_dnl_clear_detach(); usb_gadget_release(controller_index); + if (g_dnl_should_reset()) + do_reset(NULL, 0, 0, NULL); + return ret; #else pr_err("Fastboot USB not enabled\n"); diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_fastboot.c index 8ba55aab9f8f..a00d1ca571d1 100644 --- a/drivers/usb/gadget/f_fastboot.c +++ b/drivers/usb/gadget/f_fastboot.c @@ -420,7 +420,8 @@ static int fastboot_tx_write_str(const char *buffer) static void compl_do_reset(struct usb_ep *ep, struct usb_request *req) { - do_reset(NULL, 0, 0, NULL); + g_dnl_trigger_detach(); + g_dnl_trigger_reset_request(); } static unsigned int rx_bytes_expected(struct usb_ep *ep) diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c index afb7b74f3057..d083a214290d 100644 --- a/drivers/usb/gadget/g_dnl.c +++ b/drivers/usb/gadget/g_dnl.c @@ -200,6 +200,23 @@ void g_dnl_clear_detach(void) g_dnl_detach_request = false; } +static bool g_dnl_reset_request; + +bool g_dnl_should_reset(void) +{ + return g_dnl_reset_request; +} + +void g_dnl_trigger_reset_request(void) +{ + g_dnl_reset_request = true; +} + +void g_dnl_clear_reset_request(void) +{ + g_dnl_reset_request = false; +} + static int g_dnl_get_bcd_device_number(struct usb_composite_dev *cdev) { struct usb_gadget *gadget = cdev->gadget; diff --git a/include/g_dnl.h b/include/g_dnl.h index 836ee602c8da..95c772385fa6 100644 --- a/include/g_dnl.h +++ b/include/g_dnl.h @@ -43,6 +43,11 @@ void g_dnl_set_product(const char *s); bool g_dnl_detach(void); void g_dnl_trigger_detach(void); void g_dnl_clear_detach(void); + +bool g_dnl_should_reset(void); +void g_dnl_trigger_reset_request(void); +void g_dnl_clear_reset_request(void); + int run_usb_dnl_gadget(int usbctrl_index, char *usb_dnl_gadget); #endif /* __G_DOWNLOAD_H_ */ -- 2.36.1

2 6

[GIT PULL] xilinx patches for v2022.10-rc2
by Michal Simek 26 Jul '22

26 Jul '22

Hi Tom, please pull the following patches to your tree. CI loop doesn't show any problem. Thanks, Michal The following changes since commit 6e15cda270a060cf87c6c643a1cc3da65ffb242d: Prepare v2022.10-rc1 (2022-07-25 20:31:12 -0400) are available in the Git repository at: git@source.denx.de:u-boot/custodians/u-boot-microblaze.git tags/xilinx-for-v2022.10-rc2 for you to fetch changes up to 2a75bc1303b34e88745fcecfeacbe94f2a4bd1e2: spi: zynq_qspi: Fix programming qspi speed (2022-07-26 09:34:21 +0200) ---------------------------------------------------------------- Xilinx changes for v2022.10-rc2 fpga: - Convert SYS_FPGA_CHECK_CTRLC and SYS_FPGA_PROG_FEEDBACK to Kconfig - Add support for secure bitstream loading spi: - xilinx_spi: Add support for memopers and supports_op - zynq_qspi: Add support for supports_op/child_pre_probe - zynq_qspi: Fix dummy cycle and qspi speed calculations xilinx: - Get rid of #stream-id-cells - Use fixed partitions for SOM - Add support for UUID reading from FRU - Use strlcpy instead of strncpy - Add reset driver support for ZynqMP and Versal - Enable power domain driver in ZynqMP and Versal zynqmp: - Do no place BSS at 0 which have issue with NULL pointer - Enable SLG gpio driver - Disable LMB for mini configurations - Remove duplicate PMIO_NODE_ID_BASE macro versal: - Add xlnx-versal-resets.h header mmc: - zynq_sdhci: Fix macro for MMC HS relocate-rela: - Fix support for BE hosts - Define all macros for e_machine and reloc types misc: - Get rid of guard macros from ARM and RISC-V lmb: - Add support for disabling LMB serial: - zynq: Fix baudrate calculation tests: - Mark bind tests to run only on sandbox - List also dm uclass and devres ---------------------------------------------------------------- Adrian Fiergolski (1): fpga: zynqmp: support loading encrypted bitfiles Alexander Dahl (2): fpga: Convert SYS_FPGA_CHECK_CTRLC to Kconfig fpga: Convert SYS_FPGA_PROG_FEEDBACK to Kconfig Ashok Reddy Soma (11): mmc: zynq_sdhci: Fix timing macros for MMC High speed lmb: Fix lmb property's defination under struct lmb zynqmp: Run board_get_usable_ram_top() only on main U-Boot arm64: zynqmp: Enable reset driver firmware: zynqmp: Change prototype of zynqmp_pmufw_load_config_object() firmware: zynqmp: Load config overlay for core0 to pmufw arm64: zynqmp: Enable power domain driver mailbox: zynqmp: Move struct zynqmp_ipi_msg from sys_proto.h arm64: versal: Enable power domain driver and its dependencies spi: zynq_qspi: Add support for zynq_qspi_mem_exec_op spi: zynq_qspi: Fix programming qspi speed Ayan Kumar Halder (1): xilinx: Remove the legacy property "#stream-id-cells" Kunihiko Hayashi (1): serial: zynq: Use DIV_ROUND_CLOSEST() to calcurate divider value Michal Simek (13): arm64: zynqmp: Used fixed-partitions for QSPI in k26 arm64: zynqmp: Enable SLG gpo driver by default arm: riscv: Remove additional ifdef from code guarded by CONFIG_IS_ENABLED arm64: zynqmp: Disable LMB for mini configurations py: tests: Bind should run only on sandbox test/py: Run simple dm commands without checking xilinx: Remove duplicate PMIO_NODE_ID_BASE macro dt-bindings: versal: Add versal reset IDs tools: relocate-rela: Remove guard around R_AARCH64_RELATIVE tools: relocate-rela: Define all macros for e_machine and reloc types xilinx: Wire uuid reading from FRU xilinx: common: Use strlcpy instead of strncpy arm64: versal: Enable reset driver for versal Oleksandr Suvorov (12): fpga: add option for loading FPGA secure bitstreams fpga: xilinx: add missed identifier names fpga: xilinx: add bitstream flags to driver desc fpga: zynqmp: add str2flags call fpga: xilinx: pass compatible flags to xilinx_load() fpga: pass compatible flags to fpga_load() fpga: add fpga_compatible2flag spl: fit: pass real compatible flags to fpga_load() fpga: xilinx: pass compatible flags to load() callback fpga: zynqmp: reduce zynqmppl_load() code fpga: zynqmp: add bitstream compatible checking fpga: zynqmp: support loading authenticated images Samuel Holland (1): tools: relocate-rela: Fix ELF decoding on big-endian hosts Siva Durga Prasad Paladugu (1): spi: zynq_qspi: Add child pre probe function Stefan Herbrechtsmeier (1): xilinx: zynqmp: Do not use 0 as spl bss start address T Karthik Reddy (4): reset: zynqmp: Add reset driver support for versal spi: xilinx_spi: Add support for spi memory operations spi: xilinx_spi: Add support ops to axi qspi driver spi: zynq_qspi: Use dummy buswidth in dummy byte calculation README | 7 - arch/arc/lib/bootm.c | 8 +- arch/arm/dts/versal-mini-emmc0.dts | 1 - arch/arm/dts/versal-mini-emmc1.dts | 1 - arch/arm/dts/zynqmp-sm-k26-revA.dts | 163 +++++++------- arch/arm/dts/zynqmp.dtsi | 28 --- arch/arm/lib/bootm.c | 4 +- arch/arm/mach-zynq/cpu.c | 1 + arch/arm/mach-zynqmp/include/mach/sys_proto.h | 5 - arch/m68k/lib/bootm.c | 8 +- arch/microblaze/lib/bootm.c | 2 +- arch/powerpc/lib/bootm.c | 8 +- arch/riscv/lib/bootm.c | 4 +- arch/x86/lib/bootm.c | 5 +- board/xilinx/common/board.c | 26 ++- board/xilinx/versal/board.c | 5 +- board/xilinx/zynqmp/zynqmp.c | 8 +- boot/Kconfig | 4 +- boot/image-board.c | 8 +- cmd/Kconfig | 3 +- cmd/fpga.c | 8 +- common/spl/Kconfig | 3 +- common/spl/spl_fit.c | 17 +- configs/astro_mcf5373l_defconfig | 1 + configs/avnet_ultrazedev_cc_v1_0_ultrazedev_som_v1_0_defconfig | 2 - configs/xilinx_versal_virt_defconfig | 4 + configs/xilinx_zynqmp_mini_defconfig | 1 + configs/xilinx_zynqmp_mini_emmc0_defconfig | 3 +- configs/xilinx_zynqmp_mini_emmc1_defconfig | 3 +- configs/xilinx_zynqmp_mini_qspi_defconfig | 3 +- configs/xilinx_zynqmp_virt_defconfig | 7 +- doc/uImage.FIT/source_file_format.txt | 7 +- drivers/firmware/firmware-zynqmp.c | 24 +- drivers/fpga/Kconfig | 27 +++ drivers/fpga/fpga.c | 33 ++- drivers/fpga/spartan2.c | 3 +- drivers/fpga/spartan3.c | 2 +- drivers/fpga/versalpl.c | 2 +- drivers/fpga/virtex2.c | 13 +- drivers/fpga/xilinx.c | 8 +- drivers/fpga/zynqmppl.c | 99 ++++++-- drivers/fpga/zynqpl.c | 2 +- drivers/mailbox/Kconfig | 2 +- drivers/mailbox/zynqmp-ipi.c | 2 +- drivers/mmc/zynq_sdhci.c | 4 +- drivers/reset/Kconfig | 6 +- drivers/reset/reset-zynqmp.c | 10 +- drivers/serial/serial_zynq.c | 2 +- drivers/spi/xilinx_spi.c | 236 +++++++++++++------- drivers/spi/zynq_qspi.c | 73 +++++- include/configs/astro_mcf5373l.h | 1 - include/dt-bindings/reset/xlnx-versal-resets.h | 105 +++++++++ include/fpga.h | 4 +- include/lmb.h | 2 +- include/versalpl.h | 3 - include/xilinx.h | 21 +- include/zynqmp_firmware.h | 9 +- include/zynqmppl.h | 9 +- lib/lmb.c | 2 +- scripts/config_whitelist.txt | 1 - test/py/tests/test_bind.py | 3 +- test/py/tests/test_dm.py | 8 + tools/relocate-rela.c | 151 ++++++++----- 63 files changed, 836 insertions(+), 389 deletions(-) create mode 100644 include/dt-bindings/reset/xlnx-versal-resets.h -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Xilinx Microblaze Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs

2 1

[PATCH v4 1/3] drivers: tee: optee: remove unused probe local variable
by Etienne Carriere 26 Jul '22

26 Jul '22

Removes local variable child in optee_probe() that is not used. Cc: Patrick Delaunay <patrick.delaunay(a)foss.st.com> Reviewed-by: Patrick Delaunay <patrick.delaunay(a)foss.st.com> Signed-off-by: Etienne Carriere <etienne.carriere(a)linaro.org> --- Changes since v3: - Applied Patrick's R-b tag. No change since v2. New change not in v1 series. --- drivers/tee/optee/core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/tee/optee/core.c b/drivers/tee/optee/core.c index a89d62aaf0..c1f5fc4c7a 100644 --- a/drivers/tee/optee/core.c +++ b/drivers/tee/optee/core.c @@ -642,7 +642,6 @@ static int optee_probe(struct udevice *dev) { struct optee_pdata *pdata = dev_get_plat(dev); u32 sec_caps; - struct udevice *child; int ret; if (!is_optee_api(pdata->invoke_fn)) { @@ -673,7 +672,7 @@ static int optee_probe(struct udevice *dev) * only bind the drivers associated to the supported OP-TEE TA */ if (IS_ENABLED(CONFIG_RNG_OPTEE)) { - ret = device_bind_driver(dev, "optee-rng", "optee-rng", &child); + ret = device_bind_driver(dev, "optee-rng", "optee-rng", NULL); if (ret) return ret; } -- 2.25.1

3 6

[PATCH] configs: imx8mp_venice: remove unnecessary FEC_QUIRK_ENET_MAC
by Tim Harvey 26 Jul '22

26 Jul '22

FEC_QUIRK_ENET_MAC is defined in the imx-regs.h include file and thus does not need to be defined in the various board config includes. Signed-off-by: Tim Harvey <tharvey(a)gateworks.com> --- include/configs/imx8mp_venice.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/configs/imx8mp_venice.h b/include/configs/imx8mp_venice.h index d6bdab70e0df..3a8994dfe244 100644 --- a/include/configs/imx8mp_venice.h +++ b/include/configs/imx8mp_venice.h @@ -90,7 +90,4 @@ #define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + \ sizeof(CONFIG_SYS_PROMPT) + 16) -/* FEC */ -#define FEC_QUIRK_ENET_MAC - #endif -- 2.17.1

2 1

[PATCH 0/1] configs: imx8mm-cl-iot-gate: enable extension command
by Ying-Chun Liu (PaulLiu) 26 Jul '22

26 Jul '22

For imx8mm-cl-iot-gate we can use extension command to scan extension boards attached on the mainboard. We enable the extension command by default for users to detect the extension boards. Ying-Chun Liu (PaulLiu) (1): configs: imx8mm-cl-iot-gate: enable extension command configs/imx8mm-cl-iot-gate-optee_defconfig | 2 ++ configs/imx8mm-cl-iot-gate_defconfig | 2 ++ 2 files changed, 4 insertions(+) -- 2.35.1

2 2

[PATCH] imx: syscounter: support timer_get_boot_us
by Jun Nie 26 Jul '22

26 Jul '22

With supporting timer_get_boot_us, we can profile boot up time with below configs and function bootstage_mark_name(). CONFIG_BOOTSTAGE=y CONFIG_BOOTSTAGE_REPORT=y CONFIG_CMD_BOOTSTAGE=y Signed-off-by: Jun Nie <jun.nie(a)linaro.org> --- arch/arm/mach-imx/syscounter.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/arch/arm/mach-imx/syscounter.c b/arch/arm/mach-imx/syscounter.c index 7c02e199a3..df478a2326 100644 --- a/arch/arm/mach-imx/syscounter.c +++ b/arch/arm/mach-imx/syscounter.c @@ -79,6 +79,7 @@ int timer_init(void) gd->arch.tbl = 0; gd->arch.tbu = 0; + gd->arch.timer_rate_hz = freq; return 0; } #endif @@ -100,6 +101,14 @@ ulong get_timer(ulong base) return tick_to_time(get_ticks()) - base; } +ulong timer_get_boot_us(void) +{ + if (!gd->arch.timer_rate_hz) + timer_init(); + + return tick_to_time(get_ticks()); +} + void __udelay(unsigned long usec) { unsigned long long tmp; -- 2.25.1

4 3

[PATCH] MAINTAINERS: Update file list for ARM Freescale IMX
by Martyn Welch 26 Jul '22

26 Jul '22

The MAINTAINERS file currently lists files in arch/arm/include/asm/arch-imx/ being part of the IMX maintainers purview, however the arch/arm/include/asm/ directory also contains the directories arch-imx8, arch-imx8m, arch-imx8ulp and arch-imxrt which would also appear to be relevant to the team. Tweak the entry to cover these directories so that tools like get_maintainers.pl will suggest relevant maintainers when making changes just in these directories. Signed-off-by: Martyn Welch <martyn.welch(a)collabora.com> --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index a15ba7abdc..f91bb176f2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -255,7 +255,7 @@ F: arch/arm/cpu/arm926ejs/mx*/ F: arch/arm/cpu/armv7/vf610/ F: arch/arm/dts/*imx* F: arch/arm/mach-imx/ -F: arch/arm/include/asm/arch-imx/ +F: arch/arm/include/asm/arch-imx*/ F: arch/arm/include/asm/arch-mx*/ F: arch/arm/include/asm/arch-vf610/ F: arch/arm/include/asm/mach-imx/ -- 2.35.1

2 1

[PATCH] net: phy: Read physical phy IDs before phy is created
by Michal Simek 26 Jul '22

26 Jul '22

ethernet-phy-id directly specify phy ID which is present on the board but there is no code which really checks that specified phy ID is actually present on the board. That's why read phy ID regs with get_phy_id() call and compare it with required ID. If they don't match don't use this driver. Code is intentionally below reset sequence which get's phy to the right state and correct address. Signed-off-by: Michal Simek <michal.simek(a)amd.com> --- drivers/net/phy/ethernet_id.c | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/net/phy/ethernet_id.c b/drivers/net/phy/ethernet_id.c index 8864f99bb32f..7d56ad4aefa7 100644 --- a/drivers/net/phy/ethernet_id.c +++ b/drivers/net/phy/ethernet_id.c @@ -18,7 +18,7 @@ struct phy_device *phy_connect_phy_id(struct mii_dev *bus, struct udevice *dev, struct ofnode_phandle_args phandle_args; struct gpio_desc gpio; ofnode node; - u32 id, assert, deassert; + u32 id, assert, deassert, phy_id = 0xffffffff; u16 vendor, device; int ret; @@ -67,7 +67,21 @@ struct phy_device *phy_connect_phy_id(struct mii_dev *bus, struct udevice *dev, } } - id = vendor << 16 | device; + ret = get_phy_id(bus, phyaddr, phyaddr, &phy_id); + if (ret) { + dev_err(dev, "Physical phy is not detected\n"); + return NULL; + } + + id = vendor << 16 | device; + + if (phy_id != id) { + dev_err(dev, + "Required phyid %08x doesn't match with detected phyid %08x\n", + id, phy_id); + return NULL; + } + phydev = phy_device_create(bus, phyaddr, id, false); if (phydev) phydev->node = node; -- 2.36.1

2 1