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