@@ -106,8 +106,7 @@ void reset_misc(void)
printf("Synology reset...");
udelay(50000);
- b_d = ns16550_calc_divisor((NS16550_t)CONFIG_SYS_NS16550_COM2,
- CONFIG_SYS_NS16550_CLK, 9600);
+ b_d = ns16550_calc_divisor(CONFIG_SYS_NS16550_CLK, 9600);
NS16550_init((NS16550_t)CONFIG_SYS_NS16550_COM2, b_d);
NS16550_putc((NS16550_t)CONFIG_SYS_NS16550_COM2, SOFTWARE_REBOOT);
}
@@ -259,7 +259,7 @@ static inline void console_doenv(int file, struct stdio_dev *dev)
iomux_doenv(file, dev->name);
}
#endif
-#else
+#else /* !CONSOLE_MUX */
static inline int console_getc(int file)
{
return stdio_devices[file]->getc(stdio_devices[file]);
@@ -45,11 +45,13 @@ const char *log_get_cat_name(enum log_category_t cat)
if (cat >= LOGC_NONE)
return log_cat_name[cat - LOGC_NONE];
-#if CONFIG_IS_ENABLED(DM)
- name = uclass_get_name((enum uclass_id)cat);
-#else
- name = NULL;
-#endif
+ /* With tiny-dm we don't have uclasses, so cannot get the name */
+ if (CONFIG_IS_ENABLED(TINY_ONLY))
+ return "tiny";
+ if (CONFIG_IS_ENABLED(DM))
+ name = uclass_get_name((enum uclass_id)cat);
+ else
+ name = NULL;
return name ? name : "<missing>";
}
@@ -182,6 +184,21 @@ static bool log_passes_filters(struct log_device *ldev, struct log_rec *rec)
return false;
}
+void log_check(const char *msg)
+{
+ struct log_device *ldev;
+ int count = 0;
+
+ list_for_each_entry(ldev, &gd->log_head, sibling_node) {
+ count++;
+ if (count > 1) {
+ printf("%s: %s error\n", msg, __func__);
+ panic("log");
+ }
+ }
+ printf("%s: log OK\n", msg);
+}
+
/**
* log_dispatch() - Send a log record to all log devices for processing
*
@@ -296,6 +313,15 @@ int log_remove_filter(const char *drv_name, int filter_num)
return -ENOENT;
}
+void log_fixup_for_gd_move(struct global_data *new_gd)
+{
+ /* The sentinel node has moved, so update things that point to it */
+ if (gd->log_head.next) {
+ new_gd->log_head.next->prev = &new_gd->log_head;
+ new_gd->log_head.prev->next = &new_gd->log_head;
+ }
+}
+
int log_init(void)
{
struct log_driver *drv = ll_entry_start(struct log_driver, log_driver);
@@ -80,3 +80,34 @@ void malloc_simple_info(void)
log_info("malloc_simple: %lx bytes used, %lx remain\n", gd->malloc_ptr,
CONFIG_VAL(SYS_MALLOC_F_LEN) - gd->malloc_ptr);
}
+
+uint malloc_ptr_to_ofs(void *ptr)
+{
+ ulong addr = map_to_sysmem(ptr);
+ ulong offset;
+
+ offset = addr - gd->malloc_base;
+ if (offset >= gd->malloc_limit) {
+ log_debug("Invalid malloc ptr %p (base=%lx, size=%lx)\n", ptr,
+ gd->malloc_base, gd->malloc_limit);
+ panic("malloc ptr invalid");
+ }
+
+ return offset;
+}
+
+void *malloc_ofs_to_ptr(uint offset)
+{
+ void *base = map_sysmem(gd->malloc_base, gd->malloc_limit);
+ void *ptr;
+
+ if (offset >= gd->malloc_limit) {
+ log_debug("Invalid malloc offset %lx (size=%lx)\n",
+ gd->malloc_base, gd->malloc_limit);
+ panic("malloc offset invalid");
+ }
+ ptr = base + offset;
+ unmap_sysmem(base);
+
+ return ptr;
+}
@@ -399,7 +399,7 @@ static int spl_common_init(bool setup_malloc)
if (ret) {
debug("%s: Failed to set up bootstage: ret=%d\n", __func__,
ret);
- return ret;
+ return log_msg_ret("bootstage", ret);
}
#ifdef CONFIG_BOOTSTAGE_STASH
if (!u_boot_first_phase()) {
@@ -418,17 +418,17 @@ static int spl_common_init(bool setup_malloc)
ret = log_init();
if (ret) {
debug("%s: Failed to set up logging\n", __func__);
- return ret;
+ return log_msg_ret("log", ret);
}
#endif
if (CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)) {
ret = fdtdec_setup();
if (ret) {
debug("fdtdec_setup() returned error %d\n", ret);
- return ret;
+ return log_msg_ret("fdtdec", ret);
}
}
- if (CONFIG_IS_ENABLED(DM)) {
+ if (CONFIG_IS_ENABLED(DM) && !CONFIG_IS_ENABLED(TINY_ONLY)) {
bootstage_start(BOOTSTAGE_ID_ACCUM_DM_SPL,
spl_phase() == PHASE_TPL ? "dm tpl" : "dm_spl");
/* With CONFIG_SPL_OF_PLATDATA, bring in all devices */
@@ -436,7 +436,7 @@ static int spl_common_init(bool setup_malloc)
bootstage_accum(BOOTSTAGE_ID_ACCUM_DM_SPL);
if (ret) {
debug("dm_init_and_scan() returned error %d\n", ret);
- return ret;
+ return log_msg_ret("init", ret);
}
}
@@ -819,9 +819,10 @@ ulong spl_relocate_stack_gd(void)
ptr = CONFIG_SPL_STACK_R_ADDR - roundup(sizeof(gd_t),16);
new_gd = (gd_t *)ptr;
memcpy(new_gd, (void *)gd, sizeof(gd_t));
-#if CONFIG_IS_ENABLED(DM)
- dm_fixup_for_gd_move(new_gd);
-#endif
+ if (CONFIG_IS_ENABLED(DM) && !CONFIG_IS_ENABLED(TINY_ONLY))
+ dm_fixup_for_gd_move(new_gd);
+ if (CONFIG_IS_ENABLED(LOG))
+ log_fixup_for_gd_move(new_gd);
#if !defined(CONFIG_ARM) && !defined(CONFIG_RISCV)
gd = new_gd;
#endif
@@ -9,12 +9,14 @@
*/
#include <common.h>
+#include <dm.h>
#include <image.h>
#include <log.h>
#include <spi.h>
#include <spi_flash.h>
#include <errno.h>
#include <spl.h>
+#include <dm/of_extra.h>
DECLARE_GLOBAL_DATA_PTR;
@@ -55,7 +57,7 @@ static int spi_load_image_os(struct spl_image_info *spl_image,
static ulong spl_spi_fit_read(struct spl_load_info *load, ulong sector,
ulong count, void *buf)
{
- struct spi_flash *flash = load->dev;
+ struct spi_flash *flash = load->legacy_dev;
ulong ret;
ret = spi_flash_read(flash, sector, count, buf);
@@ -70,6 +72,21 @@ unsigned int __weak spl_spi_get_uboot_offs(struct spi_flash *flash)
return CONFIG_SYS_SPI_U_BOOT_OFFS;
}
+static int spl_read(struct spl_load_info *load, u32 offset, size_t len,
+ void *buf)
+{
+ int ret;
+
+ if (!CONFIG_IS_ENABLED(TINY_SPI_FLASH))
+ ret = spi_flash_read(load->legacy_dev, offset, len, buf);
+ else
+ ret = tiny_spi_flash_read(load->tdev, offset, len, buf);
+
+ if (ret)
+ return log_ret(ret);
+
+ return 0;
+}
/*
* The main entry for SPI booting. It's necessary that SDRAM is already
* configured and available since this code loads the main U-Boot image
@@ -80,41 +97,53 @@ static int spl_spi_load_image(struct spl_image_info *spl_image,
{
int err = 0;
unsigned int payload_offs;
+ struct spl_load_info load;
struct spi_flash *flash;
struct image_header *header;
+ struct tinydev *tdev;
/*
* Load U-Boot image from SPI flash into RAM
* In DM mode: defaults speed and mode will be
* taken from DT when available
*/
-
- flash = spi_flash_probe(CONFIG_SF_DEFAULT_BUS,
- CONFIG_SF_DEFAULT_CS,
- CONFIG_SF_DEFAULT_SPEED,
- CONFIG_SF_DEFAULT_MODE);
- if (!flash) {
- puts("SPI probe failed.\n");
- return -ENODEV;
+ memset(&load, '\0', sizeof(load));
+ if (!CONFIG_IS_ENABLED(TINY_SPI_FLASH)) {
+ flash = spi_flash_probe(CONFIG_SF_DEFAULT_BUS,
+ CONFIG_SF_DEFAULT_CS,
+ CONFIG_SF_DEFAULT_SPEED,
+ CONFIG_SF_DEFAULT_MODE);
+ if (!flash) {
+ puts("SPI probe failed\n");
+ return -ENODEV;
+ }
+ payload_offs = spl_spi_get_uboot_offs(flash);
+ if (CONFIG_IS_ENABLED(OF_CONTROL) &&
+ !CONFIG_IS_ENABLED(OF_PLATDATA)) {
+ payload_offs = ofnode_read_config_int(
+ "u-boot,spl-payload-offset",
+ payload_offs);
+ }
+ load.legacy_dev = flash;
+ } else {
+ tdev = tiny_dev_get(UCLASS_SPI_FLASH, 0);
+ if (!tdev) {
+ puts("SPI probe failed\n");
+ return -ENODEV;
+ }
+ load.tdev = tdev;
+ payload_offs = CONFIG_SYS_SPI_U_BOOT_OFFS;
}
- payload_offs = spl_spi_get_uboot_offs(flash);
-
header = spl_get_load_buffer(-sizeof(*header), sizeof(*header));
-#if CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)
- payload_offs = fdtdec_get_config_int(gd->fdt_blob,
- "u-boot,spl-payload-offset",
- payload_offs);
-#endif
-
#ifdef CONFIG_SPL_OS_BOOT
if (spl_start_uboot() || spi_load_image_os(spl_image, flash, header))
#endif
{
- /* Load u-boot, mkimage header is 64 bytes. */
- err = spi_flash_read(flash, payload_offs, sizeof(*header),
- (void *)header);
+ /* Load U-Boot, mkimage header is 64 bytes. */
+ err = spl_read(&load, payload_offs, sizeof(*header),
+ (void *)header);
if (err) {
debug("%s: Failed to read from SPI flash (err=%d)\n",
__func__, err);
@@ -123,30 +152,23 @@ static int spl_spi_load_image(struct spl_image_info *spl_image,
if (IS_ENABLED(CONFIG_SPL_LOAD_FIT_FULL) &&
image_get_magic(header) == FDT_MAGIC) {
- err = spi_flash_read(flash, payload_offs,
- roundup(fdt_totalsize(header), 4),
- (void *)CONFIG_SYS_LOAD_ADDR);
+ err = spl_read(&load, payload_offs,
+ roundup(fdt_totalsize(header), 4),
+ (void *)CONFIG_SYS_LOAD_ADDR);
if (err)
return err;
err = spl_parse_image_header(spl_image,
- (struct image_header *)CONFIG_SYS_LOAD_ADDR);
+ (struct image_header *)CONFIG_SYS_LOAD_ADDR);
} else if (IS_ENABLED(CONFIG_SPL_LOAD_FIT) &&
image_get_magic(header) == FDT_MAGIC) {
- struct spl_load_info load;
-
debug("Found FIT\n");
- load.dev = flash;
load.priv = NULL;
load.filename = NULL;
load.bl_len = 1;
load.read = spl_spi_fit_read;
err = spl_load_simple_fit(spl_image, &load,
- payload_offs,
- header);
+ payload_offs, header);
} else if (IS_ENABLED(CONFIG_SPL_LOAD_IMX_CONTAINER)) {
- struct spl_load_info load;
-
- load.dev = flash;
load.priv = NULL;
load.filename = NULL;
load.bl_len = 1;
@@ -158,9 +180,8 @@ static int spl_spi_load_image(struct spl_image_info *spl_image,
err = spl_parse_image_header(spl_image, header);
if (err)
return err;
- err = spi_flash_read(flash, payload_offs,
- spl_image->size,
- (void *)spl_image->load_addr);
+ err = spl_read(&load, payload_offs, spl_image->size,
+ (void *)spl_image->load_addr);
}
}
new file mode 100644
@@ -0,0 +1,35 @@
+.. SPDX-License-Identifier: GPL-2.0+
+.. Copyright (c) 2020 Heinrich Schuchardt
+
+Debugging
+=========
+
+This describes a few debugging techniques for different parts of U-Boot.
+
+Makefiles
+---------
+
+You can use $(warning) to show debugging information in a makefile::
+
+ $(warning SPL: $(CONFIG_SPL_BUILD) . $(SPL_TPL_))
+
+When make executes these they produce a message. If you put them in a rule, they
+are executed when the rule is executed. For example, to show the value of a
+variable at the point where it is used::
+
+ tools-only: scripts_basic $(version_h) $(timestamp_h) tools/version.h
+ $(warning version_h: $(version_h))
+ $(Q)$(MAKE) $(build)=tools
+
+You can use ifndef in makefiles for simple CONFIG checks::
+
+ ifndef CONFIG_DM_DEV_READ_INLINE
+ obj-$(CONFIG_OF_CONTROL) += read.o
+ endif
+
+but for those which require variable expansion you should use ifeq or ifneq::
+
+ ifeq ($(CONFIG_$(SPL_TPL_)TINY_ONLY),)
+ obj-y += device.o fdtaddr.o lists.o root.o uclass.o util.o
+ endif
+
new file mode 100644
@@ -0,0 +1,315 @@
+.. SPDX-License-Identifier: GPL-2.0+
+
+Tiny driver model (tiny-dm)
+===========================
+
+Purpose
+-------
+
+Reduce the overhead of using driver model in SPL and TPL.
+
+
+Introduction
+------------
+
+On some platforms that use SPL [1]_, SRAM is extremely limited. There is a
+need to use as little space as possible for U-Boot SPL.
+
+With the migration to driver model and devicetree, the extra software complexity
+has created more pressure on U-Boot's code and data size.
+
+A few features have been introduced to help with this problem:
+
+ - fdtgrep, introduced in 2015, automatically removes unnecessary parts of the
+ device tree, e.g. those used by drivers not present in SPL. At the time,
+ this typically reduced SPL size from about 40KB to perhaps 3KB and made
+ it feasible to look at using driver model with SPL. The minimum overhead
+ was reduced to approximately 7KB on Thumb systems, for example [2]_
+ - of-platdata, introduced in 2016 [3]_, converts the device tree into C data
+ structures which are placed in the SPL image. This saves approximately
+ 3KB of code and replaces the devicetree with something typically 30%
+ smaller.
+
+However the problem still exists. Even with of-platdata, the driver-model
+overhead is typically 3KB at the minimum. This excludes the size of allocated
+data structures, which is 84 bytes per device and 76 bytes per uclass on
+32-bit machines. On 64-bit machines these sizes approximately double.
+
+With the driver-model migration deadlines passing, a solution is needed to
+allow boards to complete migration to driver model in SPL, without taking on
+the full ~5KB overhead that this entails.
+
+
+Concept
+-------
+
+The idea of tiny devices ('tiny-dm') builds on of-platdata, but additionally
+removes most of the rich feature-set of driver model.
+
+In particular tiny-dm takes away the concept of a uclass (except that it stil
+uses uclass IDs), drastically reduces the size of a device (to 16 bytes on
+32-bit) and removes the need for a driver_info structure.
+
+With tiny-dm, dtoc outputs U_BOOT_TINY_DEVICE() instead of U_BOOT_DEVICE().
+A new 'struct tiny_dev' is used instead of 'struct udevice'. Devices can be
+located based on uclass ID and sequence number with tiny_dev_find(). Devices can
+be probed with tiny_dev_probe().
+
+In fact, tiny-dm is effectively a bypass for most of driver model. It retains
+some capability with in (chiefly by using the same device tree), but new code
+is added to implement simple features in a simple way.
+
+Tiny-dm is not suitable for complex device and interactions, but it can
+support a serial port (output only), I2C buses and other features needed to
+set up the machine just enough to load U-Boot proper.
+
+It is possible to enable Tiny-dm on a subsystem-by-subsystem basis. For example,
+enabling CONFIG_TPL_TINY_SERIAL on chromebook_coral saves about 900 bytes of
+code and data, with no perceptable difference in operation.
+
+
+Tiny devices
+------------
+
+Below is an example of a tiny device, a UART that uses NS16550. It works by
+setting up a platform structure to pass to the ns16550 driver, perhaps the
+worst driver in U-Boot.
+
+.. code-block:: c
+
+ static int apl_ns16550_tiny_probe(struct tiny_dev *tdev)
+ {
+ struct dtd_intel_apl_ns16550 *dtplat = tdev->dtplat;
+ struct ns16550_platdata *plat = tdev->priv;
+ ulong base;
+ pci_dev_t bdf;
+
+ base = dtplat->early_regs[0];
+ bdf = pci_ofplat_get_devfn(dtplat->reg[0]);
+
+ if (!CONFIG_IS_ENABLED(PCI))
+ apl_uart_init(bdf, base);
+
+ plat->base = base;
+ plat->reg_shift = dtplat->reg_shift;
+ plat->reg_width = 1;
+ plat->clock = dtplat->clock_frequency;
+ plat->fcr = UART_FCR_DEFVAL;
+
+ return ns16550_tiny_probe_plat(plat);
+ }
+
+ static int apl_ns16550_tiny_setbrg(struct tiny_dev *tdev, int baudrate)
+ {
+ struct ns16550_platdata *plat = tdev->priv;
+
+ return ns16550_tiny_setbrg(plat, baudrate);
+ }
+
+ static int apl_ns16550_tiny_putc(struct tiny_dev *tdev, const char ch)
+ {
+ struct ns16550_platdata *plat = tdev->priv;
+
+ return ns16550_tiny_putc(plat, ch);
+ }
+
+ struct tiny_serial_ops apl_ns16550_tiny_ops = {
+ .probe = apl_ns16550_tiny_probe,
+ .setbrg = apl_ns16550_tiny_setbrg,
+ .putc = apl_ns16550_tiny_putc,
+ };
+
+ U_BOOT_TINY_DRIVER(apl_ns16550) = {
+ .uclass_id = UCLASS_SERIAL,
+ .probe = apl_ns16550_tiny_probe,
+ .ops = &apl_ns16550_tiny_ops,
+ DM_TINY_PRIV(<ns16550.h>, sizeof(struct ns16550_platdata))
+ };
+
+The probe function is responsible for setting up the hardware so that the UART
+can output characters. This driver enables the device on PCI and assigns an
+address to its BAR (Base-Address Register). That code is in apl_uart_init() and
+is not show here. Then it sets up a platdata data structure for use by the
+ns16550 driver and calls its probe function.
+
+The 'tdev' device is declared like this in the device tree:
+
+.. code-block:: c
+
+ serial: serial at 18,2 {
+ reg = <0x0200c210 0 0 0 0>;
+ u-boot,dm-pre-reloc;
+ compatible = "intel,apl-ns16550";
+ early-regs = <0xde000000 0x20>;
+ reg-shift = <2>;
+ clock-frequency = <1843200>;
+ current-speed = <115200>;
+ };
+
+When dtoc runs it outputs the following code for this, into dt-platdata.c:
+
+.. code-block:: c
+
+ static struct dtd_intel_apl_ns16550 dtv_serial_at_18_2 = {
+ .clock_frequency = 0x1c2000,
+ .current_speed = 0x1c200,
+ .early_regs = {0xde000000, 0x20},
+ .reg = {0x200c210, 0x0},
+ .reg_shift = 0x2,
+ };
+
+ DM_DECL_TINY_DRIVER(apl_ns16550);
+ #include <ns16550.h>
+ u8 _serial_at_18_2_priv[sizeof(struct ns16550_platdata)] __attribute__ ((section (".data")));
+ U_BOOT_TINY_DEVICE(serial_at_18_2) = {
+ .dtplat = &dtv_serial_at_18_2,
+ .drv = DM_REF_TINY_DRIVER(apl_ns16550),
+ .priv = _serial_at_18_2_priv,
+ };
+
+This basically creates a device, with a pointer to the dtplat data (a C
+structure similar to the devicetree node) and a pointer to the driver, the
+U_BOOT_TINY_DRIVER() thing shown above.
+
+So far, tiny-dm might look pretty similar to the full driver model, but there
+are quite a few differences that may not be immediately apparent:
+
+ - Whereas U_BOOT_DEVICE() emits a driver_info structure and then allocates
+ the udevice structure at runtime, U_BOOT_TINY_DEVICE() emits an actual
+ tiny_dev device structure into the image. On platforms where SPL runs in
+ read-only memory, U-Boot automatically copies this into RAM as needed.
+ - The DM_TINY_PRIV() macro tells U-Boot about the private data needed by
+ the device. But this is not allocated at runtime. Instead it is declared
+ in the C structure above. However on platforms where SPL runs in read-only
+ memory, allocation is left until runtime.
+ - There is a corresponding 'full' driver in the same file with the same name.
+ Like of-platdata, it is not possible to use tiny-dm without 'full' support
+ added as well. This makes sense because the device needs to be supported
+ in U-Boot proper as well.
+ - While this driver is in the UCLASS_SERIAL uclass, there is in fact no
+ uclass available. The serial-uclass.c implementation has an entirely
+ separate (small) piece of code to support tiny-dm:
+
+.. code-block:: c
+
+ int serial_init(void)
+ {
+ struct tiny_dev *tdev;
+ int ret;
+
+ tdev = tiny_dev_find(UCLASS_SERIAL, 0);
+ if (!tdev) {
+ if (IS_ENABLED(CONFIG_REQUIRE_SERIAL_CONSOLE))
+ panic_str("No serial");
+ return -ENODEV;
+ }
+ ret = tiny_dev_probe(tdev);
+ if (ret)
+ return log_msg_ret("probe", ret);
+ gd->tiny_serial = tdev;
+ gd->flags |= GD_FLG_SERIAL_READY;
+ serial_setbrg();
+
+ return 0;
+ }
+
+ void serial_putc(const char ch)
+ {
+ struct tiny_dev *tdev = gd->tiny_serial;
+ struct tiny_serial_ops *ops;
+
+ if (!tdev)
+ goto err;
+
+ ops = tdev->drv->ops;
+ if (!ops->putc)
+ goto err;
+ if (ch == '\n')
+ ops->putc(tdev, '\r');
+ ops->putc(tdev, ch);
+
+ return;
+ err:
+ if (IS_ENABLED(DEBUG_UART))
+ printch(ch);
+ }
+
+ void serial_puts(const char *str)
+ {
+ for (const char *s = str; *s; s++)
+ serial_putc(*s);
+ }
+
+
+When serial_putc() is called from within U-Boot, this code looks up the tiny-dm
+device and sends it the character.
+
+
+Potential costs and benefits
+----------------------------
+
+It is hard to estimate the savings to be had by switching a subsystem over to
+tiny-dm. Further work will illuminate this. In the example above (on x86),
+about 1KB bytes is saved (code and data), but this may or may not be
+representative of other subsystems.
+
+If all devices in an image use tiny-dm then it is possible to remove all the
+core driver-model support. This is the 3KB mentioned earlier. Of course, tiny-dm
+has its own overhead, although it is substantialy less than the full driver
+model.
+
+These benefits come with some drawbacks:
+
+ - Drivers that want to use it must implement tiny-dm in addition to their
+ normal support.
+ - of-platdata must be used. This cannot be made to work with device tree.
+ - Tiny-dm drivers have none of the rich support provided by driver model.
+ There is no pre-probe support, no concept of buses holding information
+ about child devices, no automatic pin control or power control when a
+ device is probed. Tiny-dm is designed to save memory, not to make it easy
+ to write complex device drivers.
+ - Subsystems must be fully migrated to driver model with the old code
+ removed. This is partly a technical limitation (see ns16550.c for how ugly
+ it is to support both, let alone three) and partly a quid-pro-quo for
+ this feature, since it should remove existing concerns about migrating to
+ driver model.
+
+
+Next steps
+----------
+
+This is currently an RFC so the final result may change somewhat from what is
+presented here. Some features are missing, in particular the concept of sequence
+numbers is designed but not implemented. The code is extremely rough.
+
+To judge the impact of tiny-dm a suitable board needs to be fully converted to
+it. At present I am leaning towards rock2, since it already supports
+of-platdata.
+
+The goal is to sent initial patches in June 2020 with the first version in
+mainline in July 2020 ready for the October release. Refinements based on
+feedback and patches received can come after that. It isn't clear yet when this
+could become a 'stable' feature, but likely after a release or two, perhaps with
+5-10 boards converted.
+
+
+Trying it out
+-------------
+
+The source tree is available at https://github.com/sjg20/u-boot/tree/dtoc-working
+
+Only two boards are supported at present:
+
+ - sandbox_spl - run spl/u-boot-spl to try the SPL with tiny-dm
+ - chromebook_coral - TPL uses tiny-dm
+
+
+.. [1] This discussion refers to SPL but for devices that use TPL, the same
+ features are available there.
+.. [2] https://www.elinux.org/images/c/c4/\Order_at_last_-_U-Boot_driver_model_slides_%282%29.pdf
+.. [3] https://elinux.org/images/8/82/What%27s_New_with_U-Boot_%281%29.pdf
+
+
+.. Simon Glass <sjg at chromium.org>
+.. Google LLC
+.. Memorial Day 2020
@@ -36,6 +36,94 @@ config TPL_DM
CONFIG_SPL_SYS_MALLOC_F_LEN for more details on how to enable it.
Disable this for very small implementations.
+config SPL_TINY
+ bool "Support tiny drivers in TPL without full driver-model support"
+ depends on SPL_OF_PLATDATA
+ default y
+ help
+ Enable support for reducing driver-model overhead with 'tiny'
+ devices. These drivers have very basic support and do not support
+ the full driver-model infrastructure. This can be useful for saving
+ memory in SPL.
+
+config TIMYDEV_SHRINK_DATA
+ bool "Use smaller data structures"
+ help
+ Tinydev supports storing some data structures in a smaller form to
+ save memory. However this does increase code sixe so only enable
+ this option if you have quite a lot of devices used by your board
+ in SPL/TPL.
+
+config TINYDEV_DATA_MAX_COUNT
+ int "Number of tinydev data records to allow"
+ default 10
+ help
+ With tinydev each device has a single priv pointer but it is possible
+ to attach other kinds of pointers to tiny devices using a separate
+ mechanism. This sets the maximum number that can be attached. See
+ struct tinydev_info for the details. Mostly these slots are used by
+ buses, so if you have a lot of I2C or SPI devices you may need to
+ increase it.
+
+config SPL_TINY_RELOC
+ bool "Relocate devices into allocated memory before using them"
+ depends on SPL_TINY
+ help
+ Some architectures load SPL into read-only memory. U-Boot needs write
+ access to tiny devices (specifically struct tinydev) so this cannot
+ work. If this option is enabled, U-Boot relocates these devices into
+ RAM when they are needed.
+
+config SPL_TINY_ONLY
+ bool "Only support tiny drivers in SPL, not full drivers"
+ depends on SPL_TINY
+ help
+ Enable this to drop support for full drivers in SPL. This enables
+ 'tiny' drivers for all subsystems and removes the core driver-model
+ support for full drivers. This can same space, but only works if all
+ the subsystems used by your board support tiny drivers.
+
+config TPL_TINY
+ bool "Support tiny drivers in TPL without full driver-model support"
+ depends on TPL_OF_PLATDATA
+ default y
+ help
+ Enable support for reducing driver-model overhead with 'tiny'
+ devices. These drivers have very basic support and do not support
+ the full driver-model infrastructure. This can be useful for saving
+ memory in TPL.
+
+config TPL_TINY_RELOC
+ bool "Relocate devices into allocated memory before using them"
+ depends on TPL_TINY
+ default y if X86
+ help
+ Some architectures load SPL into read-only memory. U-Boot needs write
+ access to tiny devices (specifically struct tinydev) so this cannot
+ work. If this option is enabled, U-Boot relocates these devices into
+ RAM when they are needed.
+
+config TPL_TINY_ONLY
+ bool "Only support tiny drivers in SPL, not full drivers"
+ depends on TPL_TINY
+ help
+ Enable this to drop support for full drivers in SPL. This enables
+ 'tiny' drivers for all subsystems and removes the core driver-model
+ support for full drivers. This can same space, but only works if all
+ the subsystems used by your board support tiny drivers.
+
+config TINY_CHECK
+ bool "Enable run-time consistency checks"
+ default y
+ help
+ Enabling this catches some errors like drivers with required but
+ missing options. It adds slightly to code size. Provided that your
+ code is written correct and doesn't produce errors with this option
+ enabled, it is generally safe to disable it for production.
+
+ Note that this does not add checks for drivers without an operations
+ struct. A few drivers don't need operations. Otherwise, don't do that.
+
config DM_WARN
bool "Enable warnings in driver model"
depends on DM
@@ -148,6 +236,15 @@ config SPL_SYSCON
by this uclass, including accessing registers via regmap and
assigning a unique number to each.
+config SPL_TINY_SYSCON
+ bool "Support tiny syscon drivers in SPL"
+ depends on SPL_TINY
+ default y if SPL_TINY_ONLY
+ help
+ In constrained environments the driver-model overhead of several KB
+ of code and data structures can be problematic. Enable this to use a
+ tiny implementation that only supports a single driver.
+
config TPL_SYSCON
bool "Support system controllers in TPL"
depends on TPL_REGMAP
@@ -157,6 +254,15 @@ config TPL_SYSCON
by this uclass, including accessing registers via regmap and
assigning a unique number to each.
+config TPL_TINY_SYSCON
+ bool "Support tiny syscon drivers in TPL"
+ depends on TPL_TINY
+ default y if TPL_TINY_ONLY
+ help
+ In constrained environments the driver-model overhead of several KB
+ of code and data structures can be problematic. Enable this to use a
+ tiny implementation that only supports a single driver.
+
config DEVRES
bool "Managed device resources"
depends on DM
@@ -2,7 +2,9 @@
#
# Copyright (c) 2013 Google, Inc
+ifeq ($(CONFIG_$(SPL_TPL_)TINY_ONLY),)
obj-y += device.o fdtaddr.o lists.o root.o uclass.o util.o
+endif
obj-$(CONFIG_$(SPL_TPL_)ACPIGEN) += acpi.o
obj-$(CONFIG_DEVRES) += devres.o
obj-$(CONFIG_$(SPL_)DM_DEVICE_REMOVE) += device-remove.o
@@ -15,5 +17,6 @@ ifndef CONFIG_DM_DEV_READ_INLINE
obj-$(CONFIG_OF_CONTROL) += read.o
endif
obj-$(CONFIG_OF_CONTROL) += of_extra.o ofnode.o read_extra.o
+obj-$(CONFIG_$(SPL_TPL_)TINY) += tiny.o
ccflags-$(CONFIG_DM_DEBUG) += -DDEBUG
@@ -79,11 +79,9 @@ int ofnode_decode_memory_region(ofnode config_node, const char *mem_type,
ofnode node;
if (!ofnode_valid(config_node)) {
- config_node = ofnode_path("/config");
- if (!ofnode_valid(config_node)) {
- debug("%s: Cannot find /config node\n", __func__);
+ config_node = ofnode_get_config_node();
+ if (!ofnode_valid(config_node))
return -ENOENT;
- }
}
if (!suffix)
suffix = "";
@@ -127,3 +125,46 @@ int ofnode_decode_memory_region(ofnode config_node, const char *mem_type,
return 0;
}
+
+ofnode ofnode_get_config_node(void)
+{
+ ofnode node;
+
+ node = ofnode_path("/config");
+ if (!ofnode_valid(node))
+ debug("%s: Cannot find /config node\n", __func__);
+
+ return node;
+}
+
+int ofnode_read_config_int(const char *prop_name, int default_val)
+{
+ ofnode node;
+
+ log_debug("%s\n", prop_name);
+ node = ofnode_get_config_node();
+ if (!ofnode_valid(node))
+ return default_val;
+
+ return ofnode_read_u32_default(node, prop_name, default_val);
+}
+
+int ofnode_read_config_bool(const char *prop_name)
+{
+ ofnode node;
+
+ log_debug("%s\n", prop_name);
+ node = ofnode_get_config_node();
+
+ return ofnode_read_bool(node, prop_name);
+}
+
+const char *ofnode_read_config_string(const char *prop_name)
+{
+ ofnode node;
+
+ log_debug("%s\n", prop_name);
+ node = ofnode_get_config_node();
+
+ return ofnode_read_string(node, prop_name);
+}
@@ -37,8 +37,7 @@ static struct regmap *regmap_alloc(int count)
}
#if CONFIG_IS_ENABLED(OF_PLATDATA)
-int regmap_init_mem_platdata(struct udevice *dev, fdt_val_t *reg, int count,
- struct regmap **mapp)
+int regmap_init_mem_platdata(fdt_val_t *reg, int count, struct regmap **mapp)
{
struct regmap_range *range;
struct regmap *map;
@@ -16,6 +16,17 @@
#include <dm/root.h>
#include <linux/err.h>
+void *syscon_get_first_range(ulong driver_data)
+{
+ struct regmap *map;
+
+ map = syscon_get_regmap_by_driver_data(driver_data);
+ if (IS_ERR(map))
+ return map;
+ return regmap_get_range(map, 0);
+}
+
+#if !CONFIG_IS_ENABLED(TINY_SYSCON)
/*
* Caution:
* This API requires the given device has alerady been bound to syscon driver.
@@ -52,7 +63,7 @@ static int syscon_pre_probe(struct udevice *dev)
#if CONFIG_IS_ENABLED(OF_PLATDATA)
struct syscon_base_platdata *plat = dev_get_platdata(dev);
- return regmap_init_mem_platdata(dev, plat->reg, ARRAY_SIZE(plat->reg),
+ return regmap_init_mem_platdata(plat->reg, ARRAY_SIZE(plat->reg),
&priv->regmap);
#else
return regmap_init_mem(dev_ofnode(dev), &priv->regmap);
@@ -155,16 +166,6 @@ struct regmap *syscon_get_regmap_by_driver_data(ulong driver_data)
return priv->regmap;
}
-void *syscon_get_first_range(ulong driver_data)
-{
- struct regmap *map;
-
- map = syscon_get_regmap_by_driver_data(driver_data);
- if (IS_ERR(map))
- return map;
- return regmap_get_range(map, 0);
-}
-
UCLASS_DRIVER(syscon) = {
.id = UCLASS_SYSCON,
.name = "syscon",
@@ -208,3 +209,48 @@ struct regmap *syscon_node_to_regmap(ofnode node)
return r;
}
+#else
+struct tinydev *tiny_syscon_get_by_driver_data(ulong driver_data)
+{
+ struct tinydev *tdev;
+
+ tdev = tiny_dev_get_by_drvdata(UCLASS_SYSCON, driver_data);
+ if (!tdev)
+ return NULL;
+
+ return tdev;
+}
+
+struct regmap *syscon_get_regmap_by_driver_data(ulong driver_data)
+{
+ struct syscon_uc_info *uc_priv;
+ struct tinydev *tdev;
+
+ tdev = tiny_syscon_get_by_driver_data(driver_data);
+ if (!tdev)
+ return ERR_PTR(-ENODEV);
+ /*
+ * We assume that the device has struct syscon_uc_info at the start of
+ * its private data
+ */
+ uc_priv = tinydev_get_priv(tdev);
+
+ return uc_priv->regmap;
+}
+
+int tiny_syscon_setup(struct tinydev *tdev)
+{
+ struct syscon_uc_info *priv = tinydev_get_priv(tdev);
+
+ /*
+ * With OF_PLATDATA we really have no way of knowing the format of
+ * the device-specific platform data. So we assume that it starts with
+ * a 'reg' member, and this holds a single address and size. Drivers
+ * using OF_PLATDATA will need to ensure that this is true.
+ */
+ struct syscon_base_platdata *plat = tdev->dtplat;
+
+ return regmap_init_mem_platdata(plat->reg, ARRAY_SIZE(plat->reg),
+ &priv->regmap);
+}
+#endif
new file mode 100644
@@ -0,0 +1,249 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Support for tiny device (those without a fully uclass and driver)
+ *
+ * Copyright 2020 Google LLC
+ */
+
+#define LOG_CATEGORY LOGC_TINYDEV
+
+#include <common.h>
+#include <dm.h>
+#include <log.h>
+#include <malloc.h>
+
+DECLARE_GLOBAL_DATA_PTR;
+
+const char *tiny_dev_name(struct tinydev *tdev)
+{
+ return tdev->name;
+}
+
+static struct tinydev *tiny_dev_find_tail(struct tinydev *tdev)
+{
+ if (CONFIG_IS_ENABLED(TINY_RELOC)) {
+ struct tinydev *copy;
+
+ copy = malloc(sizeof(*copy));
+ if (!copy)
+ return NULL;
+ memcpy(copy, tdev, sizeof(*copy));
+ log_debug(" - found, copied to %p\n", copy);
+ return copy;
+ }
+ log_debug(" - found at %p\n", tdev);
+
+ return tdev;
+}
+
+struct tinydev *tiny_dev_find(enum uclass_id uclass_id, int seq)
+{
+ struct tinydev *info = ll_entry_start(struct tinydev, tiny_dev);
+ const int n_ents = ll_entry_count(struct tinydev, tiny_dev);
+ struct tinydev *entry;
+
+ log_debug("find %d seq %d: n_ents=%d\n", uclass_id, seq, n_ents);
+ for (entry = info; entry != info + n_ents; entry++) {
+ struct tiny_drv *drv = entry->drv;
+
+ log_content(" - entry %p, uclass %d %d\n", entry,
+ drv->uclass_id, uclass_id);
+ if (drv->uclass_id == uclass_id)
+ return tiny_dev_find_tail(entry);
+ }
+ log_debug(" - not found\n");
+
+ return NULL;
+}
+
+int tiny_dev_probe(struct tinydev *tdev)
+{
+ struct tiny_drv *drv;
+ int ret;
+
+ if (tdev->flags & DM_FLAG_ACTIVATED)
+ return 0;
+ if (tdev->parent) {
+ ret = tiny_dev_probe(tdev->parent);
+ if (ret)
+ return log_msg_ret("parent", ret);
+ /*
+ * The device might have already been probed during the call to
+ * tiny_dev_probe() on its parent device.
+ */
+ if (tdev->flags & DM_FLAG_ACTIVATED)
+ return 0;
+ }
+ drv = tdev->drv;
+
+ if (!tdev->priv && drv->priv_size) {
+ void *priv;
+
+ // This doesn't work with TINY_RELOC
+ priv = calloc(1, drv->priv_size);
+ if (!priv)
+ return -ENOMEM;
+ tdev->priv = priv;
+ log_debug("probe: %s: priv=%p\n", tiny_dev_name(tdev), priv);
+ }
+ if (drv->probe) {
+ ret = drv->probe(tdev);
+ if (ret)
+ return log_msg_ret("probe", ret);
+ }
+
+ tdev->flags |= DM_FLAG_ACTIVATED;
+
+ return 0;
+}
+
+struct tinydev *tiny_dev_get(enum uclass_id uclass_id, int seq)
+{
+ struct tinydev *dev;
+ int ret;
+
+ dev = tiny_dev_find(uclass_id, seq);
+ if (!dev)
+ return NULL;
+
+ ret = tiny_dev_probe(dev);
+ if (ret)
+ return NULL;
+
+ return dev;
+}
+
+struct tinydev *tinydev_from_dev_idx(tinydev_idx_t index)
+{
+ struct tinydev *start = U_BOOT_TINY_DEVICE_START;
+
+ return start + index;
+}
+
+tinydev_idx_t tinydev_to_dev_idx(const struct tinydev *tdev)
+{
+ struct tinydev *start = U_BOOT_TINY_DEVICE_START;
+
+ return tdev - start;
+}
+
+struct tinydev *tinydev_get_parent(const struct tinydev *tdev)
+{
+ return tdev->parent;
+}
+
+#ifndef CONFIG_SYS_MALLOC_F
+#error "Must enable CONFIG_SYS_MALLOC_F with tinydev"
+#endif
+
+static void *tinydev_lookup_data(struct tinydev *tdev, enum dm_data_t type)
+{
+ struct tinydev_info *info = &((gd_t *)gd)->tinydev_info;
+ struct tinydev_data *data;
+ int i;
+#ifdef TIMYDEV_SHRINK_DATA
+ uint idx = tinydev_to_dev_idx(tdev);
+
+ for (i = 0, data = info->data; i < info->data_count; i++, data++) {
+ if (data->type == type && data->tdev_idx == idx)
+ return malloc_ofs_to_ptr(data->ofs);
+ }
+#else
+ for (i = 0, data = info->data; i < info->data_count; i++, data++) {
+ if (data->type == type && data->tdev == tdev)
+ return data->ptr;
+ }
+#endif
+
+ return NULL;
+}
+
+void *tinydev_alloc_data(struct tinydev *tdev, enum dm_data_t type, int size)
+{
+ struct tinydev_info *info = &((gd_t *)gd)->tinydev_info;
+ struct tinydev_data *data;
+ void *ptr;
+
+ if (info->data_count == ARRAY_SIZE(info->data)) {
+ /* To fix this, increase CONFIG_TINYDEV_DATA_MAX_COUNT */
+ panic("tinydev data exhusted");
+ return NULL;
+ }
+ data = &info->data[info->data_count];
+ ptr = calloc(1, size);
+ if (!ptr)
+ return NULL; /* alloc_simple() has already written a message */
+ data->type = type;
+#ifdef TIMYDEV_SHRINK_DATA
+ data->tdev_idx = tinydev_to_dev_idx(tdev);
+ data->ofs = malloc_ptr_to_ofs(ptr);
+#else
+ data->tdev = tdev;
+ data->ptr = ptr;
+#endif
+ log_debug("alloc_data: %d: %s: tdev=%p, type=%d, size=%x, ptr=%p\n",
+ info->data_count, tiny_dev_name(tdev), tdev, type, size, ptr);
+ info->data_count++;
+
+ return ptr;
+}
+
+void *tinydev_ensure_data(struct tinydev *tdev, enum dm_data_t type, int size,
+ bool *existsp)
+{
+ bool exists = true;
+ void *ptr;
+
+ ptr = tinydev_lookup_data(tdev, type);
+ if (!ptr) {
+ exists = false;
+ ptr = tinydev_alloc_data(tdev, type, size);
+ }
+ if (existsp)
+ *existsp = exists;
+
+ return ptr;
+}
+
+void *tinydev_get_data(struct tinydev *tdev, enum dm_data_t type)
+{
+ void *ptr = tinydev_lookup_data(tdev, type);
+
+ if (!ptr) {
+ log_debug("Cannot find type %d for device %p\n", type, tdev);
+ panic("tinydev missing data");
+ }
+
+ return ptr;
+}
+
+struct tinydev *tiny_dev_get_by_drvdata(enum uclass_id uclass_id,
+ ulong driver_data)
+{
+ struct tinydev *info = ll_entry_start(struct tinydev, tiny_dev);
+ const int n_ents = ll_entry_count(struct tinydev, tiny_dev);
+ struct tinydev *entry;
+
+ log_debug("find %d driver_data %lx: n_ents=%d\n", uclass_id,
+ driver_data, n_ents);
+ for (entry = info; entry != info + n_ents; entry++) {
+ struct tiny_drv *drv = entry->drv;
+
+ log_content(" - entry %p, uclass %d, driver_data %x\n", entry,
+ drv->uclass_id, entry->driver_data);
+ if (drv->uclass_id == uclass_id &&
+ entry->driver_data == driver_data) {
+ struct tinydev *tdev = tiny_dev_find_tail(entry);
+ int ret;
+
+ if (!tdev)
+ return NULL;
+ ret = tiny_dev_probe(tdev);
+ if (ret)
+ return NULL;
+ return tdev;
+ }
+ }
+
+ return NULL;
+}
@@ -11,6 +11,7 @@
#define _DM_DEVICE_H
#include <dm/ofnode.h>
+#include <dm/tiny_struct.h>
#include <dm/uclass-id.h>
#include <fdtdec.h>
#include <linker_lists.h>
@@ -289,6 +290,126 @@ struct driver {
*/
#define U_BOOT_DRIVER_ALIAS(__name, __alias)
+/**
+ * struct tiny_drv: A tiny driver
+ *
+ * This provides a smaller driver than the full-blown driver-model. It is
+ * intended for SPL or TPL and offers just a probe() function and some basic
+ * operations. It has a limit of 256 bytes for the private size.
+ *
+ * Note that tiny drivers exist alongside normal ones. Each subsystem must be
+ * enabled for tiny drivers (e.g. CONFIG_SPL_TINY_SERIAL for serial).
+ *
+ * Naming here is changed from struct driver, to make it easier to search code.
+ *
+ * @uclass_id: Identifies the uclass we belong to
+ * @priv_size: If non-zero this is the size of the private data to be allocated
+ * in the device's ->priv pointer. If zero, then the driver is responsible for
+ * allocating any data required (but malloc() is discouraged in tiny drivers)
+ * @ops: Driver-specific operations. This is typically a list of function
+ * pointers defined by the driver, to implement driver functions required by
+ * the uclass.
+ */
+struct tiny_drv {
+ u8 uclass_id;
+ u8 priv_size;
+ int (*probe)(struct tinydev *dev);
+ struct tinydev *tdev;
+ void *ops;
+};
+
+/* Declare a new 'tiny' U-Boot driver */
+#define U_BOOT_TINY_DRIVER(__name) \
+ ll_entry_declare(struct tiny_drv, __name, tiny_drv)
+
+/* Get a pointer to a given tiny driver */
+#define DM_GET_TINY_DRIVER(__name) \
+ ll_entry_get(struct tiny_drv, __name, tiny_drv)
+
+#define DM_DECL_TINY_DRIVER(__name) \
+ ll_entry_decl(struct tiny_drv, __name, tiny_drv)
+
+/*
+ * Get a pointer to a given tiny driver, for use in data structures. This
+ * requires that the symbol be declared with DM_DECL_TINY_DRIVER() first
+ */
+#define DM_REF_TINY_DRIVER(__name) \
+ ll_entry_ref(struct tiny_drv, __name, tiny_drv)
+
+/**
+ * DM_TINY_PRIV() - Specifies the size of the private data
+ *
+ * This does not generate any code, but is parsed by dtoc. Put it inside the
+ * U_BOOT_TINY_DRIVER on its own lines to specify the amount of data to be
+ * allocated in tiny_dev->priv
+ */
+#define DM_TINY_PRIV(hdr,size) .priv_size = size,
+
+/**
+ * struct tinydev - A tiny device
+ *
+ * This does not have a separate struct driver_info like full devices. The
+ * platform data is combined into this struct, which is used by dtoc to declare
+ * devices it finds in the devicetree.
+ *
+ * @dtplat: Pointer to the platform data, as generated by dtoc
+ * @drv: Pointer to the driver
+ * @flags: Flags for this device DM_FLAG_...
+ */
+struct tinydev {
+ const char *name; /* allow this to be dropped */
+ /* TODO: Convert these into ushort offsets to a base address */
+ void *dtplat; /* u16 word index into dtd data section */
+ void *priv; /* u16 word index into device priv section */
+ struct tiny_drv *drv; /* u8 index into driver list? */
+ u16 flags; /* switch to u8? */
+ u8 driver_data;
+ struct tinydev *parent;
+};
+
+/* Declare a tiny device with a given name */
+#define U_BOOT_TINY_DEVICE(__name) \
+ ll_entry_declare(struct tinydev, __name, tiny_dev)
+
+#define DM_REF_TINY_DEVICE(__name) \
+ ll_entry_ref(struct tinydev, __name, tiny_dev)
+
+/**
+ * U_BOOT_TINY_DEVICE_START - Find the start of the list of tiny devices
+ *
+ * Use this like this:
+ * struct tinydev *start = U_BOOT_TINY_DEVICE_START;
+ */
+#define U_BOOT_TINY_DEVICE_START \
+ ll_entry_start(struct tinydev, tiny_dev)
+
+struct tinydev *tiny_dev_find(enum uclass_id uclass_id, int seq);
+
+int tiny_dev_probe(struct tinydev *tdev);
+
+struct tinydev *tiny_dev_get(enum uclass_id uclass_id, int seq);
+
+struct tinydev *tinydev_from_dev_idx(tinydev_idx_t index);
+
+tinydev_idx_t tinydev_to_dev_idx(const struct tinydev *tdev);
+
+struct tinydev *tinydev_get_parent(const struct tinydev *tdev);
+
+static inline void *tinydev_get_priv(const struct tinydev *tdev)
+{
+ return tdev->priv;
+}
+
+void *tinydev_get_data(struct tinydev *tdev, enum dm_data_t type);
+
+void *tinydev_alloc_data(struct tinydev *tdev, enum dm_data_t type, int size);
+
+void *tinydev_ensure_data(struct tinydev *tdev, enum dm_data_t type, int size,
+ bool *existsp);
+
+struct tinydev *tiny_dev_get_by_drvdata(enum uclass_id uclass_id,
+ ulong driver_data);
+
/**
* dev_get_platdata() - Get the platform data for a device
*
@@ -86,4 +86,10 @@ int ofnode_decode_memory_region(ofnode config_node, const char *mem_type,
const char *suffix, fdt_addr_t *basep,
fdt_size_t *sizep);
+ofnode ofnode_get_config_node(void);
+
+int ofnode_read_config_int(const char *prop_name, int default_val);
+int ofnode_read_config_bool(const char *prop_name);
+const char *ofnode_read_config_string(const char *prop_name);
+
#endif
@@ -45,7 +45,22 @@ struct driver_info {
#define U_BOOT_DEVICES(__name) \
ll_entry_declare_list(struct driver_info, __name, driver_info)
-/* Get a pointer to a given driver */
+/**
+ * Get a pointer to a given device info given its name
+ *
+ * With the declaration U_BOOT_DEVICE(name), DM_GET_DEVICE(name) will return a
+ * pointer to the struct driver_info created by that declaration.
+ *
+ * if OF_PLATDATA is enabled, from this it is possible to use the @dev member of
+ * struct driver_info to find the device pointer itself.
+ *
+ * TODO(sjg at chromium.org): U_BOOT_DEVICE() tells U-Boot to create a device, so
+ * the naming seems sensible, but DM_GET_DEVICE() is a bit of misnomer, since it
+ * finds the driver_info record, not the device.
+ *
+ * @__name: Driver name (C identifier, not a string. E.g. gpio7_at_ff7e0000)
+ * @return struct driver_info * to the driver that created the device
+ */
#define DM_GET_DEVICE(__name) \
ll_entry_get(struct driver_info, __name, driver_info)
@@ -57,4 +72,7 @@ struct driver_info {
* by dtoc when parsing dtb.
*/
void dm_populate_phandle_data(void);
+
+#define IF_OF_PLATDATA(x) CONFIG_IS_ENABLED(OF_PLATDATA, (x))
+
#endif
new file mode 100644
@@ -0,0 +1,42 @@
+/* SPDX-License-Identifier: GPL-2.0+ */
+/*
+ * Structures for inclusion in global_data
+ *
+ * Copyright 2020 Google LLC
+ * Written by Simon Glass <sjg at chromium.org>
+ */
+
+#ifndef __DM_TINY_STRUCT_H
+#define __DM_TINY_STRUCT_H
+
+/* A struct tinydev * stored as an index into the device linker-list */
+typedef u8 tinydev_idx_t;
+
+/* enum dm_data_t - Types of data that can be attached to devices */
+enum dm_data_t {
+ DEVDATAT_PLAT,
+ DEVDATAT_PARENT_PLAT,
+ DEVDATAT_UC_PLAT,
+
+ DEVDATAT_PRIV,
+ DEVDATAT_PARENT_PRIV,
+ DEVDATAT_UC_PRIV,
+};
+
+struct tinydev_data {
+ u8 type;
+#ifdef TIMYDEV_SHRINK_DATA
+ tinydev_idx_t tdev_idx;
+ ushort ofs;
+#else
+ struct tinydev *tdev;
+ void *ptr;
+#endif
+};
+
+struct tinydev_info {
+ int data_count;
+ struct tinydev_data data[CONFIG_TINYDEV_DATA_MAX_COUNT];
+};
+
+#endif
@@ -210,6 +210,12 @@
_ll_result; \
})
+#define ll_entry_decl(_type, _name, _list) \
+ extern _type _u_boot_list_2_##_list##_2_##_name
+
+#define ll_entry_ref(_type, _name, _list) \
+ (_type *)&_u_boot_list_2_##_list##_2_##_name
+
/**
* ll_start() - Point to first entry of first linker-generated array
* @_type: Data type of the entry
@@ -303,10 +303,12 @@ $(obj)/$(SPL_BIN).dtb: dts/dt-spl.dtb FORCE
pythonpath = PYTHONPATH=scripts/dtc/pylibfdt
quiet_cmd_dtocc = DTOC C $@
-cmd_dtocc = $(pythonpath) $(srctree)/tools/dtoc/dtoc -d $(obj)/$(SPL_BIN).dtb -o $@ platdata
+cmd_dtocc = $(pythonpath) $(srctree)/tools/dtoc/dtoc -d $(obj)/$(SPL_BIN).dtb \
+ -o $@ -c $(KCONFIG_CONFIG) -s $(srctree) platdata
quiet_cmd_dtoch = DTOC H $@
-cmd_dtoch = $(pythonpath) $(srctree)/tools/dtoc/dtoc -d $(obj)/$(SPL_BIN).dtb -o $@ struct
+cmd_dtoch = $(pythonpath) $(srctree)/tools/dtoc/dtoc -d $(obj)/$(SPL_BIN).dtb \
+ -o $@ -s $(srctree) struct
quiet_cmd_plat = PLAT $@
cmd_plat = $(CC) $(c_flags) -c $< -o $(filter-out $(PHONY),$@)
@@ -21,6 +21,7 @@ import sys
from dtoc import fdt
from dtoc import fdt_util
+from patman import tools
# When we see these properties we ignore them - i.e. do not create a structure
# member
@@ -56,6 +57,22 @@ VAL_PREFIX = 'dtv_'
# phandles is len(args). This is a list of integers.
PhandleInfo = collections.namedtuple('PhandleInfo', ['max_args', 'args'])
+class DriverInfo:
+ def __init__(self, name, uclass_id, compat):
+ self.name = name
+ self.uclass_id = uclass_id
+ self.compat = compat
+ self.priv_size = 0
+
+ def __eq__(self, other):
+ return (self.name == other.name and
+ self.uclass_id == other.uclass_id and
+ self.compat == other.compat and
+ self.priv_size == other.priv_size)
+
+ def __repr__(self):
+ return ("DriverInfo(name='%s', uclass_id='%s', compat=%s, priv_size=%s)" %
+ (self.name, self.uclass_id, self.compat, self.priv_size))
def conv_name_to_c(name):
"""Convert a device-tree name to a C identifier
@@ -144,6 +161,7 @@ class DtbPlatdata(object):
Properties:
_fdt: Fdt object, referencing the device tree
_dtb_fname: Filename of the input device tree binary file
+ _config_fname: Filename of the .config file for the build
_valid_nodes: A list of Node object with compatible strings
_include_disabled: true to include nodes marked status = "disabled"
_outfile: The current output file (sys.stdout or a real file)
@@ -158,19 +176,27 @@ class DtbPlatdata(object):
U_BOOT_DRIVER_ALIAS(driver_alias, driver_name)
value: Driver name declared with U_BOOT_DRIVER(driver_name)
_links: List of links to be included in dm_populate_phandle_data()
+ _tiny_uclasses: List of uclass names that are marked as 'tiny'
"""
- def __init__(self, dtb_fname, include_disabled, warning_disabled):
+ def __init__(self, dtb_fname, config_fname, include_disabled,
+ warning_disabled):
self._fdt = None
self._dtb_fname = dtb_fname
+ self._config_fname = config_fname
self._valid_nodes = None
self._include_disabled = include_disabled
self._outfile = None
self._warning_disabled = warning_disabled
self._lines = []
- self._aliases = {}
- self._drivers = []
+ self._compat_aliases = {}
+ self._drivers = {}
self._driver_aliases = {}
self._links = []
+ self._aliases = {}
+ self._aliases_by_path = {}
+ self._tiny_uclasses = []
+ self._of_match = {}
+ self._compat_to_driver = {}
def get_normalized_compat_name(self, node):
"""Get a node's normalized compat name
@@ -195,8 +221,8 @@ class DtbPlatdata(object):
compat_c = self._driver_aliases.get(compat_c)
if not compat_c:
if not self._warning_disabled:
- print('WARNING: the driver %s was not found in the driver list'
- % (compat_c_old))
+ print('WARNING: the driver %s was not found in the driver list. Check that your driver has the same name as one of its compatible strings' %
+ (compat_c_old))
compat_c = compat_c_old
else:
aliases_c = [compat_c_old] + aliases_c
@@ -217,6 +243,10 @@ class DtbPlatdata(object):
else:
self._outfile = open(fname, 'w')
+ def close_output(self):
+ if self._outfile is not sys.stdout:
+ self._outfile.close()
+
def out(self, line):
"""Output a string to the output file
@@ -287,8 +317,8 @@ class DtbPlatdata(object):
break
target = self._fdt.phandle_to_node.get(phandle)
if not target:
- raise ValueError("Cannot parse '%s' in node '%s'" %
- (prop.name, node_name))
+ raise ValueError("Cannot parse '%s' in node '%s' (phandle=%d)" %
+ (prop.name, node_name, phandle))
cells = None
for prop_name in ['#clock-cells', '#gpio-cells']:
cells = target.props.get(prop_name)
@@ -315,33 +345,144 @@ class DtbPlatdata(object):
with open(fn) as fd:
buff = fd.read()
- # The following re will search for driver names declared as
- # U_BOOT_DRIVER(driver_name)
- drivers = re.findall('U_BOOT_DRIVER\((.*)\)', buff)
+ drivers = {}
+
+ # Dict of compatible strings in a udevice_id array:
+ # key: udevice_id array name (e.g. 'rk3288_syscon_ids_noc')
+ # value: Dict of compatible strings in that array:
+ # key: Compatible string, e.g. 'rockchip,rk3288-grf'
+ # value: Driver data, e,g, 'ROCKCHIP_SYSCON_GRF', or None
+ of_match = {}
+
+ m_drivers = re.findall(r'U_BOOT_DRIVER\((.*)\)', buff)
+ if m_drivers:
+ driver_name = None
+
+ # Collect the uclass ID, e.g. 'UCLASS_SPI'
+ uclass_id = None
+ re_id = re.compile(r'\s*\.id\s*=\s*(UCLASS_[A-Z0-9_]+)')
+
+ # Collect the compatible string, e.g. 'rockchip,rk3288-grf'
+ compat = None
+ #re_compat = re.compile('{\s*.compatible\s*=\s*"(.*)"\s*},')
+
+ re_compat = re.compile('{\s*.compatible\s*=\s*"(.*)"\s*'
+ '(,\s*.data\s*=\s*(.*))?\s*},')
+
+ # This is a dict of compatible strings that were found:
+ # key: Compatible string, e.g. 'rockchip,rk3288-grf'
+ # value: Driver data, e,g, 'ROCKCHIP_SYSCON_GRF', or None
+ compat_dict = {}
+
+ # Holds the var nane of the udevice_id list, e.g.
+ # 'rk3288_syscon_ids_noc' in
+ # static const struct udevice_id rk3288_syscon_ids_noc[] = {
+ ids_name = None
+ re_ids = re.compile('struct udevice_id (.*)\[\]\s*=')
+
+ # Matches the references to the udevice_id list
+ re_of_match = re.compile('\.of_match\s*=\s*([a-z0-9_]+),')
+
+ # Matches the header/size information for tinydev
+ re_tiny_priv = re.compile('^\s*DM_TINY_PRIV\((.*)\)$')
+ tiny_name = None
+
+ prefix = ''
+ for line in buff.splitlines():
+ # Handle line continuation
+ if prefix:
+ line = prefix + line
+ prefix = ''
+ if line.endswith('\\'):
+ prefix = line[:-1]
+ continue
- for driver in drivers:
- self._drivers.append(driver)
+ # If we have seen U_BOOT_DRIVER()...
+ if driver_name:
+ id_m = re_id.search(line)
+ id_of_match = re_of_match.search(line)
+ if id_m:
+ uclass_id = id_m.group(1)
+ elif id_of_match:
+ compat = id_of_match.group(1)
+ elif '};' in line:
+ if uclass_id and compat:
+ if compat not in of_match:
+ raise ValueError("%s: Unknown compatible var '%s' (found %s)" %
+ (fn, compat, ','.join(of_match.keys())))
+ driver = DriverInfo(driver_name, uclass_id,
+ of_match[compat])
+ drivers[driver_name] = driver
+
+ # This needs to be deterministic, since a driver may
+ # have multiple compatible strings pointing to it.
+ # We record the one earliest in the alphabet so it
+ # will produce the same result on all machines.
+ for id in of_match[compat]:
+ old = self._compat_to_driver.get(id)
+ if not old or driver.name < old.name:
+ self._compat_to_driver[id] = driver
+ else:
+ pass
+ #print("%s: Cannot find .id/.of_match in driver '%s': uclass_id=%s, compat=%s" %
+ #(fn, driver_name, uclass_id, compat))
+ driver_name = None
+ uclass_id = None
+ ids_name = None
+ compat = None
+ compat_dict = {}
+
+ # If we have seen U_BOOT_TINY_DRIVER()...
+ elif tiny_name:
+ tiny_priv = re_tiny_priv.match(line)
+ if tiny_priv:
+ drivers[tiny_name].priv_size = tiny_priv.group(1)
+ elif '};' in line:
+ tiny_name = None
+ elif ids_name:
+ compat_m = re_compat.search(line)
+ if compat_m:
+ compat_dict[compat_m.group(1)] = compat_m.group(3)
+ elif '};' in line:
+ of_match[ids_name] = compat_dict
+ ids_name = None
+ elif 'U_BOOT_DRIVER' in line:
+ match = re.search(r'U_BOOT_DRIVER\((.*)\)', line)
+ if match:
+ driver_name = match.group(1)
+ elif 'U_BOOT_TINY_DRIVER' in line:
+ match = re.search(r'U_BOOT_TINY_DRIVER\((.*)\)', line)
+ if match:
+ tiny_name = match.group(1)
+ if tiny_name not in drivers:
+ raise ValueError("%s: Tiny driver '%s' must have a corresponding full driver in the same file (found %s)" %
+ (fn, tiny_name, drivers))
+ else:
+ ids_m = re_ids.search(line)
+ if ids_m:
+ ids_name = ids_m.group(1)
- # The following re will search for driver aliases declared as
- # U_BOOT_DRIVER_ALIAS(alias, driver_name)
- driver_aliases = re.findall('U_BOOT_DRIVER_ALIAS\(\s*(\w+)\s*,\s*(\w+)\s*\)',
- buff)
+ self._drivers.update(drivers)
+ self._of_match.update(of_match)
- for alias in driver_aliases: # pragma: no cover
- if len(alias) != 2:
- continue
- self._driver_aliases[alias[1]] = alias[0]
+ # The following re will search for driver aliases declared as
+ # U_BOOT_DRIVER_ALIAS(alias, driver_name)
+ driver_aliases = re.findall(
+ 'U_BOOT_DRIVER_ALIAS\(\s*(\w+)\s*,\s*(\w+)\s*\)',
+ buff)
- def scan_drivers(self):
+ for alias in driver_aliases: # pragma: no cover
+ if len(alias) != 2:
+ continue
+ self._driver_aliases[alias[1]] = alias[0]
+
+ def scan_drivers(self, srcpath):
"""Scan the driver folders to build a list of driver names and aliases
This procedure will populate self._drivers and self._driver_aliases
"""
- basedir = sys.argv[0].replace('tools/dtoc/dtoc', '')
- if basedir == '':
- basedir = './'
- for (dirpath, dirnames, filenames) in os.walk(basedir):
+ for (dirpath, dirnames, filenames) in os.walk(srcpath):
for fn in filenames:
if not fn.endswith('.c'):
continue
@@ -355,23 +496,30 @@ class DtbPlatdata(object):
"""
self._fdt = fdt.FdtScan(self._dtb_fname)
- def scan_node(self, root):
+ def scan_node(self, parent, level):
"""Scan a node and subnodes to build a tree of node and phandle info
This adds each node to self._valid_nodes.
Args:
- root: Root node for scan
+ parent: Parent node for scan
"""
- for node in root.subnodes:
+ for node in parent.subnodes:
if 'compatible' in node.props:
status = node.props.get('status')
if (not self._include_disabled and not status or
status.value != 'disabled'):
self._valid_nodes.append(node)
+ if level == 0 and node.name == 'aliases':
+ for prop in node.props.values():
+ self._aliases[prop.name] = prop.value
+ match = re.match('^(.*[a-z])[0-9]+', prop.name)
+ if match:
+ self._aliases_by_path[prop.value] = match.group(1)
+
# recurse to handle any subnodes
- self.scan_node(node)
+ self.scan_node(node, level + 1)
def scan_tree(self):
"""Scan the device tree for useful information
@@ -381,7 +529,26 @@ class DtbPlatdata(object):
platform data
"""
self._valid_nodes = []
- return self.scan_node(self._fdt.GetRoot())
+ self.scan_node(self._fdt.GetRoot(), 0)
+
+ def parse_config(self, config_data):
+ tiny_list = re.findall(r'CONFIG_[ST]PL_TINY_(.*)=y', config_data)
+ self._tiny_uclasses = [n.lower() for n in tiny_list
+ if n not in ['MEMSET', 'RELOC', 'ONLY']]
+
+ def scan_config(self):
+ if self._config_fname:
+ self.parse_config(tools.ReadFile(self._config_fname, binary=False))
+ unused = set(self._tiny_uclasses)
+ for node in self._valid_nodes:
+ node.is_tiny = False
+ alias = self._aliases_by_path.get(node.path)
+ if alias and alias in self._tiny_uclasses:
+ node.is_tiny = True
+ unused.discard(alias)
+ if unused:
+ print('Warning: Some tiny uclasses lack aliases or a device: %s' %
+ ', '.join(unused))
@staticmethod
def get_num_cells(node):
@@ -491,7 +658,7 @@ class DtbPlatdata(object):
struct_name, aliases = self.get_normalized_compat_name(node)
for alias in aliases:
- self._aliases[alias] = struct_name
+ self._compat_aliases[alias] = struct_name
return structs
@@ -555,7 +722,7 @@ class DtbPlatdata(object):
self.out(';\n')
self.out('};\n')
- for alias, struct_name in self._aliases.items():
+ for alias, struct_name in self._compat_aliases.items():
if alias not in sorted(structs):
self.out('#define %s%s %s%s\n'% (STRUCT_PREFIX, alias,
STRUCT_PREFIX, struct_name))
@@ -600,7 +767,8 @@ class DtbPlatdata(object):
(VAL_PREFIX, var_name, member_name, item)
# Save the the link information to be use to define
# dm_populate_phandle_data()
- self._links.append({'var_node': var_node, 'dev_name': name})
+ if not target_node.is_tiny:
+ self._links.append({'var_node': var_node, 'dev_name': name})
item += 1
for val in vals:
self.buf('\n\t\t%s,' % val)
@@ -617,6 +785,8 @@ class DtbPlatdata(object):
struct_name, _ = self.get_normalized_compat_name(node)
var_name = conv_name_to_c(node.name)
+
+ # Tiny devices don't have 'static' since it is used by the driver
self.buf('static struct %s%s %s%s = {\n' %
(STRUCT_PREFIX, struct_name, VAL_PREFIX, var_name))
for pname in sorted(node.props):
@@ -634,13 +804,59 @@ class DtbPlatdata(object):
self.buf(',\n')
self.buf('};\n')
- # Add a device declaration
- self.buf('U_BOOT_DEVICE(%s) = {\n' % var_name)
- self.buf('\t.name\t\t= "%s",\n' % struct_name)
- self.buf('\t.platdata\t= &%s%s,\n' % (VAL_PREFIX, var_name))
- self.buf('\t.platdata_size\t= sizeof(%s%s),\n' % (VAL_PREFIX, var_name))
- self.buf('};\n')
- self.buf('\n')
+ if node.is_tiny:
+ val = node.props['compatible'].value
+ if not isinstance(val, list):
+ val = [val]
+ for compat in val:
+ driver = self._compat_to_driver.get(compat)
+ if driver:
+ break
+ if not driver:
+ raise ValueError("Cant' find driver for compatible '%s' (%s)'" %
+ (', '.join(val), 'all'))
+ self.buf('DM_DECL_TINY_DRIVER(%s);\n' % driver.name);
+ priv_name = None
+ inline = True
+ if inline and driver.priv_size:
+ parts = driver.priv_size.split(',')
+ if len(parts) == 2:
+ hdr, size = parts
+ else:
+ hdr = None
+ size = parts[0]
+ priv_name = '_%s_priv' % var_name
+ if hdr:
+ self.buf('#include %s\n' % hdr)
+ section = '__attribute__ ((section (".data")))'
+
+ self.buf('u8 %s[%s] %s;\n' % (priv_name, size.strip(), section))
+
+ self.buf('U_BOOT_TINY_DEVICE(%s) = {\n' % var_name)
+ self.buf('\t.dtplat\t\t= &%s%s,\n' % (VAL_PREFIX, var_name))
+ self.buf('\t.drv\t\t= DM_REF_TINY_DRIVER(%s),\n' % driver.name)
+ driver_data = driver.compat[compat]
+ if driver_data is not None:
+ self.buf('\t.driver_data\t\t= %s,\n' % driver_data)
+ if priv_name:
+ self.buf('\t.priv\t\t= %s,\n' % priv_name)
+ self.buf('\t.name\t\t= "%s",\n' % node.name)
+ if node.parent and node.parent.name != '/':
+ parent_name = conv_name_to_c(node.parent.name)
+ self.buf('\t.parent\t\t= DM_REF_TINY_DEVICE(%s),\n' %
+ parent_name)
+ self.buf('};\n')
+ self.buf('\n')
+ else:
+ # Add a device declaration
+ self.buf('U_BOOT_DEVICE(%s) = {\n' % var_name)
+ self.buf('\t.name\t\t= "%s",\n' % struct_name)
+ self.buf('\t.platdata\t= &%s%s,\n' % (VAL_PREFIX, var_name))
+ self.buf('\t.platdata_size\t= sizeof(%s%s),\n' %
+ (VAL_PREFIX, var_name))
+ self.buf('\t.dev\t\t= NULL,\n')
+ self.buf('};\n')
+ self.buf('\n')
self.out(''.join(self.get_buf()))
@@ -659,6 +875,15 @@ class DtbPlatdata(object):
self.out('#include <dm.h>\n')
self.out('#include <dt-structs.h>\n')
self.out('\n')
+
+ self.out('/*\n')
+ self.out(' * Tiny uclasses: %s\n' % (', '.join(self._tiny_uclasses)))
+ self.out(' * Aliases with CONFIG_SPL_TINY_... enabled\n')
+ for path, alias in self._aliases_by_path.items():
+ if alias in self._tiny_uclasses:
+ self.out(' * %s: %s\n' % (path, alias))
+ self.out('*/\n')
+ self.out('\n')
nodes_to_output = list(self._valid_nodes)
# Keep outputing nodes until there is none left
@@ -682,8 +907,10 @@ class DtbPlatdata(object):
self.buf('}\n')
self.out(''.join(self.get_buf()))
+ self.close_output()
-def run_steps(args, dtb_file, include_disabled, output, warning_disabled=False):
+def run_steps(args, dtb_file, config_file, include_disabled, output, srcpath,
+ warning_disabled=False):
"""Run all the steps of the dtoc tool
Args:
@@ -697,10 +924,12 @@ def run_steps(args, dtb_file, include_disabled, output, warning_disabled=False):
if not args:
raise ValueError('Please specify a command: struct, platdata')
- plat = DtbPlatdata(dtb_file, include_disabled, warning_disabled)
- plat.scan_drivers()
+ plat = DtbPlatdata(dtb_file, config_file, include_disabled,
+ warning_disabled)
+ plat.scan_drivers(srcpath)
plat.scan_dtb()
plat.scan_tree()
+ plat.scan_config()
plat.scan_reg_sizes()
plat.setup_output(output)
structs = plat.scan_structs()
@@ -714,3 +943,4 @@ def run_steps(args, dtb_file, include_disabled, output, warning_disabled=False):
else:
raise ValueError("Unknown command '%s': (use: struct, platdata)" %
cmd)
+ return plat
@@ -10,6 +10,12 @@
/ {
#address-cells = <1>;
#size-cells = <1>;
+
+ aliases {
+ i2c0 = &i2c0;
+ serial0 = &serial0;
+ };
+
spl-test {
u-boot,dm-pre-reloc;
compatible = "sandbox,spl-test";
@@ -47,7 +53,7 @@
compatible = "sandbox,spl-test.2";
};
- i2c at 0 {
+ i2c0: i2c at 0 {
compatible = "sandbox,i2c-test";
u-boot,dm-pre-reloc;
#address-cells = <1>;
@@ -59,4 +65,8 @@
low-power;
};
};
+
+ serial0: serial {
+ compatible = "sandbox,serial";
+ };
};
@@ -60,9 +60,9 @@ def BytesToValue(data):
Type of data
Data, either a single element or a list of elements. Each element
is one of:
- Type.STRING: str/bytes value from the property
- Type.INT: a byte-swapped integer stored as a 4-byte str/bytes
- Type.BYTE: a byte stored as a single-byte str/bytes
+ Type.STRING: str value from the property
+ Type.INT: a big-endian integer stored as a 4-byte bytes
+ Type.BYTE: a byte stored as a single-byte bytes
"""
data = bytes(data)
size = len(data)
@@ -104,6 +104,7 @@ class Prop:
Properties:
name: Property name (as per the device tree)
+ bytes: Property value as raw bytes
value: Property value as a string of bytes, or a list of strings of
bytes
type: Value type
@@ -87,6 +87,8 @@ if __name__ != '__main__':
parser = OptionParser()
parser.add_option('-B', '--build-dir', type='string', default='b',
help='Directory containing the build output')
+parser.add_option('-c', '--config', action='store',
+ help='Select .config filename')
parser.add_option('-d', '--dtb-file', action='store',
help='Specify the .dtb input file')
parser.add_option('--include-disabled', action='store_true',
@@ -95,6 +97,8 @@ parser.add_option('-o', '--output', action='store', default='-',
help='Select output filename')
parser.add_option('-P', '--processes', type=int,
help='set number of processes to use for running tests')
+parser.add_option('-s', '--srcpath', type='string',
+ help='Specify the source directory for U-Boot')
parser.add_option('-t', '--test', action='store_true', dest='test',
default=False, help='run tests')
parser.add_option('-T', '--test-coverage', action='store_true',
@@ -110,5 +114,6 @@ elif options.test_coverage:
RunTestCoverage()
else:
- dtb_platdata.run_steps(args, options.dtb_file, options.include_disabled,
- options.output)
+ dtb_platdata.run_steps(args, options.dtb_file, options.config,
+ options.include_disabled, options.output,
+ options.srcpath)
@@ -11,12 +11,14 @@ tool.
import collections
import os
+import re
import struct
import sys
import unittest
from dtoc import dtb_platdata
from dtb_platdata import conv_name_to_c
+from dtb_platdata import DriverInfo
from dtb_platdata import get_compat_name
from dtb_platdata import get_value
from dtb_platdata import tab_to
@@ -52,6 +54,38 @@ C_EMPTY_POPULATE_PHANDLE_DATA = '''void dm_populate_phandle_data(void) {
}
'''
+CONFIG_FILE_DATA = '''
+CONFIG_SOMETHING=1234
+CONFIG_SPL_TINY_SPI=y
+# CONFIG_SPL_TINY_SPI is not set
+CONFIG_SPL_TINY_I2C=y
+CONFIG_SOMETHING_ELSE=5678
+'''
+
+DRIVER_FILE_DATA = '''
+static const struct udevice_id xhci_usb_ids[] = {
+ { .compatible = "rockchip,rk3328-xhci" },
+ { }
+};
+
+U_BOOT_DRIVER(usb_xhci) = {
+ .name = "xhci_rockchip",
+ .id = UCLASS_USB,
+ .of_match = xhci_usb_ids,
+ .ops = &xhci_usb_ops,
+};
+
+static const struct udevice_id usb_phy_ids[] = {
+ { .compatible = "rockchip,rk3328-usb3-phy" },
+ { }
+};
+
+U_BOOT_DRIVER(usb_phy) = {
+ .name = "usb_phy_rockchip",
+ .id = UCLASS_PHY,
+ .of_match = usb_phy_ids,
+};
+'''
def get_dtb_file(dts_fname, capture_stderr=False):
"""Compile a .dts file to a .dtb
@@ -75,7 +109,8 @@ class TestDtoc(unittest.TestCase):
@classmethod
def tearDownClass(cls):
- tools._RemoveOutputDir()
+ #tools._RemoveOutputDir()
+ pass
def _WritePythonString(self, fname, data):
"""Write a string with tabs expanded as done in this Python file
@@ -860,3 +895,57 @@ U_BOOT_DEVICE(spl_test2) = {
self.run_test(['invalid-cmd'], dtb_file, output)
self.assertIn("Unknown command 'invalid-cmd': (use: struct, platdata)",
str(e.exception))
+
+ def testAliases(self):
+ """Test we can get a list of aliases"""
+ dtb_file = get_dtb_file('dtoc_test_simple.dts')
+ plat = dtb_platdata.DtbPlatdata(dtb_file, False)
+ plat.scan_dtb()
+ plat.scan_tree()
+ self.assertEqual({'i2c0': '/i2c at 0'}, plat._aliases)
+
+ def testReadConfig(self):
+ """Test we can get a list of 'tiny' uclasses"""
+ plat = dtb_platdata.DtbPlatdata(None, None, None)
+ plat.parse_config(CONFIG_FILE_DATA)
+ self.assertEqual(['serial', 'i2c'], plat._tiny_uclasses)
+
+ def do_platdata_run(self):
+ dtb_file = get_dtb_file('dtoc_test_simple.dts')
+ output = tools.GetOutputFilename('output')
+ config_file = tools.GetOutputFilename('config')
+ tools.WriteFile(config_file, CONFIG_FILE_DATA, binary=False)
+ plat = dtb_platdata.run_steps(['platdata'], dtb_file, config_file,
+ False, output)
+ return plat, output
+
+ def testDetectTiny(self):
+ """Test we detect devices that need to be 'tiny'"""
+ plat, _ = self.do_platdata_run()
+ tiny_nodes = [node.name for node in plat._valid_nodes if node.is_tiny]
+ self.assertEqual(['i2c at 0'], tiny_nodes)
+
+ def testTinyNoStruct(self):
+ """Test we don't output U_BOOT_DEVICE for 'tiny' devices"""
+ plat, output = self.do_platdata_run()
+ data = tools.ReadFile(output)
+ dev_list = re.findall(b'U_BOOT_DEVICE\((.*)\)', data)
+ self.assertNotIn(b'i2c_at_0', dev_list)
+
+ # Find dtd declarations with 'static' in them
+ #dtv_list = re.findall(b'(.*)struct dtd.* dtv_(.*) =', data)
+ #has_static = [b for a, b in dtv_list if a == 'static ']
+ #self.assertNotIn(b'i2c_at_0', has_static)
+
+ def testTinyUclass(self):
+ plat = dtb_platdata.DtbPlatdata(None, None, None)
+ driver_file = tools.GetOutputFilename('driver.c')
+ tools.WriteFile(driver_file, DRIVER_FILE_DATA, binary=False)
+ plat.scan_driver(driver_file)
+ self.assertEqual(['usb_phy', 'usb_xhci'], sorted(plat._drivers.keys()))
+ self.assertEqual(DriverInfo(name='usb_phy', uclass_id='UCLASS_PHY',
+ compat=['rockchip,rk3328-usb3-phy']),
+ plat._drivers['usb_phy'])
+ self.assertEqual(DriverInfo(name='usb_xhci', uclass_id='UCLASS_USB',
+ compat=['rockchip,rk3328-xhci']),
+ plat._drivers['usb_xhci'])
@@ -270,7 +270,7 @@ def ReadFile(fname, binary=True):
#(fname, len(data), len(data)))
return data
-def WriteFile(fname, data):
+def WriteFile(fname, data, binary=True):
"""Write data into a file.
Args:
@@ -279,7 +279,7 @@ def WriteFile(fname, data):
"""
#self._out.Info("Write file '%s' size %d (%#0x)" %
#(fname, len(data), len(data)))
- with open(Filename(fname), 'wb') as fd:
+ with open(Filename(fname), binary and 'wb' or 'w') as fd:
fd.write(data)
def GetBytes(byte, size):
This patch includes changes to support tiny-dm in driver model and dtoc. Signed-off-by: Simon Glass <sjg at chromium.org> --- Changes in v2: - Various updates, and ported to chromebook_jerry (rockchip) board/Synology/ds109/ds109.c | 3 +- common/console.c | 2 +- common/log.c | 36 +++- common/malloc_simple.c | 31 ++++ common/spl/spl.c | 17 +- common/spl/spl_spi.c | 91 +++++---- doc/develop/debugging.rst | 35 ++++ doc/driver-model/tiny-dm.rst | 315 +++++++++++++++++++++++++++++++ drivers/core/Kconfig | 106 +++++++++++ drivers/core/Makefile | 3 + drivers/core/of_extra.c | 49 ++++- drivers/core/regmap.c | 3 +- drivers/core/syscon-uclass.c | 68 +++++-- drivers/core/tiny.c | 249 +++++++++++++++++++++++++ include/dm/device.h | 121 ++++++++++++ include/dm/of_extra.h | 6 + include/dm/platdata.h | 20 +- include/dm/tiny_struct.h | 42 +++++ include/linker_lists.h | 6 + scripts/Makefile.spl | 6 +- tools/dtoc/dtb_platdata.py | 316 +++++++++++++++++++++++++++----- tools/dtoc/dtoc_test_simple.dts | 12 +- tools/dtoc/fdt.py | 7 +- tools/dtoc/main.py | 9 +- tools/dtoc/test_dtoc.py | 91 ++++++++- tools/patman/tools.py | 4 +- 26 files changed, 1525 insertions(+), 123 deletions(-) create mode 100644 doc/develop/debugging.rst create mode 100644 doc/driver-model/tiny-dm.rst create mode 100644 drivers/core/tiny.c create mode 100644 include/dm/tiny_struct.h