On 28.07.20 13:07, Sean Anderson wrote:
On 7/28/20 6:56 AM, Heinrich Schuchardt wrote:
Hello Sean,
when I compile for Kendryte K210 I see the following build warnings:
In file included from drivers/clk/kendryte/clk.c:15: include/kendryte/pll.h:46: warning: "nop" redefined 46 | #define nop() | In file included from ./arch/riscv/include/asm/io.h:13, from drivers/clk/kendryte/clk.c:7: ./arch/riscv/include/asm/barrier.h:16: note: this is the location of the previous definition 16 | #define nop() __asm__ __volatile__ ("nop") | In file included from ./arch/riscv/include/asm/io.h:13, from drivers/clk/kendryte/pll.c:8: ./arch/riscv/include/asm/barrier.h:16: warning: "nop" redefined 16 | #define nop() __asm__ __volatile__ ("nop") | In file included from drivers/clk/kendryte/pll.c:6: include/kendryte/pll.h:46: note: this is the location of the previous definition 46 | #define nop() |
Should nop() be the assembler instruction nop or nothing?
It should be the assembler instruction. The redefinion is to allow testing the frequency calculation function on the sandbox architecture (which has no nops). The actual code which uses the nop is never called, but it still gets compiled, so I added a shim. Can you send me the config you are using?
--Sean
Please, find the config file appended.
I will provide a patch
#ifndef nop() #define nop() #endif
Another issue I found is that the memory size is not correctly set on the Kendryte as reported by bdinfo.
DRAM bank = 0x0000000000000000 -> start = 0x0000000080000000 -> size = 0x0000000000400000 DRAM bank = 0x0000000000000001 -> start = 0x0000000080400000 -> size = 0x0000000000200000 DRAM bank = 0x0000000000000002 -> start = 0x0000000080600000 -> size = 0x0000000000200000 memstart = 0x0000000000000000 memsize = 0x00000000 <<<<<<<<<<
I would have expected memsize = 0x800000.
On my MaixDuino I have an SD card slot which is connected to SPI0.
** MOSI - pin 28 ** MISO - pin 26 ** CLK - pin 27 ** CS - pin 29
Is there any ongoing work to enable the SD card slot on U-Boot yet?
Best regards
Heinrich