On 01/12/2011 11:46 PM, Marek Vasut wrote:
> Signed-off-by: Marek Vasut <marek.va...@gmail.com>
> ---
> drivers/block/Makefile | 1 +
> drivers/block/mxc_ata.c | 149
> +++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 150 insertions(+), 0 deletions(-)
> create mode 100644 drivers/block/mxc_ata.c
Hi Marek,
> +
> +/* PIO timing table */
> +#define NR_PIO_SPECS 5
> +uint16_t pio_t0[NR_PIO_SPECS] = { 600, 383, 240, 180, 120 };
> +uint16_t pio_t1[NR_PIO_SPECS] = { 70, 50, 30, 30, 25 };
> +uint16_t pio_t2_8[NR_PIO_SPECS] = { 290, 290, 290, 80, 70 };
> +uint16_t pio_t2_16[NR_PIO_SPECS] = { 165, 125, 100, 80, 70 };
> +uint16_t pio_t2i[NR_PIO_SPECS] = { 40, 0, 0, 0, 0 };
> +uint16_t pio_t4[NR_PIO_SPECS] = { 30, 20, 15, 10, 10 };
> +uint16_t pio_t9[NR_PIO_SPECS] = { 20, 15, 10, 10, 10 };
> +uint16_t pio_tA[NR_PIO_SPECS] = { 50, 50, 50, 50, 50 };
As these array are only used here, they should be static.
> +
> + /* Write TIME_2R/AX/RDX/4 */
> + val = (((pio_t2_8[mode] + T) / T) << REG2OFF(&ata_cfg_regs->time_2r))
> |
Line too long, please fix globally.
> + (((pio_tA[mode] + T) / T + 2) <<
> REG2OFF(&ata_cfg_regs->time_ax)) |
Ditto.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de
=====================================================================
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
