Re: staging: ft1000-pcmcia: remove support for v5 firmware

Ondrej Zary
2015-08-14T19:20:36Z

On Friday 14 August 2015 10:50:17 Dan Carpenter wrote:
> Hello Ondrej Zary,
>
> The patch e161a440af56: "staging: ft1000-pcmcia: remove support for
> v5 firmware" from Jul 1, 2011, leads to the following static checker
> warning:
>
> drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c:513 card_download()
> warn: condition implies we already warned.
>
> drivers/staging/ft1000/ft1000-pcmcia/ft1000_dnld.c
> 319 file_version = *(long *)pFileStart;
> 320 if (file_version != 6) {
>
> We only support version 6 now.
>
> 321 pr_err("unsupported firmware version %ld\n",
> file_version); 322 Status = FAILURE;
> 323 }
> 324
>
> [ snip ]
>
> 513 if (file_version == 5) {
>
> I think we can just delete this side of the if statement?
>
> 514 /*
> 515 * Position ASIC
> DPRAM auto-increment pointer. 516
> */
> 517
> ft1000_write_reg(dev, 518
> FT1000_REG_DPRAM_ADDR, 519
> DWNLD_PS_HDR_LOC); 520
> 521 for (; word_length
> > 0; word_length--) { /* In words */ 522
> temp = ntohs(*pUsData); 523
> ft1000_write_reg(dev, 524
>
> FT1000_REG_DPRAM_DATA, 525
> temp); 526
> pUsData++; 527
> }
> 528 } else {
> 529 /*
> 530 * Position ASIC
> DPRAM auto-increment pointer. 531
> */
> 532
> outw(DWNLD_MAG_PS_HDR_LOC, 533
> dev->base_addr + 534
> FT1000_REG_DPRAM_ADDR); 535
> if (word_length & 0x01) 536
> word_length++; 537
> 538 word_length =
> word_length / 2; 539
> 540 for (; word_length
> > 0; word_length--) { /* In words */ 541
> templong = *pUsData++; 542
> templong |= 543
> (*pUsData++ << 16); 544
> outl(templong, 545
> dev->base_addr + 546
>
> FT1000_REG_MAG_DPDATAL); 547
> }
> 548 }
>
> regards,
> dan carpenter

Most probably yes.

However, the driver will be useless soon as the Flash OFDM network is being
decommissioned here in Slovakia. AFAIK, this technology is not used anywhere
else in the world (the 2nd country was Finland). The ft1000 devices will be
left with no signal.

--
Ondrej Zary