Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys



Hi,

On 2015/08/28 16:03, Nick Hudson wrote:
> On 08/24/15 10:50, Kengo NAKAHARA wrote:
>> Hi,
>>
>> On 2015/08/24 18:24, Nick Hudson wrote:
>>> On 08/24/15 09:36, Kengo NAKAHARA wrote:
>>>> Hi,
>>>>
>>>> On 2015/08/24 16:18, Nick Hudson wrote:
>>>>
>>>>>> +__weak_alias(interrupt_construct_intrids, eopnotsupp);
>>>>> I found this by running sysctl -A on an evbarm platform
>>>> Sorry, some interrupt_* stub functions are incorrect.
>>>> Could you try below patch?
>>>>
>>> ...
>>>
>>> It works, but I don't think enomem is the function name here. nullret?
>> As you indicated, I also think enomem() is not suitable for this function.
>> It does not return ENOMEM different from enodev(), enxio(), enoioctl()
>> and so on.
>>
>> The improved patch is here.
>>
> [snip]
>> If there is no issue, should I commit above patch?
>>
>>
>> Thanks,
>>
> 
> Looks good to me

Thank you for reviewing. I committed it.


Thanks,

-- 
//////////////////////////////////////////////////////////////////////
Internet Initiative Japan Inc.

Device Engineering Section,
Core Product Development Department,
Product Division,
Technology Unit

Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>


Home | Main Index | Thread Index | Old Index