diff mbox

dt-bindings: media: xilinx: fix typo in example

Message ID 1507824214-17744-1-git-send-email-akinobu.mita@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Akinobu Mita Oct. 12, 2017, 4:03 p.m. UTC
Fix typo s/:/;/

Cc: Rob Herring <robh+dt@kernel.org>
Cc: Hyun Kwon <hyun.kwon@xilinx.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
---
 Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Rob Herring (Arm) Oct. 17, 2017, 8:52 p.m. UTC | #1
On Fri, Oct 13, 2017 at 01:03:34AM +0900, Akinobu Mita wrote:
> Fix typo s/:/;/
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Hyun Kwon <hyun.kwon@xilinx.com>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
> ---
>  Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>
Laurent Pinchart Oct. 18, 2017, 4:08 p.m. UTC | #2
Hi Akinobu,

Thank you for the patch.

On Thursday, 12 October 2017 19:03:34 EEST Akinobu Mita wrote:
> Fix typo s/:/;/
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Hyun Kwon <hyun.kwon@xilinx.com>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>

Good catch.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Should I take this patch in my tree ?

> ---
>  Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt
> b/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt index
> 9dd86b3..439351a 100644
> --- a/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt
> +++ b/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt
> @@ -66,6 +66,6 @@ Example:
>  				tpg1_out: endpoint {
>  					remote-endpoint = <&switch_in0>;
>  				};
> -			}:
> +			};
>  		};
>  	};
Akinobu Mita Oct. 19, 2017, 2:22 p.m. UTC | #3
2017-10-19 1:08 GMT+09:00 Laurent Pinchart <laurent.pinchart@ideasonboard.com>:
> Hi Akinobu,
>
> Thank you for the patch.
>
> On Thursday, 12 October 2017 19:03:34 EEST Akinobu Mita wrote:
>> Fix typo s/:/;/
>>
>> Cc: Rob Herring <robh+dt@kernel.org>
>> Cc: Hyun Kwon <hyun.kwon@xilinx.com>
>> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>> Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
>
> Good catch.
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> Should I take this patch in my tree ?

Yes, please do.
Could you also take a look at another patch that I sent at the same time
"media: xilinx-video: fix bad of_node_put() on endpoint error" ?
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt b/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt
index 9dd86b3..439351a 100644
--- a/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt
+++ b/Documentation/devicetree/bindings/media/xilinx/xlnx,v-tpg.txt
@@ -66,6 +66,6 @@  Example:
 				tpg1_out: endpoint {
 					remote-endpoint = <&switch_in0>;
 				};
-			}:
+			};
 		};
 	};