Re: [PATCH] ARM: dts: sun7i: Add dts file for pcDuino 3 Nano board

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Maxime,

On Mon, Jan 26, 2015 at 11:09:21AM +0100, Maxime Ripard wrote:
> If you read Documentation/leds/leds-class.txt, the pattern to follow is:
> "devicename:colour:function"
> so it should rather be pcduino3-nano:green:usr1 and usr2, or some
> other thing.

How about pcduino3-nano:green:usr-led3/4? Or ...:usr1/2, and a comment
in the .dts explaining how they're labelled on the board?

There are a couple of other sunxi boards that have the same problem
(multiple user-controllable LEDs of the same colour):

sun4i-a10-marsboard.dts: label = "marsboard:red1:usr";
sun4i-a10-marsboard.dts: label = "marsboard:red2:usr";
sun4i-a10-marsboard.dts: label = "marsboard:red3:usr";
sun4i-a10-marsboard.dts: label = "marsboard:red4:usr";
sun9i-a80-optimus.dts: label = "optimus:led2:usr";
sun9i-a80-optimus.dts: label = "optimus:led4:usr";

I can send a patch for those if you'd like as well?

> > Should there be aliases for I2C as well, since the same situation
> > applies (i2c0 and i2c2)?
> Yep.

Added.

> > More generally, is it correct to add nodes like these (uart2, spi0,
> > i2c2) for features that appear on the Arduino GPIO headers?
> If those pins are specifically dedicated to this feature, then yes.

They're multipurpose pins on Arduino-compatible headers:
http://learn.linksprite.com/wp-content/uploads/2014/10/pcDuino3-Nano_header-Model.jpg

How you want them to be configured will depend on what you have plugged
in; for example, some Arduino add-on boards will expect J11 P1/2 to be
serial RX/TX and some will expect them to be GPIOs. So should we
configure i2c2 etc. by default (and let the user turn them into GPIOs
if required), or just provide the GPIOs (and require the user to adjust
the .dts)?

Cheers,

-- 
Adam Sampson <ats@xxxxxxxxx>                         <http://offog.org/>
--
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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux