You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2021/12/28 12:28:00 UTC
[incubator-nuttx] branch master updated (cf2dfa8 -> baafd33)
This is an automated email from the ASF dual-hosted git repository.
xiaoxiang pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git.
from cf2dfa8 arch/arm/assert: move the arm_assert to common code
new 934a797 Use userspace chosen channel numbers when starting bl602 pwm
new baafd33 Fix a typo in description of pwm_expired
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
arch/risc-v/src/bl602/bl602_pwm_lowerhalf.c | 6 +++---
drivers/timers/pwm.c | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
[incubator-nuttx] 01/02: Use userspace chosen channel numbers when starting bl602 pwm
Posted by xi...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git
commit 934a79736aeeb5a9529e8b1d81d3ed80ec61f52a
Author: Norman Rasmussen <no...@rasmussen.co.za>
AuthorDate: Tue Dec 28 02:39:34 2021 -0800
Use userspace chosen channel numbers when starting bl602 pwm
commit 2889315c207fcf7bb0ccd46f9955ba1fe323ebed added support for pwm
but didn't read the channel numbers provided by user-space. They should
be, otherwise it's not possible to start a sub-set of channels that are
not the first "n" channels.
---
arch/risc-v/src/bl602/bl602_pwm_lowerhalf.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/risc-v/src/bl602/bl602_pwm_lowerhalf.c b/arch/risc-v/src/bl602/bl602_pwm_lowerhalf.c
index 1bce6c3..63ec627 100644
--- a/arch/risc-v/src/bl602/bl602_pwm_lowerhalf.c
+++ b/arch/risc-v/src/bl602/bl602_pwm_lowerhalf.c
@@ -381,9 +381,9 @@ static int bl602_pwm_start(struct pwm_lowerhalf_s *dev,
break;
}
- bl602_pwm_freq(priv, i, info->frequency);
- bl602_pwm_duty(priv, i, info->channels[i].duty);
- pwm_channel_enable(i);
+ bl602_pwm_freq(priv, chan, info->frequency);
+ bl602_pwm_duty(priv, chan, info->channels[i].duty);
+ pwm_channel_enable(chan);
}
#else
bl602_pwm_freq(priv, 0, info->frequency);
[incubator-nuttx] 02/02: Fix a typo in description of pwm_expired
Posted by xi...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git
commit baafd33cbe01032f6e7d196d6ac2ef2b158862d8
Author: Norman Rasmussen <no...@rasmussen.co.za>
AuthorDate: Tue Dec 28 02:53:39 2021 -0800
Fix a typo in description of pwm_expired
---
drivers/timers/pwm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/timers/pwm.c b/drivers/timers/pwm.c
index 0b2f02e..8652e3f 100644
--- a/drivers/timers/pwm.c
+++ b/drivers/timers/pwm.c
@@ -629,7 +629,7 @@ int pwm_register(FAR const char *path, FAR struct pwm_lowerhalf_s *dev)
* 1. The upper half driver calls the start method, providing the lower
* half driver with the pulse train characteristics. If a fixed
* number of pulses is required, the 'count' value will be nonzero.
- * 2. The lower half driver's start() methoc must verify that it can
+ * 2. The lower half driver's start() method must verify that it can
* support the request pulse train (frequency, duty, AND pulse count).
* If it cannot, it should return an error. If the pulse count is
* non-zero, it should set up the hardware for that number of pulses