Re: [PATCH v9 03/13] fpga: xilinx: add bitstream flags to driver desc

[Michal Simek]

On 6/1/22 10:46, Oleksandr Suvorov wrote:

Store a set of supported bitstream types in xilinx_desc structure.
It will be used to determine whether an FPGA image is able to be
loaded with a given driver.

Signed-off-by: Oleksandr Suvorov <oleksandr.suvo...@foundries.io>
---

(no changes since v1)

  include/versalpl.h | 2 +-
  include/xilinx.h   | 5 +++++
  include/zynqmppl.h | 2 +-
  3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/include/versalpl.h b/include/versalpl.h
index b94c82e6e66..26f04a2f649 100644
--- a/include/versalpl.h
+++ b/include/versalpl.h
@@ -15,6 +15,6 @@
  extern struct xilinx_fpga_op versal_op;

#define XILINX_VERSAL_DESC \

-{ xilinx_versal, csu_dma, 1, &versal_op, 0, &versal_op }
+{ xilinx_versal, csu_dma, 1, &versal_op, 0, &versal_op, NULL, FPGA_LEGACY }

#endif /* _VERSALPL_H_ */

diff --git a/include/xilinx.h b/include/xilinx.h
index 362943bc717..91179abe31f 100644
--- a/include/xilinx.h
+++ b/include/xilinx.h
@@ -37,6 +37,9 @@ typedef enum {                        /* typedef 
xilinx_family */
        max_xilinx_type         /* insert all new types before this */
  } xilinx_family;              /* end, typedef xilinx_family */

+/* FPGA bitstream supported types */

+#define FPGA_LEGACY                    BIT(0)
+
  typedef struct {              /* typedef xilinx_desc */
        xilinx_family family;   /* part type */
        xilinx_iface iface;     /* interface type */
@@ -45,6 +48,7 @@ typedef struct {              /* typedef xilinx_desc */
        int cookie;             /* implementation specific cookie */
        struct xilinx_fpga_op *operations; /* operations */
        char *name;             /* device name in bitstream */
+       int flags;              /* compatible flags */
  } xilinx_desc;                        /* end, typedef xilinx_desc */

struct xilinx_fpga_op {

@@ -56,6 +60,7 @@ struct xilinx_fpga_op {
                     struct fpga_secure_info *fpga_sec_info);
        int (*dump)(xilinx_desc *desc, const void *buf, size_t bsize);
        int (*info)(xilinx_desc *desc);
+       int (*str2flag)(xilinx_desc *desc, const char *string);

This should be in 4/13.

M