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
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
