Re: [PATCH] openrisc: Add DTS and defconfig for DE0-Nano

From: Geert Uytterhoeven
Date: Fri Nov 15 2013 - 05:22:34 EST


Hi Jonas,

On Fri, Nov 15, 2013 at 10:50 AM, Jonas Bonn <jonas@xxxxxxxxxxxx> wrote:
>> +++ b/arch/openrisc/boot/dts/de0_nano.dts
>> @@ -0,0 +1,158 @@
>> +/dts-v1/;
>> +/ {
>> + compatible = "opencores,de0_nano";
>
> This should probably be "altera,de0_nano". We also need to put a version

"terasic,de0_nano"? The board says "Designed and manufactured by Terasic".
Hmm, the sticker on the box says "Altera DE0-Nano".

> number of some kind on this thing. How are the de0_nano boards versioned?

That's a good question...

See also Andrew back's email and my response.

The supplied schematics filename is de0-nano-c4-rev-c\(release_cd_rom\).pdf,
and the drawings mention "Rev C".
The block diagram on page 2 still says "EPCS16 (16 Mb), but on page 8 it
mentions "EPCS64", which is what's on my board.

>> + i2c0: ocores@a0000000 {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + compatible = "opencores,i2c-ocores";
>
>
> Version number needed. OpenCores wanted "projectname-rtlsvn###" where ###
> is the SVN commit number of the RTL directory in the project's source
> repository.

That's gonna need some information diving, as Stefan's orpsoc git repositories
only contains references to orpsocv2 git-svn-ids, not the svn ids of
the original
component repositories.

>> + gpio0: gpio@91000000 {
>> + compatible = "opencores,jbtrivial";
>
>
> There's no project called 'jbtrivial' at OpenCores. Should this be
> "linux,jbtrivial" to indicate that this refers to a Linux driver?
>
>> + reg = <0x91000000 0x2>;
>> + #gpio-cells = <2>;
>> + gpio-controller;
>> + xlnx,data-offset = <0>;
>> + xlnx,tri-offset = <1>;
>> + xlnx,gpio-width = <8>;
>
>
> Yuck... I wrote this driver with a similar driver for a Xilinx part as a
> starting point. Apparently I left the Xilenx (xlnx) tags in there for the
> device properties. We can't really have it like this, though.
>
> Like I mentioned earlier, I'd like to see this thing disappear altogether
> and be replaced with gpio-generic.

That needs device-tree support and published bindings first. IIRC, you
had some preliminary code?

Thanks for review!

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/