Hi Tom,

On 4 July 2017 at 15:37, Tom Rini trini@konsulko.com wrote:

Enable more drivers that build currently. This will extend our build time test coverage

Cc: Simon Glass sjg@chromium.org Signed-off-by: Tom Rini trini@konsulko.com

---

configs/sandbox_defconfig | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+)

This patch patch and your other one introduce build warnings for me:

sandbox: + sandbox + regs = (uint32_t *)devfdt_get_addr_name(parent, "gpio"); + ^ + key_ptr = (uint8_t *)key_addr; + ^ + src_ptr = (uint8_t *)src_addr; + dst_ptr = (uint8_t *)dst_addr; +In file included from arch/sandbox/include/asm/io.h:74:0, + from drivers/pinctrl/pinctrl-single.c:11: + #define readw(addr) iotrace_readw((const void *)(addr)) + ^ +include/iotrace.h:31:42: note: in definition of macro ?writew? + #define writew(val, addr) iotrace_writew(val, (const void *)(addr)) + ^ +drivers/pinctrl/pinctrl-single.c:60:12: note: in expansion of macro ?readw? + writew((readw(reg) & ~pdata->mask) | val, reg); + ^ +drivers/pinctrl/pinctrl-single.c:60:4: note: in expansion of macro ?writew? + ^ + #define readl(addr) iotrace_readl((const void *)(addr)) +include/iotrace.h:25:42: note: in definition of macro ?writel? + #define writel(val, addr) iotrace_writel(val, (const void *)(addr)) +drivers/pinctrl/pinctrl-single.c:63:12: note: in expansion of macro ?readl? + writel((readl(reg) & ~pdata->mask) | val, reg); +drivers/pinctrl/pinctrl-single.c:63:4: note: in expansion of macro ?writel? + (((int)c->cmd == DS4510_CMD_DEVICE) && + ^ + switch ((int)c->cmd) { + ^ + host->ioaddr = (void *)devfdt_get_addr(dev); + ^ + priv->base = (void *)fdtdec_get_addr_size_auto_parent(gd->fdt_blob, + ^ + priv->pad_ctrl_reg = (void *)devfdt_get_addr_index(dev, 1); + ^ w+drivers/gpio/tegra186_gpio.c: In function ?tegra186_gpio_bind?: w+drivers/gpio/tegra186_gpio.c:182:9: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+cmd/aes.c: In function ?do_aes?: w+cmd/aes.c:52:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+cmd/aes.c:53:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+cmd/aes.c:54:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+drivers/pinctrl/pinctrl-single.c: In function ?single_configure_pins?: w+include/iotrace.h:28:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+include/iotrace.h:31:47: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+include/iotrace.h:22:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+include/iotrace.h:25:47: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+drivers/misc/ds4510.c: In function ?do_ds4510?: w+drivers/misc/ds4510.c:267:5: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] w+drivers/misc/ds4510.c:280:10: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] w+drivers/mmc/msm_sdhci.c: In function ?msm_ofdata_to_platdata?: w+drivers/mmc/msm_sdhci.c:174:17: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+drivers/mmc/msm_sdhci.c:177:15: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] w+drivers/mmc/xenon_sdhci.c: In function ?xenon_sdhci_ofdata_to_platdata?: w+drivers/mmc/xenon_sdhci.c:455:17: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

Regards, Simon