tiva: Fix nxstyle warnings

arch/arm/src/tiva/tm4c/tm4c129_sysctrl.c:

    * Fix nxstyle warnings. No functional changes.

arch/arm/src/tiva/tm4c/tm4c_gpio.c:

    * Fix nxstyle warnings. No functional changes.

arch/arm/src/tiva/tm4c/tm4c_gpio.h:

    * Fix nxstyle warnings. No functional changes.
This commit is contained in:
Nathan Hartman 2020-09-30 17:27:52 -04:00 committed by Xiang Xiao
parent d6210fcd84
commit 4ac2c73d82
3 changed files with 53 additions and 42 deletions

View File

@ -186,10 +186,17 @@ static uint32_t tiva_vco_frequency(uint32_t pllfreq0, uint32_t pllfreq1)
/* Extract all of the values from the hardware register values. */
mfrac = (pllfreq0 & SYSCON_PLLFREQ0_MFRAC_MASK) >> SYSCON_PLLFREQ0_MFRAC_SHIFT;
mint = (pllfreq0 & SYSCON_PLLFREQ0_MINT_MASK) >> SYSCON_PLLFREQ0_MINT_SHIFT;
q = ((pllfreq1 & SYSCON_PLLFREQ1_Q_MASK) >> SYSCON_PLLFREQ1_Q_SHIFT) + 1;
n = ((pllfreq1 & SYSCON_PLLFREQ1_N_MASK) >> SYSCON_PLLFREQ1_N_SHIFT) + 1;
mfrac = (pllfreq0 & SYSCON_PLLFREQ0_MFRAC_MASK) >>
SYSCON_PLLFREQ0_MFRAC_SHIFT;
mint = (pllfreq0 & SYSCON_PLLFREQ0_MINT_MASK) >>
SYSCON_PLLFREQ0_MINT_SHIFT;
q = ((pllfreq1 & SYSCON_PLLFREQ1_Q_MASK) >>
SYSCON_PLLFREQ1_Q_SHIFT) + 1;
n = ((pllfreq1 & SYSCON_PLLFREQ1_N_MASK) >>
SYSCON_PLLFREQ1_N_SHIFT) + 1;
/* Algorithm:
*
@ -391,8 +398,8 @@ void tiva_clock_configure(void)
uint32_t pllfreq0;
uint32_t pllfreq1;
/* Set the clocking to run with the default settings provided in the board.h
* header file
/* Set the clocking to run with the default settings provided in the
* board.h header file
*/
pllfreq0 = M2PLLFREQ0(BOARD_PLL_MINT, BOARD_PLL_MFRAC);

View File

@ -687,9 +687,9 @@ static inline void tiva_interrupt(pinconfig_t pinconfig)
* corresponding bit in the GPIO Interrupt Sense (GPIO IS) register ... is
* set to detect edges, bits set to High in GPIO IBE configure the
* corresponding pin to detect both rising and falling edges, regardless
* of the corresponding bit in the GPIO Interrupt Event (GPIO IEV) register ...
* Clearing a bit configures the pin to be controlled by GPIOIEV. All bits
* are cleared by a reset.
* of the corresponding bit in the GPIO Interrupt Event (GPIO IEV)
* register ... Clearing a bit configures the pin to be controlled by
* GPIOIEV. All bits are cleared by a reset.
*/
modifyreg32(base + TIVA_GPIO_IBE_OFFSET, ibeclr, ibeset);
@ -702,12 +702,12 @@ static inline void tiva_interrupt(pinconfig_t pinconfig)
modifyreg32(base + TIVA_GPIO_IS_OFFSET, isclr, isset);
/* "The GPIOIEV register is the interrupt event register. Bits set to
* High in GPIO IEV configure the corresponding pin to detect rising edges
* or high levels, depending on the corresponding bit value in the GPIO
* Interrupt Sense (GPIO IS) register... Clearing a bit configures the pin to
* detect falling edges or low levels, depending on the corresponding bit
* value in GPIOIS. All bits are cleared by a reset.
/* "The GPIOIEV register is the interrupt event register. Bits set to High
* in GPIO IEV configure the corresponding pin to detect rising edges or
* high levels, depending on the corresponding bit value in the GPIO
* Interrupt Sense (GPIO IS) register... Clearing a bit configures the pin
* to detect falling edges or low levels, depending on the corresponding
* bit value in GPIOIS. All bits are cleared by a reset.
*/
modifyreg32(base + TIVA_GPIO_IEV_OFFSET, ievclr, ievset);
@ -717,13 +717,13 @@ static inline void tiva_interrupt(pinconfig_t pinconfig)
gpioinfo("reg expected actual: [interrupt type=%d]\n", inttype);
regval = (getreg32(base+TIVA_GPIO_IS_OFFSET) & pin) ? pin : 0;
regval = (getreg32(base + TIVA_GPIO_IS_OFFSET) & pin) ? pin : 0;
gpioinfo("IS 0x%08x 0x%08x\n", isset, regval);
regval = (getreg32(base+TIVA_GPIO_IBE_OFFSET) & pin) ? pin : 0;
regval = (getreg32(base + TIVA_GPIO_IBE_OFFSET) & pin) ? pin : 0;
gpioinfo("IBE 0x%08x 0x%08x\n", ibeset, regval);
regval = (getreg32(base+TIVA_GPIO_IEV_OFFSET) & pin) ? pin : 0;
regval = (getreg32(base + TIVA_GPIO_IEV_OFFSET) & pin) ? pin : 0;
gpioinfo("IEV 0x%08x 0x%08x\n", ievset, regval);
#endif
}
@ -824,9 +824,9 @@ int tiva_configgpio(pinconfig_t pinconfig)
tiva_gpiofunc(base, pinno, &g_funcbits[0]);
tiva_portcontrol(base, pinno, pinconfig, &g_funcbits[0]);
/* Then set up pad strengths and pull-ups. These setups should be done before
* setting up the function because some function settings will over-ride these
* user options.
/* Then set up pad strengths and pull-ups. These setups should be done
* before setting up the function because some function settings will
* over-ride these user options.
*/
tiva_gpiopadstrength(base, pin, pinconfig);
@ -880,19 +880,20 @@ void tiva_gpiowrite(pinconfig_t pinconfig, bool value)
base = tiva_gpiobaseaddress(port);
/* "The GPIO DATA register is the data register. In software control mode,
* values written in the GPIO DATA register are transferred onto the GPIO
* port pins if the respective pins have been configured as outputs through
* the GPIO Direction (GPIO DIR) register ...
/* "The GPIO DATA register is the data register. In software control
* mode, values written in the GPIO DATA register are transferred onto
* the GPIO port pins if the respective pins have been configured as
* outputs through the GPIO Direction (GPIO DIR) register ...
*
* "In order to write to GPIO DATA, the corresponding bits in the mask,
* resulting from the address bus bits [9:2], must be High. Otherwise, the
* bit values remain unchanged by the write.
* resulting from the address bus bits [9:2], must be High. Otherwise,
* the bit values remain unchanged by the write.
*
* "... All bits are cleared by a reset."
*/
putreg32((uint32_t)value << pinno, base + TIVA_GPIO_DATA_OFFSET + (1 << (pinno + 2)));
putreg32((uint32_t)value << pinno,
base + TIVA_GPIO_DATA_OFFSET + (1 << (pinno + 2)));
}
/****************************************************************************
@ -918,18 +919,19 @@ bool tiva_gpioread(pinconfig_t pinconfig)
base = tiva_gpiobaseaddress(port);
/* "... the values read from this register are determined for each bit
* by the mask bit derived from the address used to access the data register,
* bits [9:2]. Bits that are 1 in the address mask cause the corresponding
* bits in GPIODATA to be read, and bits that are 0 in the address mask cause
* the corresponding bits in GPIO DATA to be read as 0, regardless of their
* value.
/* "... the values read from this register are determined for each bit by
* the mask bit derived from the address used to access the data
* register, bits [9:2]. Bits that are 1 in the address mask cause the
* corresponding bits in GPIODATA to be read, and bits that are 0 in the
* address mask cause the corresponding bits in GPIO DATA to be read as
* 0, regardless of their value.
*
* "A read from GPIO DATA returns the last bit value written if the respective
* pins are configured as outputs, or it returns the value on the
* corresponding input pin when these are configured as inputs. All bits
* are cleared by a reset."
* "A read from GPIO DATA returns the last bit value written if the
* respective pins are configured as outputs, or it returns the value on
* the corresponding input pin when these are configured as inputs. All
* bits are cleared by a reset."
*/
return (getreg32(base + TIVA_GPIO_DATA_OFFSET + (1 << (pinno + 2))) != 0);
}
@ -978,5 +980,6 @@ void tiva_gpio_lockport(pinconfig_t pinconfig, bool lock)
/* Restrict access to the TIVA_GPIO_CR_OFFSET register */
modifyreg32(base + TIVA_GPIO_LOCK_OFFSET, GPIO_LOCK_UNLOCK, GPIO_LOCK_LOCKED);
modifyreg32(base + TIVA_GPIO_LOCK_OFFSET, GPIO_LOCK_UNLOCK,
GPIO_LOCK_LOCKED);
}

View File

@ -115,8 +115,8 @@
*
* FFFS SPPP III. AAAA .... ...V PPPP PBBB
*
* TODO: The TM4C also supports configuration of pins to trigger ADC and/or uDMA.
* That configuration is not addressed in this this encoding.
* TODO: The TM4C also supports configuration of pins to trigger ADC and/or
* uDMA. That configuration is not addressed in this encoding.
*/
/* These bits set the primary function of the pin:
@ -200,7 +200,8 @@
# define GPIO_ALT_14 (14 << GPIO_ALT_SHIFT)
# define GPIO_ALT_15 (15 << GPIO_ALT_SHIFT)
/* If the pin is an GPIO digital output, then this identifies the initial output value:
/* If the pin is an GPIO digital output, then this identifies the initial
* output value:
* .... .... .... .... .... ...V .... ....
*/