Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
From: David Hildenbrand
Date: Wed Sep 16 2020 - 15:05:57 EST
On 16.09.20 14:10, Wei Yang wrote:
> On Wed, Sep 16, 2020 at 12:03:20PM +0200, David Hildenbrand wrote:
>> On 16.09.20 12:02, Wei Yang wrote:
>>> On Wed, Sep 16, 2020 at 09:30:41AM +0200, David Hildenbrand wrote:
>>>> "mem" in the name already indicates the root, similar to
>>>> release_mem_region() and devm_request_mem_region(). Make it implicit.
>>>> The only single caller always passes iomem_resource, other parents are
>>>> not applicable.
>>>>
>>>
>>> Looks good to me.
>>>
>>> Reviewed-by: Wei Yang <richard.weiyang@xxxxxxxxxxxxxxxxx>
>>>
>>
>> Thanks for the review!
>>
>
> Would you send another version? I didn't take a look into the following
> patches, since the 4th is missed.
Not planning to send another one as long as there are no further
comments. Seems to be an issue on your side because all patches arrived
on linux-mm (see
https://lore.kernel.org/linux-mm/20200911103459.10306-1-david@xxxxxxxxxx/)
You can find patch #4 at
https://lore.kernel.org/linux-mm/20200911103459.10306-5-david@xxxxxxxxxx/
(which has CC "Wei Yang <richardw.yang@xxxxxxxxxxxxxxx>")
--
Thanks,
David / dhildenb