From mboxrd@z Thu Jan  1 00:00:00 1970
From: Fabio Estevam <festevam@gmail.com>
Date: Fri, 30 Oct 2015 14:51:31 -0200
Subject: [U-Boot] [PATCH v4 16/16] spi: Add SPI NOR protection mechanism
In-Reply-To: <CAD6G_RQB=GW4eAu4peOcsqGJmZZCzFrN9KMwk8-BUXkd5dO0FA@mail.gmail.com>
References: <1445875918-14777-1-git-send-email-fabio.estevam@freescale.com>
 <1445875918-14777-16-git-send-email-fabio.estevam@freescale.com>
 <CAD6G_RQB=GW4eAu4peOcsqGJmZZCzFrN9KMwk8-BUXkd5dO0FA@mail.gmail.com>
Message-ID: <CAOMZO5Cq6hqUVyeH_b9NxTXnvgO-M6GxaUrEJjxKQr1cUXamJQ@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Jagan,

On Fri, Oct 30, 2015 at 2:28 PM, Jagan Teki <jteki@openedev.com> wrote:

>> +static int do_spi_protect(int argc, char * const argv[])
>> +{
>> +       int ret = 0;
>> +       loff_t start, len;
>> +       bool prot = false;
>> +
>> +       if (argc != 4)
>> +               return -1;
>> +
>> +       if (!str2off(argv[2], &start)) {
>> +               puts("start sector is not a valid number\n");
>> +               return 1;
>> +       }
>> +
>> +       if (!str2off(argv[3], &len)) {
>> +               puts("len is not a valid number\n");
>> +               return 1;
>> +       }
>> +
>> +       if (strcmp(argv[1], "lock") == 0)
>> +               prot = true;
>> +       else if (strcmp(argv[1], "unlock") == 0)
>> +               prot = false;
>> +       else
>
> Don't we have is_locked command from user? may be we can all this one as well.

Sorry, I did not understand the suggestion here.

Looks like you are happy with patches 1 to 15 of this series.

Could you please apply patches 1 to 15 and then I rework only this last one?