Re: [PATCH 1/4][v4] ARM: imx: imx7d-pinfunc: add gpio pad iomuxc settings

2015-07-27 Thread Linus Walleij
On Fri, Jul 17, 2015 at 5:51 AM, Shawn Guo  wrote:
> On Thu, Jul 16, 2015 at 03:34:44PM -0500, Adrian Alonso wrote:
>> * Add iMX7D SoC imx7d-pinfunc gpio pad settings
>>   
>> * iMX7D GPIO1_IO7 to GPIO1_IO0 encode the pad group id in the most
>>   significant bits of input_val to avoid group id overlap bweeten
>>   iomuxc and iomuxc-lpsr.
>>
>> Signed-off-by: Adrian Alonso 
>
> Unless you can convince me that this approach which hacks select input
> value is better than two instances one, it's an NACK for me on the whole
> patch series.

I'm dropping this patch series until this is resolved.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/4][v4] ARM: imx: imx7d-pinfunc: add gpio pad iomuxc settings

2015-07-16 Thread Shawn Guo
On Thu, Jul 16, 2015 at 03:34:44PM -0500, Adrian Alonso wrote:
> * Add iMX7D SoC imx7d-pinfunc gpio pad settings
>   
> * iMX7D GPIO1_IO7 to GPIO1_IO0 encode the pad group id in the most
>   significant bits of input_val to avoid group id overlap bweeten
>   iomuxc and iomuxc-lpsr.
> 
> Signed-off-by: Adrian Alonso 

Unless you can convince me that this approach which hacks select input
value is better than two instances one, it's an NACK for me on the whole
patch series.

Shawn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 1/4][v4] ARM: imx: imx7d-pinfunc: add gpio pad iomuxc settings

2015-07-16 Thread Adrian Alonso
* Add iMX7D SoC imx7d-pinfunc gpio pad settings
  
* iMX7D GPIO1_IO7 to GPIO1_IO0 encode the pad group id in the most
  significant bits of input_val to avoid group id overlap bweeten
  iomuxc and iomuxc-lpsr.

Signed-off-by: Adrian Alonso 
---
- Version 2: Use input_val upper 16 bits to represent pad group id
- Version 3: Resend
- Version 4: Resend

 arch/arm/boot/dts/imx7d-pinfunc.h | 116 ++
 1 file changed, 116 insertions(+)

diff --git a/arch/arm/boot/dts/imx7d-pinfunc.h 
b/arch/arm/boot/dts/imx7d-pinfunc.h
index a8d8149..9460d5c 100644
--- a/arch/arm/boot/dts/imx7d-pinfunc.h
+++ b/arch/arm/boot/dts/imx7d-pinfunc.h
@@ -15,6 +15,122 @@
  * 
  */
 
+#define MX7D_PAD_GPIO1_IO00__GPIO1_IO0   0x 0x0030 
0x 0x0 0x009B
+#define MX7D_PAD_GPIO1_IO00__PWM4_OUT0x 0x0030 
0x 0x1 0x009B
+#define MX7D_PAD_GPIO1_IO00__WDOD1_WDOG_ANY  0x 0x0030 
0x 0x2 0x009B
+#define MX7D_PAD_GPIO1_IO00__WDOD1_WDOG_B0x 0x0030 
0x 0x3 0x009B
+#define MX7D_PAD_GPIO1_IO00__WDOD1_WDOG__RST_B_DEB   0x 0x0030 
0x 0x4 0x009B
+#define MX7D_PAD_GPIO1_IO01__GPIO1_IO1   0x0004 0x0034 
0x 0x0 0x009C
+#define MX7D_PAD_GPIO1_IO01__PWM1_OUT0x0004 0x0034 
0x 0x1 0x009C
+#define MX7D_PAD_GPIO1_IO01__CCM_ENET_REF_CLK3   0x0004 0x0034 
0x 0x2 0x009C
+#define MX7D_PAD_GPIO1_IO01__SAI1_MCLK   0x0004 0x0034 
0x 0x3 0x009C
+#define MX7D_PAD_GPIO1_IO01__ANATOP_24M_OUT  0x0004 0x0034 
0x 0x4 0x009C
+#define MX7D_PAD_GPIO1_IO01__OBSERVE0_OUT0x0004 0x0034 
0x 0x6 0x009C
+#define MX7D_PAD_GPIO1_IO02__GPIO1_IO2   0x0008 0x0038 
0x 0x0 0x009D
+#define MX7D_PAD_GPIO1_IO02__PWM2_OUT0x0008 0x0038 
0x 0x1 0x009D
+#define MX7D_PAD_GPIO1_IO02__CCM_ENET_REF_CLK1   0x0008 0x0038 
0x0564 0x2 0x009D0003
+#define MX7D_PAD_GPIO1_IO02__SAI2_MCLK   0x0008 0x0038 
0x 0x3 0x009D
+#define MX7D_PAD_GPIO1_IO02__CCM_CLKO1   0x0008 0x0038 
0x 0x5 0x009D
+#define MX7D_PAD_GPIO1_IO02__OBSERVE1_OUT0x0008 0x0038 
0x 0x6 0x009D
+#define MX7D_PAD_GPIO1_IO02__USB_OTG1_ID 0x0008 0x0038 
0x0734 0x7 0x009D0003
+#define MX7D_PAD_GPIO1_IO03__GPIO1_IO3   0x000C 0x003C 
0x 0x0 0x009E
+#define MX7D_PAD_GPIO1_IO03__PWM3_OUT0x000C 0x003C 
0x 0x1 0x009E
+#define MX7D_PAD_GPIO1_IO03__CCM_ENET_REF_CLK2   0x000C 0x003C 
0x0570 0x2 0x009E0003
+#define MX7D_PAD_GPIO1_IO03__SAI3_MCLK   0x000C 0x003C 
0x 0x3 0x009E
+#define MX7D_PAD_GPIO1_IO03__CCM_CLKO2   0x000C 0x003C 
0x 0x5 0x009E
+#define MX7D_PAD_GPIO1_IO03__OBSERVE2_OUT0x000C 0x003C 
0x 0x6 0x009E
+#define MX7D_PAD_GPIO1_IO03__USB_OTG2_ID 0x000C 0x003C 
0x0730 0x7 0x009E0003
+#define MX7D_PAD_GPIO1_IO04__GPIO1_IO4   0x0010 0x0040 
0x 0x0 0x009F
+#define MX7D_PAD_GPIO1_IO04__USB_OTG1_OC 0x0010 0x0040 
0x072C 0x1 0x009F0001
+#define MX7D_PAD_GPIO1_IO04__FLEXTIMER1_CH4  0x0010 0x0040 
0x0594 0x2 0x009F0001
+#define MX7D_PAD_GPIO1_IO04__UART5_CTS_B 0x0010 0x0040 
0x0710 0x3 0x009F0004
+#define MX7D_PAD_GPIO1_IO04__I2C1_SCL0x0010 0x0040 
0x05D4 0x4 0x009F0002
+#define MX7D_PAD_GPIO1_IO04__OBSERVE3_OUT0x0010 0x0040 
0x 0x6 0x009F
+#define MX7D_PAD_GPIO1_IO05__GPIO1_IO5   0x0014 0x0044 
0x 0x0 0x00A0
+#define MX7D_PAD_GPIO1_IO05__USB_OTG1_PWR0x0014 0x0044 
0x 0x1 0x00A0
+#define MX7D_PAD_GPIO1_IO05__FLEXTIMER1_CH5  0x0014 0x0044 
0x0598 0x2 0x00A1
+#define MX7D_PAD_GPIO1_IO05__UART5_RTS_B 0x0014 0x0044 
0x0710 0x3 0x00A5
+#define MX7D_PAD_GPIO1_IO05__I2C1_SDA0x0014 0x0044 
0x05D8 0x4 0x00A2
+#define MX7D_PAD_GPIO1_IO05__OBSERVE4_OUT0x0014 0x0044 
0x 0x6 0x00A0
+#define MX7D_PAD_GPIO1_IO06__GPIO1_IO6   0x0018 0x0048 
0x 0x0 0x00A1
+#define MX7D_PAD_GPIO1_IO06__USB_OTG2_OC 0x0018 0x0048 
0x0728 0x1 0x00A10001
+#define MX7D_PAD_GPIO1_IO06__FLEXTIMER1_CH6  0x0018 0x0048 
0x059C 0x2 0x00A10001
+#define MX7D_PAD_GPIO1_IO06__UART5_RX_DATA   0x0018 0x0048 
0x0714 0x3 0x00A10004
+#define MX7D_PAD_GPIO1_IO06__I2C2_SCL0x0018 0x0048