Re: [PATCH 3/5] media: dt-bindings: aspeed-video: Add missing memory-region property

[Jae Hyun Yoo]

Hi Eddie,

On 4/2/2019 11:25 AM, Eddie James wrote:
> Missed documenting this property in the initial commit.
>
> Signed-off-by: Eddie James <eajames@xxxxxxxxxxxxx>
> ---
>   Documentation/devicetree/bindings/media/aspeed-video.txt | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/aspeed-video.txt b/Documentation/devicetree/bindings/media/aspeed-video.txt
> index 78b464a..346c2d3 100644
> --- a/Documentation/devicetree/bindings/media/aspeed-video.txt
> +++ b/Documentation/devicetree/bindings/media/aspeed-video.txt
> @@ -14,6 +14,11 @@ Required properties:
>   			the VE
>    - interrupts:		the interrupt associated with the VE on this platform
>   
> +Optional properties:
> + - memory-region:
> +	phandle to a memory region to allocate from, as defined in
> +	Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> +
>   Example:
>   
>   video-engine@1e700000 {
> @@ -23,4 +28,5 @@ video-engine@1e700000 {
>       clock-names = "vclk", "eclk";
>       resets = <&syscon ASPEED_RESET_VIDEO>;
>       interrupts = <7>;
> +    memory-region = <&video_engine_memory>

nit: a semicolon at the end of the line.

>   };