Messages in this thread | | | From | Elvis Wang (王军) <> | Subject | Re: [PATCH v2 1/2] dt-bingings: gce: add gce header file for mt8188 | Date | Wed, 2 Nov 2022 09:58:20 +0000 |
| |
On Mon, 2022-10-31 at 14:49 +0100, AngeloGioacchino Del Regno wrote: > Il 29/10/22 12:50, Elvis.Wang ha scritto: > > From: Elvis Wang <Elvis.Wang@mediatek.com> > > > > add gce header file to define the gce thread priority, gce subsys > > id, > > event and constant for mt8188. > > v2 - use vendor in filename, use Dual license. > > > > Signed-off-by: Elvis Wang <Elvis.Wang@mediatek.com> > > Reviewed-by: Bo-Chen Chen <rex-bc.chen@mediatek.com> > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > > > Thanks for the reviews, I have fixed them. > > I can't see where/when Krzysztof gave you his R-b tag. Drop it.
Sorry for the unnecessary misunderstanding, Regarding the "Reviewd-by" in the message, my previous understanding was wrong. I understand is who reviewed it not accepted it. I will delete it in the next version, thanks.
> > Also, please fix the typo in the commit title. s/bingings/bindings/g.
will fix in next version, thanks for the review.
> > Regards, > Angelo >
| |