Messages in this thread | | | Subject | Re: [PATCH] clk: actions: Use the correct style for SPDX License Identifier | From | Andreas Färber <> | Date | Wed, 1 May 2019 22:20:44 +0200 |
| |
+ linux-actions
Am 01.05.19 um 09:07 schrieb Nishad Kamdar: > This patch corrects the SPDX License Identifier style > in header files related to Clock Drivers for Actions Semi Socs. > For C header files Documentation/process/license-rules.rst > mandates C-like comments (opposed to C source files where > C++ style should be used) [...] > drivers/clk/actions/owl-common.h | 2 +- > drivers/clk/actions/owl-composite.h | 2 +- > drivers/clk/actions/owl-divider.h | 2 +- > drivers/clk/actions/owl-factor.h | 2 +- > drivers/clk/actions/owl-fixed-factor.h | 2 +- > drivers/clk/actions/owl-gate.h | 2 +- > drivers/clk/actions/owl-mux.h | 2 +- > drivers/clk/actions/owl-pll.h | 2 +- > drivers/clk/actions/owl-reset.h | 2 +- > 9 files changed, 9 insertions(+), 9 deletions(-)
Where's the practical benefit of this patch? These are all private headers used from C files, so they can handle C++ comments just fine, otherwise we would've seen build failures.
I could understand if you were patching files in include/ but not here.
Regards, Andreas
-- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 21284 (AG Nürnberg)
| |