From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Tue, 14 May 2019 15:55:01 +0200 Subject: [U-Boot] [PATCH 05/14] arm: socfpga: Move Stratix10 and Agilex system manager common code In-Reply-To: References: <1557467688-73706-1-git-send-email-ley.foon.tan@intel.com> <1557467688-73706-6-git-send-email-ley.foon.tan@intel.com> <9d252b99-da5e-460c-cb35-1f0b58cc3311@denx.de> Message-ID: <198802c6-d9eb-952e-9800-b0a9cc5494c5@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 5/14/19 3:52 PM, Ley Foon Tan wrote: > On Tue, May 14, 2019 at 9:47 PM Marek Vasut wrote: >> >> On 5/14/19 7:47 AM, Ley Foon Tan wrote: >>> On Fri, May 10, 2019 at 8:17 PM Marek Vasut wrote: >>>> >>>> On 5/10/19 7:54 AM, Ley Foon Tan wrote: >>>>> Move Stratix10 and Agilex system manager common code to system_manager.h. >>>>> >>>>> Signed-off-by: Ley Foon Tan >>>> >>>> This becomes quite a bit of ifdeffery . Why don't you have the >>>> system_manager.h include system_manager_$soc.h like e.g. here? >>>> >>>> http://git.denx.de/?p=u-boot.git;a=blob;f=arch/arm/mach-rmobile/include/mach/rmobile.h >>>> >>>> -- >>>> Best regards, >>>> Marek Vasut >>> We already have similar thing for Gen5. A10 and S10 includes. But, S10 >>> and Agilex share some common defines/functions. >>> I can move S10 and Agilex common code to a separate file and include >>> in their header file. >> >> I think that'd improve the code . >> > I am plan to use filename system_manager_s10_agilex_common.h, or you > have better filename suggestion? :) Whatever looks consistent is fine. -- Best regards, Marek Vasut