On 5 June 2015 at 04:26, Maxim Uvarov <maxim.uva...@linaro.org> wrote:

> why this is for api-next?
>
> Patch fixes only spelling in doxygen description. It does not modify API.
> It has to go to master.
>


All changes in the include directory go into api-next, every time we try a
short cut we open ourselves up to discussion on was it wise. See this
kernel discussion http://www.kegel.com/kerspell/.

Importantly "Correcting spelling mistakes is not as easy as it sounds; it
cannot really be automated, as many reasonable-sounding corrections could
actually change the meaning of the comments."

If it makes it in api-next it can be quickly cherry picked back to master,
but the API might have been affected if the description is now incorrect.



>
> Maxim.
>
>
> On 06/05/15 06:17, Bill Fischofer wrote:
>
>>
>>
>> On Thu, Jun 4, 2015 at 12:18 PM, Mike Holmes <mike.hol...@linaro.org
>> <mailto:mike.hol...@linaro.org>> wrote:
>>
>>     Signed-off-by: Mike Holmes <mike.hol...@linaro.org
>>     <mailto:mike.hol...@linaro.org>>
>>
>>
>> Reviewed-by: Bill Fischofer <bill.fischo...@linaro.org <mailto:
>> bill.fischo...@linaro.org>>
>>
>>
>>     ---
>>      include/odp/api/crypto.h    | 4 ++--
>>      include/odp/api/packet_io.h | 2 +-
>>      include/odp/api/pool.h      | 2 +-
>>      include/odp/api/queue.h     | 4 ++--
>>      4 files changed, 6 insertions(+), 6 deletions(-)
>>
>>     diff --git a/include/odp/api/crypto.h b/include/odp/api/crypto.h
>>     index 792549c..a9a2a1d 100644
>>     --- a/include/odp/api/crypto.h
>>     +++ b/include/odp/api/crypto.h
>>     @@ -110,7 +110,7 @@ typedef struct odp_crypto_data_range {
>>      } odp_crypto_data_range_t;
>>
>>      /**
>>     - * Crypto API session creation paramters
>>     + * Crypto API session creation parameters
>>       *
>>       * @todo Add "odp_session_proc_info_t"
>>       */
>>     @@ -133,7 +133,7 @@ typedef struct odp_crypto_session_params {
>>       *   Controls ordering of authentication and cipher operations,
>>       *   and is relative to the operation (encode vs decode).
>>       *   When encoding, @c TRUE indicates the authentication operation
>>     - *   should be peformed @b after the cipher operation else before.
>>     + *   should be performed @b after the cipher operation else before.
>>       *   When decoding, @c TRUE indicates the reverse order of operation.
>>       *
>>       * @var odp_crypto_session_params_t::compl_queue
>>     diff --git a/include/odp/api/packet_io.h b/include/odp/api/packet_io.h
>>     index 89356a6..5b04737 100644
>>     --- a/include/odp/api/packet_io.h
>>     +++ b/include/odp/api/packet_io.h
>>     @@ -64,7 +64,7 @@ extern "C" {
>>       *
>>       * @note Packets arriving via this interface assigned to a CoS by the
>>       *      classifier are received into the pool associated with
>>     that CoS. This
>>     - *      will occur either becuase this pktio is assigned a
>>     default CoS via
>>     + *      will occur either because this pktio is assigned a
>>     default CoS via
>>       *      the odp_pktio_default_cos_set() routine, or because a
>>     matching PMR
>>       *      assigned the packet to a specific CoS. The default pool
>>     specified
>>       *      here is applicable only for those packets that are not
>>     assigned to a
>>     diff --git a/include/odp/api/pool.h b/include/odp/api/pool.h
>>     index e5061ab..1ae7119 100644
>>     --- a/include/odp/api/pool.h
>>     +++ b/include/odp/api/pool.h
>>     @@ -37,7 +37,7 @@ extern "C" {
>>       * Invalid pool
>>       */
>>
>>     -/** Maximum queue name lenght in chars */
>>     +/** Maximum queue name length in chars */
>>      #define ODP_POOL_NAME_LEN  32
>>
>>      /**
>>     diff --git a/include/odp/api/queue.h b/include/odp/api/queue.h
>>     index 6a8b15f..f7a0777 100644
>>     --- a/include/odp/api/queue.h
>>     +++ b/include/odp/api/queue.h
>>     @@ -41,7 +41,7 @@ extern "C" {
>>
>>      /**
>>       * @def ODP_QUEUE_NAME_LEN
>>     - * Maximum queue name lenght in chars
>>     + * Maximum queue name length in chars
>>       */
>>
>>
>>     @@ -213,7 +213,7 @@ odp_queue_t odp_queue_lookup(const char *name);
>>      /**
>>       * Set queue context
>>       *
>>     - * Its the responsability of the interface user to make sure
>>     + * It is the responsibility of the interface user to make sure
>>       * queue context allocation is done in an area reachable for
>>       * all EOs accessing the context
>>       *
>>     --
>>     2.1.4
>>
>>     _______________________________________________
>>     lng-odp mailing list
>>     lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>>     https://lists.linaro.org/mailman/listinfo/lng-odp
>>
>>
>>
>>
>> _______________________________________________
>> lng-odp mailing list
>> lng-odp@lists.linaro.org
>> https://lists.linaro.org/mailman/listinfo/lng-odp
>>
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
>



-- 
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to