Messages in this thread | | | Date | Mon, 23 Oct 2023 09:47:10 +0200 | Subject | Re: [PATCH v2 1/9] dt-bindings: gce: mt8195: Add CMDQ_SYNC_TOKEN_SECURE_THR_EOF event id | From | Krzysztof Kozlowski <> |
| |
On 23/10/2023 06:37, Jason-JH.Lin wrote: > CMDQ_SYNC_TOKEN_SECURE_THR_EOF is used as secure irq to notify CMDQ > driver in the normal world that GCE secure thread has completed a task > in thee secure world.
s/thee/the/
> > Signed-off-by: Jason-JH.Lin <jason-jh.lin@mediatek.com> > ---
This is a new patch, so you must mention it in the changelog. There is nothing in the changelog saying about this new patch.
> include/dt-bindings/gce/mt8195-gce.h | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/include/dt-bindings/gce/mt8195-gce.h b/include/dt-bindings/gce/mt8195-gce.h > index dcfb302b8a5b..9f99da3363b9 100644 > --- a/include/dt-bindings/gce/mt8195-gce.h > +++ b/include/dt-bindings/gce/mt8195-gce.h > @@ -809,4 +809,10 @@ > /* end of hw event */ > #define CMDQ_MAX_HW_EVENT 1019 > > +/* > + * Notify normal CMDQ there are some secure task done, > + * this token sync with secure world. > + */ > +#define CMDQ_SYNC_TOKEN_SECURE_THR_EOF 980
Why is this below 1019? Your driver calls it also even, so is this an event or not?
Your driver does not use this value, so does it mean FW uses it?
Best regards, Krzysztof
| |