<br><br><div class="gmail_quote">On Sun, Nov 28, 2010 at 4:38 PM, Benjamin Herrenschmidt <span dir="ltr"><<a href="mailto:benh@kernel.crashing.org">benh@kernel.crashing.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Wed, 2010-09-08 at 09:41 -0700, Ira W. Snyder wrote:<br>
</div><div class="im">> This adds basic support for the system controller FPGA on the OVRO CARMA<br>
> board. This patch only adds infrastructure that will be used by later<br>
> drivers.<br>
<br>
</div>Oh and another comment ...<br>
<br>
I'm not sure about drivers/fpga ... in the end, one would expect such a<br>
directory to contain stuff to manipulate FPGAs in the sense of<br>
downloading bitfiles, instanciating devices (device-tree manipulation ?)<br>
etc...<br>
<br>
>From what I see in your code, the fact that these are FPGAs is almost<br>
irrelevant, you are providing support for "carma" devices, and such are<br>
no different than some other platform device, they just happen to be<br>
implemented as FPGAs. Or am I missing something ?<br>
<br>
Cheers,<br>
Ben.<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div> </div><div>Generally, I agree with this: There doesn't seem to be anything generic about</div><div>the FPGA support you are adding, it seems very specific to the CARMA hardware.</div>
<div><br></div><div>I'm generally not opposed to drivers/fpga: I think that colocating drivers for things</div><div>that are basically FPGA compute platforms isn't necessarily a bad idea, but</div><div>I think it would be better if those devices were colocated because they shared</div>
<div>some FPGA-oriented infrastructure, which this doesn't seem to do.   Currently,</div><div>generic infrastructure has been going into drivers/char, and drivers themselves have</div><div>been going wherever they seem best.  Really what you have is 2 character devices: one</div>
<div>for configuration, and one for access.</div><div><br></div><div>Steve</div></div>