Re: [PATCH v9 2/3] dt-bindings: edac: nuvoton: add NPCM memory controller

Paul Menzel
2022-05-10T06:14:38Z

Dear Medad,

Thank you for your patch.

Am 10.05.22 um 05:10 schrieb Medad CChien:
> Document devicetree bindings for the Nuvoton BMC NPCM memory controller.
>
> Signed-off-by: Medad CChien <ctcchien@nuvoton.com>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> .../edac/nuvoton,npcm-memory-controller.yaml | 61 +++++++++++++++++++
> 1 file changed, 61 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/edac/nuvoton,npcm-memory-controller.yaml
>
> diff --git a/Documentation/devicetree/bindings/edac/nuvoton,npcm-memory-controller.yaml b/Documentation/devicetree/bindings/edac/nuvoton,npcm-memory-controller.yaml
> new file mode 100644
> index 000000000000..6f37211796a3
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/edac/nuvoton,npcm-memory-controller.yaml
> @@ -0,0 +1,61 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/edac/nuvoton,npcm-memory-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Nuvoton NPCM Memory Controller
> +
> +maintainers:
> + - Medad CChien <ctcchien@nuvoton.com>

Just a side note, that in my experience functional like
<linux-npcm-memory-controller@nuvoton.com> instead of personal addresses
are useful, as you can configure on your side, who to deliver messages
to. For example, if you are on sick leave or vacation, you just
configure to deliver the message to a colleague of yours (or they get
messages in the first place anyway).

Maybe you can bring that up at Nuvoton.

[…]

Kind regards,

Paul