[U-Boot] [PATCH] OMAP3: igep0020: Add pad config for WIFI/BT GPIOs

[Enric Balletbo Serra]

2012/10/29 Anders Hedlund <anders.j.hedlund at gmail.com>:
> 2012/10/29 Enric Balletbo Serra <eballetbo at gmail.com>:
>> Hi,
>>
>> 2012/10/28 Anders Hedlund <anders.j.hedlund at gmail.com>:
>>> This adds support for WIFI/BT GPIOs which were previously missing.
>>>
>>> Signed-off-by: Anders Hedlund <anders.j.hedlund at gmail.com>
>>> Cc: Enric Balletbo i Serra <eballetbo at iseebcn.com>
>>> Cc: Jonas Zetterberg <jozz at jozz.se>
>>> ---
>>>  board/isee/igep0020/igep0020.h |    6 ++++++
>>>  1 file changed, 6 insertions(+)
>>>
>>> diff --git a/board/isee/igep0020/igep0020.h b/board/isee/igep0020/igep0020.h
>>> index 3335ecc..283b75e 100644
>>> --- a/board/isee/igep0020/igep0020.h
>>> +++ b/board/isee/igep0020/igep0020.h
>>> @@ -132,6 +132,12 @@ static void setup_net_chip(void);
>>>         MUX_VAL(CP(MMC1_DAT1),      (IEN  | PTU | EN  | M0)) /* MMC1_DAT1 */\
>>>         MUX_VAL(CP(MMC1_DAT2),      (IEN  | PTU | EN  | M0)) /* MMC1_DAT2 */\
>>>         MUX_VAL(CP(MMC1_DAT3),      (IEN  | PTU | EN  | M0)) /* MMC1_DAT3 */\
>>> +       MUX_VAL(CP(CAM_HS),         (IDIS | PTU | DIS | M4)) /* GPIO_94 */\
>>> +       MUX_VAL(CP(CAM_VS),         (IDIS | PTU | DIS | M4)) /* GPIO_95 */\
>>> +       MUX_VAL(CP(MMC2_DAT4),      (IDIS | PTU | DIS | M4)) /* GPIO_136 */\
>>> +       MUX_VAL(CP(MMC2_DAT5),      (IDIS | PTU | DIS | M4)) /* GPIO_137 */\
>>> +       MUX_VAL(CP(MMC2_DAT6),      (IDIS | PTU | DIS | M4)) /* GPIO_138 */\
>>> +       MUX_VAL(CP(MMC2_DAT7),      (IDIS | PTU | DIS | M4)) /* GPIO_139 */\
>>>         MUX_VAL(CP(UART3_TX_IRTX),  (IDIS | PTD | DIS | M0)) /* UART3_TX */\
>>>         MUX_VAL(CP(UART3_RX_IRRX),  (IEN  | PTD | DIS | M0)) /* UART3_RX */\
>>>         MUX_VAL(CP(I2C1_SCL),       (IEN  | PTU | EN  | M0)) /* I2C1_SCL */\
>>> --
>>> 1.7.10.4
>>>
>>
>> Thanks for the patch but u-boot should only mux the pins they use, as
>> u-boot does not use wifi this is not applicable here.
>>
>> Did you have problems with wifi ? Then we should check if these pins
>> are muxe'd at kernel level, I think so.
>
> Yes. In linux kernel v3.6 WIFI/BT does not work due to this issue. I
> also have a fix for this but in my oppinion this kind of HW
> configuration should be made in the boot loader.

This was discussed before in this ML, and the result of the discussion
was that bootloaders should set only the minimum required for the
uboot functionality and kernel boot. So the way to fix this problem is
muxing these pins in kernel.

Cheers,
   Enric

>
> Br,
> Anders
>
>>
>> Cheers,
>>    Enric