[PATCHv2 2/7] misc: k3_esm: Add support for Texas Instruments K3 ESM driver

[Tom Rini]

On Fri, Feb 14, 2020 at 11:18:15AM +0200, Tero Kristo wrote:

> The ESM (Error Signaling Module) is used to route error signals within
> the K3 SoCs somewhat similar to interrupts. The handling for these is
> different though, and can be routed for hardware error handling, to
> be handled by safety processor or just as error interrupts handled
> by the main processor. The u-boot level ESM driver is just used to
> configure the ESM signals so that they get routed to proper destination.
> 
> Signed-off-by: Tero Kristo <t-kristo at ti.com>
> ---
>  doc/device-tree-bindings/misc/esm-k3.txt | 25 +++++++
>  drivers/misc/Kconfig                     |  5 ++
>  drivers/misc/Makefile                    |  1 +
>  drivers/misc/k3_esm.c                    | 87 ++++++++++++++++++++++++
>  4 files changed, 118 insertions(+)
>  create mode 100644 doc/device-tree-bindings/misc/esm-k3.txt
>  create mode 100644 drivers/misc/k3_esm.c
> 
> diff --git a/doc/device-tree-bindings/misc/esm-k3.txt b/doc/device-tree-bindings/misc/esm-k3.txt
> new file mode 100644
> index 0000000000..01c8b6b294
> --- /dev/null
> +++ b/doc/device-tree-bindings/misc/esm-k3.txt
> @@ -0,0 +1,25 @@
> +Texas Instruments K3 ESM Binding
> +======================
> +
> +ESM (Error Signaling Module) is an IP block on TI K3 devices that allows
> +handling of safety events somewhat similar to what interrupt controller
> +would do. The safety signals have their separate paths within the SoC,
> +and they are handled by the ESM, which routes them to the proper
> +destination, which can be system reset, interrupt controller, etc. In
> +the simplest configuration the signals are just routed to reset the
> +SoC.
> +
> +Required properties :
> +- compatible	: "ti,j721e-esm"
> +- ti,esm-pins	: integer array of esm events IDs to route to external event
> +		  pin which can be used to reset the SoC. The array can
> +		  have arbitrary amount of event IDs listed on it.
> +
> +Example
> +=======
> +
> +	main_esm: esm at 700000 {
> +		compatible = "ti,j721e-esm";
> +		reg = <0x0 0x700000 0x0 0x1000>;
> +		ti,esm-pins = <344>, <345>;
> +	};
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index f18aa8f7ba..38588b2cbd 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -462,6 +462,11 @@ config IHS_FPGA
>  	  gdsys devices, which supply the majority of the functionality offered
>  	  by the devices. This driver supports both CON and CPU variants of the
>  	  devices, depending on the device tree entry.

missing space.

Otherwise:
Reviewed-by: Tom Rini <trini at konsulko.com>

And if there's no other feedback, Lokesh, please just fix that up when
applying, thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200214/a0511f3a/attachment.sig>