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

Groups > linux.kernel > #1207878

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

Show all headers | View raw

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/

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

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

csiph-web