Re: [PATCH v4 2/2] event: Add fpga load event

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1374 bytes --]

On Wed, Jul 19, 2023 at 03:45:44PM +0200, christian.taedcke-oss@weidmueller.com wrote:
> From: Christian Taedcke <christian.taedcke@weidmueller.com>
> 
> This enables implementing custom logic after a bitstream was loaded
> into the fpga.
> 
> Signed-off-by: Christian Taedcke <christian.taedcke@weidmueller.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
[snip]
> diff --git a/include/event.h b/include/event.h
> index fe41080fa6..77124c2e73 100644
> --- a/include/event.h
> +++ b/include/event.h
> @@ -31,6 +31,9 @@ enum event_t {
>  	/* Init hooks */
>  	EVT_MISC_INIT_F,
>  
> +	/* Fpga load hook */
> +	EVT_FPGA_LOAD,
> +
>  	/* Device tree fixups before booting */
>  	EVT_FT_FIXUP,
>  
> @@ -59,6 +62,19 @@ union event_data {
>  		struct udevice *dev;
>  	} dm;
>  
> +	/**
> +	 * struct event_fpga_load - fpga load event
> +	 *
> +	 * @buf: The buffer that was loaded into the fpga
> +	 * @bsize: The size of the buffer that was loaded into the fpga
> +	 * @result: Result of the load operation
> +	 */
> +	struct event_fpga_load {
> +		const void *buf;
> +		size_t bsize;
> +		int result;
> +	} fpga_load;
> +
>  	/**
>  	 * struct event_ft_fixup - FDT fixup before booting
>  	 *

Since here is where you're introducing a size_t to event.h, here is
where you also need to add <linux/types.h> to event.h

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]