On 03/08/2022 12:24, Jagan Teki wrote: >> Vendor in filename is needed. > > Yes this follows how similar rockchip vendor notation > > $ ls include/dt-bindings/power/rk3* -l > include/dt-bindings/power/rk3036-power.h > include/dt-bindings/power/rk3066-power.h > include/dt-bindings/power/rk3128-power.h > include/dt-bindings/power/rk3188-power.h > include/dt-bindings/power/rk3228-power.h > include/dt-bindings/power/rk3288-power.h > include/dt-bindings/power/rk3328-power.h > include/dt-bindings/power/rk3366-power.h > include/dt-bindings/power/rk3368-power.h > include/dt-bindings/power/rk3399-power.h > include/dt-bindings/power/rk3568-power.h So when are we going to switch to vendor,device pattern if you keep adding files with wrong naming? > >> >>> new file mode 100644 >>> index 000000000000..38a68e000d38 >>> --- /dev/null >>> +++ b/include/dt-bindings/power/rv1126-power.h >>> @@ -0,0 +1,35 @@ >>> +/* SPDX-License-Identifier: GPL-2.0 */ >> >> Dual license, please, unless some copyrights are no allowing this. > > Yes, as we discussed in v1 - I'm not liable to change so I keep this as it is. OK Best regards, Krzysztof