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
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
