On Thu, 21 Jan 2016, Moritz Fischer wrote:
> Hi Alan,
>
> minor nits inline:
>
> On Wed, Jan 20, 2016 at 8:24 PM, <atull@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> > v15: Add altr,fpga-bus implementation
> > Change compatible string "fpga-area" -> "altr,fpga-area"
>
> Doesn't look that way down there. Or am I reading the code wrong?
Hi Moritz,
Yes, I left that out accidentally. But hopefully we won't have
pushback around this issue and I can leave it out permanently
and also change altr,fpga-bus to fpga-bus.
Alan
>
> > +static const struct of_device_id fpga_area_of_match[] = {
> > + { .compatible = "fpga-area", },
>
> I'm fine with keeping it as fpga-area, this isn't altera specific imho
>
> Cheers,
>
> Moritz
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
