[U-Boot] [PATCH 1/5] x86: ich-spi: Remove spi_write_protect_region()

Bin Meng bmeng.cn at gmail.com
Thu Aug 24 03:17:04 UTC 2017


On Wed, Aug 16, 2017 at 2:18 PM, Stefan Roese <sr at denx.de> wrote:
> On 16.08.2017 07:38, Bin Meng wrote:
>>
>> This routine is not called anywhere.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>   drivers/spi/ich.c | 50
>> --------------------------------------------------
>>   1 file changed, 50 deletions(-)
>>
>> diff --git a/drivers/spi/ich.c b/drivers/spi/ich.c
>> index bf2e99b..46dd9a8 100644
>> --- a/drivers/spi/ich.c
>> +++ b/drivers/spi/ich.c
>> @@ -539,56 +539,6 @@ static int ich_spi_xfer(struct udevice *dev, unsigned
>> int bitlen,
>>         return 0;
>>   }
>>   -/*
>> - * This uses the SPI controller from the Intel Cougar Point and Panther
>> Point
>> - * PCH to write-protect portions of the SPI flash until reboot. The
>> changes
>> - * don't actually take effect until the HSFS[FLOCKDN] bit is set, but
>> that's
>> - * done elsewhere.
>> - */
>> -int spi_write_protect_region(struct udevice *dev, uint32_t lower_limit,
>> -                            uint32_t length, int hint)
>> -{
>> -       struct udevice *bus = dev->parent;
>> -       struct ich_spi_priv *ctlr = dev_get_priv(bus);
>> -       uint32_t tmplong;
>> -       uint32_t upper_limit;
>> -
>> -       if (!ctlr->pr) {
>> -               printf("%s: operation not supported on this chipset\n",
>> -                      __func__);
>> -               return -ENOSYS;
>> -       }
>> -
>> -       if (length == 0 ||
>> -           lower_limit > (0xFFFFFFFFUL - length) + 1 ||
>> -           hint < 0 || hint > 4) {
>> -               printf("%s(0x%x, 0x%x, %d): invalid args\n", __func__,
>> -                      lower_limit, length, hint);
>> -               return -EPERM;
>> -       }
>> -
>> -       upper_limit = lower_limit + length - 1;
>> -
>> -       /*
>> -        * Determine bits to write, as follows:
>> -        *  31     Write-protection enable (includes erase operation)
>> -        *  30:29  reserved
>> -        *  28:16  Upper Limit (FLA address bits 24:12, with 11:0 ==
>> 0xfff)
>> -        *  15     Read-protection enable
>> -        *  14:13  reserved
>> -        *  12:0   Lower Limit (FLA address bits 24:12, with 11:0 ==
>> 0x000)
>> -        */
>> -       tmplong = 0x80000000 |
>> -               ((upper_limit & 0x01fff000) << 4) |
>> -               ((lower_limit & 0x01fff000) >> 12);
>> -
>> -       printf("%s: writing 0x%08x to %p\n", __func__, tmplong,
>> -              &ctlr->pr[hint]);
>> -       ctlr->pr[hint] = tmplong;
>> -
>> -       return 0;
>> -}
>> -
>>   static int ich_spi_probe(struct udevice *dev)
>>   {
>>         struct ich_spi_platdata *plat = dev_get_platdata(dev);
>>
>
> Reviewed-by: Stefan Roese <sr at denx.de>

applied to u-boot-x86, thanks!


More information about the U-Boot mailing list