Hi AngeloGioacchino, Should I use enum instead of #define if reset ID must starts from 0? Thanks, Hsiao Chien Sung On Wed, 2023-06-07 at 09:51 +0200, AngeloGioacchino Del Regno wrote: > > External email : Please do not click links or open attachments until > you have verified the sender or the content. > > Il 07/06/23 08:11, Hsiao Chien Sung ha scritto: > > Add reset control bits for MT8188 VDOSYS1. > > > > Signed-off-by: Hsiao Chien Sung <shawn.sung@xxxxxxxxxxxx> > > --- > > include/dt-bindings/reset/mt8188-resets.h | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > > > diff --git a/include/dt-bindings/reset/mt8188-resets.h > b/include/dt-bindings/reset/mt8188-resets.h > > index 377cdfda82a9..439a9a25ca19 100644 > > --- a/include/dt-bindings/reset/mt8188-resets.h > > +++ b/include/dt-bindings/reset/mt8188-resets.h > > @@ -33,4 +33,16 @@ > > > > #define MT8188_TOPRGU_SW_RST_NUM 24 > > > > +/* VDOSYS1 */ > > +#define MT8188_VDO1_RST_MERGE0_DL_ASYNC 9 > > Sorry, I've just noticed that there's no index 0 in previous > definitions: this > is wrong, it must start from 0 and must be sequential. > >