* Re: staging: ft1000-pcmcia: remove support for v5 firmware
[not found] <20150814085017.GA30728@mwanda>
@ 2015-08-14 19:14 ` Ondrej Zary
2015-08-17 14:35 ` Dan Carpenter
0 siblings, 1 reply; 2+ messages in thread
From: Ondrej Zary @ 2015-08-14 19:14 UTC (permalink / raw)
To: Dan Carpenter; +Cc: devel, Kernel development list
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
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: staging: ft1000-pcmcia: remove support for v5 firmware
2015-08-14 19:14 ` staging: ft1000-pcmcia: remove support for v5 firmware Ondrej Zary
@ 2015-08-17 14:35 ` Dan Carpenter
0 siblings, 0 replies; 2+ messages in thread
From: Dan Carpenter @ 2015-08-17 14:35 UTC (permalink / raw)
To: Ondrej Zary; +Cc: devel, Kernel development list
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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-08-17 14:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20150814085017.GA30728@mwanda>
2015-08-14 19:14 ` staging: ft1000-pcmcia: remove support for v5 firmware Ondrej Zary
2015-08-17 14:35 ` Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox