Hi Detlev,

Detlev Zundel wrote:
> This patch adds advanced diagnosis functions for the inka4x0 board.
> 
> Signed-off-by: Andreas Pfefferle <a...@denx.de>
> Signed-off-by: Detlev Zundel <a...@denx.de>

Something went wrong here, duplicated e-mail address from
Andreas in your SoB. Also please see some style comments below.

> diff --git a/board/inka4x0/inkadiag.c b/board/inka4x0/inkadiag.c
> new file mode 100644
> index 0000000..bdbf652
> --- /dev/null
> +++ b/board/inka4x0/inkadiag.c
...
> +
> +static void inka_digio_set_output(unsigned int state, int which)
> +{
> +     if (which == 0) {
> +             /* other */
> +             CHECK_LED(0);
> +             CHECK_LED(1);
> +             CHECK_LED(2);
> +             CHECK_LED(3);
> +             CHECK_LED(4);
> +             CHECK_LED(5);
> +     } else {
> +             struct mpc5xxx_gpio *gpio = (struct mpc5xxx_gpio *)MPC5XXX_GPIO;
> +             if (which == 1) {

maybe we should add blank line before 'if' statement here?

...
> +static void ser_putc(volatile struct mpc5xxx_psc *psc, const char c)
> +{
> +     /* Wait 1 second for last character to go. */
> +     int i = 0;
> +     while (!(psc->psc_status & PSC_SR_TXEMP) && (i++ < 1000000/10))

add blank line before while().

...
> +static int ser_getc(volatile struct mpc5xxx_psc *psc)
> +{
> +     /* Wait for a character to arrive. */
> +     int i = 0;
> +     while (!(psc->psc_status & PSC_SR_RXRDY) && (i++ < 1000000/10))
> +             udelay(10);

add blank line before while().

...
> +U_BOOT_CMD(inkadiag, 6, 1, do_inkadiag,
> +        "inkadiag - inka diagnosis\n",
> +        "[inkadiag what ...]\n"
> +        "    - perform a diagnosis on inka hardware\n"
> +        "'inkadiag' performs hardware tests.\n\n");
> +
> +/* Relocate the command table functino pointers when running in RAM */

Typo, s/functino/function/

> +int inkadiag_init_r (void) {
> +     cmd_tbl_t *cmdtp;
> +
> +     for (cmdtp = &cmd_inkadiag_sub[0]; cmdtp !=
> +                  &cmd_inkadiag_sub[ARRAY_SIZE(cmd_inkadiag_sub)]; cmdtp++) {
> +             ulong addr;
> +             addr = (ulong) (cmdtp->cmd) + gd->reloc_off;

add blank line after declaration.

Best regards,
Anatolij
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to