Re: [PATCH] serial: 8250: Add support for using platform_device resources
From: Esben Haabendal
Date: Tue May 07 2019 - 07:36:52 EST
Lee Jones <lee.jones@xxxxxxxxxx> writes:
> On Thu, 02 May 2019, Esben Haabendal wrote:
>
>> Could you help clarify whether or not this patch is trying to do
>> something odd/wrong?
>>
>> I might be misunderstanding Andy (probably is), but the discussion
>> revolves around the changes I propose where I change the serial8250
>> driver to use platform_get_resource() in favour of
>> request_mem_region()/release_mem_region().
>
> Since 'serial8250' is registered as a platform device, I don't see any
> reason why it shouldn't have the capability to obtain its memory
> regions from the platform_get_*() helpers.
Good to hear. That is exactly what I am trying do with this patch.
@Andy: If you still don't like my approach, could you please advice an
acceptable method for improving the serial8250 driver to allow the use
of platform_get_*() helpers?
/Esben