Patchwork [1/2] dediprog: Reimplement target chip option

login
register
about
Submitter Nico Huber
Date 2016-05-04 11:37:10
Message ID <1462361831-30320-1-git-send-email-nico.huber@secunet.com>
Download mbox | patch
Permalink /patch/4441/
State Accepted
Headers show

Comments

Nico Huber - 2016-05-04 11:37:10
Signed-off-by: Nico Huber <nico.huber@secunet.com>
---
 dediprog.c | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)
David Hendricks - 2016-05-07 21:32:54
Looks good to me.

Acked-by: David Hendricks <david.hendricks@gmail.com>

On Wed, May 4, 2016 at 4:37 AM, Nico Huber <nico.huber@secunet.com> wrote:

> Signed-off-by: Nico Huber <nico.huber@secunet.com>
> ---
>  dediprog.c | 17 ++++++++++++++---
>  1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/dediprog.c b/dediprog.c
> index 019de46..b7276e5 100644
> --- a/dediprog.c
> +++ b/dediprog.c
> @@ -942,7 +942,7 @@ int dediprog_init(void)
>         int spispeed_idx = 1;
>         int millivolt = 3500;
>         long usedevice = 0;
> -       long target = 1;
> +       long target = FLASH_TYPE_APPLICATION_FLASH_1;
>         int i, ret;
>
>         spispeed = extract_programmer_param("spispeed");
> @@ -1014,7 +1014,18 @@ int dediprog_init(void)
>                         free(target_str);
>                         return 1;
>                 }
> -               msg_pinfo("Using target %li.\n", target);
> +               switch (target) {
> +               case 1:
> +                       msg_pinfo("Using target %s.\n",
> "FLASH_TYPE_APPLICATION_FLASH_1");
> +                       target = FLASH_TYPE_APPLICATION_FLASH_1;
> +                       break;
> +               case 2:
> +                       msg_pinfo("Using target %s.\n",
> "FLASH_TYPE_APPLICATION_FLASH_2");
> +                       target = FLASH_TYPE_APPLICATION_FLASH_2;
> +                       break;
> +               default:
> +                       break;
> +               }
>         }
>         free(target_str);
>
> @@ -1073,7 +1084,7 @@ int dediprog_init(void)
>         dediprog_set_leds(LED_ALL);
>
>         /* Select target/socket, frequency and VCC. */
> -       if (set_target_flash(FLASH_TYPE_APPLICATION_FLASH_1) ||
> +       if (set_target_flash(target) ||
>             dediprog_set_spi_speed(spispeed_idx) ||
>             dediprog_set_spi_voltage(millivolt)) {
>                 dediprog_set_leds(LED_ERROR);
> --
> 2.7.0
>
>
> _______________________________________________
> flashrom mailing list
> flashrom@flashrom.org
> https://www.flashrom.org/mailman/listinfo/flashrom
>

Patch

diff --git a/dediprog.c b/dediprog.c
index 019de46..b7276e5 100644
--- a/dediprog.c
+++ b/dediprog.c
@@ -942,7 +942,7 @@  int dediprog_init(void)
 	int spispeed_idx = 1;
 	int millivolt = 3500;
 	long usedevice = 0;
-	long target = 1;
+	long target = FLASH_TYPE_APPLICATION_FLASH_1;
 	int i, ret;
 
 	spispeed = extract_programmer_param("spispeed");
@@ -1014,7 +1014,18 @@  int dediprog_init(void)
 			free(target_str);
 			return 1;
 		}
-		msg_pinfo("Using target %li.\n", target);
+		switch (target) {
+		case 1:
+			msg_pinfo("Using target %s.\n", "FLASH_TYPE_APPLICATION_FLASH_1");
+			target = FLASH_TYPE_APPLICATION_FLASH_1;
+			break;
+		case 2:
+			msg_pinfo("Using target %s.\n", "FLASH_TYPE_APPLICATION_FLASH_2");
+			target = FLASH_TYPE_APPLICATION_FLASH_2;
+			break;
+		default:
+			break;
+		}
 	}
 	free(target_str);
 
@@ -1073,7 +1084,7 @@  int dediprog_init(void)
 	dediprog_set_leds(LED_ALL);
 
 	/* Select target/socket, frequency and VCC. */
-	if (set_target_flash(FLASH_TYPE_APPLICATION_FLASH_1) ||
+	if (set_target_flash(target) ||
 	    dediprog_set_spi_speed(spispeed_idx) ||
 	    dediprog_set_spi_voltage(millivolt)) {
 		dediprog_set_leds(LED_ERROR);