Understood, Maxim. Sorry, so many things to learn!

--
Oriol Arcas
Software Engineer
Starflow Networks

On Thu, Jun 9, 2016 at 10:04 PM, Maxim Uvarov <[email protected]>
wrote:

> Hi Oriol,
>
> I merged this path with description change.
>
> Usually we use or linux-generic or helper or api prefixes. In current case
> it has to be only helper.
>
> Thanks,
> Maxim.
>
> On 06/09/16 20:47, Oriol Arcas wrote:
>
>> Helper's chksum.h API had a void* to uint16_t* casting, which is not
>> legal in C++ (Clang and GCC).
>>
>> Signed-off-by: Oriol Arcas <[email protected]>
>> ---
>>   helper/include/odp/helper/chksum.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/helper/include/odp/helper/chksum.h
>> b/helper/include/odp/helper/chksum.h
>> index 07d0be3..520e927 100644
>> --- a/helper/include/odp/helper/chksum.h
>> +++ b/helper/include/odp/helper/chksum.h
>> @@ -44,7 +44,7 @@ typedef enum {
>>    */
>>   static inline odp_u16sum_t odph_chksum(void *buffer, int len)
>>   {
>> -       uint16_t *buf = buffer;
>> +       uint16_t *buf = (uint16_t *)buffer;
>>         uint32_t sum = 0;
>>         uint16_t result;
>>
>>
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to