Skip to content

Commit 2714129

Browse files
committed
Suggesting an error message solution
1 parent 0cc51c3 commit 2714129

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

adafruit_pca9685.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ def duty_cycle(self) -> int:
8282
@duty_cycle.setter
8383
def duty_cycle(self, value: int) -> None:
8484
if not 0 <= value <= 0xFFFF:
85-
raise ValueError("Out of range")
85+
raise ValueError(f"Out of range: value {value} not 0 <= value <= 0xFFFF")
8686

8787
if value == 0xFFFF:
8888
self._pca.pwm_regs[self._index] = (0x1000, 0)
@@ -136,7 +136,7 @@ def __init__(
136136
i2c_bus: I2C,
137137
*,
138138
address: int = 0x40,
139-
reference_clock_speed: int = 25000000
139+
reference_clock_speed: int = 25000000,
140140
) -> None:
141141
self.i2c_device = i2c_device.I2CDevice(i2c_bus, address)
142142
self.channels = PCAChannels(self)
@@ -152,6 +152,10 @@ def reset(self) -> None:
152152
@property
153153
def frequency(self) -> float:
154154
"""The overall PWM frequency in Hertz."""
155+
if self.prescale_reg < 3:
156+
raise ValueError(
157+
"The device pre_scale register (0xFE) was not read or returned a value < 3"
158+
)
155159
return self.reference_clock_speed / 4096 / self.prescale_reg
156160

157161
@frequency.setter

0 commit comments

Comments
 (0)