Djam/u-boot
1
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2025-04-17 18:34:42 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

clk: versal: Store driver data in data section

Browse source 
Line 171 in README is describing that before relocation
no code should use global variable because global variables
are placed to BSS section which is initialized to 0 after
relocation.

On Versal platforms clock driver is initialized before
relocation (via using dm,bootph-all flag in DT) and global
variables are initialized which works if this is used only
before relocation. But the variables are used after
relocation too but values are zeroed which is ending up
incorrect behavior.

That's why place variables to data section to ensure that
values are not cleared which is for now the quickest
temporary solution. The correct way to do it is to move
all global variables to private data to avoid it.

Signed-off-by: Padmarao Begari <padmarao.begari@amd.com>
Link: https://lore.kernel.org/r/20250218052419.1141139-1-padmarao.begari@amd.com
Signed-off-by: Michal Simek <michal.simek@amd.com>
This commit is contained in:
Padmarao Begari 2025-02-18 10:54:19 +05:30 • committed by Michal Simek
parent e2ef5ff765
commit 96f47ae89a
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
drivers/clk/clk_versal.c
View file

@ -106,8 +106,8 @@ struct versal_clk_priv {
struct versal_clock *clk;
};
static ulong pl_alt_ref_clk;
static ulong ref_clk;
static ulong pl_alt_ref_clk __section(".data");
static ulong ref_clk __section(".data");
struct versal_pm_query_data {
u32 qid;
@ -116,8 +116,8 @@ struct versal_pm_query_data {
u32 arg3;
};
static struct versal_clock *clock;
static unsigned int clock_max_idx;
static struct versal_clock *clock __section(".data");
static unsigned int clock_max_idx __section(".data");
#define PM_QUERY_DATA 35