From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Wed, 18 Nov 2020 15:25:30 +0100 Subject: [PATCH 4/8] spl: atf: remove helper structure from common header In-Reply-To: <20201116214731.29128-5-michael@walle.cc> References: <20201116214731.29128-1-michael@walle.cc> <20201116214731.29128-5-michael@walle.cc> Message-ID: <7d10c407-a5a6-13fa-0cae-a371f99c1c47@xilinx.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 16. 11. 20 22:47, Michael Walle wrote: > bl2_to_bl31_params_mem is just an implementation detail of the SPL ATF > support and is not needed anywhere else. Move it from the header to the > actual module. > > Signed-off-by: Michael Walle > --- > common/spl/spl_atf.c | 11 +++++++++++ > include/atf_common.h | 14 -------------- > 2 files changed, 11 insertions(+), 14 deletions(-) > > diff --git a/common/spl/spl_atf.c b/common/spl/spl_atf.c > index 63af6a6207..51b45d5dc6 100644 > --- a/common/spl/spl_atf.c > +++ b/common/spl/spl_atf.c > @@ -18,6 +18,17 @@ > #include > #include > > +/* Holds all the structures we need for bl31 parameter passing */ > +struct bl2_to_bl31_params_mem { > + struct bl31_params bl31_params; > + struct atf_image_info bl31_image_info; > + struct atf_image_info bl32_image_info; > + struct atf_image_info bl33_image_info; > + struct entry_point_info bl33_ep_info; > + struct entry_point_info bl32_ep_info; > + struct entry_point_info bl31_ep_info; > +}; > + > struct bl31_params *bl2_plat_get_bl31_params_default(uintptr_t bl32_entry, > uintptr_t bl33_entry, > uintptr_t fdt_addr) > diff --git a/include/atf_common.h b/include/atf_common.h > index fd5454c55b..e173a10ca9 100644 > --- a/include/atf_common.h > +++ b/include/atf_common.h > @@ -162,20 +162,6 @@ struct bl31_params { > struct atf_image_info *bl33_image_info; > }; > > -/******************************************************************************* > - * This structure represents the superset of information that is passed to > - * BL31, e.g. while passing control to it from BL2, bl31_params > - * and other platform specific params > - ******************************************************************************/ > -struct bl2_to_bl31_params_mem { > - struct bl31_params bl31_params; > - struct atf_image_info bl31_image_info; > - struct atf_image_info bl32_image_info; > - struct atf_image_info bl33_image_info; > - struct entry_point_info bl33_ep_info; > - struct entry_point_info bl32_ep_info; > - struct entry_point_info bl31_ep_info; > -}; > > #endif /*__ASSEMBLY__ */ > > Acked-by: Michal Simek M