<div dir="ltr">Hi all,<div><br></div><div>I hope that patch is ok, that was my first time using git send-email.</div><div><br></div><div>This patch fixes a bug in the Beaglebone BSP where the register offsets for the PWM outputs on header pins P9_14 and P9_16 are incorrectly calculated.</div><div><br></div><div>I'm about to start work on adding the QEP driver to the beaglebone BSP. Is this list where should I ask questions on BSP changes (coding style, API design etc), or should I do that on the users list?</div><div><br></div><div>Many thanks,</div><div>James Fitzsimons</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 5 Jul 2020 at 22:41, James Fitzsimons <<a href="mailto:james.fitzsimons@gmail.com">james.fitzsimons@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">---<br>
Fixed incorrect register offset values for EHRPWM1A on P9_14<br>
 and EHRPWM1B on P9_16<br>
<br>
 bsps/arm/beagle/pwm/pwm.c | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/bsps/arm/beagle/pwm/pwm.c b/bsps/arm/beagle/pwm/pwm.c<br>
index 0bc5d125bf..9a346995aa 100644<br>
--- a/bsps/arm/beagle/pwm/pwm.c<br>
+++ b/bsps/arm/beagle/pwm/pwm.c<br>
@@ -102,9 +102,9 @@ bool beagle_pwm_pinmux_setup(bbb_pwm_pin_t pin_no, BBB_PWMSS pwm_id)<br>
        } else if (pin_no == BBB_P8_36_1A) {<br>
          REG(AM335X_PADCONF_BASE + BBB_CONTROL_CONF_LCD_DATA(10)) = BBB_MUXMODE(BBB_P8_36_MUX_PWM);<br>
        } else if (pin_no == BBB_P9_14_1A) {<br>
-         REG(AM335X_PADCONF_BASE + BBB_CONTROL_CONF_GPMC_AD(2)) = BBB_MUXMODE(BBB_P9_14_MUX_PWM);<br>
+         REG(AM335X_PADCONF_BASE + BBB_CONTROL_CONF_GPMC_AD(18)) = BBB_MUXMODE(BBB_P9_14_MUX_PWM);<br>
        } else if (pin_no == BBB_P9_16_1B) {<br>
-         REG(AM335X_PADCONF_BASE + BBB_CONTROL_CONF_GPMC_AD(3)) = BBB_MUXMODE(BBB_P9_16_MUX_PWM);<br>
+         REG(AM335X_PADCONF_BASE + BBB_CONTROL_CONF_GPMC_AD(19)) = BBB_MUXMODE(BBB_P9_16_MUX_PWM);<br>
        } else { <br>
          is_valid = false;<br>
         }<br>
-- <br>
2.17.1<br>
<br>
</blockquote></div>