Am 16.06.2022 um 17:13 schrieb Michal Simek:
On 6/8/22 18:20, Stefan Herbrechtsmeier wrote:
[CAUTION: External Email]
From: Stefan Herbrechtsmeier stefan.herbrechtsmeier@weidmueller.com
A forward declaration for the serdes_illcalib function.
Signed-off-by: Stefan Herbrechtsmeier
stefan.herbrechtsmeier@weidmueller.com
tools/zynqmp_psu_init_minimize.sh | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/tools/zynqmp_psu_init_minimize.sh b/tools/zynqmp_psu_init_minimize.sh index 31fbeac327..8411065e13 100755 --- a/tools/zynqmp_psu_init_minimize.sh +++ b/tools/zynqmp_psu_init_minimize.sh @@ -108,6 +108,11 @@ cat << EOF >${TMP} #include <asm/arch/psu_init_gpl.h> #include <xil_io.h>
+static int serdes_illcalib(u32 lane3_protocol, u32 lane3_rate, + u32 lane2_protocol, u32 lane2_rate, + u32 lane1_protocol, u32 lane1_rate, + u32 lane0_protocol, u32 lane0_rate);
Better to resort that functions to avoid these additional lines.
Do you propose to move the serdes_illcalib and serdes_illcalib_pcie_gen1 functions via sed?
The psu_init_gpl.c for e-a2197-00-revA, zcu208-revA and zcu216-revA already use a forward declaration. Would it be okay to add the forward declaration if the function is used inside the source file?
Regards Stefan