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

Groups > linux.kernel > #1213492

Re: [PATCH v4 5/5] dt/bindings: bcm2835: Add binding documentation for auxiliar spi devices

Show all headers | View raw

On 08/24/2015 02:40 AM, kernel@martin.sperl.org wrote:
> From: Martin Sperl <kernel@martin.sperl.org>

Patch description?

> diff --git a/Documentation/devicetree/bindings/spi/brcm,bcm2835-aux-spi.txt b/Documentation/devicetree/bindings/spi/brcm,bcm2835-aux-spi.txt

> +Required properties:
> +- compatible: Should be "brcm,bcm2835-aux-spi".
> +- reg: Should contain register location and length for the spi block
> +       as well as for the common aux block control

Is that meant to imply that reg should contain a single value that
covers both the common aux registers and the SPI device, or two separate
values, one for the aux common registers and another for the SPI device?
Neither of those options sound correct. I would expect only a single
entry which covered solely the SPI registers. The common aux registers
are owned by the other brcm,bcm2835-aux binding.

> +Example:
> +
> +spi1@7e215080 {
> +	compatible = "brcm,bcm2835-aux-spi";
> +	reg = <0x7e215080 0x40>;

That seems to match what I'd expect, but doesn't correspond to the
description above.

> +/* the necessary syscon config referenced above*/
> +aux_enable: aux_enable@0x7e215004 {

It's not a "syscon"...

> +Note that it also requires the GPIOs to be set up with the
> +correct ALT-functions.
> +
> +For spi1 the following pins need to be set as:
> +* ALT4: 19, 20, 21 (MISO, MOSI, SCK)
> +
> +For spi2 the following pins need to be set as:
> +* ALT4: 40, 41, 42 (MISO, MOSI, SCK)
> +
> +CS-GPIOS need to get set as output - typically:
> +* spi1: 18, 17, 16 (CS0, CS1, CS2)
> +* spi2: 43, 44, 45 (CS0, CS1, CS2)

That's generally true of any HW block, and has nothing to do with the
binding for the device. I would suggest removing that chunk of text.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

Thread

[PATCH v4 0/5] bcm2835: auxiliar device support for spi kernel@martin.sperl.org - 2015-08-24 10:50 +0200
  [PATCH v4 5/5] dt/bindings: bcm2835: Add binding documentation for auxiliar spi devices kernel@martin.sperl.org - 2015-08-24 10:50 +0200
    Re: [PATCH v4 5/5] dt/bindings: bcm2835: Add binding documentation  for auxiliar spi devices Stephen Warren <swarren@wwwdotorg.org> - 2015-08-26 03:50 +0200
  [PATCH v4 4/5] spi: bcm2835: new driver implementing auxiliar spi1/spi2 on the bcm2835 soc kernel@martin.sperl.org - 2015-08-24 10:50 +0200
    Re: [PATCH v4 4/5] spi: bcm2835: new driver implementing auxiliar  spi1/spi2 on the bcm2835 soc Stephen Warren <swarren@wwwdotorg.org> - 2015-08-26 04:00 +0200
  [PATCH v4 3/5] dt/bindings: bcm2835: add binding documentation for bcm2835-aux kernel@martin.sperl.org - 2015-08-24 10:50 +0200
    Re: [PATCH v4 3/5] dt/bindings: bcm2835: add binding documentation  for bcm2835-aux Stephen Warren <swarren@wwwdotorg.org> - 2015-08-26 03:50 +0200
      Re: [PATCH v4 3/5] dt/bindings: bcm2835: add binding documentation for bcm2835-aux Martin Sperl <kernel@martin.sperl.org> - 2015-09-04 10:30 +0200

csiph-web