Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1207878 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: staging: ft1000-pcmcia: remove support for v5 firmware Ondrej Zary <linux@rainbow-software.org> - 2015-08-14 21:30 +0200
    Re: staging: ft1000-pcmcia: remove support for v5 firmware Dan Carpenter <dan.carpenter@oracle.com> - 2015-08-17 16:40 +0200

#1207878 — Re: staging: ft1000-pcmcia: remove support for v5 firmware

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1208624

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web