Re: [PATCH] Access c_ispeed and c_ospeed via APIs

[Bruce Ashfield <bruce.ashfield@windriver.com>]

On 2016-04-12 1:53 PM, Khem Raj wrote:
> FYI

Thanks. I can also confirm that when I dropped the patch, I'm
back up and running assertion free.

Bruce

>
> ---------- Forwarded message ----------
> From: Gerhard Rieger <gerhard@dest-unreach.org>
> Date: Wed, Mar 16, 2016 at 4:43 AM
> Subject: Re: [PATCH] Access c_ispeed and c_ospeed via APIs
> To: Khem Raj <raj.khem@gmail.com>
>
>
> Thank you, I will check this patch!
> - Gerhard
>
>
> Am 15.03.2016 um 22:39 schrieb Khem Raj:
>> Use cfsetispeed(), cfsetospeed(), cfgetispeed, and cfgetospeed()
>> instead of operating on c_ispeed and c_ospeed termios structure
>> members directly because they are not guaranteed to exist on all
>> libc implementations
>>
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>>   xioinitialize.c | 8 ++++----
>>   1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/xioinitialize.c b/xioinitialize.c
>> index 9f50155..632ca4c 100644
>> --- a/xioinitialize.c
>> +++ b/xioinitialize.c
>> @@ -65,10 +65,10 @@ int xioinitialize(void) {
>>   #if HAVE_TERMIOS_ISPEED && (ISPEED_OFFSET != -1) && (OSPEED_OFFSET != -1)
>>   #if defined(ISPEED_OFFSET) && (ISPEED_OFFSET != -1)
>>   #if defined(OSPEED_OFFSET) && (OSPEED_OFFSET != -1)
>> -      tdata.termarg.c_ispeed = 0x56789abc;
>> -      tdata.termarg.c_ospeed = 0x6789abcd;
>> -      assert(tdata.termarg.c_ispeed == tdata.speeds[ISPEED_OFFSET]);
>> -      assert(tdata.termarg.c_ospeed == tdata.speeds[OSPEED_OFFSET]);
>> +      cfsetispeed(&tdata.termarg, 0x56789abc);
>> +      cfsetospeed(&tdata.termarg, 0x6789abcd);
>> +      assert(cfgetispeed(&tdata.termarg) == tdata.speeds[ISPEED_OFFSET]);
>> +      assert(cfgetospeed(&tdata.termarg) == tdata.speeds[OSPEED_OFFSET]);
>>   #endif
>>   #endif
>>   #endif
>>