Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1477175

Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Tue, Sep 06, 2016 at 11:04:38AM +0800, Chen-Yu Tsai wrote:
>In short, I'm ok with it. But please put an explicit
>
>    status = "disabled";
>
>and probably a comment about how/where the peripheral can be
>used in the board dts.

Allright, I will do that in v3.

>I intended to do this for the Banana Pis. Though my original plan
>was to enable Raspberry Pi compatible peripherals by default, and
>list the other peripherals that are defined by the vendor as
>"disabled".
>
>"Defined by the vendor" means that the vendor has some sort of
>document associating the gpio header pins with the peripherals,
>as shown in:
>
>    http://www.orangepi.org/Docs/Pindefinition.html#CON3_Definition
>
>This should make it easier for the average user to enable the
>peripherals. I'm not sure we should list _all_ possible ones
>though. That would make the list very large, and some might
>end up never being used.

This is exactly what I was thinking, albeit I wanted to go a bit 
'bigger': move all unambiguous pinmux/peripheral associations from the 
H3 DTS files to the DTSI, complete them with known associations for 
which there are drivers, and have all of these disabled. Reduce the 
existing blocks in the DTS files to just "status = okay".

Peripherals requiring board-specific things (like emac, USB, voltage 
regulators, displays) stay put in the DTS.

It will explodes the scope of my proposed change a bit, but if you are 
OK with this, I will gladly do it.

Maxime, Chen-Yu: what do you think of this?

Best,

Jorik

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/7] dts: sun8i: add extra uart/i2c to H3 jorik@kippendief.biz - 2016-08-31 21:40 +0200
  [PATCH v2 3/7] dts: sun8i-h3: add i2c0/i2c1 SoC peripherals jorik@kippendief.biz - 2016-08-31 21:40 +0200
    Re: [PATCH v2 3/7] dts: sun8i-h3: add i2c0/i2c1 SoC peripherals Chen-Yu Tsai <wens@csie.org> - 2016-09-01 04:50 +0200
      Re: [PATCH v2 3/7] dts: sun8i-h3: add i2c0/i2c1 SoC peripherals Jorik Jonker <jorik@kippendief.biz> - 2016-09-01 08:40 +0200
      Re: [PATCH v2 3/7] dts: sun8i-h3: add i2c0/i2c1 SoC peripherals Chen-Yu Tsai <wens@csie.org> - 2016-09-01 08:50 +0200
  [PATCH v2 5/7] dts: sun8i-h3: add UART1-3 to Orange Pi PC jorik@kippendief.biz - 2016-08-31 21:40 +0200
  [PATCH v2 7/7] dts: sun8i-h3: add I2C0-1 to Orange Pi PC jorik@kippendief.biz - 2016-08-31 21:40 +0200
  [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus jorik@kippendief.biz - 2016-08-31 21:40 +0200
    Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-09-02 09:10 +0200
      Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Jorik Jonker <jorik@kippendief.biz> - 2016-09-03 14:10 +0200
        Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-09-05 21:40 +0200
          Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Chen-Yu Tsai <wens@csie.org> - 2016-09-06 05:10 +0200
            Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Jorik Jonker <jorik@kippendief.biz> - 2016-09-06 11:00 +0200
              Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-09-06 22:10 +0200
            Re: [PATCH v2 4/7] dts: sun8i-h3: add UART1-3 to Orange Pi Plus Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-09-06 22:10 +0200
  [PATCH v2 6/7] dts: sun8i-h3: add I2C0-1 to Orange Pi Plus jorik@kippendief.biz - 2016-08-31 21:40 +0200
  [PATCH v2 2/7] dts: sun8i-h3: add pinmux definitions for i2c0/i2c1 jorik@kippendief.biz - 2016-08-31 21:40 +0200
    Re: [PATCH v2 2/7] dts: sun8i-h3: add pinmux definitions for i2c0/i2c1 Chen-Yu Tsai <wens@csie.org> - 2016-09-01 04:50 +0200

csiph-web