[PATCH 1/2] ALSA: hda: hda_component: Add missing #include guards
Add the conventional include guards around the content of the hda_component.h header file. This prevents double-declaration of struct hda_component if the header gets included multiple times.
This isn't causing any problems with current code, so no need to backport to older kernels.
Signed-off-by: Richard Fitzgerald rf@opensource.cirrus.com --- sound/pci/hda/hda_component.h | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/sound/pci/hda/hda_component.h b/sound/pci/hda/hda_component.h index deae9dea01b4..a0fcc723483d 100644 --- a/sound/pci/hda/hda_component.h +++ b/sound/pci/hda/hda_component.h @@ -6,6 +6,9 @@ * Cirrus Logic International Semiconductor Ltd. */
+#ifndef __HDA_COMPONENT_H__ +#define __HDA_COMPONENT_H__ + #include <linux/acpi.h> #include <linux/component.h>
@@ -82,3 +85,5 @@ static inline void hda_component_manager_unbind(struct hda_codec *cdc, { component_unbind_all(hda_codec_dev(cdc), comps); } + +#endif /* ifndef __HDA_COMPONENT_H__ */
hda_component.h uses hda_codec_dev from sound/hda_codec.h. Include sound/hda_codec.h instead of assuming that it has already been included by the parent .c file.
This isn't causing any problems with current code, so no need to backport to older kernels.
Signed-off-by: Richard Fitzgerald rf@opensource.cirrus.com --- sound/pci/hda/hda_component.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/sound/pci/hda/hda_component.h b/sound/pci/hda/hda_component.h index a0fcc723483d..c80a66691b5d 100644 --- a/sound/pci/hda/hda_component.h +++ b/sound/pci/hda/hda_component.h @@ -11,6 +11,7 @@
#include <linux/acpi.h> #include <linux/component.h> +#include <sound/hda_codec.h>
#define HDA_MAX_COMPONENTS 4 #define HDA_MAX_NAME_SIZE 50
On Thu, 07 Mar 2024 12:12:15 +0100, Richard Fitzgerald wrote:
Add the conventional include guards around the content of the hda_component.h header file. This prevents double-declaration of struct hda_component if the header gets included multiple times.
This isn't causing any problems with current code, so no need to backport to older kernels.
Signed-off-by: Richard Fitzgerald rf@opensource.cirrus.com
Applied both patches now. Thanks.
Takashi
participants (2)
-
Richard Fitzgerald
-
Takashi Iwai