[PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Marc Kleine-Budde]

This patch fixes the indention of the table in the description of the
bosch,mram-cfg property.

Cc: Sriram Dash <sriram.dash@samsung.com>
Cc: Benjamin Gaignard <benjamin.gaignard@st.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
---
 Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
index 61f9e512c346..3bc1ed39a0a5 100644
--- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
+++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
@@ -69,8 +69,8 @@ properties:
       M_CAN includes the following elements according to user manual:
       11-bit Filter	0-128 elements / 0-128 words
       29-bit Filter	0-64 elements / 0-128 words
-      Rx FIFO 0	0-64 elements / 0-1152 words
-      Rx FIFO 1	0-64 elements / 0-1152 words
+      Rx FIFO 0		0-64 elements / 0-1152 words
+      Rx FIFO 1		0-64 elements / 0-1152 words
       Rx Buffers	0-64 elements / 0-1152 words
       Tx Event FIFO	0-32 elements / 0-64 words
       Tx Buffers	0-32 elements / 0-576 words
-- 
2.34.1

Re: [PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Marc Kleine-Budde]

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

Hello Chandrasekar Ramakrishnan,

I just noticed that the binding maintainer's email address bounces.
Chandrasekar Ramakrishnan can I add you as the maintainer of the DT
binding?

Marc

On 17.02.2022 11:11:11, Marc Kleine-Budde wrote:
> This patch fixes the indention of the table in the description of the
> bosch,mram-cfg property.
> 
> Cc: Sriram Dash <sriram.dash@samsung.com>
> Cc: Benjamin Gaignard <benjamin.gaignard@st.com>
> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
> ---
>  Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> index 61f9e512c346..3bc1ed39a0a5 100644
> --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> @@ -69,8 +69,8 @@ properties:
>        M_CAN includes the following elements according to user manual:
>        11-bit Filter	0-128 elements / 0-128 words
>        29-bit Filter	0-64 elements / 0-128 words
> -      Rx FIFO 0	0-64 elements / 0-1152 words
> -      Rx FIFO 1	0-64 elements / 0-1152 words
> +      Rx FIFO 0		0-64 elements / 0-1152 words
> +      Rx FIFO 1		0-64 elements / 0-1152 words
>        Rx Buffers	0-64 elements / 0-1152 words
>        Tx Event FIFO	0-32 elements / 0-64 words
>        Tx Buffers	0-32 elements / 0-576 words
> -- 
> 2.34.1
> 
> 
> 

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |

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

Re: [PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Marc Kleine-Budde]

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

On 17.02.2022 16:47:35, CHANDRASEKAR RAMAKRISHNAN wrote:
> Sure, Please add me.
> 
> I will look in to it.

The file "Documentation/devicetree/bindings/net/can/bosch,m_can.yaml"
lists Sriram Dash (email bounces), while you are already listed in the
MAINTAINERS file.

I'll send an update to the DT bindings file.

Thanks,
Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |

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

Re: [PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Marc Kleine-Budde]

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

On 17.02.2022 12:36:05, Marc Kleine-Budde wrote:
> On 17.02.2022 16:47:35, CHANDRASEKAR RAMAKRISHNAN wrote:
> > Sure, Please add me.
> > 
> > I will look in to it.
> 
> The file "Documentation/devicetree/bindings/net/can/bosch,m_can.yaml"
> lists Sriram Dash (email bounces), while you are already listed in the
> MAINTAINERS file.
> 
> I'll send an update to the DT bindings file.

-> https://lore.kernel.org/all/20220217113839.2311417-1-mkl@pengutronix.de/

Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |

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

RE: [PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Chandrasekar R]

> -----Original Message-----
> From: Marc Kleine-Budde <mkl@pengutronix.de>
> Sent: 17 February 2022 15:41
> To: linux-can@vger.kernel.org
> Cc: Rob Herring <robh+dt@kernel.org>; devicetree@vger.kernel.org; Marc
> Kleine-Budde <mkl@pengutronix.de>; Sriram Dash
> <sriram.dash@samsung.com>; Benjamin Gaignard
> <benjamin.gaignard@st.com>
> Subject: [PATCH net-next] dt-binding: can: m_can: fix indention of table
in
> bosch,mram-cfg description
> 
> This patch fixes the indention of the table in the description of the
> bosch,mram-cfg property.
> 
> Cc: Sriram Dash <sriram.dash@samsung.com>
This id does not exists anymore.

> Cc: Benjamin Gaignard <benjamin.gaignard@st.com>
> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
> ---
Changes looks fine, 
Reviewed-by: Chandrasekar Ramakrishnan <rcsekar@samsung.com>

>  Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> index 61f9e512c346..3bc1ed39a0a5 100644
> --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> @@ -69,8 +69,8 @@ properties:
>        M_CAN includes the following elements according to user manual:
>        11-bit Filter	0-128 elements / 0-128 words
>        29-bit Filter	0-64 elements / 0-128 words
> -      Rx FIFO 0	0-64 elements / 0-1152 words
> -      Rx FIFO 1	0-64 elements / 0-1152 words
> +      Rx FIFO 0		0-64 elements / 0-1152 words
> +      Rx FIFO 1		0-64 elements / 0-1152 words
>        Rx Buffers	0-64 elements / 0-1152 words
>        Tx Event FIFO	0-32 elements / 0-64 words
>        Tx Buffers	0-32 elements / 0-576 words
> --
> 2.34.1
> 
> 

Re: [PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Marc Kleine-Budde]

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

On 17.02.2022 21:28:35, Chandrasekar R wrote:
> > Subject: [PATCH net-next] dt-binding: can: m_can: fix indention of table
> in
> > bosch,mram-cfg description
> > 
> > This patch fixes the indention of the table in the description of the
> > bosch,mram-cfg property.
> > 
> > Cc: Sriram Dash <sriram.dash@samsung.com>
> This id does not exists anymore.

Thanks, already removed.

> > Cc: Benjamin Gaignard <benjamin.gaignard@st.com>
> > Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
> > ---
> Changes looks fine, 
> Reviewed-by: Chandrasekar Ramakrishnan <rcsekar@samsung.com>

Added to the patch.

regards,
Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |

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

Re: [PATCH net-next] dt-binding: can: m_can: fix indention of table in bosch,mram-cfg description

[Rob Herring]

On Thu, 17 Feb 2022 11:11:11 +0100, Marc Kleine-Budde wrote:
> This patch fixes the indention of the table in the description of the
> bosch,mram-cfg property.
> 
> Cc: Sriram Dash <sriram.dash@samsung.com>
> Cc: Benjamin Gaignard <benjamin.gaignard@st.com>
> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
> ---
>  Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>