* [PATCH v2 0/3] Add Bitstream configuration support for Versal
@ 2021-02-11 6:05 Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 1/3] drivers: firmware: Add PDI load API support Nava kishore Manne
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Nava kishore Manne @ 2021-02-11 6:05 UTC (permalink / raw)
To: mdf, trix, robh+dt, michal.simek, linux-fpga, devicetree,
linux-arm-kernel, linux-kernel, chinnikishore369
Cc: git, Nava kishore Manne
Appana Durga Kedareswara rao (1):
dt-bindings: fpga: Add binding doc for versal fpga manager
Nava kishore Manne (2):
drivers: firmware: Add PDI load API support
fpga: versal-fpga: Add versal fpga manager driver
.../bindings/fpga/xlnx,versal-fpga.yaml | 33 +++++
drivers/firmware/xilinx/zynqmp.c | 17 +++
drivers/fpga/Kconfig | 8 ++
drivers/fpga/Makefile | 1 +
drivers/fpga/versal-fpga.c | 120 ++++++++++++++++++
include/linux/firmware/xlnx-zynqmp.h | 9 ++
6 files changed, 188 insertions(+)
create mode 100644 Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
create mode 100644 drivers/fpga/versal-fpga.c
--
2.18.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 1/3] drivers: firmware: Add PDI load API support
2021-02-11 6:05 [PATCH v2 0/3] Add Bitstream configuration support for Versal Nava kishore Manne
@ 2021-02-11 6:05 ` Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver Nava kishore Manne
2 siblings, 0 replies; 8+ messages in thread
From: Nava kishore Manne @ 2021-02-11 6:05 UTC (permalink / raw)
To: mdf, trix, robh+dt, michal.simek, linux-fpga, devicetree,
linux-arm-kernel, linux-kernel, chinnikishore369
Cc: git, Nava kishore Manne
This patch adds load PDI API support to enable PDI/partial loading from
linux. Programmable Device Image (PDI) is combination of headers, images
and bitstream files to be loaded. Partial PDI is partial set of image/
images to be loaded.
Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
---
Changes for v2:
-Updated API Doc and commit msg.
No functional changes.
drivers/firmware/xilinx/zynqmp.c | 17 +++++++++++++++++
include/linux/firmware/xlnx-zynqmp.h | 9 +++++++++
2 files changed, 26 insertions(+)
diff --git a/drivers/firmware/xilinx/zynqmp.c b/drivers/firmware/xilinx/zynqmp.c
index 7eb9958662dd..9ee02655db89 100644
--- a/drivers/firmware/xilinx/zynqmp.c
+++ b/drivers/firmware/xilinx/zynqmp.c
@@ -897,6 +897,23 @@ int zynqmp_pm_set_requirement(const u32 node, const u32 capabilities,
}
EXPORT_SYMBOL_GPL(zynqmp_pm_set_requirement);
+/**
+ * zynqmp_pm_load_pdi - Load and process PDI
+ * @src: Source device where PDI is located
+ * @address: PDI src address
+ *
+ * This function provides support to load PDI from linux
+ *
+ * Return: Returns status, either success or error+reason
+ */
+int zynqmp_pm_load_pdi(const u32 src, const u64 address)
+{
+ return zynqmp_pm_invoke_fn(PM_LOAD_PDI, src,
+ lower_32_bits(address),
+ upper_32_bits(address), 0, NULL);
+}
+EXPORT_SYMBOL_GPL(zynqmp_pm_load_pdi);
+
/**
* zynqmp_pm_aes - Access AES hardware to encrypt/decrypt the data using
* AES-GCM core.
diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
index 2a0da841c942..87114ee645b1 100644
--- a/include/linux/firmware/xlnx-zynqmp.h
+++ b/include/linux/firmware/xlnx-zynqmp.h
@@ -52,6 +52,9 @@
#define ZYNQMP_PM_CAPABILITY_WAKEUP 0x4U
#define ZYNQMP_PM_CAPABILITY_UNUSABLE 0x8U
+/* Loader commands */
+#define PM_LOAD_PDI 0x701
+
/*
* Firmware FPGA Manager flags
* XILINX_ZYNQMP_PM_FPGA_FULL: FPGA full reconfiguration
@@ -354,6 +357,7 @@ int zynqmp_pm_write_pggs(u32 index, u32 value);
int zynqmp_pm_read_pggs(u32 index, u32 *value);
int zynqmp_pm_system_shutdown(const u32 type, const u32 subtype);
int zynqmp_pm_set_boot_health_status(u32 value);
+int zynqmp_pm_load_pdi(const u32 src, const u64 address);
#else
static inline struct zynqmp_eemi_ops *zynqmp_pm_get_eemi_ops(void)
{
@@ -538,6 +542,11 @@ static inline int zynqmp_pm_set_boot_health_status(u32 value)
{
return -ENODEV;
}
+
+static inline int zynqmp_pm_load_pdi(const u32 src, const u64 address)
+{
+ return -ENODEV;
+}
#endif
#endif /* __FIRMWARE_ZYNQMP_H__ */
--
2.18.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager
2021-02-11 6:05 [PATCH v2 0/3] Add Bitstream configuration support for Versal Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 1/3] drivers: firmware: Add PDI load API support Nava kishore Manne
@ 2021-02-11 6:05 ` Nava kishore Manne
2021-02-11 14:56 ` Rob Herring
2021-02-11 6:05 ` [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver Nava kishore Manne
2 siblings, 1 reply; 8+ messages in thread
From: Nava kishore Manne @ 2021-02-11 6:05 UTC (permalink / raw)
To: mdf, trix, robh+dt, michal.simek, linux-fpga, devicetree,
linux-arm-kernel, linux-kernel, chinnikishore369
Cc: git, Appana Durga Kedareswara rao, Nava kishore Manne
From: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
This patch adds binding doc for versal fpga manager driver.
Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
Signed-off-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
---
Changes for v2:
-Fixed file format and syntax issues.
.../bindings/fpga/xlnx,versal-fpga.yaml | 33 +++++++++++++++++++
1 file changed, 33 insertions(+)
create mode 100644 Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
diff --git a/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml b/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
new file mode 100644
index 000000000000..65d6877b913e
--- /dev/null
+++ b/Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
@@ -0,0 +1,33 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/fpga/xlnx,versal-fpga.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Xilinx Versal FPGA driver.
+
+maintainers:
+ - Nava kishore Manne <nava.manne@xilinx.com>
+
+description: |
+ Device Tree Versal FPGA bindings for the Versal SoC, controlled
+ using firmware interface.
+
+properties:
+ compatible:
+ items:
+ - enum:
+ - xlnx,versal-fpga
+
+required:
+ - compatible
+
+additionalProperties: false
+
+examples:
+ - |
+ versal_fpga: fpga {
+ compatible = "xlnx,versal-fpga";
+ };
+
+...
--
2.18.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver
2021-02-11 6:05 [PATCH v2 0/3] Add Bitstream configuration support for Versal Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 1/3] drivers: firmware: Add PDI load API support Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager Nava kishore Manne
@ 2021-02-11 6:05 ` Nava kishore Manne
2021-02-11 6:23 ` Randy Dunlap
2 siblings, 1 reply; 8+ messages in thread
From: Nava kishore Manne @ 2021-02-11 6:05 UTC (permalink / raw)
To: mdf, trix, robh+dt, michal.simek, linux-fpga, devicetree,
linux-arm-kernel, linux-kernel, chinnikishore369
Cc: git, Nava kishore Manne, Appana Durga Kedareswara rao
Add support for Xilinx Versal FPGA manager.
PDI source type can be DDR, OCM, QSPI flash etc..
But driver allocates memory always from DDR, Since driver supports only
DDR source type.
Signed-off-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
---
Changes for v2:
-Updated the Fpga Mgr registrations call's
to 5.11
-Fixed some minor coding issues as suggested by
Moritz.
drivers/fpga/Kconfig | 8 +++
drivers/fpga/Makefile | 1 +
drivers/fpga/versal-fpga.c | 120 +++++++++++++++++++++++++++++++++++++
3 files changed, 129 insertions(+)
create mode 100644 drivers/fpga/versal-fpga.c
diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index bf85b9a65ec2..dcd2ed5a7956 100644
--- a/drivers/fpga/Kconfig
+++ b/drivers/fpga/Kconfig
@@ -223,4 +223,12 @@ config FPGA_MGR_ZYNQMP_FPGA
to configure the programmable logic(PL) through PS
on ZynqMP SoC.
+config FPGA_MGR_VERSAL_FPGA
+ tristate "Xilinx Versal FPGA"
+ depends on ARCH_ZYNQMP || COMPILE_TEST
+ help
+ Select this option to enable FPGA manager driver support for
+ Xilinx Versal SOC. This driver uses the versal soc firmware
+ interface to load programmable logic(PL) images
+ on versal soc.
endif # FPGA
diff --git a/drivers/fpga/Makefile b/drivers/fpga/Makefile
index d8e21dfc6778..40c9adb6a644 100644
--- a/drivers/fpga/Makefile
+++ b/drivers/fpga/Makefile
@@ -18,6 +18,7 @@ obj-$(CONFIG_FPGA_MGR_TS73XX) += ts73xx-fpga.o
obj-$(CONFIG_FPGA_MGR_XILINX_SPI) += xilinx-spi.o
obj-$(CONFIG_FPGA_MGR_ZYNQ_FPGA) += zynq-fpga.o
obj-$(CONFIG_FPGA_MGR_ZYNQMP_FPGA) += zynqmp-fpga.o
+obj-$(CONFIG_FPGA_MGR_VERSAL_FPGA) += versal-fpga.o
obj-$(CONFIG_ALTERA_PR_IP_CORE) += altera-pr-ip-core.o
obj-$(CONFIG_ALTERA_PR_IP_CORE_PLAT) += altera-pr-ip-core-plat.o
diff --git a/drivers/fpga/versal-fpga.c b/drivers/fpga/versal-fpga.c
new file mode 100644
index 000000000000..22d0bfb7b1e6
--- /dev/null
+++ b/drivers/fpga/versal-fpga.c
@@ -0,0 +1,120 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (C) 2019-2021 Xilinx, Inc.
+ */
+
+#include <linux/dma-mapping.h>
+#include <linux/fpga/fpga-mgr.h>
+#include <linux/io.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/of_address.h>
+#include <linux/string.h>
+#include <linux/firmware/xlnx-zynqmp.h>
+
+/* Constant Definitions */
+#define PDI_SOURCE_TYPE 0xF
+
+/**
+ * struct versal_fpga_priv - Private data structure
+ * @dev: Device data structure
+ */
+struct versal_fpga_priv {
+ struct device *dev;
+};
+
+static int versal_fpga_ops_write_init(struct fpga_manager *mgr,
+ struct fpga_image_info *info,
+ const char *buf, size_t size)
+{
+ return 0;
+}
+
+static int versal_fpga_ops_write(struct fpga_manager *mgr,
+ const char *buf, size_t size)
+{
+ struct versal_fpga_priv *priv;
+ dma_addr_t dma_addr = 0;
+ char *kbuf;
+ int ret;
+
+ priv = mgr->priv;
+
+ kbuf = dma_alloc_coherent(priv->dev, size, &dma_addr, GFP_KERNEL);
+ if (!kbuf)
+ return -ENOMEM;
+
+ memcpy(kbuf, buf, size);
+
+ wmb(); /* ensure all writes are done before initiate FW call */
+
+ ret = zynqmp_pm_load_pdi(PDI_SOURCE_TYPE, dma_addr);
+
+ dma_free_coherent(priv->dev, size, kbuf, dma_addr);
+
+ return ret;
+}
+
+static int versal_fpga_ops_write_complete(struct fpga_manager *mgr,
+ struct fpga_image_info *info)
+{
+ return 0;
+}
+
+static enum fpga_mgr_states versal_fpga_ops_state(struct fpga_manager *mgr)
+{
+ return FPGA_MGR_STATE_UNKNOWN;
+}
+
+static const struct fpga_manager_ops versal_fpga_ops = {
+ .state = versal_fpga_ops_state,
+ .write_init = versal_fpga_ops_write_init,
+ .write = versal_fpga_ops_write,
+ .write_complete = versal_fpga_ops_write_complete,
+};
+
+static int versal_fpga_probe(struct platform_device *pdev)
+{
+ struct device *dev = &pdev->dev;
+ struct versal_fpga_priv *priv;
+ struct fpga_manager *mgr;
+ int ret;
+
+ priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
+ if (!priv)
+ return -ENOMEM;
+
+ priv->dev = dev;
+ ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
+ if (ret < 0) {
+ dev_err(dev, "no usable DMA configuration\n");
+ return ret;
+ }
+
+ mgr = devm_fpga_mgr_create(dev, "Xilinx Versal FPGA Manager",
+ &versal_fpga_ops, priv);
+ if (!mgr)
+ return -ENOMEM;
+
+ return devm_fpga_mgr_register(dev, mgr);
+}
+
+static const struct of_device_id versal_fpga_of_match[] = {
+ { .compatible = "xlnx,versal-fpga", },
+ {},
+};
+MODULE_DEVICE_TABLE(of, versal_fpga_of_match);
+
+static struct platform_driver versal_fpga_driver = {
+ .probe = versal_fpga_probe,
+ .driver = {
+ .name = "versal_fpga_manager",
+ .of_match_table = of_match_ptr(versal_fpga_of_match),
+ },
+};
+module_platform_driver(versal_fpga_driver);
+
+MODULE_AUTHOR("Nava kishore Manne <nava.manne@xilinx.com>");
+MODULE_AUTHOR("Appana Durga Kedareswara rao <appanad.durga.rao@xilinx.com>");
+MODULE_DESCRIPTION("Xilinx Versal FPGA Manager");
+MODULE_LICENSE("GPL");
--
2.18.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver
2021-02-11 6:05 ` [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver Nava kishore Manne
@ 2021-02-11 6:23 ` Randy Dunlap
0 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2021-02-11 6:23 UTC (permalink / raw)
To: Nava kishore Manne, mdf, trix, robh+dt, michal.simek, linux-fpga,
devicetree, linux-arm-kernel, linux-kernel, chinnikishore369
Cc: git, Appana Durga Kedareswara rao
Hi--
On 2/10/21 10:05 PM, Nava kishore Manne wrote:
> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
> index bf85b9a65ec2..dcd2ed5a7956 100644
> --- a/drivers/fpga/Kconfig
> +++ b/drivers/fpga/Kconfig
> @@ -223,4 +223,12 @@ config FPGA_MGR_ZYNQMP_FPGA
> to configure the programmable logic(PL) through PS
> on ZynqMP SoC.
>
> +config FPGA_MGR_VERSAL_FPGA
> + tristate "Xilinx Versal FPGA"
> + depends on ARCH_ZYNQMP || COMPILE_TEST
> + help
> + Select this option to enable FPGA manager driver support for
> + Xilinx Versal SOC. This driver uses the versal soc firmware
How about consistently capitalizing Versal and SOC (above and below)?
> + interface to load programmable logic(PL) images
> + on versal soc.
> endif # FPGA
thanks.
--
~Randy
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager
2021-02-11 6:05 ` [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager Nava kishore Manne
@ 2021-02-11 14:56 ` Rob Herring
2021-02-12 6:13 ` Nava kishore Manne
0 siblings, 1 reply; 8+ messages in thread
From: Rob Herring @ 2021-02-11 14:56 UTC (permalink / raw)
To: Nava kishore Manne
Cc: linux-arm-kernel, mdf, linux-fpga, git,
Appana Durga Kedareswara rao, devicetree, robh+dt,
chinnikishore369, linux-kernel, trix, michal.simek
On Thu, 11 Feb 2021 11:35:31 +0530, Nava kishore Manne wrote:
> From: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
>
> This patch adds binding doc for versal fpga manager driver.
>
> Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
> Signed-off-by: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
> ---
> Changes for v2:
> -Fixed file format and syntax issues.
>
> .../bindings/fpga/xlnx,versal-fpga.yaml | 33 +++++++++++++++++++
> 1 file changed, 33 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
>
My bot found errors running 'make dt_binding_check' on your patch:
yamllint warnings/errors:
./Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml:12:14: [warning] too many spaces after colon (colons)
./Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml:20:9: [warning] wrong indentation: expected 10 but found 8 (indentation)
dtschema/dtc warnings/errors:
See https://patchwork.ozlabs.org/patch/1439305
This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit.
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager
2021-02-11 14:56 ` Rob Herring
@ 2021-02-12 6:13 ` Nava kishore Manne
2021-02-17 21:06 ` Rob Herring
0 siblings, 1 reply; 8+ messages in thread
From: Nava kishore Manne @ 2021-02-12 6:13 UTC (permalink / raw)
To: Rob Herring
Cc: linux-arm-kernel, mdf, linux-fpga, git,
Appana Durga Kedareswara Rao, devicetree, robh+dt,
chinnikishore369, linux-kernel, trix, Michal Simek
Hi Rob,
Please find my response inline.
> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: Thursday, February 11, 2021 8:26 PM
> To: Nava kishore Manne <navam@xilinx.com>
> Cc: linux-arm-kernel@lists.infradead.org; mdf@kernel.org; linux-
> fpga@vger.kernel.org; git <git@xilinx.com>; Appana Durga Kedareswara Rao
> <appanad@xilinx.com>; devicetree@vger.kernel.org; robh+dt@kernel.org;
> chinnikishore369@gmail.com; linux-kernel@vger.kernel.org;
> trix@redhat.com; Michal Simek <michals@xilinx.com>
> Subject: Re: [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga
> manager
>
> On Thu, 11 Feb 2021 11:35:31 +0530, Nava kishore Manne wrote:
> > From: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
> >
> > This patch adds binding doc for versal fpga manager driver.
> >
> > Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
> > Signed-off-by: Appana Durga Kedareswara rao
> > <appana.durga.rao@xilinx.com>
> > ---
> > Changes for v2:
> > -Fixed file format and syntax issues.
> >
> > .../bindings/fpga/xlnx,versal-fpga.yaml | 33 +++++++++++++++++++
> > 1 file changed, 33 insertions(+)
> > create mode 100644
> > Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
> >
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
> ./Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml:12:14:
> [warning] too many spaces after colon (colons)
> ./Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml:20:9:
> [warning] wrong indentation: expected 10 but found 8 (indentation)
>
> dtschema/dtc warnings/errors:
>
> See https://patchwork.ozlabs.org/patch/1439305
>
> This check can fail if there are any dependencies. The base for a patch series
> is generally the most recent rc1.
>
> If you already ran 'make dt_binding_check' and didn't see the above error(s),
> then make sure 'yamllint' is installed and dt-schema is up to
> date:
>
> pip3 install dtschema --upgrade
>
> Please check and re-submit.
Initially, I couldn't see any issue when I run.
After installing yamllint and with upgraded dt-schema, I am able to reproduce the above pointed issues.
Is there any prerequisite(Other than yamllint) I need to follow to run dt-schema?
Regards,
Navakishore.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager
2021-02-12 6:13 ` Nava kishore Manne
@ 2021-02-17 21:06 ` Rob Herring
0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2021-02-17 21:06 UTC (permalink / raw)
To: Nava kishore Manne
Cc: linux-arm-kernel, mdf, linux-fpga, git,
Appana Durga Kedareswara Rao, devicetree, chinnikishore369,
linux-kernel, trix, Michal Simek
On Fri, Feb 12, 2021 at 06:13:33AM +0000, Nava kishore Manne wrote:
> Hi Rob,
>
> Please find my response inline.
>
> > -----Original Message-----
> > From: Rob Herring <robh@kernel.org>
> > Sent: Thursday, February 11, 2021 8:26 PM
> > To: Nava kishore Manne <navam@xilinx.com>
> > Cc: linux-arm-kernel@lists.infradead.org; mdf@kernel.org; linux-
> > fpga@vger.kernel.org; git <git@xilinx.com>; Appana Durga Kedareswara Rao
> > <appanad@xilinx.com>; devicetree@vger.kernel.org; robh+dt@kernel.org;
> > chinnikishore369@gmail.com; linux-kernel@vger.kernel.org;
> > trix@redhat.com; Michal Simek <michals@xilinx.com>
> > Subject: Re: [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga
> > manager
> >
> > On Thu, 11 Feb 2021 11:35:31 +0530, Nava kishore Manne wrote:
> > > From: Appana Durga Kedareswara rao <appana.durga.rao@xilinx.com>
> > >
> > > This patch adds binding doc for versal fpga manager driver.
> > >
> > > Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
> > > Signed-off-by: Appana Durga Kedareswara rao
> > > <appana.durga.rao@xilinx.com>
> > > ---
> > > Changes for v2:
> > > -Fixed file format and syntax issues.
> > >
> > > .../bindings/fpga/xlnx,versal-fpga.yaml | 33 +++++++++++++++++++
> > > 1 file changed, 33 insertions(+)
> > > create mode 100644
> > > Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml
> > >
> >
> > My bot found errors running 'make dt_binding_check' on your patch:
> >
> > yamllint warnings/errors:
> > ./Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml:12:14:
> > [warning] too many spaces after colon (colons)
> > ./Documentation/devicetree/bindings/fpga/xlnx,versal-fpga.yaml:20:9:
> > [warning] wrong indentation: expected 10 but found 8 (indentation)
> >
> > dtschema/dtc warnings/errors:
> >
> > See https://patchwork.ozlabs.org/patch/1439305
> >
> > This check can fail if there are any dependencies. The base for a patch series
> > is generally the most recent rc1.
> >
> > If you already ran 'make dt_binding_check' and didn't see the above error(s),
> > then make sure 'yamllint' is installed and dt-schema is up to
> > date:
> >
> > pip3 install dtschema --upgrade
> >
> > Please check and re-submit.
>
> Initially, I couldn't see any issue when I run.
> After installing yamllint and with upgraded dt-schema, I am able to reproduce the above pointed issues.
> Is there any prerequisite(Other than yamllint) I need to follow to run dt-schema?
No, just keeping dtschema up to date.
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-02-17 21:07 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-11 6:05 [PATCH v2 0/3] Add Bitstream configuration support for Versal Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 1/3] drivers: firmware: Add PDI load API support Nava kishore Manne
2021-02-11 6:05 ` [PATCH v2 2/3] dt-bindings: fpga: Add binding doc for versal fpga manager Nava kishore Manne
2021-02-11 14:56 ` Rob Herring
2021-02-12 6:13 ` Nava kishore Manne
2021-02-17 21:06 ` Rob Herring
2021-02-11 6:05 ` [PATCH v2 3/3] fpga: versal-fpga: Add versal fpga manager driver Nava kishore Manne
2021-02-11 6:23 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).