[PATCH] Fixes GPIO APIs Naming Convention and Comments

Gedare Bloom gedare at rtems.org
Wed Oct 14 20:28:09 UTC 2015


OK to commit, but the patch did not apply cleanly for me, something
wrong with the mail message.

On Tue, Oct 13, 2015 at 3:11 AM, Ben Gras <beng at shrike-systems.com> wrote:
> All,
>
> This change renames these gpio-specific functions:
>
> rtems_bsp_enable_interrupt
> rtems_bsp_disable_interrupt
> rtems_bsp_select_specific_io
>
> to
>
> rtems_gpio_bsp_enable_interrupt
> rtems_gpio_bsp_disable_interrupt
> rtems_gpio_bsp_select_specific_io
>
>
> +1, Definitely an improvement.
>
>
> On Mon, Oct 12, 2015 at 7:37 PM, sudarshan.rajagopalan
> <sudarshan.rajagopalan at vecna.com> wrote:
>>
>> Just found few function names to be inconsistent towards the naming
>> convention, and also few API documentation in the RTEMS GPIO files. Please
>> commit this if required.
>>
>> Thanks and Regards,
>> Sudarshan Rajagopalan
>>
>> From e17ce266ba4cfaec0159c5477697847629946ced Mon Sep 17 00:00:00 2001
>> From: Sudarshan Rajagopalan <sudarshan.rajagopalan at vecna.com>
>> Date: Mon, 12 Oct 2015 12:47:35 -0400
>> Subject: [PATCH] Fixes GPIO APIs Naming Convention and Comments
>>
>> ---
>>  c/src/lib/libbsp/arm/beagle/gpio/bbb-gpio.c      |  8 ++++----
>>  c/src/lib/libbsp/arm/raspberrypi/gpio/rpi-gpio.c |  4 ++--
>>  c/src/lib/libbsp/shared/gpio.c                   |  6 +++---
>>  c/src/lib/libbsp/shared/include/gpio.h           | 20 ++++++++++++--------
>>  4 files changed, 21 insertions(+), 17 deletions(-)
>>
>> diff --git a/c/src/lib/libbsp/arm/beagle/gpio/bbb-gpio.c
>> b/c/src/lib/libbsp/arm/beagle/gpio/bbb-gpio.c
>> index 8cf690f..9737dec 100644
>> --- a/c/src/lib/libbsp/arm/beagle/gpio/bbb-gpio.c
>> +++ b/c/src/lib/libbsp/arm/beagle/gpio/bbb-gpio.c
>> @@ -299,7 +299,7 @@ uint32_t
>> rtems_gpio_bsp_interrupt_line(rtems_vector_number vector)
>>    return event_status;
>>  }
>>
>> -rtems_status_code rtems_bsp_enable_interrupt(
>> +rtems_status_code rtems_gpio_bsp_enable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> @@ -354,7 +354,7 @@ rtems_status_code rtems_bsp_enable_interrupt(
>>    return RTEMS_SUCCESSFUL;
>>  }
>>
>> -rtems_status_code rtems_bsp_disable_interrupt(
>> +rtems_status_code rtems_gpio_bsp_disable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> @@ -528,7 +528,7 @@ uint32_t
>> rtems_gpio_bsp_interrupt_line(rtems_vector_number vector)
>>    return -1;
>>  }
>>
>> -rtems_status_code rtems_bsp_enable_interrupt(
>> +rtems_status_code rtems_gpio_bsp_enable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> @@ -536,7 +536,7 @@ rtems_status_code rtems_bsp_enable_interrupt(
>>    return RTEMS_NOT_DEFINED;
>>  }
>>
>> -rtems_status_code rtems_bsp_disable_interrupt(
>> +rtems_status_code rtems_gpio_bsp_disable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> diff --git a/c/src/lib/libbsp/arm/raspberrypi/gpio/rpi-gpio.c
>> b/c/src/lib/libbsp/arm/raspberrypi/gpio/rpi-gpio.c
>> index a782d11..4fb2c93 100644
>> --- a/c/src/lib/libbsp/arm/raspberrypi/gpio/rpi-gpio.c
>> +++ b/c/src/lib/libbsp/arm/raspberrypi/gpio/rpi-gpio.c
>> @@ -183,7 +183,7 @@ uint32_t
>> rtems_gpio_bsp_interrupt_line(rtems_vector_number vector)
>>    return event_status;
>>  }
>>
>> -rtems_status_code rtems_bsp_enable_interrupt(
>> +rtems_status_code rtems_gpio_bsp_enable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> @@ -227,7 +227,7 @@ rtems_status_code rtems_bsp_enable_interrupt(
>>    return RTEMS_SUCCESSFUL;
>>  }
>>
>> -rtems_status_code rtems_bsp_disable_interrupt(
>> +rtems_status_code rtems_gpio_bsp_disable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> diff --git a/c/src/lib/libbsp/shared/gpio.c b/c/src/lib/libbsp/shared/gpio.c
>> index 80e2727..b806556 100644
>> --- a/c/src/lib/libbsp/shared/gpio.c
>> +++ b/c/src/lib/libbsp/shared/gpio.c
>> @@ -1341,7 +1341,7 @@ rtems_status_code rtems_gpio_request_pin(
>>          return RTEMS_UNSATISFIED;
>>        }
>>
>> -      sc = rtems_bsp_select_specific_io(
>> +      sc = rtems_gpio_bsp_select_specific_io(
>>               bank,
>>               pin,
>>               bsp_data->io_function,
>> @@ -1805,7 +1805,7 @@ rtems_status_code rtems_gpio_enable_interrupt(
>>      }
>>    }
>>
>> -  sc = rtems_bsp_enable_interrupt(bank, pin, interrupt);
>> +  sc = rtems_gpio_bsp_enable_interrupt(bank, pin, interrupt);
>>
>>    if ( sc != RTEMS_SUCCESSFUL ) {
>>      RELEASE_LOCK(gpio_bank_state[bank].lock);
>> @@ -1919,7 +1919,7 @@ rtems_status_code
>> rtems_gpio_disable_interrupt(uint32_t pin_number)
>>      return RTEMS_NOT_CONFIGURED;
>>    }
>>
>> -  sc = rtems_bsp_disable_interrupt(bank, pin,
>> interrupt_state->active_interrupt);
>> +  sc = rtems_gpio_bsp_disable_interrupt(bank, pin,
>> interrupt_state->active_interrupt);
>>
>>    if ( sc != RTEMS_SUCCESSFUL ) {
>>      RELEASE_LOCK(gpio_bank_state[bank].lock);
>> diff --git a/c/src/lib/libbsp/shared/include/gpio.h
>> b/c/src/lib/libbsp/shared/include/gpio.h
>> index 54de5f1..7d8f67b 100644
>> --- a/c/src/lib/libbsp/shared/include/gpio.h
>> +++ b/c/src/lib/libbsp/shared/include/gpio.h
>> @@ -736,12 +736,16 @@ extern uint32_t rtems_gpio_bsp_multi_read(uint32_t
>> bank, uint32_t bitmask);
>>   *        does not support the feature, by returning RTEMS_NOT_DEFINED.
>>   *
>>   * @param[in] bank GPIO bank number.
>> - * @param[in] bitmask Bitmask of GPIO pins to clear in the given bank.
>> + * @param[in] pins Array filled with BSP specific pin numbers. All pins
>> belong
>> + *                 to the same select bank.
>> + * @param[in] pin_count Number of pin configurations in the @var pins
>> array.
>> + * @param[in] arg Pointer to a BSP defined structure with BSP-specific
>> + *                data. This field is handled by the BSP.
>>   *
>> - * @retval RTEMS_SUCCESSFUL All pins were cleared successfully.
>> - * @retval RTEMS_NOT_DEFINED The BSP does not support BSP specific
>> operations
>> - *                           for groups.
>> - * @retval RTEMS_UNSATISFIED Could not clear at least one of the pins.
>> + * @retval RTEMS_SUCCESSFUL Operation completed with success.
>> + * @retval RTEMS_NOT_DEFINED Group has no BSP specific pins, or the BSP
>> does not
>> + *                           support BSP specific operations for groups.
>> + * @retval RTEMS_UNSATISFIED Could not operate on at least one of the pins.
>>   */
>>  extern rtems_status_code rtems_gpio_bsp_specific_group_operation(
>>    uint32_t bank,
>> @@ -859,7 +863,7 @@ extern rtems_status_code rtems_gpio_bsp_select_output(
>>   * @retval RTEMS_SUCCESSFUL Function was assigned successfully.
>>   * @retval RTEMS_UNSATISFIED Could not assign the function to the pin.
>>   */
>> -extern rtems_status_code rtems_bsp_select_specific_io(
>> +extern rtems_status_code rtems_gpio_bsp_select_specific_io(
>>    uint32_t bank,
>>    uint32_t pin,
>>    uint32_t function,
>> @@ -918,7 +922,7 @@ extern rtems_vector_number
>> rtems_gpio_bsp_get_vector(uint32_t bank);
>>   * @retval RTEMS_SUCCESSFUL Interrupt successfully enabled for this pin.
>>   * @retval RTEMS_UNSATISFIED Could not enable the interrupt on the pin.
>>   */
>> -extern rtems_status_code rtems_bsp_enable_interrupt(
>> +extern rtems_status_code rtems_gpio_bsp_enable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> @@ -935,7 +939,7 @@ extern rtems_status_code rtems_bsp_enable_interrupt(
>>   * @retval RTEMS_SUCCESSFUL Interrupt successfully disabled for this pin.
>>   * @retval RTEMS_UNSATISFIED Could not disable interrupts on this pin.
>>   */
>> -extern rtems_status_code rtems_bsp_disable_interrupt(
>> +extern rtems_status_code rtems_gpio_bsp_disable_interrupt(
>>    uint32_t bank,
>>    uint32_t pin,
>>    rtems_gpio_interrupt interrupt
>> --
>> 1.9.1
>>
>> _______________________________________________
>> devel mailing list
>> devel at rtems.org
>> http://lists.rtems.org/mailman/listinfo/devel
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel



More information about the devel mailing list