diff --git a/Documentation/NfsHowto.html b/Documentation/NfsHowto.html index ed5e6793b8f..a6e7d3fb2ed 100644 --- a/Documentation/NfsHowto.html +++ b/Documentation/NfsHowto.html @@ -175,7 +175,7 @@ char *mountpoint; ret = mount(NULL, mountpoint, string "nfs", 0, (FAR void *)&data);

- NOTE that: (1) the block driver paramter is NULL. + NOTE that: (1) the block driver parameter is NULL. The mount() is smart enough to know that no block driver is needed with the NFS file system. (2) The NFS file system is identified with the simple string "nfs" (3) A reference to struct nfs_args is passed as an NFS-specific argument. diff --git a/Documentation/NuttShell.html b/Documentation/NuttShell.html index 071214af8f9..b73fd77c381 100644 --- a/Documentation/NuttShell.html +++ b/Documentation/NuttShell.html @@ -1988,7 +1988,7 @@ mount -t <fstype> <block-device> <dir-path>

Synopsis. The mount command performs one of two different operations. - If no paramters are provided on the command line after the mount command, then the mount command will enumerate all of the current mountpoints on the console. + If no parameters are provided on the command line after the mount command, then the mount command will enumerate all of the current mountpoints on the console.

If the mount parameters are provied on the command after the mount command, then the mount command will mount a file system in the NuttX pseudo-file system. diff --git a/Documentation/NuttXCCodingStandard.html b/Documentation/NuttXCCodingStandard.html index 4cb4a9517d0..56eea5aafa3 100755 --- a/Documentation/NuttXCCodingStandard.html +++ b/Documentation/NuttXCCodingStandard.html @@ -1665,7 +1665,7 @@ ptr = (FAR struct somestruct_s *)value;

  • * Input Parameters: followed by a description of the of each input parameter beginning on the second line. Each input parameter begins on a separator line indented by two additional spaces. - The description needs to include (1) the name of the input paramters, and (2) a short description of the input parameter. + The description needs to include (1) the name of the input parameters, and (2) a short description of the input parameter.
  • * Returned Value: followed by a description of the of returned value(s) beginning on the second line. diff --git a/arch/arm/src/sama5/sam_udphs.c b/arch/arm/src/sama5/sam_udphs.c index 5fcff8b5917..ffef30b484a 100644 --- a/arch/arm/src/sama5/sam_udphs.c +++ b/arch/arm/src/sama5/sam_udphs.c @@ -1567,7 +1567,7 @@ static void sam_req_rddisable(uint8_t epno) * 'inflight' field to hold the maximum size of the transfer; but * 'inflight' is not used with FIFO transfers. * - * When the transfer completes, the 'recvsize' paramter must be the + * When the transfer completes, the 'recvsize' parameter must be the * size of the transfer that just completed. For the case of DMA, * that is the size of the DMA transfer that has just been written to * memory; for the FIFO transfer, recvsize is the number of bytes diff --git a/configs/cloudctrl/src/up_usb.c b/configs/cloudctrl/src/up_usb.c index f38689b5c06..4442f9192ef 100644 --- a/configs/cloudctrl/src/up_usb.c +++ b/configs/cloudctrl/src/up_usb.c @@ -257,7 +257,7 @@ void stm32_usbhost_vbusdrive(int iface, bool enable) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/configs/mikroe-stm32f4/src/up_usb.c b/configs/mikroe-stm32f4/src/up_usb.c index 9a45c1a9b6e..b95413badbe 100644 --- a/configs/mikroe-stm32f4/src/up_usb.c +++ b/configs/mikroe-stm32f4/src/up_usb.c @@ -256,7 +256,7 @@ void stm32_usbhost_vbusdrive(int iface, bool enable) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/configs/olimex-stm32-p207/src/up_usb.c b/configs/olimex-stm32-p207/src/up_usb.c index 60d6f72f769..15711b32df8 100644 --- a/configs/olimex-stm32-p207/src/up_usb.c +++ b/configs/olimex-stm32-p207/src/up_usb.c @@ -207,7 +207,7 @@ int stm32_usbhost_initialize(void) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/configs/sam3u-ek/src/up_buttons.c b/configs/sam3u-ek/src/up_buttons.c index da35ef6437f..fb9f9b08eb7 100644 --- a/configs/sam3u-ek/src/up_buttons.c +++ b/configs/sam3u-ek/src/up_buttons.c @@ -78,7 +78,8 @@ static xcpt_t g_irqbutton2; ****************************************************************************/ #if defined(CONFIG_SAM34_GPIOA_IRQ) && defined(CONFIG_ARCH_IRQBUTTONS) -static xcpt_t board_button_irqx(int irq, xcpt_t irqhandler, xcpt_t *store) +static xcpt_t board_button_irqx(gpio_pinset_t pinset, int irq, + xcpt_t irqhandler, xcpt_t *store) { xcpt_t oldhandler; irqstate_t flags; @@ -100,7 +101,7 @@ static xcpt_t board_button_irqx(int irq, xcpt_t irqhandler, xcpt_t *store) { /* Configure the interrupt */ - sam_gpioirq(irq); + sam_gpioirq(pinset); (void)irq_attach(irq, irqhandler); sam_gpioirqenable(irq); } @@ -185,11 +186,13 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) { if (id == BUTTON1) { - return board_button_irqx(IRQ_BUTTON1, irqhandler, &g_irqbutton1); + return board_button_irqx(GPIO_BUTTON1, IRQ_BUTTON1, + irqhandler, &g_irqbutton1); } else if (id == BUTTON2) { - return board_button_irqx(IRQ_BUTTON2, irqhandler, &g_irqbutton2); + return board_button_irqx(GPIO_BUTTON2, IRQ_BUTTON2, + irqhandler, &g_irqbutton2); } else { diff --git a/configs/sam4e-ek/src/sam_buttons.c b/configs/sam4e-ek/src/sam_buttons.c index 6f9611829a1..2a88250b1f0 100644 --- a/configs/sam4e-ek/src/sam_buttons.c +++ b/configs/sam4e-ek/src/sam_buttons.c @@ -80,7 +80,8 @@ static xcpt_t g_irq_tamp; ****************************************************************************/ #if defined(CONFIG_SAM34_GPIOA_IRQ) && defined(CONFIG_ARCH_IRQBUTTONS) -static xcpt_t board_button_irqx(int irq, xcpt_t irqhandler, xcpt_t *store) +static xcpt_t board_button_irqx(gpio_pinset_t pinset, int irq, + xcpt_t irqhandler, xcpt_t *store) { xcpt_t oldhandler; irqstate_t flags; @@ -102,7 +103,7 @@ static xcpt_t board_button_irqx(int irq, xcpt_t irqhandler, xcpt_t *store) { /* Configure the interrupt */ - sam_gpioirq(irq); + sam_gpioirq(pinset); (void)irq_attach(irq, irqhandler); sam_gpioirqenable(irq); } @@ -130,10 +131,10 @@ static xcpt_t board_button_irqx(int irq, xcpt_t irqhandler, xcpt_t *store) * Name: board_button_initialize * * Description: - * board_button_initialize() must be called to initialize button resources. After - * that, board_buttons() may be called to collect the current state of all - * buttons or board_button_irq() may be called to register button interrupt - * handlers. + * board_button_initialize() must be called to initialize button resources. + * After that, board_buttons() may be called to collect the current state + * of all buttons or board_button_irq() may be called to register button + * interrupt handlers. * ****************************************************************************/ @@ -145,16 +146,16 @@ void board_button_initialize(void) (void)sam_configgpio(GPIO_TAMP); } -/************************************************************************************ +/**************************************************************************** * Name: board_buttons * * Description: - * After board_button_initialize() has been called, board_buttons() may be called to collect - * the state of all buttons. board_buttons() returns an 8-bit bit set with each bit - * associated with a button. See the BUTTON* definitions above for the meaning of - * each bit in the returned value. + * After board_button_initialize() has been called, board_buttons() may be + * called to collect the state of all buttons. board_buttons() returns an + * 8-bit bit set with each bit associated with a button. See the BUTTON* + * definitions above for the meaning of each bit in the returned value. * - ************************************************************************************/ + ****************************************************************************/ uint8_t board_buttons(void) { @@ -192,16 +193,20 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) switch (id) { case BUTTON_SCROLLUP: - return board_button_irqx(IRQ_SCROLLUP, irqhandler, &g_irq_scrollup); + return board_button_irqx(GPIO_SCROLLUP, IRQ_SCROLLUP, + irqhandler, &g_irq_scrollup); case BUTTON_SCROLLDOWN: - return board_button_irqx(IRQ_SCROLLDWN, irqhandler, &g_irq_scrolldown); + return board_button_irqx(GPIO_SCROLLDWN, IRQ_SCROLLDWN, + irqhandler, &g_irq_scrolldown); case BUTTON_WAKU: - return board_button_irqx(IRQ_WAKU, irqhandler, &g_irq_waku); + return board_button_irqx(GPIO_WAKU, IRQ_WAKU, + irqhandler, &g_irq_waku); case BUTTON_TAMP: - return board_button_irqx(IRQ_WAKU, irqhandler, &g_irq_tamp); + return board_button_irqx(GPIO_TAMP, IRQ_WAKU, + irqhandler, &g_irq_tamp); default: return NULL; diff --git a/configs/sam4l-xplained/src/sam_buttons.c b/configs/sam4l-xplained/src/sam_buttons.c index 8a5c0157154..2cefec40b21 100644 --- a/configs/sam4l-xplained/src/sam_buttons.c +++ b/configs/sam4l-xplained/src/sam_buttons.c @@ -76,10 +76,10 @@ static xcpt_t g_irqsw0; * Name: board_button_initialize * * Description: - * board_button_initialize() must be called to initialize button resources. After - * that, board_buttons() may be called to collect the current state of all - * buttons or board_button_irq() may be called to register button interrupt - * handlers. + * board_button_initialize() must be called to initialize button resources. + * After that, board_buttons() may be called to collect the current state + * of all buttons or board_button_irq() may be called to register button + * interrupt handlers. * ****************************************************************************/ @@ -88,16 +88,16 @@ void board_button_initialize(void) (void)sam_configgpio(GPIO_SW0); } -/************************************************************************************ +/**************************************************************************** * Name: board_buttons * * Description: - * After board_button_initialize() has been called, board_buttons() may be called to collect - * the state of all buttons. board_buttons() returns an 8-bit bit set with each bit - * associated with a button. See the BUTTON* definitions above for the meaning of - * each bit in the returned value. + * After board_button_initialize() has been called, board_buttons() may be + * called to collect the state of all buttons. board_buttons() returns an + * 8-bit bit set with each bit associated with a button. See the BUTTON* + * definitions above for the meaning of each bit in the returned value. * - ************************************************************************************/ + ****************************************************************************/ uint8_t board_buttons(void) { @@ -111,7 +111,7 @@ uint8_t board_buttons(void) * This function may be called to register an interrupt handler that will * be called when a button is depressed or released. The ID value is one * of the BUTTON* definitions provided above. The previous interrupt - * handler address isreturned (so that it may restored, if so desired). + * handler address is returned (so that it may restored, if so desired). * * Configuration Notes: * Configuration CONFIG_AVR32_GPIOIRQ must be selected to enable the @@ -148,7 +148,7 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) { /* Configure the interrupt */ - sam_gpioirq(IRQ_SW0); + sam_gpioirq(GPIO_SW0); (void)irq_attach(IRQ_SW0, irqhandler); sam_gpioirqenable(IRQ_SW0); } diff --git a/configs/sam4s-xplained-pro/src/sam_buttons.c b/configs/sam4s-xplained-pro/src/sam_buttons.c index ce2420df217..f76a86cda97 100644 --- a/configs/sam4s-xplained-pro/src/sam_buttons.c +++ b/configs/sam4s-xplained-pro/src/sam_buttons.c @@ -75,10 +75,10 @@ static xcpt_t g_irqsw0; * Name: board_button_initialize * * Description: - * board_button_initialize() must be called to initialize button resources. After - * that, board_buttons() may be called to collect the current state of all - * buttons or board_button_irq() may be called to register button interrupt - * handlers. + * board_button_initialize() must be called to initialize button resources. + * After that, board_buttons() may be called to collect the current state + * of all buttons or board_button_irq() may be called to register button + * interrupt handlers. * ****************************************************************************/ @@ -87,16 +87,16 @@ void board_button_initialize(void) (void)sam_configgpio(GPIO_SW0); } -/************************************************************************************ +/**************************************************************************** * Name: board_buttons * * Description: - * After board_button_initialize() has been called, board_buttons() may be called to collect - * the state of all buttons. board_buttons() returns an 8-bit bit set with each bit - * associated with a button. See the BUTTON* definitions above for the meaning of - * each bit in the returned value. + * After board_button_initialize() has been called, board_buttons() may be + * called to collect the state of all buttons. board_buttons() returns an + * 8-bit bit set with each bit associated with a button. See the BUTTON* + * definitions above for the meaning of each bit in the returned value. * - ************************************************************************************/ + ****************************************************************************/ uint8_t board_buttons(void) { @@ -147,7 +147,7 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) { /* Configure the interrupt */ - sam_gpioirq(IRQ_SW0); + sam_gpioirq(GPIO_SW0); (void)irq_attach(IRQ_SW0, irqhandler); sam_gpioirqenable(IRQ_SW0); } diff --git a/configs/sam4s-xplained/src/sam_buttons.c b/configs/sam4s-xplained/src/sam_buttons.c index 86b47fadd51..e69d601825f 100644 --- a/configs/sam4s-xplained/src/sam_buttons.c +++ b/configs/sam4s-xplained/src/sam_buttons.c @@ -74,10 +74,10 @@ static xcpt_t g_irqbp2; * Name: board_button_initialize * * Description: - * board_button_initialize() must be called to initialize button resources. After - * that, board_buttons() may be called to collect the current state of all - * buttons or board_button_irq() may be called to register button interrupt - * handlers. + * board_button_initialize() must be called to initialize button resources. + * After that, board_buttons() may be called to collect the current state + * of all buttons or board_button_irq() may be called to register button + * interrupt handlers. * ****************************************************************************/ @@ -86,16 +86,16 @@ void board_button_initialize(void) (void)sam_configgpio(GPIO_BP2); } -/************************************************************************************ +/**************************************************************************** * Name: board_buttons * * Description: - * After board_button_initialize() has been called, board_buttons() may be called to collect - * the state of all buttons. board_buttons() returns an 8-bit bit set with each bit - * associated with a button. See the BUTTON* definitions above for the meaning of - * each bit in the returned value. + * After board_button_initialize() has been called, board_buttons() may be + * called to collect the state of all buttons. board_buttons() returns an + * 8-bit bit set with each bit associated with a button. See the BUTTON* + * definitions above for the meaning of each bit in the returned value. * - ************************************************************************************/ + ****************************************************************************/ uint8_t board_buttons(void) { @@ -109,7 +109,7 @@ uint8_t board_buttons(void) * This function may be called to register an interrupt handler that will * be called when a button is depressed or released. The ID value is one * of the BUTTON* definitions provided above. The previous interrupt - * handler address isreturned (so that it may restored, if so desired). + * handler address is returned (so that it may restored, if so desired). * * Configuration Notes: * Configuration CONFIG_AVR32_GPIOIRQ must be selected to enable the @@ -146,7 +146,7 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) { /* Configure the interrupt */ - sam_gpioirq(IRQ_BP2); + sam_gpioirq(GPIO_BP2); (void)irq_attach(IRQ_BP2, irqhandler); sam_gpioirqenable(IRQ_BP2); } diff --git a/configs/sama5d3-xplained/src/sam_buttons.c b/configs/sama5d3-xplained/src/sam_buttons.c index e4eb35b752a..9e66cf3d9f0 100644 --- a/configs/sama5d3-xplained/src/sam_buttons.c +++ b/configs/sama5d3-xplained/src/sam_buttons.c @@ -126,7 +126,7 @@ uint8_t board_buttons(void) * This function may be called to register an interrupt handler that will * be called when a button is depressed or released. The ID value is one * of the BUTTON* definitions provided above. The previous interrupt - * handler address isreturned (so that it may restored, if so desired). + * handler address is returned (so that it may restored, if so desired). * * Configuration Notes: * Configuration CONFIG_SAMA5_PIO_IRQ must be selected to enable the @@ -155,11 +155,25 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) oldhandler = g_irquser1; g_irquser1 = irqhandler; + /* Are we attaching or detaching? */ + + if (irqhandler != NULL) + { + /* Configure the interrupt */ + + sam_pioirq(PIO_USER); + (void)irq_attach(IRQ_USER1, irqhandler); + sam_pioirqenable(IRQ_USER1); + } + else + { + /* Disable and detach the interrupt */ + + sam_pioirqdisable(IRQ_USER1); + (void)irq_detach(IRQ_USER1); + } /* Configure the interrupt */ - sam_pioirq(IRQ_USER1); - (void)irq_attach(IRQ_USER1, irqhandler); - sam_pioirqenable(IRQ_USER1); irqrestore(flags); } diff --git a/configs/sama5d3-xplained/src/sam_ethernet.c b/configs/sama5d3-xplained/src/sam_ethernet.c index 37a77b2922e..7888808055a 100644 --- a/configs/sama5d3-xplained/src/sam_ethernet.c +++ b/configs/sama5d3-xplained/src/sam_ethernet.c @@ -136,12 +136,14 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) irqstate_t flags; xcpt_t *handler; xcpt_t oldhandler; + pio_pinset_t pinset; int irq; #ifdef CONFIG_SAMA5_EMACA if (intf == EMAC_INTF) { handler = &g_emac_handler; + pinset = PIO_INT_ETH1; irq = IRQ_INT_ETH1; } else @@ -150,6 +152,7 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) if (intf == GMAC_INTF) { handler = &g_gmac_handler; + pinset = PIO_INT_ETH0; irq = IRQ_INT_ETH0; } else @@ -172,7 +175,7 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) /* Configure the interrupt */ - sam_pioirq(irq); + sam_pioirq(pinset); (void)irq_attach(irq, irqhandler); sam_pioirqenable(irq); diff --git a/configs/sama5d3-xplained/src/sam_usb.c b/configs/sama5d3-xplained/src/sam_usb.c index 08903243161..13deaea7a80 100644 --- a/configs/sama5d3-xplained/src/sam_usb.c +++ b/configs/sama5d3-xplained/src/sam_usb.c @@ -463,7 +463,7 @@ void sam_usbhost_vbusdrive(int rhport, bool enable) * REVISIT: Since this is a common signal, we will need to come up with some way * to inform both EHCI and OHCI drivers when this error occurs. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: @@ -493,7 +493,7 @@ xcpt_t sam_setup_overcurrent(xcpt_t handler) /* Configure the interrupt */ - sam_pioirq(IRQ_USBBC_VBUS_OVERCURRENT); + sam_pioirq(PIO_USBBC_VBUS_OVERCURRENT); (void)irq_attach(IRQ_USBBC_VBUS_OVERCURRENT, handler); sam_pioirqenable(IRQ_USBBC_VBUS_OVERCURRENT); diff --git a/configs/sama5d3x-ek/src/sam_buttons.c b/configs/sama5d3x-ek/src/sam_buttons.c index 9d8afcfba46..a76696847fa 100644 --- a/configs/sama5d3x-ek/src/sam_buttons.c +++ b/configs/sama5d3x-ek/src/sam_buttons.c @@ -91,10 +91,10 @@ static xcpt_t g_irquser1; * Name: board_button_initialize * * Description: - * board_button_initialize() must be called to initialize button resources. After - * that, board_buttons() may be called to collect the current state of all - * buttons or board_button_irq() may be called to register button interrupt - * handlers. + * board_button_initialize() must be called to initialize button resources. + * After that, board_buttons() may be called to collect the current state + * of all buttons or board_button_irq() may be called to register button + * interrupt handlers. * ****************************************************************************/ @@ -107,10 +107,10 @@ void board_button_initialize(void) * Name: board_buttons * * Description: - * After board_button_initialize() has been called, board_buttons() may be called to - * collect the state of all buttons. board_buttons() returns an 8-bit bit set - * with each bit associated with a button. See the BUTTON* definitions - * above for the meaning of each bit in the returned value. + * After board_button_initialize() has been called, board_buttons() may be + * called to collect the state of all buttons. board_buttons() returns an + * 8-bit bit set with each bit associated with a button. See the BUTTON* + * definitions above for the meaning of each bit in the returned value. * ****************************************************************************/ @@ -155,11 +155,24 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) oldhandler = g_irquser1; g_irquser1 = irqhandler; - /* Configure the interrupt */ + /* Are we attaching or detaching? */ + + if (irqhandler != NULL) + { + /* Configure the interrupt */ + + sam_pioirq(PIO_USER1); + (void)irq_attach(IRQ_USER1, irqhandler); + sam_pioirqenable(IRQ_USER1); + } + else + { + /* Disable and detach the interrupt */ + + sam_pioirqdisable(IRQ_USER1); + (void)irq_detach(IRQ_USER1); + } - sam_pioirq(IRQ_USER1); - (void)irq_attach(IRQ_USER1, irqhandler); - sam_pioirqenable(IRQ_USER1); irqrestore(flags); } diff --git a/configs/sama5d3x-ek/src/sam_ethernet.c b/configs/sama5d3x-ek/src/sam_ethernet.c index 4e5927d5dbf..813ef3b28af 100644 --- a/configs/sama5d3x-ek/src/sam_ethernet.c +++ b/configs/sama5d3x-ek/src/sam_ethernet.c @@ -136,12 +136,14 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) irqstate_t flags; xcpt_t *handler; xcpt_t oldhandler; + pio_pinset_t pinset; int irq; #ifdef CONFIG_SAMA5_EMACA if (intf == EMAC_INTF) { handler = &g_emac_handler; + pinset = PIO_INT_ETH1; irq = IRQ_INT_ETH1; } else @@ -150,6 +152,7 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) if (intf == GMAC_INTF) { handler = &g_gmac_handler; + pinset = PIO_INT_ETH0; irq = IRQ_INT_ETH0; } else @@ -172,7 +175,7 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) /* Configure the interrupt */ - sam_pioirq(irq); + sam_pioirq(pinset); (void)irq_attach(irq, irqhandler); sam_pioirqenable(irq); diff --git a/configs/sama5d3x-ek/src/sam_usb.c b/configs/sama5d3x-ek/src/sam_usb.c index 556ee2cf12d..1c7a86e85ce 100644 --- a/configs/sama5d3x-ek/src/sam_usb.c +++ b/configs/sama5d3x-ek/src/sam_usb.c @@ -463,7 +463,7 @@ void sam_usbhost_vbusdrive(int rhport, bool enable) * REVISIT: Since this is a common signal, we will need to come up with some way * to inform both EHCI and OHCI drivers when this error occurs. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: @@ -493,7 +493,7 @@ xcpt_t sam_setup_overcurrent(xcpt_t handler) /* Configure the interrupt */ - sam_pioirq(IRQ_USBBC_VBUS_OVERCURRENT); + sam_pioirq(PIO_USBBC_VBUS_OVERCURRENT); (void)irq_attach(IRQ_USBBC_VBUS_OVERCURRENT, handler); sam_pioirqenable(IRQ_USBBC_VBUS_OVERCURRENT); diff --git a/configs/sama5d4-ek/src/sam_buttons.c b/configs/sama5d4-ek/src/sam_buttons.c index 1d6925aeda8..2df0c8fbe6f 100644 --- a/configs/sama5d4-ek/src/sam_buttons.c +++ b/configs/sama5d4-ek/src/sam_buttons.c @@ -151,11 +151,24 @@ xcpt_t board_button_irq(int id, xcpt_t irqhandler) oldhandler = g_irquser1; g_irquser1 = irqhandler; - /* Configure the interrupt */ + /* Are we attaching or detaching? */ + + if (irqhandler != NULL) + { + /* Configure the interrupt */ + + sam_pioirq(PIO_BTN_USER); + (void)irq_attach(IRQ_BTN_USER, irqhandler); + sam_pioirqenable(IRQ_BTN_USER); + } + else + { + /* Disable and detach the interrupt */ + + sam_pioirqdisable(IRQ_BTN_USER); + (void)irq_detach(IRQ_BTN_USER); + } - sam_pioirq(IRQ_USER1); - (void)irq_attach(IRQ_USER1, irqhandler); - sam_pioirqenable(IRQ_USER1); irqrestore(flags); } diff --git a/configs/sama5d4-ek/src/sam_ethernet.c b/configs/sama5d4-ek/src/sam_ethernet.c index aa53a25e311..67bcfa54991 100644 --- a/configs/sama5d4-ek/src/sam_ethernet.c +++ b/configs/sama5d4-ek/src/sam_ethernet.c @@ -105,12 +105,14 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) irqstate_t flags; xcpt_t *handler; xcpt_t oldhandler; + pio_pinset_t pinset; int irq; #ifdef CONFIG_SAMA5_EMAC0 if (intf == EMAC0_INTF) { handler = &g_emac0_handler; + pinset = PIO_INT_ETH0; irq = IRQ_INT_ETH0; } else @@ -119,6 +121,7 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) if (intf == EMAC1_INTF) { handler = &g_emac1_handler; + pinset = PIO_INT_ETH1; irq = IRQ_INT_ETH1; } else @@ -141,7 +144,7 @@ xcpt_t sam_phyirq(int intf, xcpt_t irqhandler) /* Configure the interrupt */ - sam_pioirq(irq); + sam_pioirq(pinset); (void)irq_attach(irq, irqhandler); sam_pioirqenable(irq); diff --git a/configs/sama5d4-ek/src/sam_usb.c b/configs/sama5d4-ek/src/sam_usb.c index 3aa749a037f..b3f510b9f77 100644 --- a/configs/sama5d4-ek/src/sam_usb.c +++ b/configs/sama5d4-ek/src/sam_usb.c @@ -464,7 +464,7 @@ void sam_usbhost_vbusdrive(int rhport, bool enable) * REVISIT: Since this is a common signal, we will need to come up with some way * to inform both EHCI and OHCI drivers when this error occurs. * - * Input paramter: + * Input Parameters: * handler - New over-current interrupt handler * * Returned value: @@ -494,7 +494,7 @@ xcpt_t sam_setup_overcurrent(xcpt_t handler) /* Configure the interrupt */ - sam_pioirq(IRQ_USBBC_VBUS_OVERCURRENT); + sam_pioirq(PIO_USBBC_VBUS_OVERCURRENT); (void)irq_attach(IRQ_USBBC_VBUS_OVERCURRENT, handler); sam_pioirqenable(IRQ_USBBC_VBUS_OVERCURRENT); diff --git a/configs/shenzhou/src/up_usb.c b/configs/shenzhou/src/up_usb.c index e3416d86e70..948974c4ac3 100644 --- a/configs/shenzhou/src/up_usb.c +++ b/configs/shenzhou/src/up_usb.c @@ -256,7 +256,7 @@ void stm32_usbhost_vbusdrive(int iface, bool enable) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/configs/stm3220g-eval/src/up_usb.c b/configs/stm3220g-eval/src/up_usb.c index d1638b21011..b83d8e53d18 100644 --- a/configs/stm3220g-eval/src/up_usb.c +++ b/configs/stm3220g-eval/src/up_usb.c @@ -256,7 +256,7 @@ void stm32_usbhost_vbusdrive(int iface, bool enable) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/configs/stm3240g-eval/src/up_usb.c b/configs/stm3240g-eval/src/up_usb.c index e12e65e43ed..102054dd67f 100644 --- a/configs/stm3240g-eval/src/up_usb.c +++ b/configs/stm3240g-eval/src/up_usb.c @@ -256,7 +256,7 @@ void stm32_usbhost_vbusdrive(int iface, bool enable) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/configs/stm32f4discovery/src/stm32_usb.c b/configs/stm32f4discovery/src/stm32_usb.c index fbc77fdd59f..055c1e4272c 100644 --- a/configs/stm32f4discovery/src/stm32_usb.c +++ b/configs/stm32f4discovery/src/stm32_usb.c @@ -275,7 +275,7 @@ void stm32_usbhost_vbusdrive(int iface, bool enable) * Setup to receive an interrupt-level callback if an overcurrent condition is * detected. * - * Input paramter: + * Input Parameter: * handler - New overcurrent interrupt handler * * Returned value: diff --git a/fs/nxffs/nxffs.h b/fs/nxffs/nxffs.h index 5882cf3e32d..26e35f50658 100644 --- a/fs/nxffs/nxffs.h +++ b/fs/nxffs/nxffs.h @@ -553,7 +553,7 @@ off_t nxffs_iotell(FAR struct nxffs_volume_s *volume); * over bad blocks and block headers as necessary. * * Input Parameters: - * volume - Describes the NXFFS volume. The paramters ioblock and iooffset + * volume - Describes the NXFFS volume. The parameters ioblock and iooffset * in the volume structure determine the behavior of nxffs_getc(). * reserve - If less than this much space is available at the end of the * block, then skip to the next block. diff --git a/fs/nxffs/nxffs_cache.c b/fs/nxffs/nxffs_cache.c index 72b2ac0a642..5d87cb23938 100644 --- a/fs/nxffs/nxffs_cache.c +++ b/fs/nxffs/nxffs_cache.c @@ -195,7 +195,7 @@ off_t nxffs_iotell(FAR struct nxffs_volume_s *volume) * over bad blocks and block headers as necessary. * * Input Parameters: - * volume - Describes the NXFFS volume. The paramters ioblock and iooffset + * volume - Describes the NXFFS volume. The parameters ioblock and iooffset * in the volume structure determine the behavior of nxffs_getc(). * reserve - If less than this much space is available at the end of the * block, then skip to the next block. diff --git a/include/nuttx/poff.h b/include/nuttx/poff.h index b2b2c08c231..05349e53a87 100644 --- a/include/nuttx/poff.h +++ b/include/nuttx/poff.h @@ -425,7 +425,7 @@ typedef struct poff_debugfuncinfo_s poff_debugfuncinfo_t; struct poff_debugarginfo_s { - /* This is the size, in bytes, of one input paramter */ + /* This is the size, in bytes, of one input parameter */ uint32_t da_size; }; diff --git a/libc/README.txt b/libc/README.txt index 4ecc7c2999f..b165bd74b05 100644 --- a/libc/README.txt +++ b/libc/README.txt @@ -78,7 +78,7 @@ Each type field has a format as follows: A similar situation exists for unions. For example, the formal parameter type union sigval -- You cannot cast a uintptr_t to a union sigval, but you can cast to the type of one of the union - member types when passing the actual paramter. Similarly, we + member types when passing the actual parameter. Similarly, we cannot cast a union sigval to a uinptr_t either. Rather, we need to cast a specific union member fieldname to uintptr_t. diff --git a/syscall/README.txt b/syscall/README.txt index 11942a7b76f..39c8694f751 100644 --- a/syscall/README.txt +++ b/syscall/README.txt @@ -104,7 +104,7 @@ Each type field has a format as follows: A similar situation exists for unions. For example, the formal parameter type union sigval -- You cannot cast a uintptr_t to a union sigval, but you can cast to the type of one of the union - member types when passing the actual paramter. Similarly, we + member types when passing the actual parameter. Similarly, we cannot cast a union sigval to a uinptr_t either. Rather, we need to cast a specific union member fieldname to uintptr_t. diff --git a/tools/mkdeps.c b/tools/mkdeps.c index 1b8989e87c6..a4bc68955a9 100644 --- a/tools/mkdeps.c +++ b/tools/mkdeps.c @@ -394,7 +394,7 @@ static void parse_args(int argc, char **argv) fprintf(stderr, " Windows Native : [%s]\n", g_winnative ? "TRUE" : "FALSE"); } - /* Check for required paramters */ + /* Check for required parameters */ if (!g_cc) {