From: Anders Roxell <anders.roxell@linaro.org>
To: Dave Gerlach <d-gerlach@ti.com>
Cc: linux-kernel@vger.kernel.org, Tony Lindgren <tony@atomide.com>,
Russell King - ARM Linux <linux@armlinux.org.uk>,
Santosh Shilimkar <santosh.shilimkar@oracle.com>,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org,
linux-kbuild@vger.kernel.org,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Michal Marek <michal.lkml@markovi.net>
Subject: Re: [PATCH] ARM: omap2: Fix build when using split object directories
Date: Thu, 12 Apr 2018 10:21:57 +0200 [thread overview]
Message-ID: <20180412082157.GB4002@localhost.localdomain> (raw)
In-Reply-To: <20180411211543.7923-1-d-gerlach@ti.com>
On 2018-04-11 16:15, Dave Gerlach wrote:
> The sleep33xx and sleep43xx files should not depend on a header file
> generated in drivers/memory. Remove this dependency and instead allow
> both drivers/memory and arch/arm/mach-omap2 to generate all macros
> needed in headers local to their own paths.
>
> This fixes an issue where the build fail will when using O= to set a
> split object directory and arch/arm/mach-omap2 is built before
> drivers/memory with the following error:
>
> .../drivers/memory/emif-asm-offsets.c:1:0: fatal error: can't open drivers/memory/emif-asm-offsets.s for writing: No such file or directory
> compilation terminated.
>
> Fixes: 41d9d44d7258 ("ARM: OMAP2+: pm33xx-core: Add platform code needed for PM")
> Acked-by: Tony Lindgren <tony@atomide.com>
> Reviewed-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
Tested-by: Anders Roxell <anders.roxell@linaro.org>
Maybe we can remove drivers/memory/Makefile.asm-offsets and move those
changes into drivers/memory/Makefile ?
Cheers,
Anders
> ---
> arch/arm/mach-omap2/Makefile | 6 +--
> arch/arm/mach-omap2/pm-asm-offsets.c | 3 ++
> arch/arm/mach-omap2/sleep33xx.S | 1 -
> arch/arm/mach-omap2/sleep43xx.S | 1 -
> drivers/memory/emif-asm-offsets.c | 72 +---------------------------------
> include/linux/ti-emif-sram.h | 75 ++++++++++++++++++++++++++++++++++++
> 6 files changed, 80 insertions(+), 78 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
> index 4603c30fef73..0d9ce58bc464 100644
> --- a/arch/arm/mach-omap2/Makefile
> +++ b/arch/arm/mach-omap2/Makefile
> @@ -243,8 +243,4 @@ arch/arm/mach-omap2/pm-asm-offsets.s: arch/arm/mach-omap2/pm-asm-offsets.c
> include/generated/ti-pm-asm-offsets.h: arch/arm/mach-omap2/pm-asm-offsets.s FORCE
> $(call filechk,offsets,__TI_PM_ASM_OFFSETS_H__)
>
> -# For rule to generate ti-emif-asm-offsets.h dependency
> -include drivers/memory/Makefile.asm-offsets
> -
> -arch/arm/mach-omap2/sleep33xx.o: include/generated/ti-pm-asm-offsets.h include/generated/ti-emif-asm-offsets.h
> -arch/arm/mach-omap2/sleep43xx.o: include/generated/ti-pm-asm-offsets.h include/generated/ti-emif-asm-offsets.h
> +$(obj)/sleep33xx.o $(obj)/sleep43xx.o: include/generated/ti-pm-asm-offsets.h
> diff --git a/arch/arm/mach-omap2/pm-asm-offsets.c b/arch/arm/mach-omap2/pm-asm-offsets.c
> index 6d4392da7c11..b9846b19e5e2 100644
> --- a/arch/arm/mach-omap2/pm-asm-offsets.c
> +++ b/arch/arm/mach-omap2/pm-asm-offsets.c
> @@ -7,9 +7,12 @@
>
> #include <linux/kbuild.h>
> #include <linux/platform_data/pm33xx.h>
> +#include <linux/ti-emif-sram.h>
>
> int main(void)
> {
> + ti_emif_asm_offsets();
> +
> DEFINE(AMX3_PM_WFI_FLAGS_OFFSET,
> offsetof(struct am33xx_pm_sram_data, wfi_flags));
> DEFINE(AMX3_PM_L2_AUX_CTRL_VAL_OFFSET,
> diff --git a/arch/arm/mach-omap2/sleep33xx.S b/arch/arm/mach-omap2/sleep33xx.S
> index 218d79930b04..322b3bb868b4 100644
> --- a/arch/arm/mach-omap2/sleep33xx.S
> +++ b/arch/arm/mach-omap2/sleep33xx.S
> @@ -6,7 +6,6 @@
> * Dave Gerlach, Vaibhav Bedia
> */
>
> -#include <generated/ti-emif-asm-offsets.h>
> #include <generated/ti-pm-asm-offsets.h>
> #include <linux/linkage.h>
> #include <linux/ti-emif-sram.h>
> diff --git a/arch/arm/mach-omap2/sleep43xx.S b/arch/arm/mach-omap2/sleep43xx.S
> index b24be624e8b9..8903814a6677 100644
> --- a/arch/arm/mach-omap2/sleep43xx.S
> +++ b/arch/arm/mach-omap2/sleep43xx.S
> @@ -6,7 +6,6 @@
> * Dave Gerlach, Vaibhav Bedia
> */
>
> -#include <generated/ti-emif-asm-offsets.h>
> #include <generated/ti-pm-asm-offsets.h>
> #include <linux/linkage.h>
> #include <linux/ti-emif-sram.h>
> diff --git a/drivers/memory/emif-asm-offsets.c b/drivers/memory/emif-asm-offsets.c
> index 71a89d5d3efd..db8043019ec6 100644
> --- a/drivers/memory/emif-asm-offsets.c
> +++ b/drivers/memory/emif-asm-offsets.c
> @@ -16,77 +16,7 @@
>
> int main(void)
> {
> - DEFINE(EMIF_SDCFG_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_sdcfg_val));
> - DEFINE(EMIF_TIMING1_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_timing1_val));
> - DEFINE(EMIF_TIMING2_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_timing2_val));
> - DEFINE(EMIF_TIMING3_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_timing3_val));
> - DEFINE(EMIF_REF_CTRL_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ref_ctrl_val));
> - DEFINE(EMIF_ZQCFG_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_zqcfg_val));
> - DEFINE(EMIF_PMCR_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_pmcr_val));
> - DEFINE(EMIF_PMCR_SHDW_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_pmcr_shdw_val));
> - DEFINE(EMIF_RD_WR_LEVEL_RAMP_CTRL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_rd_wr_level_ramp_ctrl));
> - DEFINE(EMIF_RD_WR_EXEC_THRESH_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_rd_wr_exec_thresh));
> - DEFINE(EMIF_COS_CONFIG_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_cos_config));
> - DEFINE(EMIF_PRIORITY_TO_COS_MAPPING_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_priority_to_cos_mapping));
> - DEFINE(EMIF_CONNECT_ID_SERV_1_MAP_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_connect_id_serv_1_map));
> - DEFINE(EMIF_CONNECT_ID_SERV_2_MAP_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_connect_id_serv_2_map));
> - DEFINE(EMIF_OCP_CONFIG_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ocp_config_val));
> - DEFINE(EMIF_LPDDR2_NVM_TIM_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim));
> - DEFINE(EMIF_LPDDR2_NVM_TIM_SHDW_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim_shdw));
> - DEFINE(EMIF_DLL_CALIB_CTRL_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val));
> - DEFINE(EMIF_DLL_CALIB_CTRL_VAL_SHDW_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val_shdw));
> - DEFINE(EMIF_DDR_PHY_CTLR_1_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ddr_phy_ctlr_1));
> - DEFINE(EMIF_EXT_PHY_CTRL_VALS_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ext_phy_ctrl_vals));
> - DEFINE(EMIF_REGS_AMX3_SIZE, sizeof(struct emif_regs_amx3));
> -
> - BLANK();
> -
> - DEFINE(EMIF_PM_BASE_ADDR_VIRT_OFFSET,
> - offsetof(struct ti_emif_pm_data, ti_emif_base_addr_virt));
> - DEFINE(EMIF_PM_BASE_ADDR_PHYS_OFFSET,
> - offsetof(struct ti_emif_pm_data, ti_emif_base_addr_phys));
> - DEFINE(EMIF_PM_CONFIG_OFFSET,
> - offsetof(struct ti_emif_pm_data, ti_emif_sram_config));
> - DEFINE(EMIF_PM_REGS_VIRT_OFFSET,
> - offsetof(struct ti_emif_pm_data, regs_virt));
> - DEFINE(EMIF_PM_REGS_PHYS_OFFSET,
> - offsetof(struct ti_emif_pm_data, regs_phys));
> - DEFINE(EMIF_PM_DATA_SIZE, sizeof(struct ti_emif_pm_data));
> -
> - BLANK();
> -
> - DEFINE(EMIF_PM_SAVE_CONTEXT_OFFSET,
> - offsetof(struct ti_emif_pm_functions, save_context));
> - DEFINE(EMIF_PM_RESTORE_CONTEXT_OFFSET,
> - offsetof(struct ti_emif_pm_functions, restore_context));
> - DEFINE(EMIF_PM_ENTER_SR_OFFSET,
> - offsetof(struct ti_emif_pm_functions, enter_sr));
> - DEFINE(EMIF_PM_EXIT_SR_OFFSET,
> - offsetof(struct ti_emif_pm_functions, exit_sr));
> - DEFINE(EMIF_PM_ABORT_SR_OFFSET,
> - offsetof(struct ti_emif_pm_functions, abort_sr));
> - DEFINE(EMIF_PM_FUNCTIONS_SIZE, sizeof(struct ti_emif_pm_functions));
> + ti_emif_asm_offsets();
>
> return 0;
> }
> diff --git a/include/linux/ti-emif-sram.h b/include/linux/ti-emif-sram.h
> index 45bc6b376492..53604b087f2c 100644
> --- a/include/linux/ti-emif-sram.h
> +++ b/include/linux/ti-emif-sram.h
> @@ -60,6 +60,81 @@ struct ti_emif_pm_functions {
> u32 abort_sr;
> } __packed __aligned(8);
>
> +static inline void ti_emif_asm_offsets(void)
> +{
> + DEFINE(EMIF_SDCFG_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_sdcfg_val));
> + DEFINE(EMIF_TIMING1_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_timing1_val));
> + DEFINE(EMIF_TIMING2_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_timing2_val));
> + DEFINE(EMIF_TIMING3_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_timing3_val));
> + DEFINE(EMIF_REF_CTRL_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ref_ctrl_val));
> + DEFINE(EMIF_ZQCFG_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_zqcfg_val));
> + DEFINE(EMIF_PMCR_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_pmcr_val));
> + DEFINE(EMIF_PMCR_SHDW_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_pmcr_shdw_val));
> + DEFINE(EMIF_RD_WR_LEVEL_RAMP_CTRL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_rd_wr_level_ramp_ctrl));
> + DEFINE(EMIF_RD_WR_EXEC_THRESH_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_rd_wr_exec_thresh));
> + DEFINE(EMIF_COS_CONFIG_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_cos_config));
> + DEFINE(EMIF_PRIORITY_TO_COS_MAPPING_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_priority_to_cos_mapping));
> + DEFINE(EMIF_CONNECT_ID_SERV_1_MAP_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_connect_id_serv_1_map));
> + DEFINE(EMIF_CONNECT_ID_SERV_2_MAP_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_connect_id_serv_2_map));
> + DEFINE(EMIF_OCP_CONFIG_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ocp_config_val));
> + DEFINE(EMIF_LPDDR2_NVM_TIM_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim));
> + DEFINE(EMIF_LPDDR2_NVM_TIM_SHDW_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim_shdw));
> + DEFINE(EMIF_DLL_CALIB_CTRL_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val));
> + DEFINE(EMIF_DLL_CALIB_CTRL_VAL_SHDW_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val_shdw));
> + DEFINE(EMIF_DDR_PHY_CTLR_1_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ddr_phy_ctlr_1));
> + DEFINE(EMIF_EXT_PHY_CTRL_VALS_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ext_phy_ctrl_vals));
> + DEFINE(EMIF_REGS_AMX3_SIZE, sizeof(struct emif_regs_amx3));
> +
> + BLANK();
> +
> + DEFINE(EMIF_PM_BASE_ADDR_VIRT_OFFSET,
> + offsetof(struct ti_emif_pm_data, ti_emif_base_addr_virt));
> + DEFINE(EMIF_PM_BASE_ADDR_PHYS_OFFSET,
> + offsetof(struct ti_emif_pm_data, ti_emif_base_addr_phys));
> + DEFINE(EMIF_PM_CONFIG_OFFSET,
> + offsetof(struct ti_emif_pm_data, ti_emif_sram_config));
> + DEFINE(EMIF_PM_REGS_VIRT_OFFSET,
> + offsetof(struct ti_emif_pm_data, regs_virt));
> + DEFINE(EMIF_PM_REGS_PHYS_OFFSET,
> + offsetof(struct ti_emif_pm_data, regs_phys));
> + DEFINE(EMIF_PM_DATA_SIZE, sizeof(struct ti_emif_pm_data));
> +
> + BLANK();
> +
> + DEFINE(EMIF_PM_SAVE_CONTEXT_OFFSET,
> + offsetof(struct ti_emif_pm_functions, save_context));
> + DEFINE(EMIF_PM_RESTORE_CONTEXT_OFFSET,
> + offsetof(struct ti_emif_pm_functions, restore_context));
> + DEFINE(EMIF_PM_ENTER_SR_OFFSET,
> + offsetof(struct ti_emif_pm_functions, enter_sr));
> + DEFINE(EMIF_PM_EXIT_SR_OFFSET,
> + offsetof(struct ti_emif_pm_functions, exit_sr));
> + DEFINE(EMIF_PM_ABORT_SR_OFFSET,
> + offsetof(struct ti_emif_pm_functions, abort_sr));
> + DEFINE(EMIF_PM_FUNCTIONS_SIZE, sizeof(struct ti_emif_pm_functions));
> +}
> +
> struct gen_pool;
>
> int ti_emif_copy_pm_function_table(struct gen_pool *sram_pool, void *dst);
> --
> 2.16.1
>
--
Anders Roxell
anders.roxell@linaro.org
M: +46 709 71 42 85 | IRC: roxell
WARNING: multiple messages have this Message-ID (diff)
From: anders.roxell@linaro.org (Anders Roxell)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: omap2: Fix build when using split object directories
Date: Thu, 12 Apr 2018 10:21:57 +0200 [thread overview]
Message-ID: <20180412082157.GB4002@localhost.localdomain> (raw)
In-Reply-To: <20180411211543.7923-1-d-gerlach@ti.com>
On 2018-04-11 16:15, Dave Gerlach wrote:
> The sleep33xx and sleep43xx files should not depend on a header file
> generated in drivers/memory. Remove this dependency and instead allow
> both drivers/memory and arch/arm/mach-omap2 to generate all macros
> needed in headers local to their own paths.
>
> This fixes an issue where the build fail will when using O= to set a
> split object directory and arch/arm/mach-omap2 is built before
> drivers/memory with the following error:
>
> .../drivers/memory/emif-asm-offsets.c:1:0: fatal error: can't open drivers/memory/emif-asm-offsets.s for writing: No such file or directory
> compilation terminated.
>
> Fixes: 41d9d44d7258 ("ARM: OMAP2+: pm33xx-core: Add platform code needed for PM")
> Acked-by: Tony Lindgren <tony@atomide.com>
> Reviewed-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
Tested-by: Anders Roxell <anders.roxell@linaro.org>
Maybe we can remove drivers/memory/Makefile.asm-offsets and move those
changes into drivers/memory/Makefile ?
Cheers,
Anders
> ---
> arch/arm/mach-omap2/Makefile | 6 +--
> arch/arm/mach-omap2/pm-asm-offsets.c | 3 ++
> arch/arm/mach-omap2/sleep33xx.S | 1 -
> arch/arm/mach-omap2/sleep43xx.S | 1 -
> drivers/memory/emif-asm-offsets.c | 72 +---------------------------------
> include/linux/ti-emif-sram.h | 75 ++++++++++++++++++++++++++++++++++++
> 6 files changed, 80 insertions(+), 78 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
> index 4603c30fef73..0d9ce58bc464 100644
> --- a/arch/arm/mach-omap2/Makefile
> +++ b/arch/arm/mach-omap2/Makefile
> @@ -243,8 +243,4 @@ arch/arm/mach-omap2/pm-asm-offsets.s: arch/arm/mach-omap2/pm-asm-offsets.c
> include/generated/ti-pm-asm-offsets.h: arch/arm/mach-omap2/pm-asm-offsets.s FORCE
> $(call filechk,offsets,__TI_PM_ASM_OFFSETS_H__)
>
> -# For rule to generate ti-emif-asm-offsets.h dependency
> -include drivers/memory/Makefile.asm-offsets
> -
> -arch/arm/mach-omap2/sleep33xx.o: include/generated/ti-pm-asm-offsets.h include/generated/ti-emif-asm-offsets.h
> -arch/arm/mach-omap2/sleep43xx.o: include/generated/ti-pm-asm-offsets.h include/generated/ti-emif-asm-offsets.h
> +$(obj)/sleep33xx.o $(obj)/sleep43xx.o: include/generated/ti-pm-asm-offsets.h
> diff --git a/arch/arm/mach-omap2/pm-asm-offsets.c b/arch/arm/mach-omap2/pm-asm-offsets.c
> index 6d4392da7c11..b9846b19e5e2 100644
> --- a/arch/arm/mach-omap2/pm-asm-offsets.c
> +++ b/arch/arm/mach-omap2/pm-asm-offsets.c
> @@ -7,9 +7,12 @@
>
> #include <linux/kbuild.h>
> #include <linux/platform_data/pm33xx.h>
> +#include <linux/ti-emif-sram.h>
>
> int main(void)
> {
> + ti_emif_asm_offsets();
> +
> DEFINE(AMX3_PM_WFI_FLAGS_OFFSET,
> offsetof(struct am33xx_pm_sram_data, wfi_flags));
> DEFINE(AMX3_PM_L2_AUX_CTRL_VAL_OFFSET,
> diff --git a/arch/arm/mach-omap2/sleep33xx.S b/arch/arm/mach-omap2/sleep33xx.S
> index 218d79930b04..322b3bb868b4 100644
> --- a/arch/arm/mach-omap2/sleep33xx.S
> +++ b/arch/arm/mach-omap2/sleep33xx.S
> @@ -6,7 +6,6 @@
> * Dave Gerlach, Vaibhav Bedia
> */
>
> -#include <generated/ti-emif-asm-offsets.h>
> #include <generated/ti-pm-asm-offsets.h>
> #include <linux/linkage.h>
> #include <linux/ti-emif-sram.h>
> diff --git a/arch/arm/mach-omap2/sleep43xx.S b/arch/arm/mach-omap2/sleep43xx.S
> index b24be624e8b9..8903814a6677 100644
> --- a/arch/arm/mach-omap2/sleep43xx.S
> +++ b/arch/arm/mach-omap2/sleep43xx.S
> @@ -6,7 +6,6 @@
> * Dave Gerlach, Vaibhav Bedia
> */
>
> -#include <generated/ti-emif-asm-offsets.h>
> #include <generated/ti-pm-asm-offsets.h>
> #include <linux/linkage.h>
> #include <linux/ti-emif-sram.h>
> diff --git a/drivers/memory/emif-asm-offsets.c b/drivers/memory/emif-asm-offsets.c
> index 71a89d5d3efd..db8043019ec6 100644
> --- a/drivers/memory/emif-asm-offsets.c
> +++ b/drivers/memory/emif-asm-offsets.c
> @@ -16,77 +16,7 @@
>
> int main(void)
> {
> - DEFINE(EMIF_SDCFG_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_sdcfg_val));
> - DEFINE(EMIF_TIMING1_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_timing1_val));
> - DEFINE(EMIF_TIMING2_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_timing2_val));
> - DEFINE(EMIF_TIMING3_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_timing3_val));
> - DEFINE(EMIF_REF_CTRL_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ref_ctrl_val));
> - DEFINE(EMIF_ZQCFG_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_zqcfg_val));
> - DEFINE(EMIF_PMCR_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_pmcr_val));
> - DEFINE(EMIF_PMCR_SHDW_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_pmcr_shdw_val));
> - DEFINE(EMIF_RD_WR_LEVEL_RAMP_CTRL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_rd_wr_level_ramp_ctrl));
> - DEFINE(EMIF_RD_WR_EXEC_THRESH_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_rd_wr_exec_thresh));
> - DEFINE(EMIF_COS_CONFIG_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_cos_config));
> - DEFINE(EMIF_PRIORITY_TO_COS_MAPPING_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_priority_to_cos_mapping));
> - DEFINE(EMIF_CONNECT_ID_SERV_1_MAP_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_connect_id_serv_1_map));
> - DEFINE(EMIF_CONNECT_ID_SERV_2_MAP_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_connect_id_serv_2_map));
> - DEFINE(EMIF_OCP_CONFIG_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ocp_config_val));
> - DEFINE(EMIF_LPDDR2_NVM_TIM_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim));
> - DEFINE(EMIF_LPDDR2_NVM_TIM_SHDW_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim_shdw));
> - DEFINE(EMIF_DLL_CALIB_CTRL_VAL_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val));
> - DEFINE(EMIF_DLL_CALIB_CTRL_VAL_SHDW_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val_shdw));
> - DEFINE(EMIF_DDR_PHY_CTLR_1_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ddr_phy_ctlr_1));
> - DEFINE(EMIF_EXT_PHY_CTRL_VALS_OFFSET,
> - offsetof(struct emif_regs_amx3, emif_ext_phy_ctrl_vals));
> - DEFINE(EMIF_REGS_AMX3_SIZE, sizeof(struct emif_regs_amx3));
> -
> - BLANK();
> -
> - DEFINE(EMIF_PM_BASE_ADDR_VIRT_OFFSET,
> - offsetof(struct ti_emif_pm_data, ti_emif_base_addr_virt));
> - DEFINE(EMIF_PM_BASE_ADDR_PHYS_OFFSET,
> - offsetof(struct ti_emif_pm_data, ti_emif_base_addr_phys));
> - DEFINE(EMIF_PM_CONFIG_OFFSET,
> - offsetof(struct ti_emif_pm_data, ti_emif_sram_config));
> - DEFINE(EMIF_PM_REGS_VIRT_OFFSET,
> - offsetof(struct ti_emif_pm_data, regs_virt));
> - DEFINE(EMIF_PM_REGS_PHYS_OFFSET,
> - offsetof(struct ti_emif_pm_data, regs_phys));
> - DEFINE(EMIF_PM_DATA_SIZE, sizeof(struct ti_emif_pm_data));
> -
> - BLANK();
> -
> - DEFINE(EMIF_PM_SAVE_CONTEXT_OFFSET,
> - offsetof(struct ti_emif_pm_functions, save_context));
> - DEFINE(EMIF_PM_RESTORE_CONTEXT_OFFSET,
> - offsetof(struct ti_emif_pm_functions, restore_context));
> - DEFINE(EMIF_PM_ENTER_SR_OFFSET,
> - offsetof(struct ti_emif_pm_functions, enter_sr));
> - DEFINE(EMIF_PM_EXIT_SR_OFFSET,
> - offsetof(struct ti_emif_pm_functions, exit_sr));
> - DEFINE(EMIF_PM_ABORT_SR_OFFSET,
> - offsetof(struct ti_emif_pm_functions, abort_sr));
> - DEFINE(EMIF_PM_FUNCTIONS_SIZE, sizeof(struct ti_emif_pm_functions));
> + ti_emif_asm_offsets();
>
> return 0;
> }
> diff --git a/include/linux/ti-emif-sram.h b/include/linux/ti-emif-sram.h
> index 45bc6b376492..53604b087f2c 100644
> --- a/include/linux/ti-emif-sram.h
> +++ b/include/linux/ti-emif-sram.h
> @@ -60,6 +60,81 @@ struct ti_emif_pm_functions {
> u32 abort_sr;
> } __packed __aligned(8);
>
> +static inline void ti_emif_asm_offsets(void)
> +{
> + DEFINE(EMIF_SDCFG_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_sdcfg_val));
> + DEFINE(EMIF_TIMING1_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_timing1_val));
> + DEFINE(EMIF_TIMING2_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_timing2_val));
> + DEFINE(EMIF_TIMING3_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_timing3_val));
> + DEFINE(EMIF_REF_CTRL_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ref_ctrl_val));
> + DEFINE(EMIF_ZQCFG_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_zqcfg_val));
> + DEFINE(EMIF_PMCR_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_pmcr_val));
> + DEFINE(EMIF_PMCR_SHDW_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_pmcr_shdw_val));
> + DEFINE(EMIF_RD_WR_LEVEL_RAMP_CTRL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_rd_wr_level_ramp_ctrl));
> + DEFINE(EMIF_RD_WR_EXEC_THRESH_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_rd_wr_exec_thresh));
> + DEFINE(EMIF_COS_CONFIG_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_cos_config));
> + DEFINE(EMIF_PRIORITY_TO_COS_MAPPING_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_priority_to_cos_mapping));
> + DEFINE(EMIF_CONNECT_ID_SERV_1_MAP_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_connect_id_serv_1_map));
> + DEFINE(EMIF_CONNECT_ID_SERV_2_MAP_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_connect_id_serv_2_map));
> + DEFINE(EMIF_OCP_CONFIG_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ocp_config_val));
> + DEFINE(EMIF_LPDDR2_NVM_TIM_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim));
> + DEFINE(EMIF_LPDDR2_NVM_TIM_SHDW_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_lpddr2_nvm_tim_shdw));
> + DEFINE(EMIF_DLL_CALIB_CTRL_VAL_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val));
> + DEFINE(EMIF_DLL_CALIB_CTRL_VAL_SHDW_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_dll_calib_ctrl_val_shdw));
> + DEFINE(EMIF_DDR_PHY_CTLR_1_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ddr_phy_ctlr_1));
> + DEFINE(EMIF_EXT_PHY_CTRL_VALS_OFFSET,
> + offsetof(struct emif_regs_amx3, emif_ext_phy_ctrl_vals));
> + DEFINE(EMIF_REGS_AMX3_SIZE, sizeof(struct emif_regs_amx3));
> +
> + BLANK();
> +
> + DEFINE(EMIF_PM_BASE_ADDR_VIRT_OFFSET,
> + offsetof(struct ti_emif_pm_data, ti_emif_base_addr_virt));
> + DEFINE(EMIF_PM_BASE_ADDR_PHYS_OFFSET,
> + offsetof(struct ti_emif_pm_data, ti_emif_base_addr_phys));
> + DEFINE(EMIF_PM_CONFIG_OFFSET,
> + offsetof(struct ti_emif_pm_data, ti_emif_sram_config));
> + DEFINE(EMIF_PM_REGS_VIRT_OFFSET,
> + offsetof(struct ti_emif_pm_data, regs_virt));
> + DEFINE(EMIF_PM_REGS_PHYS_OFFSET,
> + offsetof(struct ti_emif_pm_data, regs_phys));
> + DEFINE(EMIF_PM_DATA_SIZE, sizeof(struct ti_emif_pm_data));
> +
> + BLANK();
> +
> + DEFINE(EMIF_PM_SAVE_CONTEXT_OFFSET,
> + offsetof(struct ti_emif_pm_functions, save_context));
> + DEFINE(EMIF_PM_RESTORE_CONTEXT_OFFSET,
> + offsetof(struct ti_emif_pm_functions, restore_context));
> + DEFINE(EMIF_PM_ENTER_SR_OFFSET,
> + offsetof(struct ti_emif_pm_functions, enter_sr));
> + DEFINE(EMIF_PM_EXIT_SR_OFFSET,
> + offsetof(struct ti_emif_pm_functions, exit_sr));
> + DEFINE(EMIF_PM_ABORT_SR_OFFSET,
> + offsetof(struct ti_emif_pm_functions, abort_sr));
> + DEFINE(EMIF_PM_FUNCTIONS_SIZE, sizeof(struct ti_emif_pm_functions));
> +}
> +
> struct gen_pool;
>
> int ti_emif_copy_pm_function_table(struct gen_pool *sram_pool, void *dst);
> --
> 2.16.1
>
--
Anders Roxell
anders.roxell at linaro.org
M: +46 709 71 42 85 | IRC: roxell
next prev parent reply other threads:[~2018-04-12 8:50 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-11 21:15 [PATCH] ARM: omap2: Fix build when using split object directories Dave Gerlach
2018-04-11 21:15 ` Dave Gerlach
2018-04-11 21:15 ` Dave Gerlach
2018-04-11 21:15 ` Dave Gerlach
2018-04-12 8:21 ` Anders Roxell [this message]
2018-04-12 8:21 ` Anders Roxell
2018-04-12 9:08 ` Masahiro Yamada
2018-04-12 9:08 ` Masahiro Yamada
2018-04-13 3:00 ` Dave Gerlach
2018-04-13 3:00 ` Dave Gerlach
2018-04-13 3:00 ` Dave Gerlach
2018-04-13 14:06 ` Dave Gerlach
2018-04-13 14:06 ` Dave Gerlach
2018-04-13 14:06 ` Dave Gerlach
2018-04-18 17:09 ` Tony Lindgren
2018-04-18 17:09 ` Tony Lindgren
2018-04-18 17:14 ` Santosh Shilimkar
2018-04-18 17:14 ` Santosh Shilimkar
2018-04-18 17:14 ` Santosh Shilimkar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180412082157.GB4002@localhost.localdomain \
--to=anders.roxell@linaro.org \
--cc=d-gerlach@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=michal.lkml@markovi.net \
--cc=santosh.shilimkar@oracle.com \
--cc=tony@atomide.com \
--cc=yamada.masahiro@socionext.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.