Messages in this thread | | | From | Doug Anderson <> | Date | Wed, 27 Jun 2018 09:28:03 -0700 | Subject | Re: [PATCH v8 2/2] regulator: add QCOM RPMh regulator driver |
| |
Hi,
On Wed, Jun 27, 2018 at 8:01 AM, Mark Brown <broonie@kernel.org> wrote: > On Tue, Jun 26, 2018 at 11:15:14AM -0700, Doug Anderson wrote: > >> I think he's still planning on re-shuffling his tree a bit. When he >> does this, do you need him to put the RPMh patches somewhere you can >> merge into your tree? > > Well, I *think* there's no actual dependency here since it's a new > driver with a Kconfig dependency. It really just needs me to get round > to trawling through what's a fairly large patch with a troubled history > now you've reviewed it.
OK, great. I guess I'm confused about the "|| COMPILE_TEST" causing problems then? I was worried that anyone trying to do "COMPILE_TEST" on your tree (or linuxnext if RPMh isn't there) would get failures due to the lack of header files. I guess if it's a problem you could just gut the "|| COMPILE_TEST" and it could be added back in later?
Hoping my reviews saved you time overall.
-Doug
| |