[v3,2/5] uio: dt-bindings: Add binding for "st,stm32f100"

Message ID 20171009224655.6173-3-chris.packham@alliedtelesis.co.nz
State Rejected
Headers show
Series
  • using uio_pdrv_genirq without module param
Related show

Commit Message

Chris Packham Oct. 9, 2017, 10:46 p.m.
The stm32f100 is a general purpose micro controller. Document a binding
that allows a user-space driver to be implemented for these devices.

Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Changes in v3:
- (replaces "uio: dt-bindings: document binding for uio-pdrv-genirq")
- split the bindings up into the devices I'm actually wanting to support. The
  first is a stm32f100 micro controller.

 Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Rob Herring Oct. 13, 2017, 8:33 p.m. | #1
On Tue, Oct 10, 2017 at 11:46:52AM +1300, Chris Packham wrote:
> The stm32f100 is a general purpose micro controller. Document a binding
> that allows a user-space driver to be implemented for these devices.

It may be general purpose, but how it is hooked up to the host is not 
and probably could be done in a variety of ways. I'd expect that to be 
described in a binding doc (that's not a suggestion to add it to this 
doc as I'm fundamentally against any binding with "UIO" in it).

Also, while the processor itself is general purpose, typically its 
function on a given board would not be except maybe on dev boards. It's 
that function (a combination of the uC and its firmware) which we want 
to describe to the host OS. There's already examples of uC used as PMICs 
or system controllers.

Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt b/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt
index 5a92b1f8825f..fb4c1b5059f2 100644
--- a/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt
+++ b/Documentation/devicetree/bindings/uio/uio-pdrv-genirq.txt
@@ -1,7 +1,8 @@ 
 *Generic UIO platform driver with interrupts
 
 Required properties:
-- compatible: Should be a value set with the of_id module parameter.
+- compatible: Should be a value set with the of_id module parameter or the
+  built in value "st,stm32f100".
 
 Optional properties:
 - interrupts: Should contain the interrupt to be associated with this device