| Message ID | 20170312063820.19313-1-mzoran@crowfest.net (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Sat, Mar 11, 2017 at 10:38:17PM -0800, Michael Zoran wrote: > Add documentation for the DMA properties required when using the > PWM controller of the bcm2835 for analog audio output. > > Signed-off-by: Michael Zoran <mzoran@crowfest.net> > --- > Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > index cf573e85b11d..08fc06fc083a 100644 > --- a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > +++ b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > @@ -1,14 +1,21 @@ > BCM2835 PWM controller (Raspberry Pi controller) > > Required properties: > -- compatible: should be "brcm,bcm2835-pwm" > +- compatible: should be "brcm,bcm2835-pwm" or "brcm,bcm2835-analog-audio" Compatibles are not for selecting usage modes. You should describe the connection to the pwm that makes it an audio output rather than change the compatible. > - reg: physical base address and length of the controller's registers > - clocks: This clock defines the base clock frequency of the PWM hardware > system, the period and the duty_cycle of the PWM signal is a multiple of > the base period. > + > +Required properties(PWM Output): > - #pwm-cells: Should be 2. See pwm.txt in this directory for a description of > the cells format. > > +Required properties(Audio Output): > +- dmas: Should contain one entry pointing to the DMA channel used to > + transfer audio data > +- dma-names: Should contain "tx" > + > Examples: > > pwm@2020c000 { > -- > 2.11.0 >
On Mon, 2017-03-20 at 11:50 -0500, Rob Herring wrote: > On Sat, Mar 11, 2017 at 10:38:17PM -0800, Michael Zoran wrote: > > Add documentation for the DMA properties required when using the > > PWM controller of the bcm2835 for analog audio output. > > > > Signed-off-by: Michael Zoran <mzoran@crowfest.net> > > --- > > Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt | 9 > > ++++++++- > > 1 file changed, 8 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > index cf573e85b11d..08fc06fc083a 100644 > > --- a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > +++ b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > @@ -1,14 +1,21 @@ > > BCM2835 PWM controller (Raspberry Pi controller) > > > > Required properties: > > -- compatible: should be "brcm,bcm2835-pwm" > > +- compatible: should be "brcm,bcm2835-pwm" or "brcm,bcm2835- > > analog-audio" > > Compatibles are not for selecting usage modes. You should describe > the > connection to the pwm that makes it an audio output rather than > change > the compatible. > Cool, that's good to know that someone let me know after all this time. And please tell how if DT nodes are an exact documentation of the hardware how are virtualized DT nodes like firmware getting into the DTs?
On Mon, Mar 20, 2017 at 12:07 PM, Michael Zoran <mzoran@crowfest.net> wrote: > On Mon, 2017-03-20 at 11:50 -0500, Rob Herring wrote: >> On Sat, Mar 11, 2017 at 10:38:17PM -0800, Michael Zoran wrote: >> > Add documentation for the DMA properties required when using the >> > PWM controller of the bcm2835 for analog audio output. >> > >> > Signed-off-by: Michael Zoran <mzoran@crowfest.net> >> > --- >> > Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt | 9 >> > ++++++++- >> > 1 file changed, 8 insertions(+), 1 deletion(-) >> > >> > diff --git a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > index cf573e85b11d..08fc06fc083a 100644 >> > --- a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > +++ b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > @@ -1,14 +1,21 @@ >> > BCM2835 PWM controller (Raspberry Pi controller) >> > >> > Required properties: >> > -- compatible: should be "brcm,bcm2835-pwm" >> > +- compatible: should be "brcm,bcm2835-pwm" or "brcm,bcm2835- >> > analog-audio" >> >> Compatibles are not for selecting usage modes. You should describe >> the >> connection to the pwm that makes it an audio output rather than >> change >> the compatible. >> > > Cool, that's good to know that someone let me know after all this time. Well, there are more submitters than reviewers and maintainers and we have travel and vacation (aka lives). > And please tell how if DT nodes are an exact documentation of the > hardware how are virtualized DT nodes like firmware getting into the > DTs? DT is software's view of the hardware. Firmware provides interfaces to the OS just like a h/w device does. And just like h/w devices, the firmware is not discoverable and needs to be described. Originally, DT was the firmware's interface to the OS with OpenFirmware. Rob
On Tue, 2017-03-21 at 07:53 -0500, Rob Herring wrote: > On Mon, Mar 20, 2017 at 12:07 PM, Michael Zoran <mzoran@crowfest.net> > wrote: > > On Mon, 2017-03-20 at 11:50 -0500, Rob Herring wrote: > > > On Sat, Mar 11, 2017 at 10:38:17PM -0800, Michael Zoran wrote: > > > > Add documentation for the DMA properties required when using > > > > the > > > > PWM controller of the bcm2835 for analog audio output. > > > > > > > > Signed-off-by: Michael Zoran <mzoran@crowfest.net> > > > > --- > > > > Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt | 9 > > > > ++++++++- > > > > 1 file changed, 8 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/Documentation/devicetree/bindings/pwm/pwm- > > > > bcm2835.txt > > > > b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > > > index cf573e85b11d..08fc06fc083a 100644 > > > > --- a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > > > +++ b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt > > > > @@ -1,14 +1,21 @@ > > > > BCM2835 PWM controller (Raspberry Pi controller) > > > > > > > > Required properties: > > > > -- compatible: should be "brcm,bcm2835-pwm" > > > > +- compatible: should be "brcm,bcm2835-pwm" or "brcm,bcm2835- > > > > analog-audio" > > > > > > Compatibles are not for selecting usage modes. You should > > > describe > > > the > > > connection to the pwm that makes it an audio output rather than > > > change > > > the compatible. > > > > > > > Cool, that's good to know that someone let me know after all this > > time. > > Well, there are more submitters than reviewers and maintainers and we > have travel and vacation (aka lives). > > > And please tell how if DT nodes are an exact documentation of the > > hardware how are virtualized DT nodes like firmware getting into > > the > > DTs? > > DT is software's view of the hardware. Firmware provides interfaces > to > the OS just like a h/w device does. And just like h/w devices, the > firmware is not discoverable and needs to be described. Originally, > DT > was the firmware's interface to the OS with OpenFirmware. > > Rob OK, I understand better know. So it sounds like what I really want to do is be able to keep the existing DT as much as possible but load an alternative driver. Presumably at the request of the end user some how. I'm not sure something like that is supported in the linux kernel. For this specific example, I don't know of any reason I couldn't write a replacement driver for the existing PWM control that does both audio and standard PWM, but I've heard replacing existing drivers is really accepted and having alternative drivers isn't something that's supported.
On Tue, Mar 21, 2017 at 8:30 AM, Michael Zoran <mzoran@crowfest.net> wrote: > On Tue, 2017-03-21 at 07:53 -0500, Rob Herring wrote: >> On Mon, Mar 20, 2017 at 12:07 PM, Michael Zoran <mzoran@crowfest.net> >> wrote: >> > On Mon, 2017-03-20 at 11:50 -0500, Rob Herring wrote: >> > > On Sat, Mar 11, 2017 at 10:38:17PM -0800, Michael Zoran wrote: >> > > > Add documentation for the DMA properties required when using >> > > > the >> > > > PWM controller of the bcm2835 for analog audio output. >> > > > >> > > > Signed-off-by: Michael Zoran <mzoran@crowfest.net> >> > > > --- >> > > > Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt | 9 >> > > > ++++++++- >> > > > 1 file changed, 8 insertions(+), 1 deletion(-) >> > > > >> > > > diff --git a/Documentation/devicetree/bindings/pwm/pwm- >> > > > bcm2835.txt >> > > > b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > > > index cf573e85b11d..08fc06fc083a 100644 >> > > > --- a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > > > +++ b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt >> > > > @@ -1,14 +1,21 @@ >> > > > BCM2835 PWM controller (Raspberry Pi controller) >> > > > >> > > > Required properties: >> > > > -- compatible: should be "brcm,bcm2835-pwm" >> > > > +- compatible: should be "brcm,bcm2835-pwm" or "brcm,bcm2835- >> > > > analog-audio" >> > > >> > > Compatibles are not for selecting usage modes. You should >> > > describe >> > > the >> > > connection to the pwm that makes it an audio output rather than >> > > change >> > > the compatible. >> > > >> > >> > Cool, that's good to know that someone let me know after all this >> > time. >> >> Well, there are more submitters than reviewers and maintainers and we >> have travel and vacation (aka lives). >> >> > And please tell how if DT nodes are an exact documentation of the >> > hardware how are virtualized DT nodes like firmware getting into >> > the >> > DTs? >> >> DT is software's view of the hardware. Firmware provides interfaces >> to >> the OS just like a h/w device does. And just like h/w devices, the >> firmware is not discoverable and needs to be described. Originally, >> DT >> was the firmware's interface to the OS with OpenFirmware. >> >> Rob > > OK, I understand better know. So it sounds like what I really want to > do is be able to keep the existing DT as much as possible but load an > alternative driver. Presumably at the request of the end user some > how. > > I'm not sure something like that is supported in the linux kernel. For > this specific example, I don't know of any reason I couldn't write a > replacement driver for the existing PWM control that does both audio > and standard PWM, but I've heard replacing existing drivers is really > accepted and having alternative drivers isn't something that's > supported. 2 drivers for the same compatible would be a bit of a problem without any other information to tell one of the drivers not to probe. What I had in mind was adding a pwm-audio node (describing what's connected to the PWM output) similar to what we have for pwm-backlight. We already have pwm-beeper as well. In theory, then a pwm-audio driver could work with any PWM. In reality, the overhead would be too high for dumb PWMs. You would need some batching support in the PWM subsystem in order to use DMA effectively, and I don't know if the PWM subsystem supports that currently. Rob
diff --git a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt index cf573e85b11d..08fc06fc083a 100644 --- a/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt +++ b/Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt @@ -1,14 +1,21 @@ BCM2835 PWM controller (Raspberry Pi controller) Required properties: -- compatible: should be "brcm,bcm2835-pwm" +- compatible: should be "brcm,bcm2835-pwm" or "brcm,bcm2835-analog-audio" - reg: physical base address and length of the controller's registers - clocks: This clock defines the base clock frequency of the PWM hardware system, the period and the duty_cycle of the PWM signal is a multiple of the base period. + +Required properties(PWM Output): - #pwm-cells: Should be 2. See pwm.txt in this directory for a description of the cells format. +Required properties(Audio Output): +- dmas: Should contain one entry pointing to the DMA channel used to + transfer audio data +- dma-names: Should contain "tx" + Examples: pwm@2020c000 {
Add documentation for the DMA properties required when using the PWM controller of the bcm2835 for analog audio output. Signed-off-by: Michael Zoran <mzoran@crowfest.net> --- Documentation/devicetree/bindings/pwm/pwm-bcm2835.txt | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)