> +/** > + * struct xilinx_fpga_core - interface between the driver and the core manager > + * of Xilinx 7 Series FPGA manager > + * @dev: device node > + * @write: write callback of the driver > + * @prog_b: PROGRAM_B gpio descriptor > + * @init_b: INIT_B gpio descriptor > + * @done: DONE gpio descriptor Please re-check the Documentation again: "Structure fields that are inside a private: area are not listed in the generated output documentation" > + */ > +struct xilinx_fpga_core { > +/* public: */ > + struct device *dev; > + int (*write)(struct xilinx_fpga_core *core, const char *buf, > + size_t count); > +/* private: handled by xilinx-core */ > + struct gpio_desc *prog_b; > + struct gpio_desc *init_b; > + struct gpio_desc *done; > +}; > + [...] > - > static int xilinx_spi_probe(struct spi_device *spi) > { > - struct xilinx_spi_conf *conf; > - struct fpga_manager *mgr; > + struct xilinx_fpga_core *conf; Why do you name it conf? Maybe "core" is better? Thanks, Yilun