On 3/12/2019 8:42 AM, Enrico Weigelt, metux IT consult wrote: > On 12.03.19 12:37, Thierry Reding wrote: > >> The driver currently doesn't request the memory described in the >> resource, so technically you're changing behaviour here and with your >> change the driver could now fail if somebody else has already claimed >> the memory. > > hmm, using w/o requesting/claiming - isn't that a bug ? Not necessarily, before regmap existed, you could have very well delegated a subset of a larger resource to a specific driver while a driver requesting that larger resource would be responsible for doing the request_mem_region(). As long as both drivers don't stomp on each other, this is a perfectly valid way to delegate, yet keep things modular/separate. -- Florian
- [PATCH 01/42] drivers: gpio: 74xx-mmio: u... Enrico Weigelt, metux IT consult
- [PATCH 07/42] drivers: gpio: clps711... Enrico Weigelt, metux IT consult
- [PATCH 42/42] drivers: gpio: use dev... Enrico Weigelt, metux IT consult
- Re: [PATCH 42/42] drivers: gpio:... Thierry Reding
- [PATCH 36/42] drivers: gpio: uniphie... Enrico Weigelt, metux IT consult
- Re: [PATCH 36/42] drivers: gpio:... Masahiro Yamada
- [PATCH 37/42] drivers: gpio: vf610: ... Enrico Weigelt, metux IT consult
- [PATCH 38/42] drivers: gpio: vr41xx:... Enrico Weigelt, metux IT consult
- Re: [PATCH 38/42] drivers: gpio:... Thierry Reding
- Re: [PATCH 38/42] drivers: g... Enrico Weigelt, metux IT consult
- Re: [PATCH 38/42] driver... Florian Fainelli
- [PATCH 30/42] drivers: gpio: sta2x11... Enrico Weigelt, metux IT consult
- [PATCH 26/42] drivers: gpio: rcar: p... Enrico Weigelt, metux IT consult
- Re: [PATCH 26/42] drivers: gpio:... Thierry Reding
- [PATCH 34/42] drivers: gpio: timberd... Enrico Weigelt, metux IT consult
- [PATCH 41/42] drivers: gpio: xlp: de... Enrico Weigelt, metux IT consult
- [PATCH 16/42] drivers: gpio: janz-tt... Enrico Weigelt, metux IT consult
- Re: [PATCH 16/42] drivers: gpio:... Ben Dooks
- Re: [PATCH 16/42] drivers: g... Enrico Weigelt, metux IT consult
- Re: [PATCH 16/42] driver... Thierry Reding
- Re: [PATCH 16/42] d... Enrico Weigelt, metux IT consult