lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20171227011936.GT7997@codeaurora.org> Date: Tue, 26 Dec 2017 17:19:36 -0800 From: Stephen Boyd <sboyd@...eaurora.org> To: sean.wang@...iatek.com Cc: mturquette@...libre.com, matthias.bgg@...il.com, jdelvare@...e.de, jamesjj.liao@...iatek.com, weiyi.lu@...iatek.com, kevin-cw.chen@...iatek.com, shunli.wang@...iatek.com, chen.zhong@...iatek.com, arnd@...db.de, linux-mediatek@...ts.infradead.org, linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] clk: mediatek: adjust dependency of reset.c to avoid unexpectedly being built On 12/26, sean.wang@...iatek.com wrote: > From: Sean Wang <sean.wang@...iatek.com> > > commit 74cb0d6dde8 ("clk: mediatek: fixup test-building of MediaTek clock > drivers") can let the build system looking into the directory where the > clock drivers resides and then allow test-building the drivers. > > But the change also gives rise to certain incorrect behavior which is > reset.c being built even not depending on either COMPILE_TEST or > ARCH_MEDIATEK alternative dependency. To get rid of reset.c being built > unexpectedly on the other platforms, it would be a good change that the > file should be built depending on its own specific configuration rather > than just on generic RESET_CONTROLLER one. > > Signed-off-by: Sean Wang <sean.wang@...iatek.com> > Cc: Jean Delvare <jdelvare@...e.de> I've typically seen vendor Kconfigs select the RESET_CONTROLLER framework if the vendor Kconfig is enabled. Any reason that same method isn't followed here? -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Powered by blists - more mailing lists