[PATCH] Fixes GPIO APIs Naming Convention and Comments

Ben Gras beng at shrike-systems.com
Wed Oct 21 15:38:43 UTC 2015


On Thu, Oct 15, 2015 at 12:02 AM, Gedare Bloom <gedare at rtems.org> wrote:
> A ticket would be preferred. I didn't notice the attachment. Can you
> please (1) make a ticket, (2) close it in the commit message, and (3)
> re-send your patch.
>
> Ben, if you get around to it, please commit the revised patch when it
> comes through. I'll be away for a few days and unable to commit.

Tested on mainline and merged. Ticket: https://devel.rtems.org/ticket/2435

>
> On Wed, Oct 14, 2015 at 5:06 PM, sudarshan.rajagopalan
> <sudarshan.rajagopalan at vecna.com> wrote:
>> On 2015-10-14 16:28, Gedare Bloom wrote:
>>>
>>> OK to commit, but the patch did not apply cleanly for me, something
>>> wrong with the mail message.
>>
>>
>>
>> Hi Gedare,
>>
>> The mail message was not sent using git send-email (this is not working for
>> us currently), so I copied the contents of the patch to this mail message.
>> Could you try using the patch attached in my first mail?
>>
>> Also, does this require a ticket?
>>
>> Thanks and Regards,
>> Sudarshan
>>
>>
>>>
>>> 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