Hi Zhengwang, On Tue, 7 May 2019 at 08:54, 阮正旺 <rua...@xiaopeng.com> wrote: > > Hi Vincent, Lius, > > How do you think about this patch? :-)
It looks good to me. I'm going to merge it Thanks Vincent > > Kind Regards, > > Zhengwang > > > -------- Original Message -------- > From: 阮正旺 > Sent: Thu, 2 May 2019 15:49:04 +0800 > To: Vincent Guittot, Luis Machado > Cc: Linaro-dev > Subject: Re: [PATCH PM-QA v2] Android.mk: fix compiling errors caused by > unused parameters > > Hi Vincent, > > This is version 2nd, please help review it again, thanks! > > Kind Regards, > > Zhengwang > > > -------- Original Message -------- > From: Zhengwang Ruan > Sent: Thu, 2 May 2019 15:44:13 +0800 > To: Vincent Guittot, Luis Machado > Cc: Linaro-dev, Zhengwang Ruan > Subject: [PATCH PM-QA v2] Android.mk: fix compiling errors caused by unused > parameters > > Android enables '-Wall -Werror' compiling parameters by default, as a result > compiling errors are caused in cpuidle and utils, so add '-Wno-error' to > disable > this check. > > Signed-off-by: Zhengwang Ruan <rua...@xiaopeng.com> > --- > cpuidle/Android.mk | 1 + > utils/Android.mk | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/cpuidle/Android.mk b/cpuidle/Android.mk > index f0dc52d..041b7f6 100644 > --- a/cpuidle/Android.mk > +++ b/cpuidle/Android.mk > @@ -23,6 +23,7 @@ LOCAL_MODULE := cpuidle_killer > systemtarball: cpuidle_killer > systemimage: cpuidle_killer > LOCAL_SRC_FILES:= cpuidle_killer.c > +LOCAL_CFLAGS += -Wno-error > LOCAL_SHARED_LIBRARIES := libcutils libc > LOCAL_MODULE_TAGS := tests > LOCAL_MODULE_PATH := $(TARGET_OUT_EXECUTABLES)/pm-qa/$(module_name) > diff --git a/utils/Android.mk b/utils/Android.mk > index 0fee7bf..6484bfc 100644 > --- a/utils/Android.mk > +++ b/utils/Android.mk > @@ -11,6 +11,7 @@ define $(module_name)_etc_add_executable > systemtarball: $1 > systemimage: $1 > LOCAL_SRC_FILES := $1.c > + LOCAL_CFLAGS += -Wno-error > LOCAL_MODULE_PATH := $(TARGET_OUT_EXECUTABLES)/pm-qa/$(module_name) > include $(BUILD_EXECUTABLE) > endef > > _______________________________________________ > linaro-dev mailing list > linaro-dev@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/linaro-dev > > _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org https://lists.linaro.org/mailman/listinfo/linaro-dev