Messages in this thread Patches in this message | | | Date | Fri, 13 Dec 2013 17:18:00 -0800 | From | John Stultz <> | Subject | Re: [PATCH 001/115] gpu: ion: Add ION Memory Manager |
| |
On 12/13/2013 03:50 PM, Greg KH wrote: > On Fri, Dec 13, 2013 at 02:23:35PM -0800, John Stultz wrote: >> From: Rebecca Schultz Zavin <rebecca@android.com> >> >> Signed-off-by: Rebecca Schultz Zavin <rebecca@android.com> >> [jstultz: Squished in Colin Cross' move to staging change] >> Signed-off-by: John Stultz <john.stultz@linaro.org> >> --- >> drivers/staging/android/Kconfig | 2 + >> drivers/staging/android/Makefile | 2 + > This patch breaks the build if this option is enabled. > > I suggest only adding the driver to the build _after_ it can be built, > for obvious reasons :) > > Other than this "minor" thing, I have no objection to these patches at > all. Can you redo this patch to fix this up, and maybe send another > patch adding it to the build and let me know at what point in the series > it should be applied?
Ok. Two patches attached. I ended up just yanking the build directory since disabling the config option caused trivial collisions in the series.
0001-ion-Disable-ION-as-it-doesn-t-build.patch - This can be folded in after the first patch.
0117-ion-Reenable-the-build.patch - This can be applied to the end of the series, or folded into the last patch "ion: Update system heap shrinker to use the new count/scan interface"
Let me know if you'd rather this be done some other way. Also let me know if you've still not seen the other patches in the series.
thanks -john
From eba11fdcb50d53906f1b159969448803178c173f Mon Sep 17 00:00:00 2001 From: John Stultz <john.stultz@linaro.org> Date: Fri, 13 Dec 2013 16:56:53 -0800 Subject: [PATCH] ion: Disable ION, as it doesn't build
Since the ion patches are forward ported from 3.10, they won't build. So disable it from the build for now and we'll re-enable it after things are building again.
Signed-off-by: John Stultz <john.stultz@linaro.org> --- drivers/staging/android/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/android/Makefile b/drivers/staging/android/Makefile index 0a01e191..29c5ff0 100644 --- a/drivers/staging/android/Makefile +++ b/drivers/staging/android/Makefile @@ -1,6 +1,7 @@ ccflags-y += -I$(src) # needed for trace events -obj-y += ion/ +# ION doesn't build just yet, so disable it from the build +#obj-y += ion/ obj-$(CONFIG_ANDROID_BINDER_IPC) += binder.o obj-$(CONFIG_ASHMEM) += ashmem.o -- 1.8.3.2 From 60a715a53406a68c399dcf706bdb33c83e9c42ff Mon Sep 17 00:00:00 2001 From: John Stultz <john.stultz@linaro.org> Date: Fri, 13 Dec 2013 17:12:18 -0800 Subject: [PATCH] ion: Reenable the build
Now that ION builds, reenable it in the build.
Signed-off-by: John Stultz <john.stultz@linaro.org> --- drivers/staging/android/Makefile | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/android/Makefile b/drivers/staging/android/Makefile index 29c5ff0..0a01e191 100644 --- a/drivers/staging/android/Makefile +++ b/drivers/staging/android/Makefile @@ -1,7 +1,6 @@ ccflags-y += -I$(src) # needed for trace events -# ION doesn't build just yet, so disable it from the build -#obj-y += ion/ +obj-y += ion/ obj-$(CONFIG_ANDROID_BINDER_IPC) += binder.o obj-$(CONFIG_ASHMEM) += ashmem.o -- 1.8.3.2
| |