| Message ID | 1607498532-19518-1-git-send-email-spujar@nvidia.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | ASoC: audio-graph-card: Drop remote-endpoint as required property | expand |
On Wed, Dec 09, 2020 at 12:52:12PM +0530, Sameer Pujar wrote: > The remote-endpoint may not be available if it is part of some > pluggable module. One such example would be an audio card, the > Codec endpoint will not be available until it is plugged in. > Hence drop 'remote-endpoint' as a required property. Please hold off on this. I have more changes coming. > > Cc: Rob Herring <robh@kernel.org> > Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > Signed-off-by: Sameer Pujar <spujar@nvidia.com> > --- > Documentation/devicetree/bindings/sound/audio-graph-port.yaml | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > index 2005014..766e910 100644 > --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > @@ -71,9 +71,6 @@ properties: > description: CPU to Codec rate channels. > $ref: /schemas/types.yaml#/definitions/uint32 > > - required: > - - remote-endpoint > - > ports: > description: multi OF-Graph subnode > type: object > -- > 2.7.4 >
Hi Rob, >> The remote-endpoint may not be available if it is part of some >> pluggable module. One such example would be an audio card, the >> Codec endpoint will not be available until it is plugged in. >> Hence drop 'remote-endpoint' as a required property. > Please hold off on this. I have more changes coming. OK, I will wait for your patch. Kindly note that this is currently blocking series https://patchwork.kernel.org/project/alsa-devel/list/?series=391735&state=* Thanks, Sameer.
Hi Rob, >>> The remote-endpoint may not be available if it is part of some >>> pluggable module. One such example would be an audio card, the >>> Codec endpoint will not be available until it is plugged in. >>> Hence drop 'remote-endpoint' as a required property. >> Please hold off on this. I have more changes coming. Is this a good time to push for audio-graph-port changes?
Hi Rob, On 12/10/2020 8:14 PM, Sameer Pujar wrote: > Hi Rob, > >>> The remote-endpoint may not be available if it is part of some >>> pluggable module. One such example would be an audio card, the >>> Codec endpoint will not be available until it is plugged in. >>> Hence drop 'remote-endpoint' as a required property. >> Please hold off on this. I have more changes coming. Sorry to bother you again. Is it possible if we take this patch now and your remaining changes can come later? This would help to unblock below series, which is pending quite some time now. > > OK, I will wait for your patch. Kindly note that this is currently > blocking series > https://patchwork.kernel.org/project/alsa-devel/list/?series=391735&state=*
diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml index 2005014..766e910 100644 --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml @@ -71,9 +71,6 @@ properties: description: CPU to Codec rate channels. $ref: /schemas/types.yaml#/definitions/uint32 - required: - - remote-endpoint - ports: description: multi OF-Graph subnode type: object
The remote-endpoint may not be available if it is part of some pluggable module. One such example would be an audio card, the Codec endpoint will not be available until it is plugged in. Hence drop 'remote-endpoint' as a required property. Cc: Rob Herring <robh@kernel.org> Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Signed-off-by: Sameer Pujar <spujar@nvidia.com> --- Documentation/devicetree/bindings/sound/audio-graph-port.yaml | 3 --- 1 file changed, 3 deletions(-)