fix: Correct parsing of ImpinjPeakRSSI parameter

[ajcasagrande]

Signed-off-by: Anthony Casagrande anthony.j.casagrande@intel.com

PR Type

What kind of change does this PR introduce?

• Bugfix

Other information

Fixed a typo. ImpinjEnablePeakRSSI is what you send to turn on reporting of that value. ImpinjPeakRSSI is the actual subtype of the reported parameter. Verified working on my SpeedwayR420 RFID reader.

[ajcasagrande]

Definitions:

app-rfid-llrp-inventory/internal/llrp/vendors.go

Lines 40 to 42 in 64ced19

	const (
	ImpinjEnablePeakRSSI = ImpinjParamSubtype(53)
	ImpinjPeakRSSI = ImpinjParamSubtype(57)

Verification:

for _, c := range rt.Custom {
    fmt.Printf("Got Custom -- vendor: %s (%d), subtype: %s (%d)\n",
	    VendorPEN(c.VendorID).String(), c.VendorID, ImpinjParamSubtype(c.Subtype).String(), c.Subtype)
    if c.Is(PENImpinj, ImpinjPeakRSSI) && len(c.Data) == 2 {
	    fmt.Printf("Got ImpinjPeakRSSI: %v\n", float64(int16(binary.BigEndian.Uint16(c.Data))) / 100.0)
	    return float64(int16(binary.BigEndian.Uint16(c.Data))) / 100.0, true // dBm x100
    }
}

Got Custom -- vendor: PENImpinj (25882), subtype: ImpinjPeakRSSI (57)
Got ImpinjPeakRSSI: -19.5
Got Custom -- vendor: PENImpinj (25882), subtype: ImpinjPeakRSSI (57)
Got ImpinjPeakRSSI: -24
Got Custom -- vendor: PENImpinj (25882), subtype: ImpinjPeakRSSI (57)
Got ImpinjPeakRSSI: -23.5
Got Custom -- vendor: PENImpinj (25882), subtype: ImpinjPeakRSSI (57)
Got ImpinjPeakRSSI: -19.5
Got Custom -- vendor: PENImpinj (25882), subtype: ImpinjPeakRSSI (57)
Got ImpinjPeakRSSI: -24.5
Got Custom -- vendor: PENImpinj (25882), subtype: ImpinjPeakRSSI (57)
Got ImpinjPeakRSSI: -23.5

[lenny-goodell]

LGTM