于 2012年07月27日 05:58, Kevin Hilman 写道:
"zumeng.chen"<zchen@xxxxxxxxxxxxx> writes:
On 2012年07月27日 03:30, Kevin Hilman wrote:
+ Zumeng Chen
Igor Grinberg<grinberg@xxxxxxxxxxxxxx> writes:
Hi Kevin,
I've just noticed that the patch has been modified by Arnd in a way
that of course will trigger GPIO use without being requested.
I'm sorry, I was not available by that time Arnd changed the patch.
Your right, your original patch isn't the problem. I found the root
cause.
The real problem is actually introduced by the merge of your patch from
the arm-soc/cleanup branch, and this one from Zumeng Chen: commit
16aced80f6 (ARM: OMAP3530evm: set pendown_state and debounce time for
ads7846) from the arm-soc/boards branch.
However, looking closer at the one from Zumeng, that one is clearly not
right. It unconditionally adds a *board-specific* ->get_pendown_state
function to the pdata that is common to *all* boards. That's just wrong
and has the side-effect of making ->get_pendown_state() wrong on every
board except the OMAP3EVM. Oops.
So, IMO, in addition to $SUBJECT patch, in order to get the touchscreen
GPIO working on non OMAP3EVM boards, we also need something like this as
well.
Definitely, thanks Kevin.
Igor, Zumeng, could you try this out on your boards anc confirm if it's
working? I currently don't have a board setup with a touchscreen in my
board farm.
Acked
Did you test this on your board? If so, could you respond with a
Tested-by tag? Thanks.
Hi Kevin,
Your patch self has no problem, feel free to add
"Tested-by: Zumeng Chen <zumeng.chen@xxxxxxxxxxxxx>
But obviously, the current ads7846 doesn't work, I have found
one reason, another reason is about spi-dma update, I'm trying
to fix it.
Regards,
Zumeng
Kevin
--
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
