oem/smc: address linter

Signed-off-by: Matt Vandermeulen <[email protected]>
stmcginnis · Oct 15, 2024 · f776189 · f776189
1 parent 0e46b21
commit f776189
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/oem/smc/drive.go b/oem/smc/drive.go
@@ -61,15 +61,15 @@ func FromDrive(drive *redfish.Drive) (Drive, error) {
 	// in the Oem sections - certain models and bmc firmwares will mix
 	// these up, so we check both
 	smcDrive.indicateTarget = t.Actions.Oem.DriveIndicate.Target
-	if len(t.Actions.Oem.SmcDriveIndicate.Target) > 0 {
+	if t.Actions.Oem.SmcDriveIndicate.Target != "" {
 		smcDrive.indicateTarget = t.Actions.Oem.SmcDriveIndicate.Target
 	}
 
 	return smcDrive, nil
 }
 
 // Indicate will set the indicator light activity, true for on, false for off
-func (d Drive) Indicate(active bool) error {
+func (d *Drive) Indicate(active bool) error {
 	// Return a common error to let the user try falling back on the normal gofish path
 	if d.indicateTarget == "" {
 		return ErrActionNotSupported

diff --git a/oem/smc/drive_test.go b/oem/smc/drive_test.go
@@ -58,7 +58,7 @@ func TestSmcDriveOem(t *testing.T) {
 	}
 
 	if smcDrive.Temperature != 33 {
-		t.Errorf("unexpected oem drive temerature: %d", smcDrive.Temperature)
+		t.Errorf("unexpected oem drive temperature: %d", smcDrive.Temperature)
 	}
 
 	if smcDrive.indicateTarget != "/redfish/v1/Chassis/NVMeSSD.0.Group.0.StorageBackplane/Drives/Disk.Bay.22/Actions/Oem/Drive.Indicate" {