> > Adds samsung,boot-mode.h header file which contains boot mode > > definitions for bootloader. As for now, there are only boot mode > > definitions for Exynos Auto v9 SoC. > > > > Signed-off-by: Chanho Park <chanho61.park@xxxxxxxxxxx> > > Thank you for your patch. There is something to discuss/improve. > > > --- > > include/dt-bindings/soc/samsung,boot-mode.h | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > create mode 100644 include/dt-bindings/soc/samsung,boot-mode.h > > > > diff --git a/include/dt-bindings/soc/samsung,boot-mode.h > > b/include/dt-bindings/soc/samsung,boot-mode.h > > new file mode 100644 > > index 000000000000..f1d03d96f45c > > --- /dev/null > > +++ b/include/dt-bindings/soc/samsung,boot-mode.h > > @@ -0,0 +1,12 @@ > > +/* SPDX-License-Identifier: GPL-2.0 */ > > Dual license: > GPL-2.0-only OR BSD-2-Clause Okay. I'll update this on v3. > > > + > > +#ifndef __SAMSUNG_BOOT_MODE_H > > __DT_BINDINGS_SAMSUNG_BOOT_MODE_H > > > +#define __SAMSUNG_BOOT_MODE_H > > + > > +/* Boot mode definitions for Exynos Auto v9 SoC */ > > + > > +#define EXYNOSAUTOV9_BOOT_FASTBOOT (0xfa) > > +#define EXYNOSAUTOV9_BOOT_BOOTLOADER (0xfc) > > +#define EXYNOSAUTOV9_BOOT_RECOVERY (0xff) > > No need for (). Thanks. Will be removed on v3. Best Regards, Chanho Park