On 2020-08-11 2:17 PM, Amadeusz Sławiński wrote:
On 8/11/2020 12:00 PM, Cezary Rojewski wrote:
Declare base structures, registers and device routines for the catpt solution. Catpt deprecates and is a direct replacement for sound/soc/intel/haswell. Supports Lynxpoint and Wildcat Point both.
Signed-off-by: Cezary Rojewski cezary.rojewski@intel.com
...
diff --git a/sound/soc/intel/catpt/registers.h b/sound/soc/intel/catpt/registers.h new file mode 100644 index 000000000000..62c968a4721a --- /dev/null +++ b/sound/soc/intel/catpt/registers.h @@ -0,0 +1,191 @@ +/* SPDX-License-Identifier: GPL-2.0-only
If we are making sure that SPDX headers are correct, from what I remember SPDX identifier should be a separate comment line, even if followed by comment. Checkpatch doesn't seem to care and I can't find anything explicitly saying that's a rule in Documentation/process/license-rules.rst (apart from one line "C Header" example, which is one line comment). Can someone else chime in? Also all other header that are included in patchset seem to follow this scheme...
Sorry for the repeated mistake. Mark already did mention it. Fixed core.h, traces.h and messages.h but the register.h was omitted during the update.
Ack.