Hi,<br><br><div class="gmail_quote">On Fri, May 6, 2011 at 2:47 AM, Hans J. Koch <span dir="ltr"><<a href="mailto:hjk@hansjkoch.de">hjk@hansjkoch.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Thu, May 05, 2011 at 08:40:44AM +0200, Michal Simek wrote:<br>
<br>
I think we've got it.<br>
<br>
Thanks,<br>
Hans<br>
<div class="im"><br>
> Adding OF binding to genirq.<br>
> Version string is setup to the "devicetree".<br>
><br>
> Compatible string is not setup for now but you can add your<br>
> custom compatible string to uio_of_genirq_match structure.<br>
><br>
> For example with "vendor,device" compatible string:<br>
> static const struct of_device_id __devinitconst uio_of_genirq_match[] = {<br>
>       { .compatible = "vendor,device", },<br>
>       { /* empty for now */ },<br>
> };<br>
><br>
> Signed-off-by: Michal Simek <<a href="mailto:monstr@monstr.eu">monstr@monstr.eu</a>><br>
<br>
</div>Signed-off-by: Hans J. Koch <<a href="mailto:hjk@hansjkoch.de">hjk@hansjkoch.de</a>><br>
<div><div></div><div class="h5"><br>
> CC: Hans J. Koch <<a href="mailto:hjk@hansjkoch.de">hjk@hansjkoch.de</a>><br>
> CC: Arnd Bergmann <<a href="mailto:arnd@arndb.de">arnd@arndb.de</a>><br>
> CC: John Williams <<a href="mailto:john.williams@petalogix.com">john.williams@petalogix.com</a>><br>
> CC: Grant Likely <<a href="mailto:grant.likely@secretlab.ca">grant.likely@secretlab.ca</a>><br>
> CC: Wolfram Sang <<a href="mailto:w.sang@pengutronix.de">w.sang@pengutronix.de</a>><br>
><br></div></div></blockquote><div><br>Sorry for adding to this discussion so late...<br><br>I fail to see how a
 device described in the device tree can be bound to this generic 
driver, as the match table is empty. I read in the description:<div class="im"><br>
<br>> Compatible string is not setup for now but you can add your<br>
> custom compatible string to uio_of_genirq_match structure.<br>
><br>
> For example with "vendor,device" compatible string:<br>
> static const struct of_device_id __devinitconst uio_of_genirq_match[] = {<br>
>       { .compatible = "vendor,device", },<br>
>       { /* empty for now */ },<br>
> };<br><br></div>So, this would mean that for a given project, you need to modify the sources of the *generic* driver ?<br><br>Thanks for clarifying, <br>Thomas<br><br></div></div>