Re: [PATCH v3 9/9] drm/mediatek: reduce clear event

2020-07-08 Thread Bibby Hsieh
ent always clear before wait. > This fix depend on patch: > "soc: mediatek: cmdq: add clear option in cmdq_pkt_wfe api" > > Fixes: 2f965be7f9008 ("drm/mediatek: apply CMDQ control flow") > Signed-off-by: Dennis YC Hsieh Reviewed-by: Bibby Hsieh > --- &

Re: [PATCH v3 1/9] soc: mediatek: cmdq: add address shift in jump

2020-07-08 Thread Bibby Hsieh
Reviewed-by: Bibby Hsieh On Tue, 2020-07-07 at 23:45 +0800, Dennis YC Hsieh wrote: > Add address shift when compose jump instruction > to compatible with 35bit format. > > Change since v1: > - Rename cmdq_mbox_shift() to cmdq_get_shift_pa(). > > Signed-of

Re: [PATCH v1 01/11] soc: mediatek: cmdq: add address shift in jump

2020-06-21 Thread Bibby Hsieh
Reviewed-by: Bibby Hsieh Thanks. On Sun, 2020-06-21 at 22:18 +0800, Dennis YC Hsieh wrote: > Add address shift when compose jump instruction > to compatible with 35bit format. > > Signed-off-by: Dennis YC Hsieh > --- > drivers/soc/mediatek/mtk-cmdq-helper.c |3 ++- &g

Re: [PATCH v1 0/11] support cmdq helper function on mt6779 platform

2020-06-21 Thread Bibby Hsieh
Hi, Dennis, Please add "depends on patch: support gce on mt6779 platform" in cover letter. Thanks Bibby On Sun, 2020-06-21 at 22:18 +0800, Dennis YC Hsieh wrote: > This patch support cmdq helper function on mt6779 platform, > based on "support gce on mt6779 platform" patchset. > > > Dennis YC

Re: [PATCH v7 4/4] mailbox: mediatek: cmdq: clear task in channel before shutdown

2020-06-21 Thread Bibby Hsieh
Reviewed-by: Bibby Hsieh Thanks. On Sun, 2020-06-21 at 21:22 +0800, Dennis YC Hsieh wrote: > Do success callback in channel when shutdown. For those task not finish, > callback with error code thus client has chance to cleanup or reset. > > Signed-off-by: Dennis YC Hsieh > Rev

Re: [PATCH v7 3/4] mailbox: cmdq: support mt6779 gce platform definition

2020-06-21 Thread Bibby Hsieh
Reviewed-by: Bibby Hsieh Thanks. On Sun, 2020-06-21 at 21:22 +0800, Dennis YC Hsieh wrote: > Add gce v4 hardware support with different thread number and shift. > > Signed-off-by: Dennis YC Hsieh > Reviewed-by: CK Hu > Reviewed-by: Matthias Brugger > --- > dri

Re: [PATCH v7 1/4] dt-binding: gce: add gce header file for mt6779

2020-06-21 Thread Bibby Hsieh
Reviewed-by: Bibby Hsieh Thanks. On Sun, 2020-06-21 at 21:22 +0800, Dennis YC Hsieh wrote: > Add documentation for the mt6779 gce. > > Add gce header file defined the gce hardware event, > subsys number and constant for mt6779. > > Signed-off-by: Dennis YC Hsieh > Rev

Re: [PATCH v7 2/4] mailbox: cmdq: variablize address shift in platform

2020-06-21 Thread Bibby Hsieh
On Sun, 2020-06-21 at 21:22 +0800, Dennis YC Hsieh wrote: > Some gce hardware shift pc and end address in register to support > large dram addressing. > Implement gce address shift when write or read pc and end register. > And add shift bit in platform definition. > > Signed-off-by: Dennis YC

Re: [PATCH 1/3] arm64: dts: mt8183: Add gce setting in display node

2020-05-21 Thread Bibby Hsieh
On Thu, 2020-05-21 at 12:10 +0200, Matthias Brugger wrote: > On 14/02/2020 11:06, Matthias Brugger wrote: > > > > > > On 14/02/2020 05:49, Bibby Hsieh wrote: > >> In order to use GCE function, we need add some information > >> into display node (mboxes

[PATCH v15 1/4] soc: mediatek: cmdq: define the instruction struct

2019-09-27 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Houlong Wei --- drivers/soc/mediatek/mtk-cmdq-helper.c | 106 +-- include/linux

[PATCH v15 4/4] arm64: dts: add gce node for mt8183

2019-09-27 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 66aaa07f6cec..52b9af38a00a 100644

[PATCH v15 3/4] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-09-27 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Houlong Wei --- drivers/soc/mediatek/mtk-cmdq-helper.c | 29 ++ include/linux/soc/mediatek/mtk

[PATCH v15 0/4] support gce on mt8183 platform

2019-09-27 Thread Bibby Hsieh
snip ...] Bibby Hsieh (4): soc: mediatek: cmdq: define the instruction struct soc: mediatek: cmdq: add polling function soc: mediatek: cmdq: add cmdq_dev_get_client_reg function arm64: dts: add gce node for mt8183 arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ drivers/soc/mediatek/mtk-

[PATCH v15 2/4] soc: mediatek: cmdq: add polling function

2019-09-27 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Houlong Wei --- drivers/soc/mediatek/mtk-cmdq-helper.c | 41 include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 32

[PATCH 0/2] Support CMDQ interface and control flow

2019-08-30 Thread Bibby Hsieh
://patchwork.kernel.org/cover/11123119/) add drm support for MT8183 (https://patchwork.kernel.org/cover/11121519/) support gce on mt8183 platform (https://patchwork.kernel.org/cover/11120153/) Bibby Hsieh (2): drm/mediatek: Support CMDQ interface in ddp component drm/mediatek: Apply CMDQ control

[PATCH v14 08/10] soc: mediatek: cmdq: add polling function

2019-08-28 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Houlong Wei --- drivers/soc/mediatek/mtk-cmdq-helper.c | 30 ++ include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 32

[PATCH v14 03/10] dt-binding: gce: add binding for gce client reg property

2019-08-28 Thread Bibby Hsieh
cmdq driver provide a function that get the relationship of sub system number from device node for client. add specification for #subsys-cells, mediatek,gce-client-reg. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 16

[PATCH v14 02/10] dt-binding: gce: add gce header file for mt8183

2019-08-28 Thread Bibby Hsieh
Add documentation for the mt8183 gce. Add gce header file defined the gce hardware event, subsys number and constant for mt8183. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 6 +- include/dt-bindings/gce/mt8183-gce.h | 175

[PATCH v14 07/10] soc: mediatek: cmdq: define the instruction struct

2019-08-28 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Houlong Wei --- drivers/soc/mediatek/mtk-cmdq-helper.c | 77 include/linux

[PATCH v14 05/10] mailbox: mediatek: cmdq: support mt8183 gce function

2019-08-28 Thread Bibby Hsieh
add mt8183 compatible name for supporting gce function Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c index 8fddd26288e8

[PATCH v14 09/10] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-08-28 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Houlong Wei --- drivers/soc/mediatek/mtk-cmdq-helper.c | 29 ++ include/linux/soc/mediatek/mtk

[PATCH v14 06/10] mailbox: mediatek: cmdq: clear the event in cmdq initial flow

2019-08-28 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Matthias Brugger --- drivers/mailbox/mtk-cmdq-mailbox.c

[PATCH v14 10/10] arm64: dts: add gce node for mt8183

2019-08-28 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 66aaa07f6cec..52b9af38a00a 100644

[PATCH v14 01/10] dt-binding: gce: remove thread-num property

2019-08-28 Thread Bibby Hsieh
"thread-num" is an unused property so we remove it from example. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mailbox/mtk-

[PATCH v14 06/10] soc: mediatek: cmdq: clear the event in cmdq initial flow

2019-08-28 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 5 + include/linux/m

[PATCH v14 00/10] support gce on mt8183 platform

2019-08-28 Thread Bibby Hsieh
tek: cmdq: add cmdq_dev_get_client_reg function Bibby Hsieh (10): dt-binding: gce: remove thread-num property dt-binding: gce: add gce header file for mt8183 dt-binding: gce: add binding for gce client reg property mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data mailbox:

[PATCH v14 04/10] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-08-28 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu Reviewed-by: Matthias Brugger --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file

Re: [RESEND, PATCH v13 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-08-28 Thread Bibby Hsieh
On Tue, 2019-08-27 at 12:13 +0200, Matthias Brugger wrote: > > On 27/08/2019 05:59, Bibby Hsieh wrote: > > On Fri, 2019-08-23 at 16:21 +0200, Matthias Brugger wrote: > >> > >> On 20/08/2019 10:49, Bibby Hsieh wrote: > >>> GCE cannot know the register

Re: [RESEND, PATCH v13 09/12] soc: mediatek: cmdq: define the instruction struct

2019-08-26 Thread Bibby Hsieh
On Fri, 2019-08-23 at 15:50 +0200, Matthias Brugger wrote: > > On 20/08/2019 10:49, Bibby Hsieh wrote: > > Define an instruction structure for gce driver to append command. > > This structure can make the client's code more readability. > > > > Signed-off-by: Bibby

Re: [RESEND, PATCH v13 10/12] soc: mediatek: cmdq: add polling function

2019-08-26 Thread Bibby Hsieh
On Fri, 2019-08-23 at 16:05 +0200, Matthias Brugger wrote: > > On 20/08/2019 10:49, Bibby Hsieh wrote: > > add polling function in cmdq helper functions > > > > Signed-off-by: Bibby Hsieh > > Reviewed-by: CK Hu > > --- > > dri

Re: [RESEND, PATCH v13 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-08-26 Thread Bibby Hsieh
On Fri, 2019-08-23 at 16:21 +0200, Matthias Brugger wrote: > > On 20/08/2019 10:49, Bibby Hsieh wrote: > > GCE cannot know the register base address, this function > > can help cmdq client to get the cmdq_client_reg structure. > > > > Signed-off-by: Bibby

[RESEND, PATCH v13 07/12] soc: mediatek: cmdq: reorder the parameter

2019-08-20 Thread Bibby Hsieh
The order of gce instructions is [subsys offset value] so reorder the parameter of cmdq_pkt_write_mask and cmdq_pkt_write function. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 6 +++--- include/linux/soc/mediatek/mtk-cmdq.h | 10 +- 2

[RESEND, PATCH v13 01/12] dt-binding: gce: remove thread-num property

2019-08-20 Thread Bibby Hsieh
"thread-num" is an unused property so we remove it from example. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mailbox/mtk-

[RESEND, PATCH v13 00/12] support gce on mt8183 platform

2019-08-20 Thread Bibby Hsieh
thread-num property dt-binding: gce: add binding for gce client reg property soc: mediatek: cmdq: define the instruction struct soc: mediatek: cmdq: add polling function soc: mediatek: cmdq: add cmdq_dev_get_client_reg function Bibby Hsieh (12): dt-binding: gce: remove thread-num property

[RESEND, PATCH v13 09/12] soc: mediatek: cmdq: define the instruction struct

2019-08-20 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 106 +++ include/linux/mailbox/mtk-cmdq-mailbox.h | 2

[RESEND, PATCH v13 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-08-20 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 29 ++ include/linux/soc/mediatek/mtk-cmdq.h | 21

[RESEND, PATCH v13 03/12] dt-binding: gce: add binding for gce client reg property

2019-08-20 Thread Bibby Hsieh
cmdq driver provide a function that get the relationship of sub system number from device node for client. add specification for #subsys-cells, mediatek,gce-client-reg. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 16

[RESEND, PATCH v13 05/12] mailbox: mediatek: cmdq: support mt8183 gce function

2019-08-20 Thread Bibby Hsieh
add mt8183 compatible name for supporting gce function Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c index 8fddd26288e8

[RESEND, PATCH v13 04/12] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-08-20 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file changed, 7 insertions(+), 5

[RESEND, PATCH v13 02/12] dt-binding: gce: add gce header file for mt8183

2019-08-20 Thread Bibby Hsieh
Add documentation for the mt8183 gce. Add gce header file defined the gce hardware event, subsys number and constant for mt8183. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 6 +- include/dt-bindings/gce/mt8183-gce.h | 175

[RESEND, PATCH v13 10/12] soc: mediatek: cmdq: add polling function

2019-08-20 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 28 include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 15 + 3 files changed

[RESEND, PATCH v13 06/12] soc: mediatek: cmdq: clear the event in cmdq initial flow

2019-08-20 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 5 + include/linux/m

[RESEND, PATCH v13 12/12] arm64: dts: add gce node for mt8183

2019-08-20 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 08274bfcebd8..a81c995bbea9 100644

[RESEND, PATCH v13 08/12] soc: mediatek: cmdq: change the type of input parameter

2019-08-20 Thread Bibby Hsieh
According to the cmdq hardware design, the subsys is u8, the offset is u16 and the event id is u16. This patch changes the type of subsys, offset and event id to the correct type. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 10 +- include

[PATCH v13 03/12] dt-binding: gce: add binding for gce client reg property

2019-08-20 Thread Bibby Hsieh
cmdq driver provide a function that get the relationship of sub system number from device node for client. add specification for #subsys-cells, mediatek,gce-client-reg. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 16

[PATCH v13 01/12] dt-binding: gce: remove thread-num property

2019-08-20 Thread Bibby Hsieh
"thread-num" is an unused property so we remove it from example. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mailbox/mtk-

[PATCH v13 12/12] arm64: dts: add gce node for mt8183

2019-08-20 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 08274bfcebd8..a81c995bbea9 100644

[PATCH v13 04/12] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-08-20 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file changed, 7 insertions(+), 5

[PATCH v13 10/12] soc: mediatek: cmdq: add polling function

2019-08-20 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 28 include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 15 + 3 files changed

[PATCH v13 00/12] support gce on mt8183 platform

2019-08-20 Thread Bibby Hsieh
thread-num property dt-binding: gce: add binding for gce client reg property soc: mediatek: cmdq: define the instruction struct soc: mediatek: cmdq: add polling function soc: mediatek: cmdq: add cmdq_dev_get_client_reg function Bibby Hsieh (12): dt-binding: gce: remove thread-num property

[PATCH v13 05/12] mailbox: mediatek: cmdq: support mt8183 gce function

2019-08-20 Thread Bibby Hsieh
add mt8183 compatible name for supporting gce function Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c index 8fddd26288e8

[PATCH v13 09/12] soc: mediatek: cmdq: define the instruction struct

2019-08-20 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 106 +++ include/linux/mailbox/mtk-cmdq-mailbox.h | 2

[PATCH v13 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-08-20 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 29 ++ include/linux/soc/mediatek/mtk-cmdq.h | 21

[PATCH v13 08/12] soc: mediatek: cmdq: change the type of input parameter

2019-08-20 Thread Bibby Hsieh
According to the cmdq hardware design, the subsys is u8, the offset is u16 and the event id is u16. This patch changes the type of subsys, offset and event id to the correct type. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 10 +- include

[PATCH v13 07/12] soc: mediatek: cmdq: reorder the parameter

2019-08-20 Thread Bibby Hsieh
The order of gce instructions is [subsys offset value] so reorder the parameter of cmdq_pkt_write_mask and cmdq_pkt_write function. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 6 +++--- include/linux/soc/mediatek/mtk-cmdq.h | 10 +- 2

[PATCH v13 02/12] dt-binding: gce: add gce header file for mt8183

2019-08-20 Thread Bibby Hsieh
Add documentation for the mt8183 gce. Add gce header file defined the gce hardware event, subsys number and constant for mt8183. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 6 +- include/dt-bindings/gce/mt8183-gce.h | 175

[PATCH v13 06/12] soc: mediatek: cmdq: clear the event in cmdq initial flow

2019-08-20 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 5 + include/linux/m

[PATCH v12 02/12] dt-binding: gce: add gce header file for mt8183

2019-08-18 Thread Bibby Hsieh
Add documentation for the mt8183 gce. Add gce header file defined the gce hardware event, subsys number and constant for mt8183. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 6 +- include/dt-bindings/gce/mt8183-gce.h | 177

[PATCH v12 10/12] soc: mediatek: cmdq: add polling function

2019-08-18 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 28 include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 15 + 3 files changed

[PATCH v12 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-08-18 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh --- drivers/soc/mediatek/mtk-cmdq-helper.c | 29 ++ include/linux/soc/mediatek/mtk-cmdq.h | 21 +++ 2 files

[PATCH v12 07/12] soc: mediatek: cmdq: reorder the parameter

2019-08-18 Thread Bibby Hsieh
The order of gce instructions is [subsys offset value] so reorder the parameter of cmdq_pkt_write_mask and cmdq_pkt_write function. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 6 +++--- include/linux/soc/mediatek/mtk-cmdq.h | 10 +- 2

[PATCH v12 12/12] arm64: dts: add gce node for mt8183

2019-08-18 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 08274bfcebd8..a81c995bbea9 100644

[PATCH v12 09/12] soc: mediatek: cmdq: define the instruction struct

2019-08-18 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 105 +++ include/linux/mailbox/mtk-cmdq-mailbox.h | 2

[PATCH v12 01/12] dt-binding: gce: remove thread-num property

2019-08-18 Thread Bibby Hsieh
"thread-num" is an unused property so we remove it from example. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mailbox/mtk-

[PATCH v12 00/12] support gce on mt8183 platform

2019-08-18 Thread Bibby Hsieh
nstruction struct soc: mediatek: cmdq: add polling function soc: mediatek: cmdq: add cmdq_dev_get_client_reg function Bibby Hsieh (12): dt-binding: gce: remove thread-num property dt-binding: gce: add gce header file for mt8183 dt-binding: gce: add binding for gce client reg property

[PATCH v12 06/12] soc: mediatek: cmdq: clear the event in cmdq initial flow

2019-08-18 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 5 + include/linux/m

[PATCH v12 04/12] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-08-18 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file changed, 7 insertions(+), 5

[PATCH v12 03/12] dt-binding: gce: add binding for gce client reg property

2019-08-18 Thread Bibby Hsieh
cmdq driver provide a function that get the relationship of sub system number from device node for client. add specification for #subsys-cells, mediatek,gce-client-reg. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 16

[PATCH v12 08/12] soc: mediatek: cmdq: change the type of input parameter

2019-08-18 Thread Bibby Hsieh
According to the cmdq hardware design, the subsys is u8, the offset is u16 and the event id is u16. This patch changes the type of subsys, offset and event id to the correct type. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 10 +- include

[PATCH v12 05/12] mailbox: mediatek: cmdq: support mt8183 gce function

2019-08-18 Thread Bibby Hsieh
add mt8183 compatible name for supporting gce function Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c index 8fddd26288e8

[PATCH v11 08/12] soc: mediatek: cmdq: change the type of input parameter

2019-07-29 Thread Bibby Hsieh
According to the cmdq hardware design, the subsys is u8, the offset is u16 and the event id is u16. This patch changes the type of subsys, offset and event id to the correct type. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 10 +- include

[PATCH v11 09/12] soc: mediatek: cmdq: define the instruction struct

2019-07-29 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 103 +++ include/linux/mailbox/mtk-cmdq-mailbox.h | 2

[PATCH v11 05/12] mailbox: mediatek: cmdq: support mt8183 gce function

2019-07-29 Thread Bibby Hsieh
add mt8183 compatible name for supporting gce function Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c index 8fddd26288e8

[PATCH v11 04/12] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-07-29 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file changed, 7 insertions(+), 5

[PATCH v11 12/12] arm64: dts: add gce node for mt8183

2019-07-29 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 08274bfcebd8..98d17d0bdebf 100644

[PATCH v11 03/12] dt-binding: gce: add binding for gce client reg property

2019-07-29 Thread Bibby Hsieh
cmdq driver provide a function that get the relationship of sub system number from device node for client. add specification for #subsys-cells, mediatek,gce-client-reg. Signed-off-by: Bibby Hsieh --- .../devicetree/bindings/mailbox/mtk-gce.txt | 16 1 file changed, 12

[PATCH v11 00/12] support gce on mt8183 platform

2019-07-29 Thread Bibby Hsieh
atek: cmdq: add cmdq_dev_get_client_reg function Bibby Hsieh (12): dt-binding: gce: remove thread-num property dt-binding: gce: add gce header file for mt8183 dt-binding: gce: add binding for gce client reg property mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

[PATCH v11 02/12] dt-binding: gce: add gce header file for mt8183

2019-07-29 Thread Bibby Hsieh
Add documentation for the mt8183 gce. Add gce header file defined the gce hardware event, subsys number and constant for mt8183. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 6 +- include/dt-bindings/gce/mt8183-gce.h | 177

[PATCH v11 10/12] soc: mediatek: cmdq: add polling function

2019-07-29 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 28 include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 15 + 3 files changed

[PATCH v11 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-07-29 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh --- drivers/soc/mediatek/mtk-cmdq-helper.c | 29 ++ include/linux/soc/mediatek/mtk-cmdq.h | 21 +++ 2 files

[PATCH v11 07/12] soc: mediatek: cmdq: reorder the parameter

2019-07-29 Thread Bibby Hsieh
The order of gce instructions is [subsys offset value] so reorder the parameter of cmdq_pkt_write_mask and cmdq_pkt_write function. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 6 +++--- include/linux/soc/mediatek/mtk-cmdq.h | 10 +- 2

[PATCH v11 01/12] dt-binding: gce: remove thread-num property

2019-07-29 Thread Bibby Hsieh
"thread-num" is an unused property so we remove it from example. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mailbox/mtk-

[PATCH v11 06/12] soc: mediatek: cmdq: clear the event in cmdq initial flow

2019-07-29 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 5 + include/linux/m

Re: [PATCH v10 03/12] dt-binding: gce: add binding for gce client reg property

2019-07-10 Thread Bibby Hsieh
Hi, Rob, Sorry to bother you, could you please review this patch when you are available? Thanks. On Mon, 2019-07-01 at 15:48 +0800, Bibby Hsieh wrote: > cmdq driver provide a function that get the relationship > of sub system number from device node for client. > add specification fo

Re: [PATCH v10 03/12] dt-binding: gce: add binding for gce client reg property

2019-07-01 Thread Bibby Hsieh
Hi, Rob, Sorry to bother you, could you please review this patch when you are available? Thanks. On Mon, 2019-07-01 at 15:48 +0800, Bibby Hsieh wrote: > cmdq driver provide a function that get the relationship > of sub system number from device node for client. > add specification fo

[PATCH v10 08/12] soc: mediatek: cmdq: change the type of input parameter

2019-07-01 Thread Bibby Hsieh
According to the cmdq hardware design, the subsys is u8, the offset is u16 and the event id is u16. This patch changes the type of subsys, offset and event id to the correct type. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 10 +- include

[PATCH v10 05/12] mailbox: mediatek: cmdq: support mt8183 gce function

2019-07-01 Thread Bibby Hsieh
add mt8183 compatible name for supporting gce function Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c index 8fddd26288e8

[PATCH v10 00/12] support gce on mt8183 platform

2019-07-01 Thread Bibby Hsieh
nding: gce: add binding for gce client reg property soc: mediatek: cmdq: define the instruction struct soc: mediatek: cmdq: add polling function soc: mediatek: cmdq: add cmdq_dev_get_client_reg function Bibby Hsieh (12): dt-binding: gce: remove thread-num property dt-binding: gce: add gce h

[PATCH v10 07/12] soc: mediatek: cmdq: reorder the parameter

2019-07-01 Thread Bibby Hsieh
The order of gce instructions is [subsys offset value] so reorder the parameter of cmdq_pkt_write_mask and cmdq_pkt_write function. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 6 +++--- include/linux/soc/mediatek/mtk-cmdq.h | 10 +- 2

[PATCH v10 01/12] dt-binding: gce: remove thread-num property

2019-07-01 Thread Bibby Hsieh
"thread-num" is an unused property so we remove it from example. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/mailbox/mtk-gce.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mailbox/mtk-

[PATCH v10 10/12] soc: mediatek: cmdq: add polling function

2019-07-01 Thread Bibby Hsieh
add polling function in cmdq helper functions Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 28 include/linux/mailbox/mtk-cmdq-mailbox.h | 1 + include/linux/soc/mediatek/mtk-cmdq.h| 15 + 3 files changed

[PATCH v10 11/12] soc: mediatek: cmdq: add cmdq_dev_get_client_reg function

2019-07-01 Thread Bibby Hsieh
GCE cannot know the register base address, this function can help cmdq client to get the cmdq_client_reg structure. Signed-off-by: Bibby Hsieh --- drivers/soc/mediatek/mtk-cmdq-helper.c | 28 ++ include/linux/soc/mediatek/mtk-cmdq.h | 21 +++ 2 files

[PATCH v10 12/12] arm64: dts: add gce node for mt8183

2019-07-01 Thread Bibby Hsieh
add gce device node for mt8183 Signed-off-by: Bibby Hsieh --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 11 +++ 1 file changed, 11 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi index 08274bfcebd8..42b7cc9e7304 100644

[PATCH v10 06/12] soc: mediatek: cmdq: clear the event in cmdq initial flow

2019-07-01 Thread Bibby Hsieh
that the wait event function is exactly correct, we need to clear the sysram value in cmdq initial flow. Fixes: 623a6143a845 ("mailbox: mediatek: Add Mediatek CMDQ driver") Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 5 + include/linux/m

[PATCH v10 03/12] dt-binding: gce: add binding for gce client reg property

2019-07-01 Thread Bibby Hsieh
cmdq driver provide a function that get the relationship of sub system number from device node for client. add specification for #subsys-cells, mediatek,gce-client-reg. Signed-off-by: Bibby Hsieh --- .../devicetree/bindings/mailbox/mtk-gce.txt| 18 ++ 1 file changed, 14

[PATCH v10 04/12] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-07-01 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file changed, 7 insertions(+), 5

[PATCH v10 02/12] dt-binding: gce: add gce header file for mt8183

2019-07-01 Thread Bibby Hsieh
Add documentation for the mt8183 gce. Add gce header file defined the gce hardware event, subsys number and constant for mt8183. Signed-off-by: Bibby Hsieh Reviewed-by: Rob Herring --- .../devicetree/bindings/mailbox/mtk-gce.txt | 6 +- include/dt-bindings/gce/mt8183-gce.h | 177

[PATCH v10 09/12] soc: mediatek: cmdq: define the instruction struct

2019-07-01 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 103 +++ include/linux/mailbox/mtk-cmdq-mailbox.h | 2

[PATCH v9 09/12] soc: mediatek: cmdq: define the instruction struct

2019-06-27 Thread Bibby Hsieh
Define an instruction structure for gce driver to append command. This structure can make the client's code more readability. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 103 +++ include/linux/mailbox/mtk-cmdq-mailbox.h | 2

[PATCH v9 04/12] mailbox: mediatek: cmdq: move the CMDQ_IRQ_MASK into cmdq driver data

2019-06-27 Thread Bibby Hsieh
The interrupt mask and thread number has positive correlation, so we move the CMDQ_IRQ_MASK into cmdq driver data and calculate it by thread number. Signed-off-by: Bibby Hsieh Reviewed-by: CK Hu --- drivers/mailbox/mtk-cmdq-mailbox.c | 12 +++- 1 file changed, 7 insertions(+), 5

  1   2   3   4   5   >