| Message ID | 20191113124734.27984-2-peter.ujfalusi@ti.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | cdacc761dae1cbd6475ac79f0e732f2b1ca021e0 |
| Headers | show |
| Series | ASoC: pcm3168a: Poor man's RST gpio handling | expand |
On Wed, Nov 13, 2019 at 02:47:33PM +0200, Peter Ujfalusi wrote: > On boards where the RST line is not pulled up, but it is connected to a > GPIO line this property must present in order to be able to enable the > codec. Please submit patches using subject lines reflecting the style for the subsystem, this makes it easier for people to identify relevant patches. Look at what existing commits in the area you're changing are doing and make sure your subject lines visually resemble what they're doing. There's no need to resubmit to fix this alone.
Mark, On 18/11/2019 15.08, Mark Brown wrote: > On Wed, Nov 13, 2019 at 02:47:33PM +0200, Peter Ujfalusi wrote: >> On boards where the RST line is not pulled up, but it is connected to a >> GPIO line this property must present in order to be able to enable the >> codec. > > Please submit patches using subject lines reflecting the style for the > subsystem, this makes it easier for people to identify relevant patches. > Look at what existing commits in the area you're changing are doing and > make sure your subject lines visually resemble what they're doing. > There's no need to resubmit to fix this alone. What would be the appropriate subject line for Documentation/devicetree/bindings/sound Oops, I have missed the dt- prefix for the bindings for sure. - Péter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On Mon, Nov 18, 2019 at 04:46:26PM +0200, Peter Ujfalusi wrote: > On 18/11/2019 15.08, Mark Brown wrote: > > Please submit patches using subject lines reflecting the style for the > > subsystem, this makes it easier for people to identify relevant patches. > > Look at what existing commits in the area you're changing are doing and > > make sure your subject lines visually resemble what they're doing. > > There's no need to resubmit to fix this alone. > What would be the appropriate subject line for > Documentation/devicetree/bindings/sound > Oops, I have missed the dt- prefix for the bindings for sure. I prefer ASoC: but yeah, you missed dt-.
On Wed, Nov 13, 2019 at 02:47:33PM +0200, Peter Ujfalusi wrote: > On boards where the RST line is not pulled up, but it is connected to a > GPIO line this property must present in order to be able to enable the > codec. > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com> > --- > Documentation/devicetree/bindings/sound/ti,pcm3168a.txt | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt b/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt > index 5d9cb84c661d..f30aebc7603a 100644 > --- a/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt > +++ b/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt > @@ -25,6 +25,12 @@ Required properties: > > For required properties on SPI/I2C, consult SPI/I2C device tree documentation > > +Optional properties: > + > + - rst-gpios : Optional RST gpio line for the codec > + RST = low: device power-down > + RST = high: device is enabled 'reset-gpios' is the standard naming for reset lines. > + > Examples: > > i2c0: i2c0@0 { > @@ -34,6 +40,7 @@ i2c0: i2c0@0 { > pcm3168a: audio-codec@44 { > compatible = "ti,pcm3168a"; > reg = <0x44>; > + rst-gpios = <&gpio0 4 GPIO_ACTIVE_HIGH>; > clocks = <&clk_core CLK_AUDIO>; > clock-names = "scki"; > VDD1-supply = <&supply3v3>; > -- > Peter > > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki >
diff --git a/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt b/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt index 5d9cb84c661d..f30aebc7603a 100644 --- a/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt +++ b/Documentation/devicetree/bindings/sound/ti,pcm3168a.txt @@ -25,6 +25,12 @@ Required properties: For required properties on SPI/I2C, consult SPI/I2C device tree documentation +Optional properties: + + - rst-gpios : Optional RST gpio line for the codec + RST = low: device power-down + RST = high: device is enabled + Examples: i2c0: i2c0@0 { @@ -34,6 +40,7 @@ i2c0: i2c0@0 { pcm3168a: audio-codec@44 { compatible = "ti,pcm3168a"; reg = <0x44>; + rst-gpios = <&gpio0 4 GPIO_ACTIVE_HIGH>; clocks = <&clk_core CLK_AUDIO>; clock-names = "scki"; VDD1-supply = <&supply3v3>;
On boards where the RST line is not pulled up, but it is connected to a GPIO line this property must present in order to be able to enable the codec. Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com> --- Documentation/devicetree/bindings/sound/ti,pcm3168a.txt | 7 +++++++ 1 file changed, 7 insertions(+)