Hi Tomeu,

On Thu, Jan 15, 2015 at 9:11 AM, Tomeu Vizoso <[email protected]> wrote:
> I have seen that coreboot configures these ones differently:
>
> -    ('sdmmc3_clk_pa6',         'sdmmc3',      None,      'none',
> False, False, False, False),
> +    ('sdmmc3_clk_pa6',         'sdmmc3',      None,      'none',
> False, True,  False, False),
> -    ('sdmmc3_clk_lb_out_pee4', 'sdmmc3',      None,      'none',
> False, False, False, False),
> +    ('sdmmc3_clk_lb_out_pee4', 'sdmmc3',      None,      'down',
> False, True,  False, False),
> -    ('sdmmc4_cmd_pt7',         'sdmmc4',      None,      'none',
> False, True,  False, False),
> +    ('sdmmc4_cmd_pt7',         'sdmmc4',      None,      'up',
> False, True,  False, False),
> -    ('dp_hpd_pff0',            'dp',          None,      'up',
> False, True,  False, False),
> +    ('dp_hpd_pff0',            'dp',          None,      'none',
> False, True,  False, False),
>
> May be good to have them match?

I believe these should all be the same between Norrin (Nyan), Big, and
Blaze, so I'd go with whatever is in the Norrin pinmux config (which I
believe is what Simon's patch is based on).

> Besides, I have collected these differences between blaze and big,
> could you check with what you know?

These are the differences you've collected between blaze and big from
the schematic or... ?

> -    ('usb_vbus_en2_pff1',      'rsvd2',       None,      'down',
> True,  False, False, False),
> +    ('usb_vbus_en2_pff1',      'usb',         None,      'up',
> False, True,  False, False),

usb_vbus_en2 is not connected on all Nyan-based boards, so I would
just use whatever the Norrin config uses.

> -    ('usb_vbus_en0_pn4',       'usb',         None,      'none',
> False, True,  True,  False),
> -    ('usb_vbus_en1_pn5',       'usb',         None,      'none',
> False, True,  True,  False),
> +    ('usb_vbus_en0_pn4',       'usb',         None,      'up',
> False, True,  False, False),
> +    ('usb_vbus_en1_pn5',       'usb',         None,      'up',
> False, True,  False, False),

I would expect these to be like Venice2/Norrin: input enabled,
open-drain, no pull.

Thanks,
Andrew
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to