Re: [PATCH v2 02/10] dt-binding: gce: add binding for gce event property
From: CK Hu <hidden>
Date: 2019-03-08 08:47:55
Also in:
linux-devicetree, linux-mediatek, lkml
Hi, Bibby: On Wed, 2019-03-06 at 17:50 +0800, Bibby Hsieh wrote:
cmdq driver provide a function that get event number from device node for client.
I think device tree is based on the view of hardware design, so you need not to mention how the driver design. Client hardware would send event to GCE hardware, so this property present this event.
quoted hunk ↗ jump to hunk
add specification for #event-cells, mediatek,gce-event-names, mediatek,gce-events. Signed-off-by: Bibby Hsieh <redacted> --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-)diff --git a/Documentation/devicetree/bindings/mailbox/mtk-gce.txt b/Documentation/devicetree/bindings/mailbox/mtk-gce.txt index 9c0d982..812698f 100644 --- a/Documentation/devicetree/bindings/mailbox/mtk-gce.txt +++ b/Documentation/devicetree/bindings/mailbox/mtk-gce.txt@@ -21,12 +21,20 @@ Required properties: priority: Priority of GCE thread. atomic_exec: GCE processing continuous packets of commands in atomic way. +- #event-cells: Should be 1. + <&phandle event_number> + phandle: Label name of a gce node. + event_number: the event number defined in 'dt-bindings/gce/mt8173-gce.h' + or 'dt-binding/gce/mt8183-gce.h'. Required properties for a client device: - mboxes: Client use mailbox to communicate with GCE, it should have this property and list of phandle, mailbox specifiers. - mediatek,gce-subsys: u32, specify the sub-system id which is corresponding to the register address. +- mediatek,gce-event-names: the event name can be defined by user. +- mediatek,gce-events: u32, the event number defined in + 'dt-bindings/gce/mt8173-gce.h' or 'dt-binding/gce/mt8183-gce.h'.
I think both gce-subsys and gce-event are optional and depend on the hardware design. Regards, CK
quoted hunk ↗ jump to hunk
Some vaules of properties are defined in 'dt-bindings/gce/mt8173-gce.h' or 'dt-binding/gce/mt8183-gce.h'. Such as sub-system ids, thread priority, event ids.@@ -41,6 +49,7 @@ Example: clock-names = "gce"; thread-num = CMDQ_THR_MAX_COUNT; #mbox-cells = <3>; + #event-cells = <1>; }; Example for a client device:@@ -50,8 +59,9 @@ Example for a client device: mboxes = <&gce 0 CMDQ_THR_PRIO_LOWEST 1>, <&gce 1 CMDQ_THR_PRIO_LOWEST 1>; mediatek,gce-subsys = <SUBSYS_1400XXXX>; - mutex-event-eof = <CMDQ_EVENT_MUTEX0_STREAM_EOF - CMDQ_EVENT_MUTEX1_STREAM_EOF>; - + mediatek,gce-event-names = "rdma0_sof", + "rsz0_sof"; + mediatek,gce-events = <&gce CMDQ_EVENT_MDP_RDMA0_SOF>, + <&gce CMDQ_EVENT_MDP_RSZ0_SOF>; ... };
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel