From: Mircea Caprioru <[email protected]>
All (newer) FPGA IP cores supported by Analog Devices, store information in
the synthesized designs. This information describes various parameters,
including the family of boards on which this is deployed, speed-grade, and
so on.
Currently, some of these definitions are deployed mostly on Xilinx boards,
but they have been considered also for FPGA boards from other vendors.
The register definitions are described at this link:
https://wiki.analog.com/resources/fpga/docs/hdl/regmap
(the 'Base (common to all cores)' section).
Acked-by: Moritz Fischer <[email protected]>
Signed-off-by: Mircea Caprioru <[email protected]>
Signed-off-by: Alexandru Ardelean <[email protected]>
---
This is a continuation of this old set:
https://lore.kernel.org/linux-clk/[email protected]/
Particularly patches:
https://lore.kernel.org/linux-clk/[email protected]/
https://lore.kernel.org/linux-clk/[email protected]/
That was v4, but this patchset was split away from it, to resolve
discussion on some other patches in that set.
The other patches were accepted here:
https://lore.kernel.org/linux-clk/[email protected]/
include/linux/fpga/adi-axi-common.h | 103 ++++++++++++++++++++++++++++
1 file changed, 103 insertions(+)
diff --git a/include/linux/fpga/adi-axi-common.h b/include/linux/fpga/adi-axi-common.h
index 141ac3f251e6..1a7f18e3a384 100644
--- a/include/linux/fpga/adi-axi-common.h
+++ b/include/linux/fpga/adi-axi-common.h
@@ -13,6 +13,9 @@
#define ADI_AXI_REG_VERSION 0x0000
+#define ADI_AXI_REG_FPGA_INFO 0x001C
+#define ADI_AXI_REG_FPGA_VOLTAGE 0x0140
+
#define ADI_AXI_PCORE_VER(major, minor, patch) \
(((major) << 16) | ((minor) << 8) | (patch))
@@ -20,4 +23,104 @@
#define ADI_AXI_PCORE_VER_MINOR(version) (((version) >> 8) & 0xff)
#define ADI_AXI_PCORE_VER_PATCH(version) ((version) & 0xff)
+#define ADI_AXI_INFO_FPGA_VOLTAGE(val) ((val) & 0xffff)
+
+#define ADI_AXI_INFO_FPGA_TECH(info) (((info) >> 24) & 0xff)
+#define ADI_AXI_INFO_FPGA_FAMILY(info) (((info) >> 16) & 0xff)
+#define ADI_AXI_INFO_FPGA_SPEED_GRADE(info) (((info) >> 8) & 0xff)
+#define ADI_AXI_INFO_FPGA_DEV_PACKAGE(info) ((info) & 0xff)
+
+/**
+ * FPGA Technology definitions
+ */
+#define ADI_AXI_FPGA_TECH_XILINX_UNKNOWN 0
+#define ADI_AXI_FPGA_TECH_XILINS_SERIES7 1
+#define ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE 2
+#define ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE_PLUS 3
+
+#define ADI_AXI_FPGA_TECH_INTEL_UNKNOWN 100
+#define ADI_AXI_FPGA_TECH_INTEL_CYCLONE_5 101
+#define ADI_AXI_FPGA_TECH_INTEL_CYCLONE_10 102
+#define ADI_AXI_FPGA_TECH_INTEL_ARRIA_10 103
+#define ADI_AXI_FPGA_TECH_INTEL_STRATIX_10 104
+
+/**
+ * FPGA Family definitions
+ */
+#define ADI_AXI_FPGA_FAMILY_UNKNOWN 0
+
+#define ADI_AXI_FPGA_FAMILY_XILINX_ARTIX 1
+#define ADI_AXI_FPGA_FAMILY_XILINX_KINTEX 2
+#define ADI_AXI_FPGA_FAMILY_XILINX_VIRTEX 3
+#define ADI_AXI_FPGA_FAMILY_XILINX_ZYNQ 4
+
+#define ADI_AXI_FPGA_FAMILY_INTEL_SX 1
+#define ADI_AXI_FPGA_FAMILY_INTEL_GX 2
+#define ADI_AXI_FPGA_FAMILY_INTEL_GT 3
+#define ADI_AXI_FPGA_FAMILY_INTEL_GZ 4
+
+/**
+ * FPGA Speed-grade definitions
+ */
+#define ADI_AXI_FPGA_SPEED_GRADE_UNKNOWN 0
+
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1 10
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1L 11
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1H 12
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1HV 13
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1LV 14
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2 20
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2L 21
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2LV 22
+#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_3 30
+
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_1 1
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_2 2
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_3 3
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_4 4
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_5 5
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_6 6
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_7 7
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_8 8
+#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_9 9
+
+/**
+ * FPGA Device Package definitions
+ */
+#define ADI_AXI_FPGA_DEV_PACKAGE_UNKNOWN 0
+
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RF 1
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FL 2
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FF 3
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FB 4
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_HC 5
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FH 6
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CS 7
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CP 8
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FT 9
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FG 10
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_SB 11
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RB 12
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RS 13
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CL 14
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_SF 15
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_BA 16
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FA 17
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FS 18
+#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FI 19
+
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_BGA 1
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PGA 2
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_FBGA 3
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_HBGA 4
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PDIP 5
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_EQFP 6
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PLCC 7
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PQFP 8
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_RQFP 9
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_TQFP 10
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_UBGA 11
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_UFBGA 12
+#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_MBGA 13
+
#endif /* ADI_AXI_COMMON_H_ */
--
2.17.1
From: Mircea Caprioru <[email protected]>
This patch adds support for vco maximum and minimum ranges in accordance
with fpga speed grade, voltage, device package, technology and family. This
new information is extracted from two new registers implemented in the ip
core: ADI_REG_FPGA_INFO and ADI_REG_FPGA_VOLTAGE, which are stored in the
'include/linux/fpga/adi-axi-common.h' file as they are common to all ADI
FPGA cores.
Signed-off-by: Mircea Caprioru <[email protected]>
Signed-off-by: Alexandru Ardelean <[email protected]>
---
drivers/clk/clk-axi-clkgen.c | 52 +++++++++++++++++++++++++++++++++++-
1 file changed, 51 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c
index ac6ff736ac8f..e4d6c87f8a07 100644
--- a/drivers/clk/clk-axi-clkgen.c
+++ b/drivers/clk/clk-axi-clkgen.c
@@ -8,6 +8,7 @@
#include <linux/platform_device.h>
#include <linux/clk-provider.h>
+#include <linux/fpga/adi-axi-common.h>
#include <linux/slab.h>
#include <linux/io.h>
#include <linux/of.h>
@@ -240,6 +241,50 @@ static void axi_clkgen_read(struct axi_clkgen *axi_clkgen,
*val = readl(axi_clkgen->base + reg);
}
+static void axi_clkgen_setup_ranges(struct axi_clkgen *axi_clkgen)
+{
+ struct axi_clkgen_limits *limits = &axi_clkgen->limits;
+ unsigned int reg_value;
+ unsigned int tech, family, speed_grade, voltage;
+
+ axi_clkgen_read(axi_clkgen, ADI_AXI_REG_FPGA_INFO, ®_value);
+ tech = ADI_AXI_INFO_FPGA_TECH(reg_value);
+ family = ADI_AXI_INFO_FPGA_FAMILY(reg_value);
+ speed_grade = ADI_AXI_INFO_FPGA_SPEED_GRADE(reg_value);
+
+ axi_clkgen_read(axi_clkgen, ADI_AXI_REG_FPGA_VOLTAGE, ®_value);
+ voltage = ADI_AXI_INFO_FPGA_VOLTAGE(reg_value);
+
+ switch (speed_grade) {
+ case ADI_AXI_FPGA_SPEED_GRADE_XILINX_1 ... ADI_AXI_FPGA_SPEED_GRADE_XILINX_1LV:
+ limits->fvco_max = 1200000;
+ limits->fpfd_max = 450000;
+ break;
+ case ADI_AXI_FPGA_SPEED_GRADE_XILINX_2 ... ADI_AXI_FPGA_SPEED_GRADE_XILINX_2LV:
+ limits->fvco_max = 1440000;
+ limits->fpfd_max = 500000;
+ if ((family == ADI_AXI_FPGA_FAMILY_XILINX_KINTEX) |
+ (family == ADI_AXI_FPGA_FAMILY_XILINX_ARTIX)) {
+ if (voltage < 950) {
+ limits->fvco_max = 1200000;
+ limits->fpfd_max = 450000;
+ }
+ }
+ break;
+ case ADI_AXI_FPGA_SPEED_GRADE_XILINX_3:
+ limits->fvco_max = 1600000;
+ limits->fpfd_max = 550000;
+ break;
+ default:
+ break;
+ };
+
+ if (tech == ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE_PLUS) {
+ limits->fvco_max = 1600000;
+ limits->fvco_min = 800000;
+ }
+}
+
static int axi_clkgen_wait_non_busy(struct axi_clkgen *axi_clkgen)
{
unsigned int timeout = 10000;
@@ -510,7 +555,7 @@ static int axi_clkgen_probe(struct platform_device *pdev)
struct clk_init_data init;
const char *parent_names[2];
const char *clk_name;
- unsigned int i;
+ unsigned int i, ver;
int ret;
dflt_limits = device_get_match_data(&pdev->dev);
@@ -537,6 +582,11 @@ static int axi_clkgen_probe(struct platform_device *pdev)
memcpy(&axi_clkgen->limits, dflt_limits, sizeof(axi_clkgen->limits));
+ axi_clkgen_read(axi_clkgen, ADI_AXI_REG_VERSION, &ver);
+
+ if (ADI_AXI_PCORE_VER_MAJOR(ver) > 0x04)
+ axi_clkgen_setup_ranges(axi_clkgen);
+
clk_name = pdev->dev.of_node->name;
of_property_read_string(pdev->dev.of_node, "clock-output-names",
&clk_name);
--
2.17.1
Alexandru,
On Wed, Feb 10, 2021 at 12:15:34PM +0200, Alexandru Ardelean wrote:
> From: Mircea Caprioru <[email protected]>
>
> All (newer) FPGA IP cores supported by Analog Devices, store information in
Nit: extra ',' ?
> the synthesized designs. This information describes various parameters,
> including the family of boards on which this is deployed, speed-grade, and
> so on.
>
> Currently, some of these definitions are deployed mostly on Xilinx boards,
> but they have been considered also for FPGA boards from other vendors.
Let's add them together with the code that uses them.
>
> The register definitions are described at this link:
> https://wiki.analog.com/resources/fpga/docs/hdl/regmap
> (the 'Base (common to all cores)' section).
>
> Acked-by: Moritz Fischer <[email protected]>
This patchset is very different from the reviewed one earlier. Please
don't just copy Acked-by's.
> Signed-off-by: Mircea Caprioru <[email protected]>
> Signed-off-by: Alexandru Ardelean <[email protected]>
> ---
>
> This is a continuation of this old set:
> https://lore.kernel.org/linux-clk/[email protected]/
>
> Particularly patches:
> https://lore.kernel.org/linux-clk/[email protected]/
> https://lore.kernel.org/linux-clk/[email protected]/
>
> That was v4, but this patchset was split away from it, to resolve
> discussion on some other patches in that set.
>
> The other patches were accepted here:
> https://lore.kernel.org/linux-clk/[email protected]/
>
> include/linux/fpga/adi-axi-common.h | 103 ++++++++++++++++++++++++++++
> 1 file changed, 103 insertions(+)
>
> diff --git a/include/linux/fpga/adi-axi-common.h b/include/linux/fpga/adi-axi-common.h
> index 141ac3f251e6..1a7f18e3a384 100644
> --- a/include/linux/fpga/adi-axi-common.h
> +++ b/include/linux/fpga/adi-axi-common.h
> @@ -13,6 +13,9 @@
>
> #define ADI_AXI_REG_VERSION 0x0000
>
> +#define ADI_AXI_REG_FPGA_INFO 0x001C
> +#define ADI_AXI_REG_FPGA_VOLTAGE 0x0140
> +
> #define ADI_AXI_PCORE_VER(major, minor, patch) \
> (((major) << 16) | ((minor) << 8) | (patch))
>
> @@ -20,4 +23,104 @@
> #define ADI_AXI_PCORE_VER_MINOR(version) (((version) >> 8) & 0xff)
> #define ADI_AXI_PCORE_VER_PATCH(version) ((version) & 0xff)
>
> +#define ADI_AXI_INFO_FPGA_VOLTAGE(val) ((val) & 0xffff)
> +
> +#define ADI_AXI_INFO_FPGA_TECH(info) (((info) >> 24) & 0xff)
> +#define ADI_AXI_INFO_FPGA_FAMILY(info) (((info) >> 16) & 0xff)
> +#define ADI_AXI_INFO_FPGA_SPEED_GRADE(info) (((info) >> 8) & 0xff)
> +#define ADI_AXI_INFO_FPGA_DEV_PACKAGE(info) ((info) & 0xff)
Do we really need all the macros?
> +
> +/**
> + * FPGA Technology definitions
> + */
> +#define ADI_AXI_FPGA_TECH_XILINX_UNKNOWN 0
> +#define ADI_AXI_FPGA_TECH_XILINS_SERIES7 1
> +#define ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE 2
> +#define ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE_PLUS 3
> +
> +#define ADI_AXI_FPGA_TECH_INTEL_UNKNOWN 100
> +#define ADI_AXI_FPGA_TECH_INTEL_CYCLONE_5 101
> +#define ADI_AXI_FPGA_TECH_INTEL_CYCLONE_10 102
> +#define ADI_AXI_FPGA_TECH_INTEL_ARRIA_10 103
> +#define ADI_AXI_FPGA_TECH_INTEL_STRATIX_10 104
> +
> +/**
> + * FPGA Family definitions
> + */
> +#define ADI_AXI_FPGA_FAMILY_UNKNOWN 0
> +
> +#define ADI_AXI_FPGA_FAMILY_XILINX_ARTIX 1
> +#define ADI_AXI_FPGA_FAMILY_XILINX_KINTEX 2
> +#define ADI_AXI_FPGA_FAMILY_XILINX_VIRTEX 3
> +#define ADI_AXI_FPGA_FAMILY_XILINX_ZYNQ 4
> +
> +#define ADI_AXI_FPGA_FAMILY_INTEL_SX 1
> +#define ADI_AXI_FPGA_FAMILY_INTEL_GX 2
> +#define ADI_AXI_FPGA_FAMILY_INTEL_GT 3
> +#define ADI_AXI_FPGA_FAMILY_INTEL_GZ 4
> +
> +/**
> + * FPGA Speed-grade definitions
> + */
> +#define ADI_AXI_FPGA_SPEED_GRADE_UNKNOWN 0
> +
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1 10
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1L 11
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1H 12
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1HV 13
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1LV 14
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2 20
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2L 21
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2LV 22
> +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_3 30
> +
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_1 1
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_2 2
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_3 3
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_4 4
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_5 5
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_6 6
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_7 7
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_8 8
> +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_9 9
> +
> +/**
> + * FPGA Device Package definitions
> + */
> +#define ADI_AXI_FPGA_DEV_PACKAGE_UNKNOWN 0
> +
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RF 1
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FL 2
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FF 3
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FB 4
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_HC 5
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FH 6
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CS 7
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CP 8
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FT 9
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FG 10
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_SB 11
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RB 12
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RS 13
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CL 14
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_SF 15
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_BA 16
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FA 17
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FS 18
> +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FI 19
> +
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_BGA 1
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PGA 2
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_FBGA 3
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_HBGA 4
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PDIP 5
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_EQFP 6
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PLCC 7
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PQFP 8
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_RQFP 9
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_TQFP 10
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_UBGA 11
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_UFBGA 12
> +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_MBGA 13
What is using those? Do these package impact anything behavioral?
> +
> #endif /* ADI_AXI_COMMON_H_ */
> --
> 2.17.1
>
- Moritz
PS: The subject line could use a bit of work, too :)
On Wed, Feb 10, 2021 at 12:15:35PM +0200, Alexandru Ardelean wrote:
> From: Mircea Caprioru <[email protected]>
>
> This patch adds support for vco maximum and minimum ranges in accordance
VCO
> with fpga speed grade, voltage, device package, technology and family. This
FPGA
> new information is extracted from two new registers implemented in the ip
> core: ADI_REG_FPGA_INFO and ADI_REG_FPGA_VOLTAGE, which are stored in the
> 'include/linux/fpga/adi-axi-common.h' file as they are common to all ADI
> FPGA cores.
Either all caps FPGA or all non-caps.
>
> Signed-off-by: Mircea Caprioru <[email protected]>
> Signed-off-by: Alexandru Ardelean <[email protected]>
> ---
> drivers/clk/clk-axi-clkgen.c | 52 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 51 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c
> index ac6ff736ac8f..e4d6c87f8a07 100644
> --- a/drivers/clk/clk-axi-clkgen.c
> +++ b/drivers/clk/clk-axi-clkgen.c
> @@ -8,6 +8,7 @@
>
> #include <linux/platform_device.h>
> #include <linux/clk-provider.h>
> +#include <linux/fpga/adi-axi-common.h>
> #include <linux/slab.h>
> #include <linux/io.h>
> #include <linux/of.h>
> @@ -240,6 +241,50 @@ static void axi_clkgen_read(struct axi_clkgen *axi_clkgen,
> *val = readl(axi_clkgen->base + reg);
> }
>
> +static void axi_clkgen_setup_ranges(struct axi_clkgen *axi_clkgen)
> +{
> + struct axi_clkgen_limits *limits = &axi_clkgen->limits;
> + unsigned int reg_value;
> + unsigned int tech, family, speed_grade, voltage;
Reverse xmas-tree please.
xxxxxx
xxx
x
> +
> + axi_clkgen_read(axi_clkgen, ADI_AXI_REG_FPGA_INFO, ®_value);
> + tech = ADI_AXI_INFO_FPGA_TECH(reg_value);
> + family = ADI_AXI_INFO_FPGA_FAMILY(reg_value);
> + speed_grade = ADI_AXI_INFO_FPGA_SPEED_GRADE(reg_value);
> +
> + axi_clkgen_read(axi_clkgen, ADI_AXI_REG_FPGA_VOLTAGE, ®_value);
> + voltage = ADI_AXI_INFO_FPGA_VOLTAGE(reg_value);
> +
> + switch (speed_grade) {
> + case ADI_AXI_FPGA_SPEED_GRADE_XILINX_1 ... ADI_AXI_FPGA_SPEED_GRADE_XILINX_1LV:
> + limits->fvco_max = 1200000;
> + limits->fpfd_max = 450000;
> + break;
> + case ADI_AXI_FPGA_SPEED_GRADE_XILINX_2 ... ADI_AXI_FPGA_SPEED_GRADE_XILINX_2LV:
> + limits->fvco_max = 1440000;
> + limits->fpfd_max = 500000;
> + if ((family == ADI_AXI_FPGA_FAMILY_XILINX_KINTEX) |
> + (family == ADI_AXI_FPGA_FAMILY_XILINX_ARTIX)) {
> + if (voltage < 950) {
> + limits->fvco_max = 1200000;
> + limits->fpfd_max = 450000;
> + }
> + }
> + break;
> + case ADI_AXI_FPGA_SPEED_GRADE_XILINX_3:
> + limits->fvco_max = 1600000;
> + limits->fpfd_max = 550000;
> + break;
> + default:
No warning? Does PCORE_VER_MAJOR(ver) > 0x04 not imply this to be known
or valid?
> + break;
> + };
> +
Maybe:
"For Ultrascale+ the speedgrades don't matter" or something as a comment?
> + if (tech == ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE_PLUS) {
> + limits->fvco_max = 1600000;
> + limits->fvco_min = 800000;
> + }
> +}
> +
> static int axi_clkgen_wait_non_busy(struct axi_clkgen *axi_clkgen)
> {
> unsigned int timeout = 10000;
> @@ -510,7 +555,7 @@ static int axi_clkgen_probe(struct platform_device *pdev)
> struct clk_init_data init;
> const char *parent_names[2];
> const char *clk_name;
> - unsigned int i;
> + unsigned int i, ver;
> int ret;
>
> dflt_limits = device_get_match_data(&pdev->dev);
> @@ -537,6 +582,11 @@ static int axi_clkgen_probe(struct platform_device *pdev)
>
> memcpy(&axi_clkgen->limits, dflt_limits, sizeof(axi_clkgen->limits));
>
> + axi_clkgen_read(axi_clkgen, ADI_AXI_REG_VERSION, &ver);
> +
> + if (ADI_AXI_PCORE_VER_MAJOR(ver) > 0x04)
Nit: 0x4 is fine?
> + axi_clkgen_setup_ranges(axi_clkgen);
> +
> clk_name = pdev->dev.of_node->name;
> of_property_read_string(pdev->dev.of_node, "clock-output-names",
> &clk_name);
> --
> 2.17.1
>
Thanks,
Moritz
On Thu, Feb 11, 2021 at 6:00 AM Moritz Fischer <[email protected]> wrote:
>
> Alexandru,
>
> On Wed, Feb 10, 2021 at 12:15:34PM +0200, Alexandru Ardelean wrote:
> > From: Mircea Caprioru <[email protected]>
> >
> > All (newer) FPGA IP cores supported by Analog Devices, store information in
>
> Nit: extra ',' ?
> > the synthesized designs. This information describes various parameters,
> > including the family of boards on which this is deployed, speed-grade, and
> > so on.
> >
> > Currently, some of these definitions are deployed mostly on Xilinx boards,
> > but they have been considered also for FPGA boards from other vendors.
> Let's add them together with the code that uses them.
> >
> > The register definitions are described at this link:
> > https://wiki.analog.com/resources/fpga/docs/hdl/regmap
> > (the 'Base (common to all cores)' section).
> >
> > Acked-by: Moritz Fischer <[email protected]>
> This patchset is very different from the reviewed one earlier. Please
> don't just copy Acked-by's.
Apologies
I think it got some more reviews and I just kept the tag.
Sloppy on my part.
>
> > Signed-off-by: Mircea Caprioru <[email protected]>
> > Signed-off-by: Alexandru Ardelean <[email protected]>
> > ---
> >
> > This is a continuation of this old set:
> > https://lore.kernel.org/linux-clk/[email protected]/
> >
> > Particularly patches:
> > https://lore.kernel.org/linux-clk/[email protected]/
> > https://lore.kernel.org/linux-clk/[email protected]/
> >
> > That was v4, but this patchset was split away from it, to resolve
> > discussion on some other patches in that set.
> >
> > The other patches were accepted here:
> > https://lore.kernel.org/linux-clk/[email protected]/
> >
> > include/linux/fpga/adi-axi-common.h | 103 ++++++++++++++++++++++++++++
> > 1 file changed, 103 insertions(+)
> >
> > diff --git a/include/linux/fpga/adi-axi-common.h b/include/linux/fpga/adi-axi-common.h
> > index 141ac3f251e6..1a7f18e3a384 100644
> > --- a/include/linux/fpga/adi-axi-common.h
> > +++ b/include/linux/fpga/adi-axi-common.h
> > @@ -13,6 +13,9 @@
> >
> > #define ADI_AXI_REG_VERSION 0x0000
> >
> > +#define ADI_AXI_REG_FPGA_INFO 0x001C
> > +#define ADI_AXI_REG_FPGA_VOLTAGE 0x0140
> > +
> > #define ADI_AXI_PCORE_VER(major, minor, patch) \
> > (((major) << 16) | ((minor) << 8) | (patch))
> >
> > @@ -20,4 +23,104 @@
> > #define ADI_AXI_PCORE_VER_MINOR(version) (((version) >> 8) & 0xff)
> > #define ADI_AXI_PCORE_VER_PATCH(version) ((version) & 0xff)
> >
> > +#define ADI_AXI_INFO_FPGA_VOLTAGE(val) ((val) & 0xffff)
> > +
> > +#define ADI_AXI_INFO_FPGA_TECH(info) (((info) >> 24) & 0xff)
> > +#define ADI_AXI_INFO_FPGA_FAMILY(info) (((info) >> 16) & 0xff)
> > +#define ADI_AXI_INFO_FPGA_SPEED_GRADE(info) (((info) >> 8) & 0xff)
> > +#define ADI_AXI_INFO_FPGA_DEV_PACKAGE(info) ((info) & 0xff)
>
> Do we really need all the macros?
No.
I can trim them to a minimum.
> > +
> > +/**
> > + * FPGA Technology definitions
> > + */
> > +#define ADI_AXI_FPGA_TECH_XILINX_UNKNOWN 0
> > +#define ADI_AXI_FPGA_TECH_XILINS_SERIES7 1
> > +#define ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE 2
> > +#define ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE_PLUS 3
> > +
> > +#define ADI_AXI_FPGA_TECH_INTEL_UNKNOWN 100
> > +#define ADI_AXI_FPGA_TECH_INTEL_CYCLONE_5 101
> > +#define ADI_AXI_FPGA_TECH_INTEL_CYCLONE_10 102
> > +#define ADI_AXI_FPGA_TECH_INTEL_ARRIA_10 103
> > +#define ADI_AXI_FPGA_TECH_INTEL_STRATIX_10 104
> > +
> > +/**
> > + * FPGA Family definitions
> > + */
> > +#define ADI_AXI_FPGA_FAMILY_UNKNOWN 0
> > +
> > +#define ADI_AXI_FPGA_FAMILY_XILINX_ARTIX 1
> > +#define ADI_AXI_FPGA_FAMILY_XILINX_KINTEX 2
> > +#define ADI_AXI_FPGA_FAMILY_XILINX_VIRTEX 3
> > +#define ADI_AXI_FPGA_FAMILY_XILINX_ZYNQ 4
> > +
> > +#define ADI_AXI_FPGA_FAMILY_INTEL_SX 1
> > +#define ADI_AXI_FPGA_FAMILY_INTEL_GX 2
> > +#define ADI_AXI_FPGA_FAMILY_INTEL_GT 3
> > +#define ADI_AXI_FPGA_FAMILY_INTEL_GZ 4
> > +
> > +/**
> > + * FPGA Speed-grade definitions
> > + */
> > +#define ADI_AXI_FPGA_SPEED_GRADE_UNKNOWN 0
> > +
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1 10
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1L 11
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1H 12
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1HV 13
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_1LV 14
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2 20
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2L 21
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_2LV 22
> > +#define ADI_AXI_FPGA_SPEED_GRADE_XILINX_3 30
> > +
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_1 1
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_2 2
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_3 3
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_4 4
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_5 5
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_6 6
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_7 7
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_8 8
> > +#define ADI_AXI_FPGA_SPEED_GRADE_INTEL_9 9
> > +
> > +/**
> > + * FPGA Device Package definitions
> > + */
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_UNKNOWN 0
> > +
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RF 1
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FL 2
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FF 3
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FB 4
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_HC 5
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FH 6
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CS 7
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CP 8
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FT 9
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FG 10
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_SB 11
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RB 12
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_RS 13
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_CL 14
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_SF 15
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_BA 16
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FA 17
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FS 18
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_XILINX_FI 19
> > +
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_BGA 1
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PGA 2
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_FBGA 3
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_HBGA 4
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PDIP 5
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_EQFP 6
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PLCC 7
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_PQFP 8
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_RQFP 9
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_TQFP 10
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_UBGA 11
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_UFBGA 12
> > +#define ADI_AXI_FPGA_DEV_PACKAGE_INTEL_MBGA 13
>
> What is using those? Do these package impact anything behavioral?
So, these should get defined in ADI FPGA projects.
The initial idea was to be able to read this in drivers and adjust
behavior (usually speed) according to each parameter.
Then some drivers could have used this to do some tunings.
Maybe also pass these in userspace and have the application decide.
I don't think there was some finalization on the idea (yet).
[I know how this sounds, but] The more I try to explain it, the less I
get convinced by this patch.
I pick up some of the patches in our tree and try to make sense of
them through upstreaming.
Sometimes when I pick them, it's not always obvious to me whether
they're worth it or not.
¯\_(ツ)_/¯
> > +
> > #endif /* ADI_AXI_COMMON_H_ */
> > --
> > 2.17.1
> >
>
> - Moritz
>
> PS: The subject line could use a bit of work, too :)
Will think about it :)
On 2/10/21 2:15 AM, Alexandru Ardelean wrote:
> From: Mircea Caprioru <[email protected]>
>
> This patch adds support for vco maximum and minimum ranges in accordance
> with fpga speed grade, voltage, device package, technology and family. This
> new information is extracted from two new registers implemented in the ip
> core: ADI_REG_FPGA_INFO and ADI_REG_FPGA_VOLTAGE, which are stored in the
> 'include/linux/fpga/adi-axi-common.h' file as they are common to all ADI
> FPGA cores.
>
> Signed-off-by: Mircea Caprioru <[email protected]>
> Signed-off-by: Alexandru Ardelean <[email protected]>
> ---
> drivers/clk/clk-axi-clkgen.c | 52 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 51 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c
> index ac6ff736ac8f..e4d6c87f8a07 100644
> --- a/drivers/clk/clk-axi-clkgen.c
> +++ b/drivers/clk/clk-axi-clkgen.c
> @@ -8,6 +8,7 @@
>
> #include <linux/platform_device.h>
> #include <linux/clk-provider.h>
> +#include <linux/fpga/adi-axi-common.h>
> #include <linux/slab.h>
> #include <linux/io.h>
> #include <linux/of.h>
> @@ -240,6 +241,50 @@ static void axi_clkgen_read(struct axi_clkgen *axi_clkgen,
> *val = readl(axi_clkgen->base + reg);
> }
>
> +static void axi_clkgen_setup_ranges(struct axi_clkgen *axi_clkgen)
> +{
> + struct axi_clkgen_limits *limits = &axi_clkgen->limits;
> + unsigned int reg_value;
> + unsigned int tech, family, speed_grade, voltage;
> +
> + axi_clkgen_read(axi_clkgen, ADI_AXI_REG_FPGA_INFO, ®_value);
> + tech = ADI_AXI_INFO_FPGA_TECH(reg_value);
> + family = ADI_AXI_INFO_FPGA_FAMILY(reg_value);
> + speed_grade = ADI_AXI_INFO_FPGA_SPEED_GRADE(reg_value);
> +
> + axi_clkgen_read(axi_clkgen, ADI_AXI_REG_FPGA_VOLTAGE, ®_value);
> + voltage = ADI_AXI_INFO_FPGA_VOLTAGE(reg_value);
> +
> + switch (speed_grade) {
> + case ADI_AXI_FPGA_SPEED_GRADE_XILINX_1 ... ADI_AXI_FPGA_SPEED_GRADE_XILINX_1LV:
> + limits->fvco_max = 1200000;
> + limits->fpfd_max = 450000;
> + break;
> + case ADI_AXI_FPGA_SPEED_GRADE_XILINX_2 ... ADI_AXI_FPGA_SPEED_GRADE_XILINX_2LV:
> + limits->fvco_max = 1440000;
> + limits->fpfd_max = 500000;
> + if ((family == ADI_AXI_FPGA_FAMILY_XILINX_KINTEX) |
> + (family == ADI_AXI_FPGA_FAMILY_XILINX_ARTIX)) {
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
3a419c9317b157ef06ca347b7e6bbab846a3d605 clk: axi-clkgen: Add support for FPGA info
CHECK: Unnecessary parentheses around 'family == ADI_AXI_FPGA_FAMILY_XILINX_KINTEX'
#57: FILE: drivers/clk/clk-axi-clkgen.c:266:
+ if ((family == ADI_AXI_FPGA_FAMILY_XILINX_KINTEX) |
+ (family == ADI_AXI_FPGA_FAMILY_XILINX_ARTIX)) {
Please use checkpatch -strict to find this problem.
Likely the '|', should be '||'
Tom
> + if (voltage < 950) {
> + limits->fvco_max = 1200000;
> + limits->fpfd_max = 450000;
> + }
> + }
> + break;
> + case ADI_AXI_FPGA_SPEED_GRADE_XILINX_3:
> + limits->fvco_max = 1600000;
> + limits->fpfd_max = 550000;
> + break;
> + default:
> + break;
> + };
> +
> + if (tech == ADI_AXI_FPGA_TECH_XILINX_ULTRASCALE_PLUS) {
> + limits->fvco_max = 1600000;
> + limits->fvco_min = 800000;
> + }
> +}
> +
> static int axi_clkgen_wait_non_busy(struct axi_clkgen *axi_clkgen)
> {
> unsigned int timeout = 10000;
> @@ -510,7 +555,7 @@ static int axi_clkgen_probe(struct platform_device *pdev)
> struct clk_init_data init;
> const char *parent_names[2];
> const char *clk_name;
> - unsigned int i;
> + unsigned int i, ver;
> int ret;
>
> dflt_limits = device_get_match_data(&pdev->dev);
> @@ -537,6 +582,11 @@ static int axi_clkgen_probe(struct platform_device *pdev)
>
> memcpy(&axi_clkgen->limits, dflt_limits, sizeof(axi_clkgen->limits));
>
> + axi_clkgen_read(axi_clkgen, ADI_AXI_REG_VERSION, &ver);
> +
> + if (ADI_AXI_PCORE_VER_MAJOR(ver) > 0x04)
> + axi_clkgen_setup_ranges(axi_clkgen);
> +
> clk_name = pdev->dev.of_node->name;
> of_property_read_string(pdev->dev.of_node, "clock-output-names",
> &clk_name);