| Message ID | 20190508144252.31001-2-jagan@amarulasolutions.com |
|---|---|
| State | New |
| Headers | show |
| Series |
|
| Related | show |
On Wed, May 8, 2019 at 10:43 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > Like clk_get_by_index, there is requirement for > clk_get_by_index_nodev. In this case to make common > code functionalities for dev and nodev, clk_get_by_index > is trying to get the index of clock by passing ofnode > instead of actual dev like current gpio uclass does. > > In these scenarios with current order of include files > the fu540-prci driver is unable to find CONFIG_ENV_SIZE. > > In file included from arch/riscv/include/asm/u-boot.h:23:0, > from include/dm/of.h:10, > from include/dm/ofnode.h:12, > from include/clk.h:11, > from include/clk-uclass.h:13, > from drivers/clk/sifive/fu540-prci.c:32: > include/environment.h:145:19: error: 'CONFIG_ENV_SIZE' > undeclared here (not in a function); did you mean 'CONFIG_CMD_XIMG'? > #define ENV_SIZE (CONFIG_ENV_SIZE - ENV_HEADER_SIZE) > > So, fix consists of changing the order of include files > in fu540-prci.c to include first common.h file. > > Cc: Anup Patel <Anup.Patel@wdc.com> > Cc: Rick Chen <rick@andestech.com> > Cc: Paul Walmsley <paul.walmsley@sifive.com> > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > --- > drivers/clk/sifive/fu540-prci.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
On Wed, May 8, 2019 at 8:13 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > Like clk_get_by_index, there is requirement for > clk_get_by_index_nodev. In this case to make common > code functionalities for dev and nodev, clk_get_by_index > is trying to get the index of clock by passing ofnode > instead of actual dev like current gpio uclass does. > > In these scenarios with current order of include files > the fu540-prci driver is unable to find CONFIG_ENV_SIZE. > > In file included from arch/riscv/include/asm/u-boot.h:23:0, > from include/dm/of.h:10, > from include/dm/ofnode.h:12, > from include/clk.h:11, > from include/clk-uclass.h:13, > from drivers/clk/sifive/fu540-prci.c:32: > include/environment.h:145:19: error: 'CONFIG_ENV_SIZE' > undeclared here (not in a function); did you mean 'CONFIG_CMD_XIMG'? > #define ENV_SIZE (CONFIG_ENV_SIZE - ENV_HEADER_SIZE) > > So, fix consists of changing the order of include files > in fu540-prci.c to include first common.h file. > > Cc: Anup Patel <Anup.Patel@wdc.com> > Cc: Rick Chen <rick@andestech.com> > Cc: Paul Walmsley <paul.walmsley@sifive.com> > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > --- Applied to u-boot-sunxi/master
diff --git a/drivers/clk/sifive/fu540-prci.c b/drivers/clk/sifive/fu540-prci.c index e1b5f8e6a9..2d47ebc6b1 100644 --- a/drivers/clk/sifive/fu540-prci.c +++ b/drivers/clk/sifive/fu540-prci.c @@ -28,10 +28,10 @@ * - SiFive FU540-C000 manual v1p0, Chapter 7 "Clocking and Reset" */ +#include <common.h> #include <asm/io.h> #include <clk-uclass.h> #include <clk.h> -#include <common.h> #include <div64.h> #include <dm.h> #include <errno.h>
Like clk_get_by_index, there is requirement for clk_get_by_index_nodev. In this case to make common code functionalities for dev and nodev, clk_get_by_index is trying to get the index of clock by passing ofnode instead of actual dev like current gpio uclass does. In these scenarios with current order of include files the fu540-prci driver is unable to find CONFIG_ENV_SIZE. In file included from arch/riscv/include/asm/u-boot.h:23:0, from include/dm/of.h:10, from include/dm/ofnode.h:12, from include/clk.h:11, from include/clk-uclass.h:13, from drivers/clk/sifive/fu540-prci.c:32: include/environment.h:145:19: error: 'CONFIG_ENV_SIZE' undeclared here (not in a function); did you mean 'CONFIG_CMD_XIMG'? #define ENV_SIZE (CONFIG_ENV_SIZE - ENV_HEADER_SIZE) So, fix consists of changing the order of include files in fu540-prci.c to include first common.h file. Cc: Anup Patel <Anup.Patel@wdc.com> Cc: Rick Chen <rick@andestech.com> Cc: Paul Walmsley <paul.walmsley@sifive.com> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> --- drivers/clk/sifive/fu540-prci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)