* [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass
@ 2018-03-28 12:37 Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 2/3] fpgamap/misc: Add ihs_fpga and gdsys_soc drivers Mario Six
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Mario Six @ 2018-03-28 12:37 UTC (permalink / raw)
To: u-boot
The FPGA (as a device) and the register map supplied by a FPGA are two
different entities. There are U-Boot drivers for the former, but not for
the later.
Add a uclass that makes it possible to read from and write to FPGA
memory maps. The interface provided emulates the PCI interface, with one
function for reading/writing plus a size parameter.
Signed-off-by: Mario Six <mario.six@gdsys.cc>
---
drivers/Kconfig | 2 +
drivers/Makefile | 1 +
drivers/fpgamap/Kconfig | 9 +++
drivers/fpgamap/Makefile | 8 +++
drivers/fpgamap/fpgamap-uclass.c | 53 ++++++++++++++++
include/dm/uclass-id.h | 1 +
include/fpgamap.h | 131 +++++++++++++++++++++++++++++++++++++++
7 files changed, 205 insertions(+)
create mode 100644 drivers/fpgamap/Kconfig
create mode 100644 drivers/fpgamap/Makefile
create mode 100644 drivers/fpgamap/fpgamap-uclass.c
create mode 100644 include/fpgamap.h
diff --git a/drivers/Kconfig b/drivers/Kconfig
index c2e813f5ad..29dac7b50e 100644
--- a/drivers/Kconfig
+++ b/drivers/Kconfig
@@ -32,6 +32,8 @@ source "drivers/firmware/Kconfig"
source "drivers/fpga/Kconfig"
+source "drivers/fpgamap/Kconfig"
+
source "drivers/gpio/Kconfig"
source "drivers/i2c/Kconfig"
diff --git a/drivers/Makefile b/drivers/Makefile
index 6846d181aa..32f0527001 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -74,6 +74,7 @@ obj-$(CONFIG_CPU) += cpu/
obj-y += crypto/
obj-y += firmware/
obj-$(CONFIG_FPGA) += fpga/
+obj-$(CONFIG_FPGAMAP) += fpgamap/
obj-y += misc/
obj-$(CONFIG_MMC) += mmc/
obj-$(CONFIG_NVME) += nvme/
diff --git a/drivers/fpgamap/Kconfig b/drivers/fpgamap/Kconfig
new file mode 100644
index 0000000000..b57d31e3f7
--- /dev/null
+++ b/drivers/fpgamap/Kconfig
@@ -0,0 +1,9 @@
+menuconfig FPGAMAP
+ bool "FPGA register map drivers"
+ help
+ Support drivers to operate on memory-mapped register maps of FPGAs.
+ Drivers are necessarily very device-dependant.
+
+ Generic read/write operations with varying widths (8, 16, and 32 for
+ now), as well as read/write access to named registers (defined via
+ device tree entries) are supported
diff --git a/drivers/fpgamap/Makefile b/drivers/fpgamap/Makefile
new file mode 100644
index 0000000000..c32efba771
--- /dev/null
+++ b/drivers/fpgamap/Makefile
@@ -0,0 +1,8 @@
+#
+# (C) Copyright 2017
+# Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-$(CONFIG_FPGAMAP) += fpgamap-uclass.o
diff --git a/drivers/fpgamap/fpgamap-uclass.c b/drivers/fpgamap/fpgamap-uclass.c
new file mode 100644
index 0000000000..8f39e22dad
--- /dev/null
+++ b/drivers/fpgamap/fpgamap-uclass.c
@@ -0,0 +1,53 @@
+/*
+ * (C) Copyright 2017
+ * Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <fpgamap.h>
+
+int fpgamap_set_reg(struct udevice *dev, const char *compat, uint value)
+{
+ struct fpgamap_ops *ops = fpgamap_get_ops(dev);
+
+ return ops->set_reg(dev, compat, value);
+}
+
+int fpgamap_get_reg(struct udevice *dev, const char *compat, uint *value)
+{
+ struct fpgamap_ops *ops = fpgamap_get_ops(dev);
+
+ return ops->get_reg(dev, compat, value);
+}
+
+int fpgamap_read(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size)
+{
+ struct fpgamap_ops *ops = fpgamap_get_ops(dev);
+
+ if (!ops->read)
+ return -ENOSYS;
+
+ return ops->read(dev, address, data, size);
+}
+
+int fpgamap_write(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size)
+{
+ struct fpgamap_ops *ops = fpgamap_get_ops(dev);
+
+ if (!ops->write)
+ return -ENOSYS;
+
+ return ops->write(dev, address, data, size);
+}
+
+UCLASS_DRIVER(fpgamap) = {
+ .id = UCLASS_FPGAMAP,
+ .name = "fpgamap",
+ .post_bind = dm_scan_fdt_dev,
+ .flags = DM_UC_FLAG_SEQ_ALIAS,
+};
diff --git a/include/dm/uclass-id.h b/include/dm/uclass-id.h
index 07fabc3ce6..8c47ae9d00 100644
--- a/include/dm/uclass-id.h
+++ b/include/dm/uclass-id.h
@@ -36,6 +36,7 @@ enum uclass_id {
UCLASS_DMA, /* Direct Memory Access */
UCLASS_EFI, /* EFI managed devices */
UCLASS_ETH, /* Ethernet device */
+ UCLASS_FPGAMAP, /* FPGA register maps */
UCLASS_GPIO, /* Bank of general-purpose I/O pins */
UCLASS_FIRMWARE, /* Firmware */
UCLASS_I2C, /* I2C bus */
diff --git a/include/fpgamap.h b/include/fpgamap.h
new file mode 100644
index 0000000000..a04ac4d5ae
--- /dev/null
+++ b/include/fpgamap.h
@@ -0,0 +1,131 @@
+/*
+ * (C) Copyright 2017
+ * Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+
+#ifndef _FPGAMAP_H_
+#define _FPGAMAP_H_
+
+/* Access sizes for fpgamap reads and writes */
+enum fpgamap_size_t {
+ FPGAMAP_SIZE_8,
+ FPGAMAP_SIZE_16,
+ FPGAMAP_SIZE_32,
+};
+
+/**
+ * struct fpgamap_ops - driver operations for FPGA register map uclass.
+ *
+ * Drivers should support these operations unless otherwise noted. These
+ * operations are intended to be used by uclass code, not directly from
+ * other code.
+ */
+struct fpgamap_ops {
+ /**
+ * get_reg() - Read a single integer from a named register of a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to read from.
+ * @compat: Compatible string of a named register of the FPGA map.
+ * @value: Pointer to a variable that takes the data value read
+ * from the register of the FPGA register map.
+ * @return 0 if OK, -ve on error.
+ */
+ int (*get_reg)(struct udevice *dev, const char *compat, uint *value);
+
+ /**
+ * set_reg() - Write a single integer to a named register of a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to write to.
+ * @compat: Compatible string of a named register of the FPGA map.
+ * @value: Data value to be written to the register of the FPGA
+ * register map.
+ * @return 0 if OK, -ve on error.
+ */
+ int (*set_reg)(struct udevice *dev, const char *compat, uint value);
+
+ /**
+ * read() - Read a single value from a specified address on a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to read from.
+ * @address: The address to read from.
+ * @data: Pointer to a variable that takes the data value read
+ * from the address on the FPGA register map.
+ * @return 0 if OK, -ve on error.
+ */
+ int (*read)(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size);
+
+ /**
+ * write() - Write a single value to a specified address on a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to write to.
+ * @address: The address to write to.
+ * @data: Data value to be written to the address on the FPGA
+ * register map.
+ * @return 0 if OK, -ve on error.
+ */
+ int (*write)(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size);
+
+};
+
+#define fpgamap_get_ops(dev) ((struct fpgamap_ops *)(dev)->driver->ops)
+
+/**
+ * fpgamap_get_reg() - Read a single integer from a named register of a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to read from.
+ * @compat: Compatible string of a named register of the FPGA map.
+ * @value: Pointer to a variable that takes the data value read
+ * from the register of the FPGA register map.
+ * @return 0 if OK, -ve on error.
+ */
+int fpgamap_get_reg(struct udevice *dev, const char *compat, uint *value);
+
+/**
+ * fpgamap_set_reg() - Write a single integer to a named register of a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to write to.
+ * @compat: Compatible string of a named register of the FPGA map.
+ * @value: Data value to be written to the register of the FPGA
+ * register map.
+ * @return 0 if OK, -ve on error.
+ */
+int fpgamap_set_reg(struct udevice *dev, const char *compat, uint value);
+/**
+ * fpgamap_read() - Read a single value from a specified address on a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to read from.
+ * @address: The address to read from.
+ * @data: Pointer to a variable that takes the data value read from the
+ * address on the FPGA register map.
+ * @size: The size of the data to be read
+ * @return 0 if OK, -ve on error.
+ */
+int fpgamap_read(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size);
+
+/**
+ * fpgamap_write() - Write a single value to a specified address on a FPGA
+ * register map
+ *
+ * @dev: FPGA register map to write to.
+ * @address: The address to write to.
+ * @data: Data value to be written to the address on the FPGA register
+ * map.
+ * @size: The size of the data to be written
+ * @return 0 if OK, -ve on error.
+ */
+int fpgamap_write(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size);
+
+#endif
--
2.16.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 2/3] fpgamap/misc: Add ihs_fpga and gdsys_soc drivers
2018-03-28 12:37 [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Mario Six
@ 2018-03-28 12:37 ` Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 3/3] cmd: Add fpgamap command Mario Six
2018-03-30 8:40 ` [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Simon Glass
2 siblings, 0 replies; 5+ messages in thread
From: Mario Six @ 2018-03-28 12:37 UTC (permalink / raw)
To: u-boot
This patch adds drivers for IHS FPGAs and their associated busses.
Signed-off-by: Mario Six <mario.six@gdsys.cc>
---
drivers/fpgamap/Kconfig | 11 +
drivers/fpgamap/Makefile | 1 +
drivers/fpgamap/ihs_fpgamap.c | 811 ++++++++++++++++++++++++++++++++++++++++++
drivers/misc/Kconfig | 10 +
drivers/misc/Makefile | 2 +
drivers/misc/gdsys_soc.c | 67 ++++
drivers/misc/gdsys_soc.h | 24 ++
7 files changed, 926 insertions(+)
create mode 100644 drivers/fpgamap/ihs_fpgamap.c
create mode 100644 drivers/misc/gdsys_soc.c
create mode 100644 drivers/misc/gdsys_soc.h
diff --git a/drivers/fpgamap/Kconfig b/drivers/fpgamap/Kconfig
index b57d31e3f7..d32043ebfd 100644
--- a/drivers/fpgamap/Kconfig
+++ b/drivers/fpgamap/Kconfig
@@ -7,3 +7,14 @@ menuconfig FPGAMAP
Generic read/write operations with varying widths (8, 16, and 32 for
now), as well as read/write access to named registers (defined via
device tree entries) are supported
+
+if FPGAMAP
+
+config IHS_FPGAMAP
+ bool "Enable IHS FPGA register map driver"
+ depends on DM
+ help
+ Support for IHS FPGA register map on a gdsys IHS FPGA used on gdsys
+ boards.
+
+endif
diff --git a/drivers/fpgamap/Makefile b/drivers/fpgamap/Makefile
index c32efba771..80153b34a0 100644
--- a/drivers/fpgamap/Makefile
+++ b/drivers/fpgamap/Makefile
@@ -6,3 +6,4 @@
#
obj-$(CONFIG_FPGAMAP) += fpgamap-uclass.o
+obj-$(CONFIG_IHS_FPGAMAP) += ihs_fpgamap.o
diff --git a/drivers/fpgamap/ihs_fpgamap.c b/drivers/fpgamap/ihs_fpgamap.c
new file mode 100644
index 0000000000..7cac696e5b
--- /dev/null
+++ b/drivers/fpgamap/ihs_fpgamap.c
@@ -0,0 +1,811 @@
+/*
+ * (C) Copyright 2017
+ * Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+ *
+ * based on the ioep-fpga driver, which is
+ *
+ * (C) Copyright 2014
+ * Dirk Eibach, Guntermann & Drunck GmbH, eibach at gdsys.de
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <dm/lists.h>
+#include <fpgamap.h>
+#include <misc.h>
+#include <mapmem.h>
+#include <asm/gpio.h>
+#include <linux/bitops.h>
+
+DECLARE_GLOBAL_DATA_PTR;
+
+struct reg_spec {
+ uint addr;
+ uint shift;
+ ulong mask;
+};
+
+struct ihs_fpga_priv {
+ u8 *regs;
+ fdt_addr_t addr;
+ struct gpio_desc reset_gpio;
+ struct gpio_desc done_gpio;
+ struct gpio_desc startupfin_gpios[2];
+ ofnode regmap_node;
+ bool has_osd;
+};
+
+const u16 reflection_testpattern = 0xdede;
+
+enum pcb_video_type {
+ PCB_DVI_SL,
+ PCB_DP_165MPIX,
+ PCB_DP_300MPIX,
+ PCB_HDMI,
+ PCB_DP_1_2,
+ PCB_HDMI_2_0,
+};
+
+enum pcb_transmission_type {
+ PCB_CAT_1G,
+ PCB_FIBER_3G,
+ PCB_CAT_10G,
+ PCB_FIBER_10G,
+};
+
+enum carrier_speed {
+ CARRIER_SPEED_1G,
+ CARRIER_SPEED_3G,
+ CARRIER_SPEED_2_5G = CARRIER_SPEED_3G,
+ CARRIER_SPEED_10G,
+};
+
+enum ram_config {
+ RAM_DDR2_32BIT_295MBPS,
+ RAM_DDR3_32BIT_590MBPS,
+ RAM_DDR3_48BIT_590MBPS,
+ RAM_DDR3_64BIT_1800MBPS,
+ RAM_DDR3_48BIT_1800MBPS,
+};
+
+enum sysclock {
+ SYSCLK_147456,
+};
+
+struct fpga_versions {
+ bool video_channel;
+ bool con_side;
+ enum pcb_video_type pcb_video_type;
+ enum pcb_transmission_type pcb_transmission_type;
+ unsigned int hw_version;
+};
+
+struct fpga_features {
+ u8 video_channels;
+ u8 carriers;
+ enum carrier_speed carrier_speed;
+ enum ram_config ram_config;
+ enum sysclock sysclock;
+
+ bool pcm_tx;
+ bool pcm_rx;
+ bool spdif_tx;
+ bool spdif_rx;
+ bool usb2;
+ bool rs232;
+ bool compression_type1;
+ bool compression_type2;
+ bool compression_type3;
+ bool interlace;
+ bool osd;
+ bool compression_pipes;
+};
+
+#ifdef CONFIG_SYS_FPGA_FLAVOR_GAZERBEAM
+
+static int get_versions(struct udevice *dev, struct fpga_versions *versions)
+{
+ enum {
+ VERSIONS_FPGA_VIDEO_CHANNEL = BIT(12),
+ VERSIONS_FPGA_CON_SIDE = BIT(13),
+ VERSIONS_FPGA_SC = BIT(14),
+ VERSIONS_PCB_CON = BIT(9),
+ VERSIONS_PCB_SC = BIT(8),
+ VERSIONS_PCB_VIDEO_MASK = 0x3 << 6,
+ VERSIONS_PCB_VIDEO_DP_1_2 = 0x0 << 6,
+ VERSIONS_PCB_VIDEO_HDMI_2_0 = 0x1 << 6,
+ VERSIONS_PCB_TRANSMISSION_MASK = 0x3 << 4,
+ VERSIONS_PCB_TRANSMISSION_FIBER_10G = 0x0 << 4,
+ VERSIONS_PCB_TRANSMISSION_CAT_10G = 0x1 << 4,
+ VERSIONS_PCB_TRANSMISSION_FIBER_3G = 0x2 << 4,
+ VERSIONS_PCB_TRANSMISSION_CAT_1G = 0x3 << 4,
+ VERSIONS_HW_VER_MASK = 0xf << 0,
+ };
+ uint raw_versions;
+
+ memset(versions, 0, sizeof(struct fpga_versions));
+
+ fpgamap_get_reg(dev, "versions", &raw_versions);
+
+ versions->video_channel = raw_versions & VERSIONS_FPGA_VIDEO_CHANNEL;
+ versions->con_side = raw_versions & VERSIONS_FPGA_CON_SIDE;
+
+ switch (raw_versions & VERSIONS_PCB_VIDEO_MASK) {
+ case VERSIONS_PCB_VIDEO_DP_1_2:
+ versions->pcb_video_type = PCB_DP_1_2;
+ break;
+
+ case VERSIONS_PCB_VIDEO_HDMI_2_0:
+ versions->pcb_video_type = PCB_HDMI_2_0;
+ break;
+ }
+
+ switch (raw_versions & VERSIONS_PCB_TRANSMISSION_MASK) {
+ case VERSIONS_PCB_TRANSMISSION_FIBER_10G:
+ versions->pcb_transmission_type = PCB_FIBER_10G;
+ break;
+
+ case VERSIONS_PCB_TRANSMISSION_CAT_10G:
+ versions->pcb_transmission_type = PCB_CAT_10G;
+ break;
+
+ case VERSIONS_PCB_TRANSMISSION_FIBER_3G:
+ versions->pcb_transmission_type = PCB_FIBER_3G;
+ break;
+
+ case VERSIONS_PCB_TRANSMISSION_CAT_1G:
+ versions->pcb_transmission_type = PCB_CAT_1G;
+ break;
+ }
+
+ versions->hw_version = raw_versions & VERSIONS_HW_VER_MASK;
+
+ return 0;
+}
+
+static int get_features(struct udevice *dev, struct fpga_features *features)
+{
+ enum {
+ FEATURE_SPDIF_RX = BIT(15),
+ FEATURE_SPDIF_TX = BIT(14),
+ FEATURE_PCM_RX = BIT(13),
+ FEATURE_PCM_TX = BIT(12),
+ FEATURE_RAM_MASK = GENMASK(11, 8),
+ FEATURE_RAM_DDR2_32BIT_295MBPS = 0x0 << 8,
+ FEATURE_RAM_DDR3_32BIT_590MBPS = 0x1 << 8,
+ FEATURE_RAM_DDR3_48BIT_590MBPS = 0x2 << 8,
+ FEATURE_RAM_DDR3_64BIT_1800MBPS = 0x3 << 8,
+ FEATURE_RAM_DDR3_48BIT_1800MBPS = 0x4 << 8,
+ FEATURE_CARRIER_SPEED_MASK = GENMASK(7, 6),
+ FEATURE_CARRIER_SPEED_1G = 0x0 << 6,
+ FEATURE_CARRIER_SPEED_2_5G = 0x1 << 6,
+ FEATURE_CARRIER_SPEED_10G = 0x2 << 6,
+ FEATURE_CARRIERS_MASK = GENMASK(5, 4),
+ FEATURE_CARRIERS_0 = 0x0 << 4,
+ FEATURE_CARRIERS_1 = 0x1 << 4,
+ FEATURE_CARRIERS_2 = 0x2 << 4,
+ FEATURE_CARRIERS_4 = 0x3 << 4,
+ FEATURE_USB2 = BIT(3),
+ FEATURE_VIDEOCHANNELS_MASK = GENMASK(2, 0),
+ FEATURE_VIDEOCHANNELS_0 = 0x0 << 0,
+ FEATURE_VIDEOCHANNELS_1 = 0x1 << 0,
+ FEATURE_VIDEOCHANNELS_1_1 = 0x2 << 0,
+ FEATURE_VIDEOCHANNELS_2 = 0x3 << 0,
+ };
+
+ enum {
+ EXT_FEATURE_OSD = BIT(15),
+ EXT_FEATURE_ETHERNET = BIT(9),
+ EXT_FEATURE_INTERLACE = BIT(8),
+ EXT_FEATURE_RS232 = BIT(7),
+ EXT_FEATURE_COMPRESSION_PERF_MASK = GENMASK(6, 4),
+ EXT_FEATURE_COMPRESSION_PERF_1X = 0x0 << 4,
+ EXT_FEATURE_COMPRESSION_PERF_2X = 0x1 << 4,
+ EXT_FEATURE_COMPRESSION_PERF_4X = 0x2 << 4,
+ EXT_FEATURE_COMPRESSION_TYPE1 = BIT(0),
+ EXT_FEATURE_COMPRESSION_TYPE2 = BIT(1),
+ EXT_FEATURE_COMPRESSION_TYPE3 = BIT(2),
+ };
+
+ uint raw_features;
+ uint raw_extended_features;
+
+ memset(features, 0, sizeof(struct fpga_features));
+
+ fpgamap_get_reg(dev, "fpga-features", &raw_features);
+ fpgamap_get_reg(dev, "fpga-ext-features", &raw_extended_features);
+
+ switch (raw_features & FEATURE_VIDEOCHANNELS_MASK) {
+ case FEATURE_VIDEOCHANNELS_0:
+ features->video_channels = 0;
+ break;
+
+ case FEATURE_VIDEOCHANNELS_1:
+ features->video_channels = 1;
+ break;
+
+ case FEATURE_VIDEOCHANNELS_1_1:
+ case FEATURE_VIDEOCHANNELS_2:
+ features->video_channels = 2;
+ break;
+ };
+
+ switch (raw_features & FEATURE_CARRIERS_MASK) {
+ case FEATURE_CARRIERS_0:
+ features->carriers = 0;
+ break;
+
+ case FEATURE_CARRIERS_1:
+ features->carriers = 1;
+ break;
+
+ case FEATURE_CARRIERS_2:
+ features->carriers = 2;
+ break;
+
+ case FEATURE_CARRIERS_4:
+ features->carriers = 4;
+ break;
+ }
+
+ switch (raw_features & FEATURE_CARRIER_SPEED_MASK) {
+ case FEATURE_CARRIER_SPEED_1G:
+ features->carrier_speed = CARRIER_SPEED_1G;
+ break;
+ case FEATURE_CARRIER_SPEED_2_5G:
+ features->carrier_speed = CARRIER_SPEED_2_5G;
+ break;
+ case FEATURE_CARRIER_SPEED_10G:
+ features->carrier_speed = CARRIER_SPEED_10G;
+ break;
+ }
+
+ switch (raw_features & FEATURE_RAM_MASK) {
+ case FEATURE_RAM_DDR2_32BIT_295MBPS:
+ features->ram_config = RAM_DDR2_32BIT_295MBPS;
+ break;
+
+ case FEATURE_RAM_DDR3_32BIT_590MBPS:
+ features->ram_config = RAM_DDR3_32BIT_590MBPS;
+ break;
+
+ case FEATURE_RAM_DDR3_48BIT_590MBPS:
+ features->ram_config = RAM_DDR3_48BIT_590MBPS;
+ break;
+
+ case FEATURE_RAM_DDR3_64BIT_1800MBPS:
+ features->ram_config = RAM_DDR3_64BIT_1800MBPS;
+ break;
+
+ case FEATURE_RAM_DDR3_48BIT_1800MBPS:
+ features->ram_config = RAM_DDR3_48BIT_1800MBPS;
+ break;
+ }
+
+ features->pcm_tx = raw_features & FEATURE_PCM_TX;
+ features->pcm_rx = raw_features & FEATURE_PCM_RX;
+ features->spdif_tx = raw_features & FEATURE_SPDIF_TX;
+ features->spdif_rx = raw_features & FEATURE_SPDIF_RX;
+ features->usb2 = raw_features & FEATURE_USB2;
+ features->rs232 = raw_extended_features & EXT_FEATURE_RS232;
+ features->compression_type1 = raw_extended_features &
+ EXT_FEATURE_COMPRESSION_TYPE1;
+ features->compression_type2 = raw_extended_features &
+ EXT_FEATURE_COMPRESSION_TYPE2;
+ features->compression_type3 = raw_extended_features &
+ EXT_FEATURE_COMPRESSION_TYPE3;
+ features->interlace = raw_extended_features & EXT_FEATURE_INTERLACE;
+ features->osd = raw_extended_features & EXT_FEATURE_OSD;
+ features->compression_pipes = raw_extended_features &
+ EXT_FEATURE_COMPRESSION_PERF_MASK;
+
+ return 0;
+}
+
+#else
+
+static int get_versions(unsigned int fpga, struct fpga_versions *versions)
+{
+ enum {
+ /* HW version encoding is a mess, leave it for the moment */
+ VERSIONS_HW_VER_MASK = 0xf << 0,
+ VERSIONS_PIX_CLOCK_GEN_IDT8N3QV01 = BIT(4),
+ VERSIONS_SFP = BIT(5),
+ VERSIONS_VIDEO_MASK = 0x7 << 6,
+ VERSIONS_VIDEO_DVI = 0x0 << 6,
+ VERSIONS_VIDEO_DP_165 = 0x1 << 6,
+ VERSIONS_VIDEO_DP_300 = 0x2 << 6,
+ VERSIONS_VIDEO_HDMI = 0x3 << 6,
+ VERSIONS_UT_MASK = 0xf << 12,
+ VERSIONS_UT_MAIN_SERVER = 0x0 << 12,
+ VERSIONS_UT_MAIN_USER = 0x1 << 12,
+ VERSIONS_UT_VIDEO_SERVER = 0x2 << 12,
+ VERSIONS_UT_VIDEO_USER = 0x3 << 12,
+ };
+ u16 raw_versions;
+
+ memset(versions, 0, sizeof(struct fpga_versions));
+
+ FPGA_GET_REG(fpga, versions, &raw_versions);
+
+ switch (raw_versions & VERSIONS_UT_MASK) {
+ case VERSIONS_UT_MAIN_SERVER:
+ versions->video_channel = false;
+ versions->con_side = false;
+ break;
+
+ case VERSIONS_UT_MAIN_USER:
+ versions->video_channel = false;
+ versions->con_side = true;
+ break;
+
+ case VERSIONS_UT_VIDEO_SERVER:
+ versions->video_channel = true;
+ versions->con_side = false;
+ break;
+
+ case VERSIONS_UT_VIDEO_USER:
+ versions->video_channel = true;
+ versions->con_side = true;
+ break;
+ }
+
+ switch (raw_versions & VERSIONS_VIDEO_MASK) {
+ case VERSIONS_VIDEO_DVI:
+ versions->pcb_video_type = PCB_DVI_SL;
+ break;
+
+ case VERSIONS_VIDEO_DP_165:
+ versions->pcb_video_type = PCB_DP_165MPIX;
+ break;
+
+ case VERSIONS_VIDEO_DP_300:
+ versions->pcb_video_type = PCB_DP_300MPIX;
+ break;
+
+ case VERSIONS_VIDEO_HDMI:
+ versions->pcb_video_type = PCB_HDMI;
+ break;
+ }
+
+ versions->hw_version = raw_versions & VERSIONS_HW_VER_MASK;
+
+ if (raw_versions & VERSIONS_SFP)
+ versions->pcb_transmission_type = PCB_FIBER_3G;
+ else
+ versions->pcb_transmission_type = PCB_CAT_1G;
+
+ return 0;
+}
+
+static int get_features(unsigned int fpga, struct fpga_features *features)
+{
+ enum {
+ FEATURE_CARRIER_SPEED_2_5 = BIT(4),
+ FEATURE_RAM_MASK = 0x7 << 5,
+ FEATURE_RAM_DDR2_32BIT = 0x0 << 5,
+ FEATURE_RAM_DDR3_32BIT = 0x1 << 5,
+ FEATURE_RAM_DDR3_48BIT = 0x2 << 5,
+ FEATURE_PCM_AUDIO_TX = BIT(9),
+ FEATURE_PCM_AUDIO_RX = BIT(10),
+ FEATURE_OSD = BIT(11),
+ FEATURE_USB20 = BIT(12),
+ FEATURE_COMPRESSION_MASK = 7 << 13,
+ FEATURE_COMPRESSION_TYPE1 = 0x1 << 13,
+ FEATURE_COMPRESSION_TYPE1_TYPE2 = 0x3 << 13,
+ FEATURE_COMPRESSION_TYPE1_TYPE2_TYPE3 = 0x7 << 13,
+ };
+
+ enum {
+ EXTENDED_FEATURE_SPDIF_AUDIO_TX = BIT(0),
+ EXTENDED_FEATURE_SPDIF_AUDIO_RX = BIT(1),
+ EXTENDED_FEATURE_RS232 = BIT(2),
+ EXTENDED_FEATURE_COMPRESSION_PIPES = BIT(3),
+ EXTENDED_FEATURE_INTERLACE = BIT(4),
+ };
+
+ u16 raw_features;
+ u16 raw_extended_features;
+
+ memset(features, 0, sizeof(struct fpga_features));
+
+ FPGA_GET_REG(fpga, fpga_features, &raw_features);
+ FPGA_GET_REG(fpga, fpga_ext_features, &raw_extended_features);
+
+ features->video_channels = raw_features & 0x3;
+ features->carriers = (raw_features >> 2) & 0x3;
+
+ features->carrier_speed = (raw_features & FEATURE_CARRIER_SPEED_2_5)
+ ? CARRIER_SPEED_2_5G : CARRIER_SPEED_1G;
+
+ switch (raw_features & FEATURE_RAM_MASK) {
+ case FEATURE_RAM_DDR2_32BIT:
+ features->ram_config = RAM_DDR2_32BIT_295MBPS;
+ break;
+
+ case FEATURE_RAM_DDR3_32BIT:
+ features->ram_config = RAM_DDR3_32BIT_590MBPS;
+ break;
+
+ case FEATURE_RAM_DDR3_48BIT:
+ features->ram_config = RAM_DDR3_48BIT_590MBPS;
+ break;
+ }
+
+ features->pcm_tx = raw_features & FEATURE_PCM_AUDIO_TX;
+ features->pcm_rx = raw_features & FEATURE_PCM_AUDIO_RX;
+ features->spdif_tx = raw_extended_features &
+ EXTENDED_FEATURE_SPDIF_AUDIO_TX;
+ features->spdif_rx = raw_extended_features &
+ EXTENDED_FEATURE_SPDIF_AUDIO_RX;
+
+ features->usb2 = raw_features & FEATURE_USB20;
+ features->rs232 = raw_extended_features & EXTENDED_FEATURE_RS232;
+
+ features->compression_type1 = false;
+ features->compression_type2 = false;
+ features->compression_type3 = false;
+ switch (raw_features & FEATURE_COMPRESSION_MASK) {
+ case FEATURE_COMPRESSION_TYPE1_TYPE2_TYPE3:
+ features->compression_type3 = true;
+ case FEATURE_COMPRESSION_TYPE1_TYPE2:
+ features->compression_type2 = true;
+ case FEATURE_COMPRESSION_TYPE1:
+ features->compression_type1 = true;
+ break;
+ }
+
+ features->interlace = raw_extended_features &
+ EXTENDED_FEATURE_INTERLACE;
+ features->osd = raw_features & FEATURE_OSD;
+ features->compression_pipes = raw_extended_features &
+ EXTENDED_FEATURE_COMPRESSION_PIPES;
+
+ return 0;
+}
+
+#endif
+
+static void fpga_print_info(struct udevice *dev)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ uint fpga_version;
+ struct fpga_versions versions;
+ struct fpga_features features;
+
+ fpgamap_get_reg(dev, "fpga-version", &fpga_version);
+ get_versions(dev, &versions);
+ get_features(dev, &features);
+
+ priv->has_osd = features.osd;
+
+ if (versions.video_channel)
+ printf("Videochannel");
+ else
+ printf("Mainchannel");
+
+ if (versions.con_side)
+ printf(" User");
+ else
+ printf(" Server");
+
+// FIXME
+#if 0
+ if (versions & (1<<4))
+ printf(" UC");
+#endif
+
+ switch (versions.pcb_transmission_type) {
+ case PCB_CAT_1G:
+ case PCB_CAT_10G:
+ printf(" CAT");
+ break;
+ case PCB_FIBER_3G:
+ case PCB_FIBER_10G:
+ printf(" Fiber");
+ break;
+ };
+
+ switch (versions.pcb_video_type) {
+ case PCB_DVI_SL:
+ printf(" DVI,");
+ break;
+ case PCB_DP_165MPIX:
+ printf(" DP 165MPix/s,");
+ break;
+ case PCB_DP_300MPIX:
+ printf(" DP 300MPix/s,");
+ break;
+ case PCB_HDMI:
+ printf(" HDMI,");
+ break;
+ case PCB_DP_1_2:
+ printf(" DP 1.2,");
+ break;
+ case PCB_HDMI_2_0:
+ printf(" HDMI 2.0,");
+ break;
+ }
+
+ printf(" FPGA V %d.%02d\n features: ",
+ fpga_version / 100, fpga_version % 100);
+
+ if (!features.compression_type1 &&
+ !features.compression_type2 &&
+ !features.compression_type3)
+ printf("no compression, ");
+
+ if (features.compression_type1)
+ printf("type1, ");
+
+ if (features.compression_type2)
+ printf("type2, ");
+
+ if (features.compression_type3)
+ printf("type3, ");
+
+ printf("%sosd", features.osd ? "" : "no ");
+
+ if (features.pcm_rx && features.pcm_tx)
+ printf(", pcm rx+tx");
+ else if (features.pcm_rx)
+ printf(", pcm rx");
+ else if (features.pcm_tx)
+ printf(", pcm tx");
+
+ if (features.spdif_rx && features.spdif_tx)
+ printf(", spdif rx+tx");
+ else if (features.spdif_rx)
+ printf(", spdif rx");
+ else if (features.spdif_tx)
+ printf(", spdif tx");
+
+ puts(",\n ");
+
+ switch (features.sysclock) {
+ case SYSCLK_147456:
+ printf("clock 147.456 MHz");
+ break;
+ }
+
+ switch (features.ram_config) {
+ case RAM_DDR2_32BIT_295MBPS:
+ printf(", RAM 32 bit DDR2");
+ break;
+ case RAM_DDR3_32BIT_590MBPS:
+ printf(", RAM 32 bit DDR3");
+ break;
+ case RAM_DDR3_48BIT_590MBPS:
+ case RAM_DDR3_48BIT_1800MBPS:
+ printf(", RAM 48 bit DDR3");
+ break;
+ case RAM_DDR3_64BIT_1800MBPS:
+ printf(", RAM 64 bit DDR3");
+ break;
+ }
+
+ printf(", %d carrier(s)", features.carriers);
+
+ switch (features.carrier_speed) {
+ case CARRIER_SPEED_1G:
+ printf(", 1Gbit/s");
+ break;
+ case CARRIER_SPEED_3G:
+ printf(", 3Gbit/s");
+ break;
+ case CARRIER_SPEED_10G:
+ printf(", 10Gbit/s");
+ break;
+ }
+
+ printf(", %d video channel(s)\n", features.video_channels);
+}
+
+static int do_reflection_test(struct udevice *dev)
+{
+ int ctr = 0;
+
+ while (1) {
+ uint val;
+
+ fpgamap_set_reg(dev, "reflection-low", reflection_testpattern);
+
+ fpgamap_get_reg(dev, "reflection-low", &val);
+ if (val == (~reflection_testpattern & 0xffff))
+ return 1;
+
+ mdelay(100);
+ if (ctr++ > 5)
+ return 0;
+ }
+}
+
+static int get_reg_spec(struct udevice *dev, const char *compat,
+ struct reg_spec *spec)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ ofnode subnode;
+
+ for (subnode = ofnode_first_subnode(priv->regmap_node);
+ ofnode_valid(subnode);
+ subnode = ofnode_next_subnode(subnode)) {
+ const char *compatprop;
+ u32 reg[3];
+ uint start_bit, end_bit;
+
+ compatprop = ofnode_get_property(subnode, "compatible", NULL);
+
+ if (!strcmp(compatprop, compat)) {
+ ofnode_read_u32_array(subnode, "reg", reg, 3);
+
+ start_bit = reg[1];
+ end_bit = reg[1] - reg[2] + 1;
+
+ spec->mask = GENMASK(start_bit, end_bit);
+ spec->addr = reg[0];
+ spec->shift = end_bit;
+
+ return 0;
+ }
+ }
+
+ return 1;
+}
+
+static int ihs_fpga_set_reg(struct udevice *dev, const char *compat,
+ uint value)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ struct reg_spec spec;
+
+ if (get_reg_spec(dev, compat, &spec)) {
+ printf("%s: Could not get %s regspec for '%s'.\n", __func__,
+ dev->name, compat);
+ return -ENODEV;
+ }
+
+ out_le16((void *)(priv->regs + spec.addr), value << spec.shift);
+
+ return 0;
+}
+
+static int ihs_fpga_get_reg(struct udevice *dev, const char *compat,
+ uint *value)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ struct reg_spec spec;
+ uint tmp;
+
+ if (get_reg_spec(dev, compat, &spec)) {
+ printf("%s: Could not get %s regspec for '%s'.\n", __func__,
+ dev->name, compat);
+ return -ENODEV;
+ }
+
+ tmp = in_le16((void *)(priv->regs + spec.addr));
+ *value = (tmp & spec.mask) >> spec.shift;
+
+ return 0;
+}
+
+static int ihs_fpga_read(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ u16 *d = data;
+
+ /* TODO: MCLink transfer */
+
+ if (size != FPGAMAP_SIZE_16)
+ return -ENODEV;
+
+ *d = in_le16((void *)(priv->regs + address));
+
+ return 0;
+}
+
+static int ihs_fpga_write(struct udevice *dev, ulong address, void *data,
+ enum fpgamap_size_t size)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ u16 *d = data;
+
+ if (size != FPGAMAP_SIZE_16)
+ return -ENODEV;
+
+ /* TODO: MCLink transfer */
+
+ out_le16((void *)(priv->regs + address), *d);
+
+ return 0;
+}
+
+static const struct fpgamap_ops ihs_fpga_ops = {
+ .set_reg = ihs_fpga_set_reg,
+ .get_reg = ihs_fpga_get_reg,
+ .read = ihs_fpga_read,
+ .write = ihs_fpga_write,
+};
+
+static int ihs_fpga_probe(struct udevice *dev)
+{
+ struct ihs_fpga_priv *priv = dev_get_priv(dev);
+ struct ofnode_phandle_args args;
+ u32 reg[3];
+ u32 addr;
+
+ if (dev_read_phandle_with_args(dev, "regmap", NULL, 0, 0, &args)) {
+ printf("%s: Could not get regmap.\n", dev->name);
+ return 1;
+ }
+
+ priv->regmap_node = args.node;
+
+ /* TODO: FPGA attached to MCLink bus */
+
+ dev_read_u32_array(dev, "reg", reg, 3);
+ addr = dev_translate_address(dev, reg);
+ priv->regs = map_sysmem(addr, reg[2]);
+
+ gpio_request_by_name(dev, "reset-gpios", 0, &priv->reset_gpio,
+ GPIOD_IS_OUT);
+ if (!priv->reset_gpio.dev) {
+ printf("%s: Could not get reset-GPIO.\n", dev->name);
+ return 1;
+ }
+
+ gpio_request_by_name(dev, "done-gpios", 0, &priv->done_gpio,
+ GPIOD_IS_IN);
+ if (!priv->done_gpio.dev) {
+ printf("%s: Could not get done-GPIO.\n", dev->name);
+ return 1;
+ }
+
+ dm_gpio_set_value(&priv->reset_gpio, 1);
+
+ if (!do_reflection_test(dev)) {
+ int ctr = 0;
+
+ dm_gpio_set_value(&priv->reset_gpio, 0);
+
+ while (!dm_gpio_get_value(&priv->done_gpio)) {
+ mdelay(100);
+ if (ctr++ > 5) {
+ printf("Initializing FPGA failed\n");
+ break;
+ }
+ }
+
+ udelay(10);
+
+ dm_gpio_set_value(&priv->reset_gpio, 1);
+
+ if (!do_reflection_test(dev)) {
+ printf("%s: Reflection test FAILED.\n", dev->name);
+ return -1;
+ }
+ }
+
+ printf("%s: Reflection test passed.\n", dev->name);
+
+ fpga_print_info(dev);
+
+ return 0;
+}
+
+static const struct udevice_id ihs_fpga_ids[] = {
+ { .compatible = "gdsys,iocon_fpga" },
+ { .compatible = "gdsys,iocpu_fpga" },
+ { }
+};
+
+U_BOOT_DRIVER(ihs_fpga_bus) = {
+ .name = "ihs_fpga_bus",
+ .id = UCLASS_FPGAMAP,
+ .ops = &ihs_fpga_ops,
+ .of_match = ihs_fpga_ids,
+ .probe = ihs_fpga_probe,
+ .priv_auto_alloc_size = sizeof(struct ihs_fpga_priv),
+};
diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index d774569cbc..b2e0cf2f83 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -263,5 +263,15 @@ config SYS_I2C_EEPROM_ADDR_OVERFLOW
endif
+config GDSYS_IOEP
+ bool "Enable gdsys IOEP driver"
+ depends on MISC
+ help
+ Support gdsys FPGA's IO endpoint driver.
+config GDSYS_SOC
+ bool "Enable gdsys SOC driver"
+ depends on MISC
+ help
+ Support for IHS SOC.
endmenu
diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
index e8d598cd47..2e1c1021d2 100644
--- a/drivers/misc/Makefile
+++ b/drivers/misc/Makefile
@@ -54,3 +54,5 @@ obj-$(CONFIG_QFW) += qfw.o
obj-$(CONFIG_ROCKCHIP_EFUSE) += rockchip-efuse.o
obj-$(CONFIG_STM32_RCC) += stm32_rcc.o
obj-$(CONFIG_SYS_DPAA_QBMAN) += fsl_portals.o
+obj-$(CONFIG_GDSYS_IOEP) += gdsys_ioep.o
+obj-$(CONFIG_GDSYS_SOC) += gdsys_soc.o
diff --git a/drivers/misc/gdsys_soc.c b/drivers/misc/gdsys_soc.c
new file mode 100644
index 0000000000..3ae0eb4209
--- /dev/null
+++ b/drivers/misc/gdsys_soc.c
@@ -0,0 +1,67 @@
+/*
+ * (C) Copyright 2017
+ * Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <dm/lists.h>
+
+#include "gdsys_soc.h"
+
+struct gdsys_soc_priv {
+ struct udevice *fpga;
+};
+
+static const struct udevice_id gdsys_soc_ids[] = {
+ { .compatible = "gdsys,soc" },
+ { /* sentinel */ }
+};
+
+int gdsys_soc_get_fpga(struct udevice *child, struct udevice **fpga)
+{
+ struct gdsys_soc_priv *bus_priv;
+
+ if (!child->parent)
+ return -EINVAL;
+
+ if (!device_is_compatible(child->parent, "gdsys,soc"))
+ return -EINVAL;
+
+ bus_priv = dev_get_priv(child->parent);
+
+ *fpga = bus_priv->fpga;
+
+ return 0;
+}
+
+static int gdsys_soc_probe(struct udevice *dev)
+{
+ struct gdsys_soc_priv *priv = dev_get_priv(dev);
+ struct udevice *fpga;
+ int res = uclass_get_device_by_phandle(UCLASS_FPGAMAP, dev, "fpga",
+ &fpga);
+ if (res == -ENOENT) {
+ printf("%s: Could not find 'fpga' phandle.\n", dev->name);
+ return -EINVAL;
+ }
+
+ if (res == -ENODEV) {
+ printf("%s: Could not get FPGA device.\n", dev->name);
+ return -EINVAL;
+ }
+
+ priv->fpga = fpga;
+
+ return 0;
+}
+
+U_BOOT_DRIVER(gdsys_soc_bus) = {
+ .name = "gdsys_soc_bus",
+ .id = UCLASS_SIMPLE_BUS,
+ .of_match = gdsys_soc_ids,
+ .probe = gdsys_soc_probe,
+ .priv_auto_alloc_size = sizeof(struct gdsys_soc_priv),
+};
diff --git a/drivers/misc/gdsys_soc.h b/drivers/misc/gdsys_soc.h
new file mode 100644
index 0000000000..0f88149fd6
--- /dev/null
+++ b/drivers/misc/gdsys_soc.h
@@ -0,0 +1,24 @@
+/*
+ * (C) Copyright 2017
+ * Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+
+#ifndef _GDSYS_SOC_H_
+#define _GDSYS_SOC_H_
+
+/**
+ * gdsys_soc_get_fpga() - Retrieve pointer to parent bus' FPGA device
+ *
+ * To access their register maps, devices on gdsys soc buses usually have
+ * facilitate the accessor function of the IHS FPGA their parent bus is
+ * attached to. To access the FPGA device from within the bus' children, this
+ * function returns a pointer to it.
+ *
+ * @child: The child device on the FPGA bus needing access to the FPGA.
+ * @fpga: Pointer to the retrieved FPGA device.
+ * @return 0 on success, -ve on failure
+ */
+int gdsys_soc_get_fpga(struct udevice *child, struct udevice **fpga);
+#endif /* _GDSYS_SOC_H_ */
--
2.16.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 3/3] cmd: Add fpgamap command
2018-03-28 12:37 [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 2/3] fpgamap/misc: Add ihs_fpga and gdsys_soc drivers Mario Six
@ 2018-03-28 12:37 ` Mario Six
2018-03-30 8:40 ` [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Simon Glass
2 siblings, 0 replies; 5+ messages in thread
From: Mario Six @ 2018-03-28 12:37 UTC (permalink / raw)
To: u-boot
Add a command to debug the IHS FPGA's bus.
Signed-off-by: Mario Six <mario.six@gdsys.cc>
---
cmd/Kconfig | 5 +
cmd/Makefile | 2 +
cmd/fpgamap.c | 306 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 313 insertions(+)
create mode 100644 cmd/fpgamap.c
diff --git a/cmd/Kconfig b/cmd/Kconfig
index 136836d146..b669b9d6ac 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -976,6 +976,11 @@ config CMD_USB_MASS_STORAGE
help
USB mass storage support
+config CMD_FPGAMAP
+ bool "fpgamap"
+ help
+ Enable the command "fpgamap" to access register maps of FPGAs.
+
endmenu
diff --git a/cmd/Makefile b/cmd/Makefile
index 9a358e4801..6fcb89e118 100644
--- a/cmd/Makefile
+++ b/cmd/Makefile
@@ -144,6 +144,8 @@ obj-$(CONFIG_CMD_DFU) += dfu.o
obj-$(CONFIG_CMD_GPT) += gpt.o
obj-$(CONFIG_CMD_ETHSW) += ethsw.o
+obj-$(CONFIG_CMD_FPGAMAP) += fpgamap.o
+
# Power
obj-$(CONFIG_CMD_PMIC) += pmic.o
obj-$(CONFIG_CMD_REGULATOR) += regulator.o
diff --git a/cmd/fpgamap.c b/cmd/fpgamap.c
new file mode 100644
index 0000000000..56af558361
--- /dev/null
+++ b/cmd/fpgamap.c
@@ -0,0 +1,306 @@
+/*
+ * (C) Copyright 2017
+ * Mario Six, Guntermann & Drunck GmbH, mario.six at gdsys.cc
+ *
+ * based on the fpgad command, which is
+ *
+ * (C) Copyright 2013
+ * Dirk Eibach, Guntermann & Drunck GmbH, dirk.eibach at gdsys.cc
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <command.h>
+#include <console.h>
+#include <fpgamap.h>
+
+static struct udevice *fpgamap_cur;
+static uint dp_last_size;
+static uint dp_last_addr;
+static uint dp_last_length = 0x40;
+
+static void show_fpgamap(struct udevice *fpgamap)
+{
+ struct udevice *dev;
+
+ printf("Bus %d:\t%s", fpgamap->req_seq, fpgamap->name);
+ if (device_active(fpgamap))
+ printf(" (active %d)", fpgamap->seq);
+ printf("\n");
+ for (device_find_first_child(fpgamap, &dev);
+ dev;
+ device_find_next_child(&dev))
+ printf(" %s\n", dev->name);
+}
+
+static int do_fpgamap_show_fpgamap(cmd_tbl_t *cmdtp, int flag, int argc,
+ char * const argv[])
+{
+ if (argc == 1) {
+ /* show all FPGAs */
+ struct udevice *fpgamap;
+ struct uclass *uc;
+ int ret;
+
+ ret = uclass_get(UCLASS_FPGAMAP, &uc);
+ if (ret)
+ return CMD_RET_FAILURE;
+ uclass_foreach_dev(fpgamap, uc)
+ show_fpgamap(fpgamap);
+ } else {
+ int i;
+
+ /* show specific fpgamap */
+ i = simple_strtoul(argv[1], NULL, 10);
+
+ struct udevice *fpgamap;
+ int ret;
+
+ ret = uclass_get_device_by_seq(UCLASS_FPGAMAP, i, &fpgamap);
+ if (ret) {
+ printf("Invalid fpgamap %d: err=%d\n", i, ret);
+ return CMD_RET_FAILURE;
+ }
+ show_fpgamap(fpgamap);
+ }
+
+ return 0;
+}
+
+static int cmd_fpgamap_set_fpgamap_num(unsigned int fpgamapnum)
+{
+ struct udevice *fpgamap;
+ int ret;
+
+ ret = uclass_get_device_by_seq(UCLASS_FPGAMAP, fpgamapnum, &fpgamap);
+ if (ret) {
+ debug("%s: No fpgamap %d\n", __func__, fpgamapnum);
+ return ret;
+ }
+ fpgamap_cur = fpgamap;
+
+ return 0;
+}
+
+static int fpgamap_get_cur_fpgamap(struct udevice **fpgamapp)
+{
+ if (!fpgamap_cur) {
+ puts("No fpgamap selected\n");
+ return -ENODEV;
+ }
+ *fpgamapp = fpgamap_cur;
+
+ return 0;
+}
+
+static int do_fpgamap_fpgamap_num(cmd_tbl_t *cmdtp, int flag, int argc,
+ char * const argv[])
+{
+ int ret = 0;
+ int fpgamap_no;
+
+ if (argc == 1) {
+ /* querying current setting */
+ struct udevice *fpgamap;
+
+ if (!fpgamap_get_cur_fpgamap(&fpgamap))
+ fpgamap_no = fpgamap->seq;
+ else
+ fpgamap_no = -1;
+ printf("Current fpgamap is %d\n", fpgamap_no);
+ } else {
+ fpgamap_no = simple_strtoul(argv[1], NULL, 10);
+ printf("Setting fpgamap to %d\n", fpgamap_no);
+
+ ret = cmd_fpgamap_set_fpgamap_num(fpgamap_no);
+
+ if (ret)
+ printf("Failure changing fpgamap number (%d)\n", ret);
+ }
+
+ return ret ? CMD_RET_FAILURE : 0;
+}
+
+#define DISP_LINE_LEN 16
+
+int do_fpgamap_md(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+{
+ unsigned int k;
+ ulong addr, size, length;
+ int rc = 0;
+ u16 linebuf[DISP_LINE_LEN / sizeof(u16)];
+ ulong nbytes;
+ enum fpgamap_size_t size_param;
+
+ /*
+ * We use the last specified parameters, unless new ones are
+ * entered.
+ */
+ size = dp_last_size;
+ addr = dp_last_addr;
+ length = dp_last_length;
+
+ if (argc < 3)
+ return CMD_RET_USAGE;
+
+ if (!fpgamap_cur) {
+ puts("No fpgamap selected\n");
+ return CMD_RET_FAILURE;
+ }
+
+ if ((flag & CMD_FLAG_REPEAT) == 0) {
+ /*
+ * Size is specified since argc > 3
+ */
+ size = simple_strtoul(argv[1], NULL, 10);
+
+ /*
+ * Address is specified since argc > 3
+ */
+ addr = simple_strtoul(argv[2], NULL, 16);
+
+ /*
+ * If another parameter, it is the length to display.
+ * Length is the number of objects, not number of bytes.
+ */
+ if (argc > 3)
+ length = simple_strtoul(argv[3], NULL, 16);
+ }
+
+ switch (size) {
+ case 1:
+ size_param = FPGAMAP_SIZE_8;
+ break;
+ case 2:
+ size_param = FPGAMAP_SIZE_16;
+ break;
+ case 4:
+ size_param = FPGAMAP_SIZE_32;
+ break;
+ default:
+ printf("Unknown size: %lu\n", size);
+ return CMD_RET_FAILURE;
+ }
+
+ nbytes = length * size;
+ do {
+ ulong linebytes = min(nbytes, (ulong)DISP_LINE_LEN);
+
+ for (k = 0; k < linebytes / size; ++k)
+ rc = fpgamap_read(fpgamap_cur, addr + k * size,
+ &linebuf[k], size_param);
+ if (rc)
+ break;
+
+ print_buffer(addr, (void *)linebuf, size,
+ linebytes / size,
+ DISP_LINE_LEN / size);
+
+ nbytes -= linebytes;
+ addr += linebytes;
+ if (ctrlc()) {
+ rc = 1;
+ break;
+ }
+ } while (nbytes > 0);
+
+ dp_last_size = size;
+ dp_last_addr = addr;
+ dp_last_length = length;
+
+ return rc ? CMD_RET_FAILURE : 0;
+}
+
+static int do_fpgamap_mw(cmd_tbl_t *cmdtp, int flag, int argc,
+ char * const argv[])
+{
+ u16 writeval;
+ ulong size, addr, count;
+ enum fpgamap_size_t size_param;
+
+ if (argc < 4 || argc > 5)
+ return CMD_RET_USAGE;
+
+ if (!fpgamap_cur) {
+ puts("No fpgamap selected\n");
+ return CMD_RET_FAILURE;
+ }
+
+ /* Size is specified since argc > 4 */
+ size = simple_strtoul(argv[1], NULL, 10);
+
+ /* Address is specified since argc > 4 */
+ addr = simple_strtoul(argv[2], NULL, 16);
+
+ /* Get the value to write. */
+ writeval = simple_strtoul(argv[3], NULL, 16);
+
+ /* Count ? */
+ if (argc == 4)
+ count = simple_strtoul(argv[4], NULL, 16);
+ else
+ count = 1;
+
+ switch (size) {
+ case 1:
+ size_param = FPGAMAP_SIZE_8;
+ break;
+ case 2:
+ size_param = FPGAMAP_SIZE_16;
+ break;
+ case 4:
+ size_param = FPGAMAP_SIZE_32;
+ break;
+ default:
+ printf("Unknown size: %lu\n", size);
+ return CMD_RET_FAILURE;
+ }
+
+ while (count-- > 0)
+ fpgamap_write(fpgamap_cur, addr + count * size, &writeval,
+ size_param);
+
+ return 0;
+}
+
+static cmd_tbl_t cmd_fpgamap_sub[] = {
+ U_BOOT_CMD_MKENT(show, 1, 1, do_fpgamap_show_fpgamap, "", ""),
+ U_BOOT_CMD_MKENT(dev, 1, 1, do_fpgamap_fpgamap_num, "", ""),
+ U_BOOT_CMD_MKENT(md, 3, 1, do_fpgamap_md, "", ""),
+ U_BOOT_CMD_MKENT(mw, 4, 1, do_fpgamap_mw, "", ""),
+};
+
+static int do_fpgamap(cmd_tbl_t *cmdtp, int flag, int argc,
+ char * const argv[])
+{
+ cmd_tbl_t *c;
+
+ if (argc < 2)
+ return CMD_RET_USAGE;
+
+ /* Strip off leading 'fpgamap' command argument */
+ argc--;
+ argv++;
+
+ c = find_cmd_tbl(argv[0], &cmd_fpgamap_sub[0],
+ ARRAY_SIZE(cmd_fpgamap_sub));
+
+ if (c)
+ return c->cmd(cmdtp, flag, argc, argv);
+ else
+ return CMD_RET_USAGE;
+}
+
+static char fpgamap_help_text[] =
+ "show - show FPGA register map info\n"
+ "fpgamap dev [dev] - show or set current FPGA register map\n"
+ "fpgamap md size address [# of objects] - read from FPGA register map\n"
+ "fpgamap mw size address value [count] - write to FPGA register map (fill)\n";
+
+U_BOOT_CMD(
+ fpgamap, 7, 1, do_fpgamap,
+ "FPGA sub-system",
+ fpgamap_help_text
+);
--
2.16.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass
2018-03-28 12:37 [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 2/3] fpgamap/misc: Add ihs_fpga and gdsys_soc drivers Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 3/3] cmd: Add fpgamap command Mario Six
@ 2018-03-30 8:40 ` Simon Glass
2018-04-11 6:23 ` Mario Six
2 siblings, 1 reply; 5+ messages in thread
From: Simon Glass @ 2018-03-30 8:40 UTC (permalink / raw)
To: u-boot
Hi Mario,
On 28 March 2018 at 20:37, Mario Six <mario.six@gdsys.cc> wrote:
> The FPGA (as a device) and the register map supplied by a FPGA are two
> different entities. There are U-Boot drivers for the former, but not for
> the later.
>
> Add a uclass that makes it possible to read from and write to FPGA
> memory maps. The interface provided emulates the PCI interface, with one
> function for reading/writing plus a size parameter.
>
> Signed-off-by: Mario Six <mario.six@gdsys.cc>
> ---
> drivers/Kconfig | 2 +
> drivers/Makefile | 1 +
> drivers/fpgamap/Kconfig | 9 +++
> drivers/fpgamap/Makefile | 8 +++
> drivers/fpgamap/fpgamap-uclass.c | 53 ++++++++++++++++
> include/dm/uclass-id.h | 1 +
> include/fpgamap.h | 131 +++++++++++++++++++++++++++++++++++++++
> 7 files changed, 205 insertions(+)
> create mode 100644 drivers/fpgamap/Kconfig
> create mode 100644 drivers/fpgamap/Makefile
> create mode 100644 drivers/fpgamap/fpgamap-uclass.c
> create mode 100644 include/fpgamap.h
>
Could we use regmap for this? That uclass really could use being
enhanced to do the things you do here.
Regards,
Simon
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass
2018-03-30 8:40 ` [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Simon Glass
@ 2018-04-11 6:23 ` Mario Six
0 siblings, 0 replies; 5+ messages in thread
From: Mario Six @ 2018-04-11 6:23 UTC (permalink / raw)
To: u-boot
Hi Simon,
On Fri, Mar 30, 2018 at 10:40 AM, Simon Glass <sjg@chromium.org> wrote:
> Hi Mario,
>
> On 28 March 2018 at 20:37, Mario Six <mario.six@gdsys.cc> wrote:
>> The FPGA (as a device) and the register map supplied by a FPGA are two
>> different entities. There are U-Boot drivers for the former, but not for
>> the later.
>>
>> Add a uclass that makes it possible to read from and write to FPGA
>> memory maps. The interface provided emulates the PCI interface, with one
>> function for reading/writing plus a size parameter.
>>
>> Signed-off-by: Mario Six <mario.six@gdsys.cc>
>> ---
>> drivers/Kconfig | 2 +
>> drivers/Makefile | 1 +
>> drivers/fpgamap/Kconfig | 9 +++
>> drivers/fpgamap/Makefile | 8 +++
>> drivers/fpgamap/fpgamap-uclass.c | 53 ++++++++++++++++
>> include/dm/uclass-id.h | 1 +
>> include/fpgamap.h | 131 +++++++++++++++++++++++++++++++++++++++
>> 7 files changed, 205 insertions(+)
>> create mode 100644 drivers/fpgamap/Kconfig
>> create mode 100644 drivers/fpgamap/Makefile
>> create mode 100644 drivers/fpgamap/fpgamap-uclass.c
>> create mode 100644 include/fpgamap.h
>>
>
> Could we use regmap for this? That uclass really could use being
> enhanced to do the things you do here.
>
OK, that should work, I'll move the functionality over to regmap then.
> Regards,
> Simon
>
Best regards,
Mario
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-04-11 6:23 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-28 12:37 [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 2/3] fpgamap/misc: Add ihs_fpga and gdsys_soc drivers Mario Six
2018-03-28 12:37 ` [U-Boot] [PATCH 3/3] cmd: Add fpgamap command Mario Six
2018-03-30 8:40 ` [U-Boot] [PATCH 1/3] drivers: Add FPGA register map uclass Simon Glass
2018-04-11 6:23 ` Mario Six
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.