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 2020/11/27 11:19:12 UTC
[incubator-nuttx] 05/48: arch/arm/src/stm32/stm32_pwm.c: Fix syslog
formats
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 96f012cc733bec919a2498681b3634f5e67c7912
Author: YAMAMOTO Takashi <ya...@midokura.com>
AuthorDate: Thu Nov 26 07:04:09 2020 +0900
arch/arm/src/stm32/stm32_pwm.c: Fix syslog formats
---
arch/arm/src/stm32/stm32_pwm.c | 28 ++++++++++++++++------------
1 file changed, 16 insertions(+), 12 deletions(-)
diff --git a/arch/arm/src/stm32/stm32_pwm.c b/arch/arm/src/stm32/stm32_pwm.c
index 7bad56f..7dc9f3e 100644
--- a/arch/arm/src/stm32/stm32_pwm.c
+++ b/arch/arm/src/stm32/stm32_pwm.c
@@ -42,6 +42,7 @@
#include <nuttx/config.h>
+#include <inttypes.h>
#include <stdint.h>
#include <stdio.h>
#include <assert.h>
@@ -2291,7 +2292,7 @@ static int pwm_duty_update(FAR struct pwm_lowerhalf_s *dev, uint8_t channel,
DEBUGASSERT(priv != NULL);
- pwminfo("TIM%u channel: %u duty: %08x\n",
+ pwminfo("TIM%u channel: %u duty: %08" PRIx32 "\n",
priv->timid, channel, duty);
#ifndef CONFIG_PWM_MULTICHAN
@@ -2310,7 +2311,7 @@ static int pwm_duty_update(FAR struct pwm_lowerhalf_s *dev, uint8_t channel,
ccr = b16toi(duty * reload + b16HALF);
- pwminfo("ccr: %u\n", ccr);
+ pwminfo("ccr: %" PRIu32 "\n", ccr);
/* Write corresponding CCR register */
@@ -2418,8 +2419,9 @@ static int pwm_frequency_update(FAR struct pwm_lowerhalf_s *dev,
reload--;
}
- pwminfo("TIM%u PCLK: %u frequency: %u TIMCLK: %u "
- "prescaler: %u reload: %u\n",
+ pwminfo("TIM%u PCLK: %" PRIu32" frequency: %" PRIu32
+ " TIMCLK: %" PRIu32 " "
+ "prescaler: %" PRIu32 " reload: %" PRIu32 "\n",
priv->timid, priv->pclk, frequency, timclk, prescaler, reload);
/* Set the reload and prescaler values */
@@ -3668,10 +3670,10 @@ static int pwm_timer(FAR struct pwm_lowerhalf_s *dev,
DEBUGASSERT(priv != NULL && info != NULL);
#if defined(CONFIG_PWM_MULTICHAN)
- pwminfo("TIM%u frequency: %u\n",
+ pwminfo("TIM%u frequency: %" PRIu32 "\n",
priv->timid, info->frequency);
#else
- pwminfo("TIM%u channel: %u frequency: %u duty: %08x\n",
+ pwminfo("TIM%u channel: %u frequency: %" PRIu32 " duty: %08" PRIx32 "\n",
priv->timid, priv->channels[0].channel,
info->frequency, info->duty);
#endif
@@ -4078,7 +4080,8 @@ static int pwm_set_apb_clock(FAR struct stm32_pwmtimer_s *priv, bool on)
/* Enable/disable APB 1/2 clock for timer */
- pwminfo("RCC_APBxENR base: %08x bits: %04x\n", regaddr, en_bit);
+ pwminfo("RCC_APBxENR base: %08" PRIx32 " bits: %04" PRIx32 "\n",
+ regaddr, en_bit);
if (on)
{
@@ -4146,7 +4149,7 @@ static int pwm_setup(FAR struct pwm_lowerhalf_s *dev)
pincfg = priv->channels[i].out1.pincfg;
if (pincfg != 0)
{
- pwminfo("pincfg: %08x\n", pincfg);
+ pwminfo("pincfg: %08" PRIx32 "\n", pincfg);
stm32_configgpio(pincfg);
pwm_dumpgpio(pincfg, "PWM setup");
@@ -4165,7 +4168,7 @@ static int pwm_setup(FAR struct pwm_lowerhalf_s *dev)
if (pincfg != 0)
{
- pwminfo("pincfg: %08x\n", pincfg);
+ pwminfo("pincfg: %08" PRIx32 "\n", pincfg);
stm32_configgpio(pincfg);
pwm_dumpgpio(pincfg, "PWM setup");
@@ -4240,7 +4243,7 @@ static int pwm_shutdown(FAR struct pwm_lowerhalf_s *dev)
pincfg = priv->channels[i].out1.pincfg;
if (pincfg != 0)
{
- pwminfo("pincfg: %08x\n", pincfg);
+ pwminfo("pincfg: %08" PRIx32 "\n", pincfg);
pincfg &= (GPIO_PORT_MASK | GPIO_PIN_MASK);
pincfg |= PINCFG_DEFAULT;
@@ -4252,7 +4255,7 @@ static int pwm_shutdown(FAR struct pwm_lowerhalf_s *dev)
pincfg = priv->channels[i].out2.pincfg;
if (pincfg != 0)
{
- pwminfo("pincfg: %08x\n", pincfg);
+ pwminfo("pincfg: %08" PRIx32 "\n", pincfg);
pincfg &= (GPIO_PORT_MASK | GPIO_PIN_MASK);
pincfg |= PINCFG_DEFAULT;
@@ -4561,7 +4564,8 @@ static int pwm_stop(FAR struct pwm_lowerhalf_s *dev)
putreg32(regval, regaddr);
leave_critical_section(flags);
- pwminfo("regaddr: %08x resetbit: %08x\n", regaddr, resetbit);
+ pwminfo("regaddr: %08" PRIx32 " resetbit: %08" PRIx32 "\n",
+ regaddr, resetbit);
pwm_dumpregs(dev, "After stop");
errout: