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
On Fri, Aug 14, 2015 at 09:14:05PM +0200, Ondrej Zary wrote:
> 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.
Wikipedia says they are still used on some German trains [citation
needed] and parts of Dublin and few other places possibly...
https://en.wikipedia.org/wiki/Orthogonal_frequency-division_multiplexing#FLASH-OFDM
That page seems to be have been written about eight years ago.
regards,
dan carpenter