diff mbox series

[1/4] ASoC: dt-bindings: qcom,lpass-rx-macro: Add X1E80100 LPASS RX

Message ID 20231211123104.72963-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State Not Applicable
Headers show
Series [1/4] ASoC: dt-bindings: qcom,lpass-rx-macro: Add X1E80100 LPASS RX | expand

Commit Message

Krzysztof Kozlowski Dec. 11, 2023, 12:31 p.m. UTC
Add bindings for Qualcomm X1E80100 SoC Low Power Audio SubSystem (LPASS)
RX macro codec, which looks like compatible with earlier SM8550.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/sound/qcom,lpass-rx-macro.yaml        | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Conor Dooley Dec. 11, 2023, 5:29 p.m. UTC | #1
On Mon, Dec 11, 2023 at 01:31:01PM +0100, Krzysztof Kozlowski wrote:
> Add bindings for Qualcomm X1E80100 SoC Low Power Audio SubSystem (LPASS)
> RX macro codec, which looks like compatible with earlier SM8550.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

I wish you'd send cover letters when you send series of trivial patches
like this that could be acked in one go.

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.

> ---
>  .../devicetree/bindings/sound/qcom,lpass-rx-macro.yaml        | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml b/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml
> index cbc36646100f..b8540b30741e 100644
> --- a/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml
> +++ b/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml
> @@ -19,7 +19,9 @@ properties:
>            - qcom,sm8550-lpass-rx-macro
>            - qcom,sc8280xp-lpass-rx-macro
>        - items:
> -          - const: qcom,sm8650-lpass-rx-macro
> +          - enum:
> +              - qcom,sm8650-lpass-rx-macro
> +              - qcom,x1e80100-lpass-rx-macro
>            - const: qcom,sm8550-lpass-rx-macro
>  
>    reg:
> -- 
> 2.34.1
>
Konrad Dybcio Dec. 11, 2023, 5:31 p.m. UTC | #2
On 11.12.2023 18:29, Conor Dooley wrote:
> On Mon, Dec 11, 2023 at 01:31:01PM +0100, Krzysztof Kozlowski wrote:
>> Add bindings for Qualcomm X1E80100 SoC Low Power Audio SubSystem (LPASS)
>> RX macro codec, which looks like compatible with earlier SM8550.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> I wish you'd send cover letters when you send series of trivial patches
> like this that could be acked in one go.
(+1)

Konrad
Mark Brown Dec. 11, 2023, 5:36 p.m. UTC | #3
On Mon, Dec 11, 2023 at 05:29:50PM +0000, Conor Dooley wrote:
> On Mon, Dec 11, 2023 at 01:31:01PM +0100, Krzysztof Kozlowski wrote:
> > Add bindings for Qualcomm X1E80100 SoC Low Power Audio SubSystem (LPASS)
> > RX macro codec, which looks like compatible with earlier SM8550.

> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

> I wish you'd send cover letters when you send series of trivial patches
> like this that could be acked in one go.

In general any patch series should have at least a brief cover letter,
it makes everything work much better.
Rob Herring (Arm) Dec. 14, 2023, 9:36 p.m. UTC | #4
On Mon, Dec 11, 2023 at 05:29:50PM +0000, Conor Dooley wrote:
> On Mon, Dec 11, 2023 at 01:31:01PM +0100, Krzysztof Kozlowski wrote:
> > Add bindings for Qualcomm X1E80100 SoC Low Power Audio SubSystem (LPASS)
> > RX macro codec, which looks like compatible with earlier SM8550.
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> I wish you'd send cover letters when you send series of trivial patches
> like this that could be acked in one go.

Or just one patch because it's basically all the same changes and it's 
all going to 1 tree and the same set of people that will care to look 
at it.

Rob
Mark Brown Dec. 18, 2023, 5:28 p.m. UTC | #5
On Mon, 11 Dec 2023 13:31:01 +0100, Krzysztof Kozlowski wrote:
> Add bindings for Qualcomm X1E80100 SoC Low Power Audio SubSystem (LPASS)
> RX macro codec, which looks like compatible with earlier SM8550.
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/4] ASoC: dt-bindings: qcom,lpass-rx-macro: Add X1E80100 LPASS RX
      commit: 2f4734f3f48fff9437e4d38531c699b0a62ff8a8
[2/4] ASoC: dt-bindings: qcom,lpass-rx-macro: Add X1E80100 LPASS TX
      commit: 7de2109ce1619951e957eaafab83545a4cab8609
[3/4] ASoC: dt-bindings: qcom,lpass-rx-macro: Add X1E80100 LPASS VA
      commit: f990306adf2715cc2bdb85470065f7326a42081b
[4/4] ASoC: dt-bindings: qcom,lpass-rx-macro: Add X1E80100 LPASS WSA
      commit: 173a3b20a4980265bab52dbc60b616e739664b0d

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml b/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml
index cbc36646100f..b8540b30741e 100644
--- a/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml
+++ b/Documentation/devicetree/bindings/sound/qcom,lpass-rx-macro.yaml
@@ -19,7 +19,9 @@  properties:
           - qcom,sm8550-lpass-rx-macro
           - qcom,sc8280xp-lpass-rx-macro
       - items:
-          - const: qcom,sm8650-lpass-rx-macro
+          - enum:
+              - qcom,sm8650-lpass-rx-macro
+              - qcom,x1e80100-lpass-rx-macro
           - const: qcom,sm8550-lpass-rx-macro
 
   reg: