]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
arm_ffa: introduce armffa command
authorAbdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
Thu, 13 Jul 2023 13:28:42 +0000 (14:28 +0100)
committerTom Rini <trini@konsulko.com>
Mon, 24 Jul 2023 19:30:03 +0000 (15:30 -0400)
Provide armffa command showcasing the use of the U-Boot FF-A support

armffa is a command showcasing how to invoke FF-A operations.
This provides a guidance to the client developers on how to
call the FF-A bus interfaces. The command also allows to gather secure
partitions information and ping these  partitions. The command is also
helpful in testing the communication with secure partitions.

For more details please refer to the command documentation [1].

[1]: doc/usage/cmd/armffa.rst

Signed-off-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Cc: Jens Wiklander <jens.wiklander@linaro.org>
Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
MAINTAINERS
cmd/Kconfig
cmd/Makefile
cmd/armffa.c [new file with mode: 0644]
doc/arch/arm64.ffa.rst
doc/usage/cmd/armffa.rst [new file with mode: 0644]
doc/usage/index.rst
drivers/firmware/arm-ffa/Kconfig

index ab26e0fe3ed775391abb656d41b91cd8c0b6c7f6..d1cfe38476fac5d329f08b5244b24dcaa8a2897d 100644 (file)
@@ -269,7 +269,9 @@ F:  configs/cortina_presidio-asic-pnand_defconfig
 ARM FF-A
 M:     Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
 S:     Maintained
+F:     cmd/armffa.c
 F:     doc/arch/arm64.ffa.rst
+F:     doc/usage/cmd/armffa.rst
 F:     drivers/firmware/arm-ffa/
 F:     include/arm_ffa.h
 F:     include/sandbox_arm_ffa.h
index 2d6e5f993f043459348c5826d85de2bc96131bbc..6cc3bf6c2d06c41dcfad0b54c2b673053b0bc5b8 100644 (file)
@@ -952,6 +952,16 @@ endmenu
 
 menu "Device access commands"
 
+config CMD_ARMFFA
+       bool "Arm FF-A test command"
+       depends on ARM_FFA_TRANSPORT
+       help
+         Provides a test command for the FF-A support
+         supported options:
+               - Listing the partition(s) info
+               - Sending a data pattern to the specified partition
+               - Displaying the arm_ffa device info
+
 config CMD_ARMFLASH
        #depends on FLASH_CFI_DRIVER
        bool "armflash"
index 9f8c0b058beae4dba57777393d7f20988732e154..9bebf321c397058c0f814aad7f6b2b703a169148 100644 (file)
@@ -12,6 +12,7 @@ obj-y += panic.o
 obj-y += version.o
 
 # command
+obj-$(CONFIG_CMD_ARMFFA) += armffa.o
 obj-$(CONFIG_CMD_2048) += 2048.o
 obj-$(CONFIG_CMD_ACPI) += acpi.o
 obj-$(CONFIG_CMD_ADDRMAP) += addrmap.o
diff --git a/cmd/armffa.c b/cmd/armffa.c
new file mode 100644 (file)
index 0000000..7e6eafc
--- /dev/null
@@ -0,0 +1,202 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright 2022-2023 Arm Limited and/or its affiliates <open-source-office@arm.com>
+ *
+ * Authors:
+ *   Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>
+ */
+#include <common.h>
+#include <arm_ffa.h>
+#include <command.h>
+#include <dm.h>
+#include <mapmem.h>
+#include <stdlib.h>
+#include <asm/io.h>
+
+/* Select the right physical address formatting according to the platform */
+#ifdef CONFIG_PHYS_64BIT
+#define PhysAddrLength "ll"
+#else
+#define PhysAddrLength ""
+#endif
+#define PHYS_ADDR_LN "%" PhysAddrLength "x"
+
+/**
+ * ffa_get_dev() - Return the FF-A device
+ * @devp:      pointer to the FF-A device
+ *
+ * Search for the FF-A device.
+ *
+ * Return:
+ * 0 on success. Otherwise, failure
+ */
+static int ffa_get_dev(struct udevice **devp)
+{
+       int ret;
+
+       ret = uclass_first_device_err(UCLASS_FFA, devp);
+       if (ret) {
+               log_err("Cannot find FF-A bus device\n");
+               return ret;
+       }
+
+       return 0;
+}
+
+/**
+ * do_ffa_getpart() - implementation of the getpart subcommand
+ * @cmdtp:             Command Table
+ * @flag:              flags
+ * @argc:              number of arguments
+ * @argv:              arguments
+ *
+ * Query a secure partition information. The secure partition UUID is provided
+ * as an argument. The function uses the arm_ffa driver
+ * partition_info_get operation which implements FFA_PARTITION_INFO_GET
+ * ABI to retrieve the data. The input UUID string is expected to be in big
+ * endian format.
+ *
+ * Return:
+ *
+ * CMD_RET_SUCCESS: on success, otherwise failure
+ */
+static int do_ffa_getpart(struct cmd_tbl *cmdtp, int flag, int argc,
+                         char *const argv[])
+{
+       u32 count = 0;
+       int ret;
+       struct ffa_partition_desc *descs;
+       u32 i;
+       struct udevice *dev;
+
+       if (argc != 2) {
+               log_err("Missing argument\n");
+               return CMD_RET_USAGE;
+       }
+
+       ret = ffa_get_dev(&dev);
+       if (ret)
+               return CMD_RET_FAILURE;
+
+       /* Ask the driver to fill the buffer with the SPs info */
+
+       ret = ffa_partition_info_get(dev, argv[1], &count, &descs);
+       if (ret) {
+               log_err("Failure in querying partition(s) info (error code: %d)\n", ret);
+               return CMD_RET_FAILURE;
+       }
+
+       /* SPs found , show the partition information */
+       for (i = 0; i < count ; i++) {
+               log_info("Partition: id = %x , exec_ctxt %x , properties %x\n",
+                        descs[i].info.id,
+                        descs[i].info.exec_ctxt,
+                        descs[i].info.properties);
+       }
+
+       return CMD_RET_SUCCESS;
+}
+
+/**
+ * do_ffa_ping() - implementation of the ping subcommand
+ * @cmdtp:             Command Table
+ * @flag:              flags
+ * @argc:              number of arguments
+ * @argv:              arguments
+ *
+ * Send data to a secure partition. The secure partition UUID is provided
+ * as an argument. Use the arm_ffa driver sync_send_receive operation
+ * which implements FFA_MSG_SEND_DIRECT_{REQ,RESP} ABIs to send/receive data.
+ *
+ * Return:
+ *
+ * CMD_RET_SUCCESS: on success, otherwise failure
+ */
+static int do_ffa_ping(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
+{
+       struct ffa_send_direct_data msg = {
+                       .data0 = 0xaaaaaaaa,
+                       .data1 = 0xbbbbbbbb,
+                       .data2 = 0xcccccccc,
+                       .data3 = 0xdddddddd,
+                       .data4 = 0xeeeeeeee,
+       };
+       u16 part_id;
+       int ret;
+       struct udevice *dev;
+
+       if (argc != 2) {
+               log_err("Missing argument\n");
+               return CMD_RET_USAGE;
+       }
+
+       part_id = strtoul(argv[1], NULL, 16);
+       if (!part_id) {
+               log_err("Partition ID can not be 0\n");
+               return CMD_RET_USAGE;
+       }
+
+       ret = ffa_get_dev(&dev);
+       if (ret)
+               return CMD_RET_FAILURE;
+
+       ret = ffa_sync_send_receive(dev, part_id, &msg, 1);
+       if (!ret) {
+               u8 cnt;
+
+               log_info("SP response:\n[LSB]\n");
+               for (cnt = 0;
+                    cnt < sizeof(struct ffa_send_direct_data) / sizeof(u64);
+                    cnt++)
+                       log_info("%llx\n", ((u64 *)&msg)[cnt]);
+               return CMD_RET_SUCCESS;
+       }
+
+       log_err("Sending direct request error (%d)\n", ret);
+       return CMD_RET_FAILURE;
+}
+
+/**
+ *do_ffa_devlist() - implementation of the devlist subcommand
+ * @cmdtp: [in]                Command Table
+ * @flag:              flags
+ * @argc:              number of arguments
+ * @argv:              arguments
+ *
+ * Query the device belonging to the UCLASS_FFA
+ * class.
+ *
+ * Return:
+ *
+ * CMD_RET_SUCCESS: on success, otherwise failure
+ */
+static int do_ffa_devlist(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
+{
+       struct udevice *dev;
+       int ret;
+
+       ret = ffa_get_dev(&dev);
+       if (ret)
+               return CMD_RET_FAILURE;
+
+       log_info("device %s, addr " PHYS_ADDR_LN ", driver %s, ops " PHYS_ADDR_LN "\n",
+                dev->name,
+                map_to_sysmem(dev),
+                dev->driver->name,
+                map_to_sysmem(dev->driver->ops));
+
+       return CMD_RET_SUCCESS;
+}
+
+static char armffa_help_text[] =
+       "getpart <partition UUID>\n"
+       "       - lists the partition(s) info\n"
+       "ping <partition ID>\n"
+       "       - sends a data pattern to the specified partition\n"
+       "devlist\n"
+       "       - displays information about the FF-A device/driver\n";
+
+U_BOOT_CMD_WITH_SUBCMDS(armffa, "Arm FF-A test command", armffa_help_text,
+                       U_BOOT_SUBCMD_MKENT(getpart, 2, 1, do_ffa_getpart),
+                       U_BOOT_SUBCMD_MKENT(ping, 2, 1, do_ffa_ping),
+                       U_BOOT_SUBCMD_MKENT(devlist, 1, 1, do_ffa_devlist));
index 4f817f053cc647c966e12db1158ba8672cc7d11e..aefd527447037da8add6a2922f065a2cb3f8247f 100644 (file)
@@ -205,6 +205,13 @@ The following features are provided:
 
 - FF-A bus can be compiled and used without EFI
 
+The armffa command
+-----------------------------------
+
+armffa is a command showcasing how to use the FF-A bus and how to invoke the driver operations.
+
+Please refer the command documentation at :doc:`../usage/cmd/armffa`
+
 Example of boot logs with FF-A enabled
 --------------------------------------
 
diff --git a/doc/usage/cmd/armffa.rst b/doc/usage/cmd/armffa.rst
new file mode 100644 (file)
index 0000000..3d42268
--- /dev/null
@@ -0,0 +1,93 @@
+.. SPDX-License-Identifier: GPL-2.0+:
+
+armffa command
+==============
+
+Synopsis
+--------
+
+::
+
+   armffa [sub-command] [arguments]
+
+   sub-commands:
+
+        getpart [partition UUID]
+
+            lists the partition(s) info
+
+        ping [partition ID]
+
+            sends a data pattern to the specified partition
+
+        devlist
+
+            displays information about the FF-A device/driver
+
+Description
+-----------
+
+armffa is a command showcasing how to use the FF-A bus and how to invoke its operations.
+
+This provides a guidance to the client developers on how to call the FF-A bus interfaces.
+
+The command also allows to gather secure partitions information and ping these  partitions.
+
+The command is also helpful in testing the communication with secure partitions.
+
+Example
+-------
+
+The following examples are run on Corstone-1000 platform.
+
+* ping
+
+::
+
+   corstone1000# armffa ping 0x8003
+   SP response:
+   [LSB]
+   fffffffe
+   0
+   0
+   0
+   0
+
+* ping (failure case)
+
+::
+
+   corstone1000# armffa ping 0
+   Sending direct request error (-22)
+
+* getpart
+
+::
+
+   corstone1000# armffa getpart 33d532ed-e699-0942-c09c-a798d9cd722d
+   Partition: id = 8003 , exec_ctxt 1 , properties 3
+
+* getpart (failure case)
+
+::
+
+   corstone1000# armffa getpart 33d532ed-e699-0942-c09c-a798d9cd7221
+   INVALID_PARAMETERS: Unrecognized UUID
+   Failure in querying partitions count (error code: -22)
+
+* devlist
+
+::
+
+   corstone1000# armffa devlist
+   device name arm_ffa, dev 00000000fdf41c30, driver name arm_ffa, ops 00000000fffc0e98
+
+Configuration
+-------------
+
+The command is available if CONFIG_CMD_ARMFFA=y and CONFIG_ARM_FFA_TRANSPORT=y.
+
+Return value
+------------
+
+The return value $? is 0 (true) on success, 1 (false) on failure.
index 072db53614c565454f7ec30ae096fdf9e727b37c..3326ec82fffd18a705d7eeca9fe781846e685768 100644 (file)
@@ -22,6 +22,7 @@ Shell commands
 
    cmd/acpi
    cmd/addrmap
+   cmd/armffa
    cmd/askenv
    cmd/base
    cmd/bdinfo
index 9200c8028b860bfa7e90811c542715e873a4b1cf..a7d539285988205431f5c3cfe8de8e37c89fee8b 100644 (file)
@@ -5,6 +5,7 @@ config ARM_FFA_TRANSPORT
        depends on DM && ARM64
        select ARM_SMCCC
        select ARM_SMCCC_FEATURES
+       imply CMD_ARMFFA
        select LIB_UUID
        select DEVRES
        help