port-i386/51016: src/sys/arch/x86/x86/powernow.c:910: pointless assignment ?

[dcb314%hotmail.com@localhost]

>Number:         51016
>Category:       port-i386
>Synopsis:       src/sys/arch/x86/x86/powernow.c:910: pointless assignment ?
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    port-i386-maintainer
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Mar 26 15:40:00 +0000 2016
>Originator:     David Binderman
>Release:        cvs dated 20160326
>Organization:
>Environment:
>Description:

src/sys/arch/x86/x86/powernow.c:910]: (style) Assignment of function parameter has no effect outside the function.

Source code is

     if (cfid == fid || cvid == vid)
        freq = sc->sc_state->state_table[i].freq;


Suggest either remove the if and the assignment or use it in some way.

>How-To-Repeat:

>Fix: