Elliot Berman
573c7f061d
ANDROID_VENDOR_HOOKS has implicit dependencies on the subsystems which
it hooks into whenever it directly uses an "internal" header for that
system. For instance, drivers/scsi/ufs/ufshcd.h doesn't compile unless
CONFIG_SCSI_UFSHCD is enabled:
In file included from drivers/android/vendor_hooks.c:41:
In file included from include/trace/hooks/ufshcd.h:21:
include/../drivers/scsi/ufs/ufshcd.h:675:38: error: field has incomplete type 'struct devfreq_simple_ondemand_data'
struct devfreq_simple_ondemand_data ondemand_data;
^
include/../drivers/scsi/ufs/ufshcd.h:675:9: note: forward declaration of 'struct devfreq_simple_ondemand_data'
struct devfreq_simple_ondemand_data ondemand_data;
To avoid these implicit dependencies, guard any "internal" header
includes with the respective CONFIG_ options when applicable. If the
CONFIG_ option is not enabled, then the forward-declared structs are
used. This is acceptable because those hooks would not have been called
anyway since the underlying subsytem wasn't enabled.
Fixes: commit 1590a0e8e1 ("ANDROID: GKI: include more type definitions in vendor hooks")
Bug: 240404657
Change-Id: I43d19136fdb1b534e80630067f5db92c379adc67
Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>
54 lines
1.5 KiB
C
54 lines
1.5 KiB
C
/* SPDX-License-Identifier: GPL-2.0 */
|
|
#undef TRACE_SYSTEM
|
|
#define TRACE_SYSTEM mmc_core
|
|
|
|
#define TRACE_INCLUDE_PATH trace/hooks
|
|
|
|
#if !defined(_TRACE_HOOK_MMC_CORE_H) || defined(TRACE_HEADER_MULTI_READ)
|
|
#define _TRACE_HOOK_MMC_CORE_H
|
|
|
|
#include <linux/tracepoint.h>
|
|
#include <trace/hooks/vendor_hooks.h>
|
|
|
|
#if defined(__GENKSYMS__) || !IS_ENABLED(CONFIG_MMC_SDHCI)
|
|
struct sdhci_host;
|
|
#else
|
|
/* struct sdhci_host */
|
|
#include <../drivers/mmc/host/sdhci.h>
|
|
#endif
|
|
|
|
#ifdef __GENKSYMS__
|
|
struct mmc_card;
|
|
struct mmc_host;
|
|
#else
|
|
/* struct mmc_card */
|
|
#include <linux/mmc/card.h>
|
|
/* struct mmc_host */
|
|
#include <linux/mmc/host.h>
|
|
#endif /* __GENKSYMS__ */
|
|
|
|
DECLARE_HOOK(android_vh_mmc_blk_reset,
|
|
TP_PROTO(struct mmc_host *host, int err, bool *allow),
|
|
TP_ARGS(host, err, allow));
|
|
DECLARE_HOOK(android_vh_mmc_blk_mq_rw_recovery,
|
|
TP_PROTO(struct mmc_card *card),
|
|
TP_ARGS(card));
|
|
DECLARE_HOOK(android_vh_sd_update_bus_speed_mode,
|
|
TP_PROTO(struct mmc_card *card),
|
|
TP_ARGS(card));
|
|
DECLARE_HOOK(android_vh_mmc_attach_sd,
|
|
TP_PROTO(struct mmc_host *host, u32 ocr, int err),
|
|
TP_ARGS(host, ocr, err));
|
|
DECLARE_HOOK(android_vh_sdhci_get_cd,
|
|
TP_PROTO(struct sdhci_host *host, bool *allow),
|
|
TP_ARGS(host, allow));
|
|
DECLARE_HOOK(android_vh_mmc_gpio_cd_irqt,
|
|
TP_PROTO(struct mmc_host *host, bool *allow),
|
|
TP_ARGS(host, allow));
|
|
|
|
/* macro versions of hooks are no longer required */
|
|
|
|
#endif /* _TRACE_HOOK_MMC_CORE_H */
|
|
/* This part must be outside protection */
|
|
#include <trace/define_trace.h>
|