linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH -next] staging/android/ion: remove useless document file
@ 2017-05-06  9:49 Yisheng Xie
  2017-05-08 14:32 ` Laura Abbott
  0 siblings, 1 reply; 2+ messages in thread
From: Yisheng Xie @ 2017-05-06  9:49 UTC (permalink / raw)
  To: labbott, sumit.semwal, gregkh, arve, riandrews, wporter82
  Cc: devel, linux-kernel

From: Yisheng Xie <xieyisheng1@huawei.com>

After commit 9828282e33a0 ("staging: android: ion: Remove old platform
support"), the document about devicetree of ion is no need anymore, so
just remove it.

Signed-off-by: Yisheng Xie <xieyisheng1@huawei.com>
---
 .../devicetree/bindings/staging/ion/hi6220-ion.txt | 31 -------------
 MAINTAINERS                                        |  1 -
 drivers/staging/android/ion/devicetree.txt         | 51 ----------------------
 3 files changed, 83 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt
 delete mode 100644 drivers/staging/android/ion/devicetree.txt

diff --git a/Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt b/Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt
deleted file mode 100644
index c59e27c..0000000
--- a/Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt
+++ /dev/null
@@ -1,31 +0,0 @@
-Hi6220 SoC ION
-===================================================================
-Required properties:
-- compatible : "hisilicon,hi6220-ion"
-- list of the ION heaps
-	- heap name : maybe heap_sys_user@0
-	- heap id   : id should be unique in the system.
-	- heap base : base ddr address of the heap,0 means that
-	it is dynamic.
-	- heap size : memory size and 0 means it is dynamic.
-	- heap type : the heap type of the heap, please also
-	see the define in ion.h(drivers/staging/android/uapi/ion.h)
--------------------------------------------------------------------
-Example:
-	hi6220-ion {
-		compatible = "hisilicon,hi6220-ion";
-		heap_sys_user@0 {
-			heap-name = "sys_user";
-			heap-id   = <0x0>;
-			heap-base = <0x0>;
-			heap-size = <0x0>;
-			heap-type = "ion_system";
-		};
-		heap_sys_contig@0 {
-			heap-name = "sys_contig";
-			heap-id   = <0x1>;
-			heap-base = <0x0>;
-			heap-size = <0x0>;
-			heap-type = "ion_system_contig";
-		};
-	};
diff --git a/MAINTAINERS b/MAINTAINERS
index 58590cf..9ec008c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -846,7 +846,6 @@ M:	Laura Abbott <labbott@redhat.com>
 M:	Sumit Semwal <sumit.semwal@linaro.org>
 L:	devel@driverdev.osuosl.org
 S:	Supported
-F:	Documentation/devicetree/bindings/staging/ion/
 F:	drivers/staging/android/ion
 F:	drivers/staging/android/uapi/ion.h
 F:	drivers/staging/android/uapi/ion_test.h
diff --git a/drivers/staging/android/ion/devicetree.txt b/drivers/staging/android/ion/devicetree.txt
deleted file mode 100644
index 16871527..0000000
--- a/drivers/staging/android/ion/devicetree.txt
+++ /dev/null
@@ -1,51 +0,0 @@
-Ion Memory Manager
-
-Ion is a memory manager that allows for sharing of buffers via dma-buf.
-Ion allows for different types of allocation via an abstraction called
-a 'heap'. A heap represents a specific type of memory. Each heap has
-a different type. There can be multiple instances of the same heap
-type.
-
-Specific heap instances are tied to heap IDs. Heap IDs are not to be specified
-in the devicetree.
-
-Required properties for Ion
-
-- compatible: "linux,ion" PLUS a compatible property for the device
-
-All child nodes of a linux,ion node are interpreted as heaps
-
-required properties for heaps
-
-- compatible: compatible string for a heap type PLUS a compatible property
-for the specific instance of the heap. Current heap types
--- linux,ion-heap-system
--- linux,ion-heap-system-contig
--- linux,ion-heap-carveout
--- linux,ion-heap-chunk
--- linux,ion-heap-dma
--- linux,ion-heap-custom
-
-Optional properties
-- memory-region: A phandle to a memory region. Required for DMA heap type
-(see reserved-memory.txt for details on the reservation)
-
-Example:
-
-	ion {
-		compatbile = "hisilicon,ion", "linux,ion";
-
-		ion-system-heap {
-			compatbile = "hisilicon,system-heap", "linux,ion-heap-system"
-		};
-
-		ion-camera-region {
-			compatible = "hisilicon,camera-heap", "linux,ion-heap-dma"
-			memory-region = <&camera_region>;
-		};
-
-		ion-fb-region {
-			compatbile = "hisilicon,fb-heap", "linux,ion-heap-dma"
-			memory-region = <&fb_region>;
-		};
-	}
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH -next] staging/android/ion: remove useless document file
  2017-05-06  9:49 [PATCH -next] staging/android/ion: remove useless document file Yisheng Xie
@ 2017-05-08 14:32 ` Laura Abbott
  0 siblings, 0 replies; 2+ messages in thread
From: Laura Abbott @ 2017-05-08 14:32 UTC (permalink / raw)
  To: Yisheng Xie, sumit.semwal, gregkh, arve, riandrews, wporter82
  Cc: devel, linux-kernel

On 05/06/2017 02:49 AM, Yisheng Xie wrote:
> From: Yisheng Xie <xieyisheng1@huawei.com>
> 
> After commit 9828282e33a0 ("staging: android: ion: Remove old platform
> support"), the document about devicetree of ion is no need anymore, so
> just remove it.
> 

Acked-by: Laura Abbott <labbott@redhat.com>

> Signed-off-by: Yisheng Xie <xieyisheng1@huawei.com>
> ---
>  .../devicetree/bindings/staging/ion/hi6220-ion.txt | 31 -------------
>  MAINTAINERS                                        |  1 -
>  drivers/staging/android/ion/devicetree.txt         | 51 ----------------------
>  3 files changed, 83 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt
>  delete mode 100644 drivers/staging/android/ion/devicetree.txt
> 
> diff --git a/Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt b/Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt
> deleted file mode 100644
> index c59e27c..0000000
> --- a/Documentation/devicetree/bindings/staging/ion/hi6220-ion.txt
> +++ /dev/null
> @@ -1,31 +0,0 @@
> -Hi6220 SoC ION
> -===================================================================
> -Required properties:
> -- compatible : "hisilicon,hi6220-ion"
> -- list of the ION heaps
> -	- heap name : maybe heap_sys_user@0
> -	- heap id   : id should be unique in the system.
> -	- heap base : base ddr address of the heap,0 means that
> -	it is dynamic.
> -	- heap size : memory size and 0 means it is dynamic.
> -	- heap type : the heap type of the heap, please also
> -	see the define in ion.h(drivers/staging/android/uapi/ion.h)
> --------------------------------------------------------------------
> -Example:
> -	hi6220-ion {
> -		compatible = "hisilicon,hi6220-ion";
> -		heap_sys_user@0 {
> -			heap-name = "sys_user";
> -			heap-id   = <0x0>;
> -			heap-base = <0x0>;
> -			heap-size = <0x0>;
> -			heap-type = "ion_system";
> -		};
> -		heap_sys_contig@0 {
> -			heap-name = "sys_contig";
> -			heap-id   = <0x1>;
> -			heap-base = <0x0>;
> -			heap-size = <0x0>;
> -			heap-type = "ion_system_contig";
> -		};
> -	};
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 58590cf..9ec008c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -846,7 +846,6 @@ M:	Laura Abbott <labbott@redhat.com>
>  M:	Sumit Semwal <sumit.semwal@linaro.org>
>  L:	devel@driverdev.osuosl.org
>  S:	Supported
> -F:	Documentation/devicetree/bindings/staging/ion/
>  F:	drivers/staging/android/ion
>  F:	drivers/staging/android/uapi/ion.h
>  F:	drivers/staging/android/uapi/ion_test.h
> diff --git a/drivers/staging/android/ion/devicetree.txt b/drivers/staging/android/ion/devicetree.txt
> deleted file mode 100644
> index 16871527..0000000
> --- a/drivers/staging/android/ion/devicetree.txt
> +++ /dev/null
> @@ -1,51 +0,0 @@
> -Ion Memory Manager
> -
> -Ion is a memory manager that allows for sharing of buffers via dma-buf.
> -Ion allows for different types of allocation via an abstraction called
> -a 'heap'. A heap represents a specific type of memory. Each heap has
> -a different type. There can be multiple instances of the same heap
> -type.
> -
> -Specific heap instances are tied to heap IDs. Heap IDs are not to be specified
> -in the devicetree.
> -
> -Required properties for Ion
> -
> -- compatible: "linux,ion" PLUS a compatible property for the device
> -
> -All child nodes of a linux,ion node are interpreted as heaps
> -
> -required properties for heaps
> -
> -- compatible: compatible string for a heap type PLUS a compatible property
> -for the specific instance of the heap. Current heap types
> --- linux,ion-heap-system
> --- linux,ion-heap-system-contig
> --- linux,ion-heap-carveout
> --- linux,ion-heap-chunk
> --- linux,ion-heap-dma
> --- linux,ion-heap-custom
> -
> -Optional properties
> -- memory-region: A phandle to a memory region. Required for DMA heap type
> -(see reserved-memory.txt for details on the reservation)
> -
> -Example:
> -
> -	ion {
> -		compatbile = "hisilicon,ion", "linux,ion";
> -
> -		ion-system-heap {
> -			compatbile = "hisilicon,system-heap", "linux,ion-heap-system"
> -		};
> -
> -		ion-camera-region {
> -			compatible = "hisilicon,camera-heap", "linux,ion-heap-dma"
> -			memory-region = <&camera_region>;
> -		};
> -
> -		ion-fb-region {
> -			compatbile = "hisilicon,fb-heap", "linux,ion-heap-dma"
> -			memory-region = <&fb_region>;
> -		};
> -	}
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-05-08 14:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-06  9:49 [PATCH -next] staging/android/ion: remove useless document file Yisheng Xie
2017-05-08 14:32 ` Laura Abbott

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).