diff mbox

pinctrl: always include all subdirectories in build

Message ID 1444578044-17804-1-git-send-email-jogo@openwrt.org
State New
Headers show

Commit Message

Jonas Gorski Oct. 11, 2015, 3:40 p.m. UTC
All pinctrl drivers in subdirectories are properly guarded by individual
configuration symbols, so there is no need to limit the directories to
the belonging machines.

This allows wider COMPILE_TEST coverage on "foreign" targets.

Signed-off-by: Jonas Gorski <jogo@openwrt.org>
---
 drivers/pinctrl/Makefile | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

Comments

Viresh Kumar Oct. 11, 2015, 4:27 p.m. UTC | #1
On 11-10-15, 17:40, Jonas Gorski wrote:
> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
> 
> This allows wider COMPILE_TEST coverage on "foreign" targets.
> 
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>
> ---
>  drivers/pinctrl/Makefile | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
> 

For SPEAr:

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Matthias Brugger Oct. 11, 2015, 6:36 p.m. UTC | #2
On 11/10/15 17:40, Jonas Gorski wrote:
> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
>
> This allows wider COMPILE_TEST coverage on "foreign" targets.
>
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>
> ---
>   drivers/pinctrl/Makefile | 24 ++++++++++++------------
>   1 file changed, 12 insertions(+), 12 deletions(-)

For Mediatek:

Acked-by: Matthias Brugger <matthias.bgg@gmail.com>

>
> diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
> index cad077c..2eb25a8 100644
> --- a/drivers/pinctrl/Makefile
> +++ b/drivers/pinctrl/Makefile
> @@ -39,17 +39,17 @@ obj-$(CONFIG_PINCTRL_TB10X)	+= pinctrl-tb10x.o
>   obj-$(CONFIG_PINCTRL_ST) 	+= pinctrl-st.o
>   obj-$(CONFIG_PINCTRL_ZYNQ)	+= pinctrl-zynq.o
>
> -obj-$(CONFIG_ARCH_BCM)		+= bcm/
> -obj-$(CONFIG_ARCH_BERLIN)	+= berlin/
> +obj-y				+= bcm/
> +obj-y				+= berlin/
>   obj-y				+= freescale/
> -obj-$(CONFIG_X86)		+= intel/
> -obj-$(CONFIG_PLAT_ORION)        += mvebu/
> +obj-y				+= intel/
> +obj-y				+= mvebu/
>   obj-y				+= nomadik/
> -obj-$(CONFIG_ARCH_QCOM)		+= qcom/
> -obj-$(CONFIG_PINCTRL_SAMSUNG)	+= samsung/
> -obj-$(CONFIG_PINCTRL_SH_PFC)	+= sh-pfc/
> -obj-$(CONFIG_PLAT_SPEAR)	+= spear/
> -obj-$(CONFIG_ARCH_SUNXI)	+= sunxi/
> -obj-$(CONFIG_ARCH_UNIPHIER)	+= uniphier/
> -obj-$(CONFIG_ARCH_VT8500)	+= vt8500/
> -obj-$(CONFIG_ARCH_MEDIATEK)	+= mediatek/
> +obj-y				+= qcom/
> +obj-y				+= samsung/
> +obj-y				+= sh-pfc/
> +obj-y				+= spear/
> +obj-y				+= sunxi/
> +obj-y				+= uniphier/
> +obj-y				+= vt8500/
> +obj-y				+= mediatek/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Laurent Pinchart Oct. 11, 2015, 7:15 p.m. UTC | #3
Hi Jonas,

(Reducing the CC list a bit to avoid spamming people)

Thank you for the patch.

On Sunday 11 October 2015 17:40:44 Jonas Gorski wrote:
> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
> 
> This allows wider COMPILE_TEST coverage on "foreign" targets.
> 
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>

For sh-pfc,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  drivers/pinctrl/Makefile | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
> index cad077c..2eb25a8 100644
> --- a/drivers/pinctrl/Makefile
> +++ b/drivers/pinctrl/Makefile
> @@ -39,17 +39,17 @@ obj-$(CONFIG_PINCTRL_TB10X)	+= pinctrl-tb10x.o
>  obj-$(CONFIG_PINCTRL_ST) 	+= pinctrl-st.o
>  obj-$(CONFIG_PINCTRL_ZYNQ)	+= pinctrl-zynq.o
> 
> -obj-$(CONFIG_ARCH_BCM)		+= bcm/
> -obj-$(CONFIG_ARCH_BERLIN)	+= berlin/
> +obj-y				+= bcm/
> +obj-y				+= berlin/
>  obj-y				+= freescale/
> -obj-$(CONFIG_X86)		+= intel/
> -obj-$(CONFIG_PLAT_ORION)        += mvebu/
> +obj-y				+= intel/
> +obj-y				+= mvebu/
>  obj-y				+= nomadik/
> -obj-$(CONFIG_ARCH_QCOM)		+= qcom/
> -obj-$(CONFIG_PINCTRL_SAMSUNG)	+= samsung/
> -obj-$(CONFIG_PINCTRL_SH_PFC)	+= sh-pfc/
> -obj-$(CONFIG_PLAT_SPEAR)	+= spear/
> -obj-$(CONFIG_ARCH_SUNXI)	+= sunxi/
> -obj-$(CONFIG_ARCH_UNIPHIER)	+= uniphier/
> -obj-$(CONFIG_ARCH_VT8500)	+= vt8500/
> -obj-$(CONFIG_ARCH_MEDIATEK)	+= mediatek/
> +obj-y				+= qcom/
> +obj-y				+= samsung/
> +obj-y				+= sh-pfc/
> +obj-y				+= spear/
> +obj-y				+= sunxi/
> +obj-y				+= uniphier/
> +obj-y				+= vt8500/
> +obj-y				+= mediatek/
Thomas Petazzoni Oct. 12, 2015, 7:17 a.m. UTC | #4
Dear Jonas Gorski,

On Sun, 11 Oct 2015 17:40:44 +0200, Jonas Gorski wrote:
> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
> 
> This allows wider COMPILE_TEST coverage on "foreign" targets.
> 
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>
> ---
>  drivers/pinctrl/Makefile | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)

For mvebu:

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Thanks,

Thomas
Mika Westerberg Oct. 12, 2015, 10:55 a.m. UTC | #5
On Sun, Oct 11, 2015 at 05:40:44PM +0200, Jonas Gorski wrote:
> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
> 
> This allows wider COMPILE_TEST coverage on "foreign" targets.
> 
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>

...

> +obj-y				+= intel/

Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Eric Anholt Oct. 12, 2015, 7:26 p.m. UTC | #6
Jonas Gorski <jogo@openwrt.org> writes:

> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
>
> This allows wider COMPILE_TEST coverage on "foreign" targets.

Thanks!  Unrelatedly, I actually ended up being surprised by clk-bcm2835
being gated on CONFIG_ARCH_BCM today.  For bcm2835:

Acked-by: Eric Anholt <eric@anholt.net>
Bjorn Andersson Oct. 13, 2015, 3:59 p.m. UTC | #7
On Sun, Oct 11, 2015 at 8:40 AM, Jonas Gorski <jogo@openwrt.org> wrote:
[..]
> diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
[..]
> +obj-y                          += qcom/

Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>

Regards,
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ray Jui Oct. 15, 2015, 5:08 a.m. UTC | #8
On 10/11/2015 8:40 AM, Jonas Gorski wrote:
> All pinctrl drivers in subdirectories are properly guarded by individual
> configuration symbols, so there is no need to limit the directories to
> the belonging machines.
> 
> This allows wider COMPILE_TEST coverage on "foreign" targets.
> 
> Signed-off-by: Jonas Gorski <jogo@openwrt.org>
> ---
>  drivers/pinctrl/Makefile | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)

For ARCH_BCM,

Acked-by: Ray Jui <rjui@broadcom.com>

Thanks,

Ray
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Masahiro Yamada Oct. 15, 2015, 6:10 a.m. UTC | #9
SGkuDQoNCg0KPiBGcm9tOiBsaW51eC1ncGlvLW93bmVyQHZnZXIua2VybmVsLm9yZw0KPiBbbWFp
bHRvOmxpbnV4LWdwaW8tb3duZXJAdmdlci5rZXJuZWwub3JnXSBPbiBCZWhhbGYgT2YgSm9uYXMg
R29yc2tpDQo+IFNlbnQ6IE1vbmRheSwgT2N0b2JlciAxMiwgMjAxNSAxMjo0MSBBTQ0KPiBUbzog
bGludXgtZ3Bpb0B2Z2VyLmtlcm5lbC5vcmcNCj4gQ2M6IExpbnVzIFdhbGxlaWo7IEFsZXhhbmRy
ZSBDb3VyYm90OyBUaG9tYXMgUGV0YXp6b25pOyBTdGVwaGVuIFdhcnJlbjsNCj4gTGVlIEpvbmVz
OyBFcmljIEFuaG9sdDsgRmxvcmlhbiBGYWluZWxsaTsgUmF5IEp1aTsgU2NvdHQgQnJhbmRlbjsg
TWlrYQ0KPiBXZXN0ZXJiZXJnOyBIZWlra2kgS3JvZ2VydXM7IE1hdHRoaWFzIEJydWdnZXI7IE1h
eGltZSBDb3F1ZWxpbjsgS3VtYXIgR2FsYTsNCj4gQW5keSBHcm9zczsgRGF2aWQgQnJvd247IFZp
cmVzaCBLdW1hcjsgVG9tYXN6IEZpZ2E7IFRvbnkgUHJpc2s7IFNlYmFzdGlhbg0KPiBIZXNzZWxi
YXJ0aDsgTGF1cmVudCBQaW5jaGFydDsgWWFtYWRhLCBNYXNhaGlyby8bJEI7M0VEGyhCIBskQj8/
OTAbKEINCj4gU3ViamVjdDogW1BBVENIXSBwaW5jdHJsOiBhbHdheXMgaW5jbHVkZSBhbGwgc3Vi
ZGlyZWN0b3JpZXMgaW4gYnVpbGQNCj4gDQo+IEFsbCBwaW5jdHJsIGRyaXZlcnMgaW4gc3ViZGly
ZWN0b3JpZXMgYXJlIHByb3Blcmx5IGd1YXJkZWQgYnkgaW5kaXZpZHVhbA0KPiBjb25maWd1cmF0
aW9uIHN5bWJvbHMsIHNvIHRoZXJlIGlzIG5vIG5lZWQgdG8gbGltaXQgdGhlIGRpcmVjdG9yaWVz
IHRvIHRoZQ0KPiBiZWxvbmdpbmcgbWFjaGluZXMuDQo+IA0KPiBUaGlzIGFsbG93cyB3aWRlciBD
T01QSUxFX1RFU1QgY292ZXJhZ2Ugb24gImZvcmVpZ24iIHRhcmdldHMuDQo+IA0KPiBTaWduZWQt
b2ZmLWJ5OiBKb25hcyBHb3Jza2kgPGpvZ29Ab3BlbndydC5vcmc+DQo+IC0tLQ0KPiAgZHJpdmVy
cy9waW5jdHJsL01ha2VmaWxlIHwgMjQgKysrKysrKysrKysrLS0tLS0tLS0tLS0tDQo+ICAxIGZp
bGUgY2hhbmdlZCwgMTIgaW5zZXJ0aW9ucygrKSwgMTIgZGVsZXRpb25zKC0pDQo+IA0KPiBkaWZm
IC0tZ2l0IGEvZHJpdmVycy9waW5jdHJsL01ha2VmaWxlIGIvZHJpdmVycy9waW5jdHJsL01ha2Vm
aWxlIGluZGV4DQo+IGNhZDA3N2MuLjJlYjI1YTggMTAwNjQ0DQo+IC0tLSBhL2RyaXZlcnMvcGlu
Y3RybC9NYWtlZmlsZQ0KPiArKysgYi9kcml2ZXJzL3BpbmN0cmwvTWFrZWZpbGUNCj4gQEAgLTM5
LDE3ICszOSwxNyBAQCBvYmotJChDT05GSUdfUElOQ1RSTF9UQjEwWCkJKz0gcGluY3RybC10YjEw
eC5vDQo+ICBvYmotJChDT05GSUdfUElOQ1RSTF9TVCkgCSs9IHBpbmN0cmwtc3Qubw0KPiAgb2Jq
LSQoQ09ORklHX1BJTkNUUkxfWllOUSkJKz0gcGluY3RybC16eW5xLm8NCj4gDQo+IC1vYmotJChD
T05GSUdfQVJDSF9CQ00pCQkrPSBiY20vDQo+IC1vYmotJChDT05GSUdfQVJDSF9CRVJMSU4pCSs9
IGJlcmxpbi8NCj4gK29iai15CQkJCSs9IGJjbS8NCj4gK29iai15CQkJCSs9IGJlcmxpbi8NCj4g
IG9iai15CQkJCSs9IGZyZWVzY2FsZS8NCj4gLW9iai0kKENPTkZJR19YODYpCQkrPSBpbnRlbC8N
Cj4gLW9iai0kKENPTkZJR19QTEFUX09SSU9OKSAgICAgICAgKz0gbXZlYnUvDQo+ICtvYmoteQkJ
CQkrPSBpbnRlbC8NCj4gK29iai15CQkJCSs9IG12ZWJ1Lw0KPiAgb2JqLXkJCQkJKz0gbm9tYWRp
ay8NCj4gLW9iai0kKENPTkZJR19BUkNIX1FDT00pCQkrPSBxY29tLw0KPiAtb2JqLSQoQ09ORklH
X1BJTkNUUkxfU0FNU1VORykJKz0gc2Ftc3VuZy8NCj4gLW9iai0kKENPTkZJR19QSU5DVFJMX1NI
X1BGQykJKz0gc2gtcGZjLw0KPiAtb2JqLSQoQ09ORklHX1BMQVRfU1BFQVIpCSs9IHNwZWFyLw0K
PiAtb2JqLSQoQ09ORklHX0FSQ0hfU1VOWEkpCSs9IHN1bnhpLw0KPiAtb2JqLSQoQ09ORklHX0FS
Q0hfVU5JUEhJRVIpCSs9IHVuaXBoaWVyLw0KPiAtb2JqLSQoQ09ORklHX0FSQ0hfVlQ4NTAwKQkr
PSB2dDg1MDAvDQo+IC1vYmotJChDT05GSUdfQVJDSF9NRURJQVRFSykJKz0gbWVkaWF0ZWsvDQo+
ICtvYmoteQkJCQkrPSBxY29tLw0KPiArb2JqLXkJCQkJKz0gc2Ftc3VuZy8NCj4gK29iai15CQkJ
CSs9IHNoLXBmYy8NCj4gK29iai15CQkJCSs9IHNwZWFyLw0KPiArb2JqLXkJCQkJKz0gc3VueGkv
DQo+ICtvYmoteQkJCQkrPSB1bmlwaGllci8NCj4gK29iai15CQkJCSs9IHZ0ODUwMC8NCj4gK29i
ai15CQkJCSs9IG1lZGlhdGVrLw0KDQoNCkN1cnJlbnRseSwgSSBvbmx5IHNlZSBDT01QSUxFX1RF
U1QgdW5kZXIgIGJjbS8sIG1lZGlhdGVrLywgYW5kIHFjb20vLg0KRm9yIHRoZSBvdGhlcnMsIEti
dWlsZCB3b3VsZCBkZXNjZW5kIGludG8gdW5uZWVkZWQgc3ViZGlyZWN0b3JpZXMNCmp1c3QgZm9y
IGNyZWF0aW5nIGVtcHR5IGJ1aWx0LWluLm8gb2JqZWN0cy4NCg0KTW9yZW92ZXIsIHNhbXN1bmcv
IGFuZCBzaC1wZmMvIGFyZSBhbHJlYWR5IGVuYWJsZWQgYnkgQ09ORklHX1BJTkNUUkxfKi4NCk5v
IG5lZWQgdG8gY2hhbmdlIHRoZXNlIHR3byBhdCBsZWFzdC4NCg0KQmVzdCBSZWdhcmRzDQpNYXNh
aGlybyBZYW1hZGENCg0K
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
index cad077c..2eb25a8 100644
--- a/drivers/pinctrl/Makefile
+++ b/drivers/pinctrl/Makefile
@@ -39,17 +39,17 @@  obj-$(CONFIG_PINCTRL_TB10X)	+= pinctrl-tb10x.o
 obj-$(CONFIG_PINCTRL_ST) 	+= pinctrl-st.o
 obj-$(CONFIG_PINCTRL_ZYNQ)	+= pinctrl-zynq.o
 
-obj-$(CONFIG_ARCH_BCM)		+= bcm/
-obj-$(CONFIG_ARCH_BERLIN)	+= berlin/
+obj-y				+= bcm/
+obj-y				+= berlin/
 obj-y				+= freescale/
-obj-$(CONFIG_X86)		+= intel/
-obj-$(CONFIG_PLAT_ORION)        += mvebu/
+obj-y				+= intel/
+obj-y				+= mvebu/
 obj-y				+= nomadik/
-obj-$(CONFIG_ARCH_QCOM)		+= qcom/
-obj-$(CONFIG_PINCTRL_SAMSUNG)	+= samsung/
-obj-$(CONFIG_PINCTRL_SH_PFC)	+= sh-pfc/
-obj-$(CONFIG_PLAT_SPEAR)	+= spear/
-obj-$(CONFIG_ARCH_SUNXI)	+= sunxi/
-obj-$(CONFIG_ARCH_UNIPHIER)	+= uniphier/
-obj-$(CONFIG_ARCH_VT8500)	+= vt8500/
-obj-$(CONFIG_ARCH_MEDIATEK)	+= mediatek/
+obj-y				+= qcom/
+obj-y				+= samsung/
+obj-y				+= sh-pfc/
+obj-y				+= spear/
+obj-y				+= sunxi/
+obj-y				+= uniphier/
+obj-y				+= vt8500/
+obj-y				+= mediatek/