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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <201508142114.06209.linux@rainbow-software.org>
Date:	Fri, 14 Aug 2015 21:14:05 +0200
From:	Ondrej Zary <linux@...nbow-software.org>
To:	Dan Carpenter <dan.carpenter@...cle.com>
Cc:	devel@...verdev.osuosl.org,
	Kernel development list <linux-kernel@...r.kernel.org>
Subject: 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/