* Optional regulators in bulk operations - reverted patch
@ 2017-01-31 21:43 Heiner Kallweit
2017-01-31 21:59 ` Heiner Kallweit
0 siblings, 1 reply; 2+ messages in thread
From: Heiner Kallweit @ 2017-01-31 21:43 UTC (permalink / raw)
To: bjorn.andersson; +Cc: Mark Brown, Linux Kernel Mailing List
Hi Bjorn,
I just came across a use case where I wanted to deal with optional
regulators in bulk operations. I was about to submit a related
patch when I saw that you submitted basically the same in
3ff3f518a135 "regulator: Make bulk API support optional supplies"
and reverted it later stating that you misunderstood the API.
How does this extension break the API? To me it sounds like a
neat feature.
The only thing I can think of if that we have to assume that
struct regulator_bulk_data is zero-initialized to ensure
backwards-compatibility (use "normal" get as default).
Rgds, Heiner
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Optional regulators in bulk operations - reverted patch
2017-01-31 21:43 Optional regulators in bulk operations - reverted patch Heiner Kallweit
@ 2017-01-31 21:59 ` Heiner Kallweit
0 siblings, 0 replies; 2+ messages in thread
From: Heiner Kallweit @ 2017-01-31 21:59 UTC (permalink / raw)
To: bjorn.andersson; +Cc: Mark Brown, Linux Kernel Mailing List
Am 31.01.2017 um 22:43 schrieb Heiner Kallweit:
> Hi Bjorn,
>
> I just came across a use case where I wanted to deal with optional
> regulators in bulk operations. I was about to submit a related
> patch when I saw that you submitted basically the same in
> 3ff3f518a135 "regulator: Make bulk API support optional supplies"
> and reverted it later stating that you misunderstood the API.
>
> How does this extension break the API? To me it sounds like a
> neat feature.
>
> The only thing I can think of if that we have to assume that
> struct regulator_bulk_data is zero-initialized to ensure
> backwards-compatibility (use "normal" get as default).
>
> Rgds, Heiner
>
Sorry for the noise, I got it meanwhile.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-01-31 23:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-31 21:43 Optional regulators in bulk operations - reverted patch Heiner Kallweit
2017-01-31 21:59 ` Heiner Kallweit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®