diff mbox series

[U-Boot] drivers/ddr/fsl: Update fsl_ddr_board_options as weak function

Message ID 1540805755-24359-1-git-send-email-priyanka.jain@nxp.com
State Accepted
Delegated to: Prabhakar Kushwaha
Headers show
Series [U-Boot] drivers/ddr/fsl: Update fsl_ddr_board_options as weak function | expand

Commit Message

Priyanka Jain Oct. 29, 2018, 9:37 a.m. UTC
fsl_ddr_board_options is generally defined in board
board's ddr.c, but some boards like lx2160ardb board
does not need this function.
Defining fsl_ddr_board_options as weak function to
resolve compilation errors for such boards.

Signed-off-by: Priyanka Jain <priyanka.jain@nxp.com>
---
 drivers/ddr/fsl/options.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

Comments

Prabhakar Kushwaha Feb. 20, 2019, 7:35 a.m. UTC | #1
> -----Original Message-----
> From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Priyanka Jain
> Sent: Monday, October 29, 2018 3:08 PM
> To: u-boot@lists.denx.de; York Sun <york.sun@nxp.com>
> Cc: Priyanka Jain <priyanka.jain@nxp.com>
> Subject: [U-Boot] [PATCH] drivers/ddr/fsl: Update fsl_ddr_board_options as
> weak function
> 
> fsl_ddr_board_options is generally defined in board board's ddr.c, but some
> boards like lx2160ardb board does not need this function.
> Defining fsl_ddr_board_options as weak function to resolve compilation
> errors for such boards.
> 
> Signed-off-by: Priyanka Jain <priyanka.jain@nxp.com>
> ---

Applied u-boot-fsl-qoriq master

Awaiting acceptance.

--pk
diff mbox series

Patch

diff --git a/drivers/ddr/fsl/options.c b/drivers/ddr/fsl/options.c
index 7639a8b..e406d17 100644
--- a/drivers/ddr/fsl/options.c
+++ b/drivers/ddr/fsl/options.c
@@ -22,9 +22,12 @@ 
  */
 
 /* Board-specific functions defined in each board's ddr.c */
-extern void fsl_ddr_board_options(memctl_options_t *popts,
+void __weak fsl_ddr_board_options(memctl_options_t *popts,
 		dimm_params_t *pdimm,
-		unsigned int ctrl_num);
+		unsigned int ctrl_num)
+{
+	return;
+}
 
 struct dynamic_odt {
 	unsigned int odt_rd_cfg;