Skip to content

Commit a22cd5f

Browse files
committed
Fix seg fault of PWM in Python 3.6 (#189)
PWM.set_frequency() and PWM.set_duty_cycle() were resulting in a seg fault inside the call to PyArg_ParseTupleAndKeywords(). Signed-off-by: Drew Fustini <drew@pdp7.com>
1 parent a666e50 commit a22cd5f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

source/py_pwm.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -217,8 +217,8 @@ static const char moduledocstring[] = "PWM functionality of a BeagleBone using P
217217
PyMethodDef pwm_methods[] = {
218218
{"start", (PyCFunction)py_start_channel, METH_VARARGS | METH_KEYWORDS, "Set up and start the PWM channel. channel can be in the form of 'P8_10', or 'EHRPWM2A'"},
219219
{"stop", (PyCFunction)py_stop_channel, METH_VARARGS | METH_KEYWORDS, "Stop the PWM channel. channel can be in the form of 'P8_10', or 'EHRPWM2A'"},
220-
{ "set_duty_cycle", (PyCFunction)py_set_duty_cycle, METH_VARARGS, "Change the duty cycle\ndutycycle - between 0.0 and 100.0" },
221-
{ "set_frequency", (PyCFunction)py_set_frequency, METH_VARARGS, "Change the frequency\nfrequency - frequency in Hz (freq > 0.0)" },
220+
{ "set_duty_cycle", (PyCFunction)py_set_duty_cycle, METH_VARARGS | METH_KEYWORDS, "Change the duty cycle\ndutycycle - between 0.0 and 100.0" },
221+
{ "set_frequency", (PyCFunction)py_set_frequency, METH_VARARGS | METH_KEYWORDS, "Change the frequency\nfrequency - frequency in Hz (freq > 0.0)" },
222222
{"cleanup", py_cleanup, METH_VARARGS, "Clean up by resetting all GPIO channels that have been used by this program to INPUT with no pullup/pulldown and no event detection"},
223223
//{"setwarnings", py_setwarnings, METH_VARARGS, "Enable or disable warning messages"},
224224
{NULL, NULL, 0, NULL}

0 commit comments

Comments
 (0)