Hi, On 16.11.2016 01:50, Andre Przywara wrote: > It seems pretty odd that the minimum supported SDHCI frequency is > the maximum frequency shifted _left_ by 9 bits. > Shifting it right by that amount seems to make much more sense. > > Pointed out by GCC 6.2 as the value needs more than 32 bits. > > Signed-off-by: Andre Przywara <andre.przyw...@arm.com> > --- > include/configs/xilinx_zynqmp_ep.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/configs/xilinx_zynqmp_ep.h > b/include/configs/xilinx_zynqmp_ep.h > index 8e4b960..d0ce768 100644 > --- a/include/configs/xilinx_zynqmp_ep.h > +++ b/include/configs/xilinx_zynqmp_ep.h > @@ -14,7 +14,7 @@ > #define __CONFIG_ZYNQMP_EP_H > > #define CONFIG_ZYNQ_SDHCI_MAX_FREQ 52000000 > -#define CONFIG_ZYNQ_SDHCI_MIN_FREQ (CONFIG_ZYNQ_SDHCI_MAX_FREQ << 9) > +#define CONFIG_ZYNQ_SDHCI_MIN_FREQ (CONFIG_ZYNQ_SDHCI_MAX_FREQ >> 9) > #define CONFIG_ZYNQ_EEPROM > #define CONFIG_SATA_CEVA > #define CONFIG_ZYNQMP_XHCI_LIST {ZYNQMP_USB0_XHCI_BASEADDR, \ >
thanks for the patch. We have fixed that in our internal repo but didn't send this out yet. Here is the link https://github.com/Xilinx/u-boot-xlnx/commit/299ceaf77ee6d5a555ecb5f129bd9248aa981837 Definitely it is good patch and will apply. Thanks, Michal _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot