<<snip>>
Missed this one in the previous threads.
>> > +
>> > eth_cs = OMAP3EVM_SMSC911X_CS;
>> >
>> > l3ck = clk_get(NULL, "l3_ck");
>> > @@ -136,6 +141,22 @@ static inline void __init
>> omap3evm_init_smsc911x(void)
>> > else
>> > rate = clk_get_rate(l3ck);
>> >
>> > + /* Configure ethernet controller reset gpio */
>> > + if (cpu_is_omap3430()) {
>>
>> cpu_is_omap3430() is not required, as this board init would not be
>> called otherwise.
> [Hiremath, Vaibhav] That is not quite true, why do you say this?
The board file init means that the cpu info is already identified.
Do you think that omap3evm_init_smsc911x() would be called for
other than OMAP3430?
>
>>
>> > + if (gpio_request(eth_rst, "SMSC911x gpio") < 0) {
>> > + pr_err(KERN_ERR "Failed to request GPIO7 for
>> smsc911x\n");
>> > + return;
>> > + }
>> > +
>> > + gpio_direction_output(eth_rst, 1);
>>
<<snip>>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
