[U-Boot] [PATCH v3 2/2] ventana: Add Gateworks Ventana family support
Wolfgang Denk
wd at denx.de
Thu Feb 6 00:43:07 CET 2014
Dear Tim Harvey,
In message <1391642553-25778-1-git-send-email-tharvey at gateworks.com> you wrote:
> Gateworks Ventana is a product family based on the i.MX6. This
> patch adds support for all boards in the Ventana family. Where
> possible, data from the boards EEPROM is used to determine various
> details about the board at runtime.
...
> +#define MINMAX(n, percent) n*(1.0-percent/100), n*(1.0+percent/100)
U-Boot is integer arithmetics only. You must not use any FP
calculations in the code.
> +/*
> + * Hardware Monitor
> + */
> +#define GSC_HWMON_TEMP 0x00
> +#define GSC_HWMON_VIN 0x02
> +#define GSC_HWMON_VDD_3P3 0x05
> +#define GSC_HWMON_VBATT 0x08
> +#define GSC_HWMON_VDD_5P0 0x0b
> +#define GSC_HWMON_VDD_CORE 0x0e
> +#define GSC_HWMON_VDD_HIGH 0x14
> +#define GSC_HWMON_VDD_DDR 0x17
> +#define GSC_HWMON_VDD_SOC 0x11
> +#define GSC_HWMON_VDD_1P8 0x1d
> +#define GSC_HWMON_VDD_2P5 0x23
> +#define GSC_HWMON_VDD_1P0 0x20
I'd really like to see a C struct here to describe the register
layout...
> diff --git a/include/configs/gw_ventana.h b/include/configs/gw_ventana.h
> new file mode 100644
> index 0000000..1a18d93
...
> + #define CONFIG_SPI_FLASH_WINBOND_ERASESIZE 64*1024 /* 4,32,64K for W26Q256 */
This causes a checkpatch error:
ERROR: Macros with complex values should be enclosed in parenthesis
Please fix!
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The use of anthropomorphic terminology when dealing with computing
systems is a symptom of professional immaturity. -- Edsger Dijkstra
More information about the U-Boot
mailing list