On Thu, Feb 6, 2014 at 9:28 AM, Ivan T. Ivanov <iivanov@xxxxxxxxxx> wrote: > + msmgpio: pinctrl@fd510000 { > + compatible = "qcom,msm8974-pinctrl"; > + reg = <0xfd510000 0x4000>; > + gpio-controller; > + #gpio-cells = <2>; > + interrupt-controller; > + #interrupt-cells = <2>; > + interrupts = <0 208 0>; > + > + spi8_default: spi8_default { > + mosi { > + pins = "gpio45"; > + function = "blsp_spi8"; > + }; > + miso { > + pins = "gpio46"; > + function = "blsp_spi8"; > + }; > + cs { > + pins = "gpio47"; > + function = "blsp_spi8"; > + }; > + clk { > + pins = "gpio48"; > + function = "blsp_spi8"; > + }; > + }; > + }; > }; > }; I'm confused by this. Isn't this information already defined in the pinctrl-msm8x74.c driver? static const char * const blsp_spi8_groups[] = { "gpio45", "gpio46", "gpio47", "gpio48" }; -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html