1. 1
2. 2
3. 3
4. 4
5. 5
6. 6
7. 7
8. 8
9. 9
10. 10
11. 11
12. 12
13. 13
14. 14
15. 15
16. 16
17. 17
18. 18
19. 19
20. 20
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
35. 35
36. 36
37. 37
38. 38
39. 39
40. 40
41. 41
42. 42
43. 43
44. 44
45. 45
46. 46
47. 47
48. 48
49. 49
50. 50
51. 51
52. 52
53. 53
54. 54
55. 55
56. 56
57. 57
58. 58
59. 59
60. 60
61. 61
62. 62
63. 63
64. 64
65. 65
66. 66
67. 67
68. 68
69. 69
70. 70

diff --git a/rocm_smi.py b/rocm_smi.py
index 7b36028..8ba2bd0 100755
--- a/rocm_smi.py
+++ b/rocm_smi.py
@@ -739,7 +739,7 @@ def getCurrentClock(device, clock, clocktype):
     # Hack: In the kernel, FCLK doesn't have an * at all if DPM is disabled.
     # If there is only 1 speed (1 line total, meaning 0 levels), just print it
     if len(currClocks.splitlines()) == 1 and len(currClocks) > 1:
-        if clocktype is 'freq':
+        if clocktype == 'freq':
             if currClocks.find('DPM disabled'):
                 logging.debug('Only 1 level for clock %s; DPM is disabled for this specific clock' % clock)
             return currClocks.split(' *')[0][3:]
@@ -824,7 +824,7 @@ def getVersion(deviceList, component):
     deviceList -- List of DRM devices (can be a single-item list)
     component - Component (currently only driver)
     """
-    if component is 'driver':
+    if component == 'driver':
         # Only 1 version, so report it for GPU 0
         driver = getSysfsValue(None, 'driver')
         if driver is None:
@@ -846,10 +846,10 @@ def getRetiredPages(device, retiredType):
         return None
     for line in pages.split('\n'):
         pgType = line.split(' : ')[-1]
-        if (retiredType is 'all' or \
-           retiredType is 'retired' and pgType is 'R' or \
-           retiredType is 'pending' and pgType is 'P' or \
-           retiredType is 'unreservable' and pgType is 'F'):
+        if (retiredType == 'all' or \
+           retiredType == 'retired' and pgType == 'R' or \
+           retiredType == 'pending' and pgType == 'P' or \
+           retiredType == 'unreservable' and pgType == 'F'):
             returnPages += '\n%s' % line
     return returnPages.lstrip('\n')
 
@@ -1424,7 +1424,7 @@ def showVersion(deviceList, component):
     if component not in validVersionComponents:
         printLog(device, 'Unable to display version information for unsupported component %s' % component)
         return
-    if component is 'driver':
+    if component == 'driver':
         driver = getVersion(deviceList, component)
         printSysLog('%s version: %s' % (component.capitalize(), driver))
 
@@ -1824,9 +1824,9 @@ def showRetiredPages(deviceList, retiredType='all'):
                 addr = line.split(' : ')[0]
                 size = line.split(' : ')[1]
                 ptype = line.split(' : ')[2]
-                if ptype is 'R':
+                if ptype == 'R':
                     pgType = 'Retired'
-                elif ptype is 'P':
+                elif ptype == 'P':
                     pgType = 'Pending'
                 else:
                     pgType = 'Unreservable'
@@ -2281,9 +2281,9 @@ def setClockRange(deviceList, clktype, level, value, autoRespond):
         logging.error('Non-integer characters are present in %s', value)
         RETCODE = 1
         return
-    if clkType is 'sclk':
+    if clkType == 'sclk':
         sysvalue = 's %s %s' % (level, value)
-    elif clkType is 'mclk':
+    elif clkType == 'mclk':
         sysvalue = 'm %s %s' % (level, value)
     else:
         printLogNoDev('Invalid clock type %s' % clkType)