All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-18 10:16 ` Tzung-Bi Shih
  0 siblings, 0 replies; 9+ messages in thread
From: Tzung-Bi Shih @ 2021-10-18 10:16 UTC (permalink / raw)
  To: broonie, robh+dt, matthias.bgg
  Cc: alsa-devel, tzungbi, devicetree, enric.balletbo, linux-watchdog,
	linux-mediatek

Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
constants into common location") moves the reset controller headers.
However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.

Renames the DT example to pass dt_binding_check.

Fixes: f07c776f6d7e ("arm64: dts: mediatek: Move reset controller constants into common location")
Signed-off-by: Tzung-Bi Shih <tzungbi@google.com>
---
The patch bases on next-20211018.

In Mark's tree[1], the commit f07c776f6d7e hasn't shown up.
In Matthias's tree[2], mt8192-afe-pcm.yaml hasn't applied.

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/
[2]: https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/

Also, I am not sure if the commit hash "f07c776f6d7e" would change
or not after it applies to mainline.

 Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
index 5e9fe067f440..7a25bc9b8060 100644
--- a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
+++ b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
@@ -74,7 +74,7 @@ examples:
     #include <dt-bindings/interrupt-controller/arm-gic.h>
     #include <dt-bindings/interrupt-controller/irq.h>
     #include <dt-bindings/power/mt8192-power.h>
-    #include <dt-bindings/reset-controller/mt8192-resets.h>
+    #include <dt-bindings/reset/mt8192-resets.h>
 
     afe: mt8192-afe-pcm {
         compatible = "mediatek,mt8192-audio";
-- 
2.33.0.1079.g6e70778dc9-goog


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-18 10:16 ` Tzung-Bi Shih
  0 siblings, 0 replies; 9+ messages in thread
From: Tzung-Bi Shih @ 2021-10-18 10:16 UTC (permalink / raw)
  To: broonie, robh+dt, matthias.bgg
  Cc: devicetree, alsa-devel, linux-watchdog, tzungbi, linux-mediatek,
	enric.balletbo

Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
constants into common location") moves the reset controller headers.
However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.

Renames the DT example to pass dt_binding_check.

Fixes: f07c776f6d7e ("arm64: dts: mediatek: Move reset controller constants into common location")
Signed-off-by: Tzung-Bi Shih <tzungbi@google.com>
---
The patch bases on next-20211018.

In Mark's tree[1], the commit f07c776f6d7e hasn't shown up.
In Matthias's tree[2], mt8192-afe-pcm.yaml hasn't applied.

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/
[2]: https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/

Also, I am not sure if the commit hash "f07c776f6d7e" would change
or not after it applies to mainline.

 Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
index 5e9fe067f440..7a25bc9b8060 100644
--- a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
+++ b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
@@ -74,7 +74,7 @@ examples:
     #include <dt-bindings/interrupt-controller/arm-gic.h>
     #include <dt-bindings/interrupt-controller/irq.h>
     #include <dt-bindings/power/mt8192-power.h>
-    #include <dt-bindings/reset-controller/mt8192-resets.h>
+    #include <dt-bindings/reset/mt8192-resets.h>
 
     afe: mt8192-afe-pcm {
         compatible = "mediatek,mt8192-audio";
-- 
2.33.0.1079.g6e70778dc9-goog


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-18 10:16 ` Tzung-Bi Shih
  0 siblings, 0 replies; 9+ messages in thread
From: Tzung-Bi Shih @ 2021-10-18 10:16 UTC (permalink / raw)
  To: broonie, robh+dt, matthias.bgg
  Cc: alsa-devel, tzungbi, devicetree, enric.balletbo, linux-watchdog,
	linux-mediatek

Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
constants into common location") moves the reset controller headers.
However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.

Renames the DT example to pass dt_binding_check.

Fixes: f07c776f6d7e ("arm64: dts: mediatek: Move reset controller constants into common location")
Signed-off-by: Tzung-Bi Shih <tzungbi@google.com>
---
The patch bases on next-20211018.

In Mark's tree[1], the commit f07c776f6d7e hasn't shown up.
In Matthias's tree[2], mt8192-afe-pcm.yaml hasn't applied.

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/
[2]: https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/

Also, I am not sure if the commit hash "f07c776f6d7e" would change
or not after it applies to mainline.

 Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
index 5e9fe067f440..7a25bc9b8060 100644
--- a/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
+++ b/Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml
@@ -74,7 +74,7 @@ examples:
     #include <dt-bindings/interrupt-controller/arm-gic.h>
     #include <dt-bindings/interrupt-controller/irq.h>
     #include <dt-bindings/power/mt8192-power.h>
-    #include <dt-bindings/reset-controller/mt8192-resets.h>
+    #include <dt-bindings/reset/mt8192-resets.h>
 
     afe: mt8192-afe-pcm {
         compatible = "mediatek,mt8192-audio";
-- 
2.33.0.1079.g6e70778dc9-goog


_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
  2021-10-18 10:16 ` Tzung-Bi Shih
  (?)
@ 2021-10-19 13:47   ` Rob Herring
  -1 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2021-10-19 13:47 UTC (permalink / raw)
  To: Tzung-Bi Shih
  Cc: Mark Brown, Matthias Brugger, Linux-ALSA, devicetree,
	Enric Balletbo i Serra, LINUX-WATCHDOG,
	moderated list:ARM/Mediatek SoC support

On Mon, Oct 18, 2021 at 5:16 AM Tzung-Bi Shih <tzungbi@google.com> wrote:
>
> Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
> constants into common location") moves the reset controller headers.
> However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.
>
> Renames the DT example to pass dt_binding_check.
>
> Fixes: f07c776f6d7e ("arm64: dts: mediatek: Move reset controller constants into common location")
> Signed-off-by: Tzung-Bi Shih <tzungbi@google.com>
> ---
> The patch bases on next-20211018.
>
> In Mark's tree[1], the commit f07c776f6d7e hasn't shown up.
> In Matthias's tree[2], mt8192-afe-pcm.yaml hasn't applied.
>
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/
> [2]: https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/
>
> Also, I am not sure if the commit hash "f07c776f6d7e" would change
> or not after it applies to mainline.
>
>  Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Not great as things will break again in the merge window temporarily.
You could just not use the 1 reset define in the example and avoid all
that, but this is fine:

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-19 13:47   ` Rob Herring
  0 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2021-10-19 13:47 UTC (permalink / raw)
  To: Tzung-Bi Shih
  Cc: devicetree, Linux-ALSA, LINUX-WATCHDOG, Enric Balletbo i Serra,
	Mark Brown, moderated list:ARM/Mediatek SoC support,
	Matthias Brugger

On Mon, Oct 18, 2021 at 5:16 AM Tzung-Bi Shih <tzungbi@google.com> wrote:
>
> Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
> constants into common location") moves the reset controller headers.
> However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.
>
> Renames the DT example to pass dt_binding_check.
>
> Fixes: f07c776f6d7e ("arm64: dts: mediatek: Move reset controller constants into common location")
> Signed-off-by: Tzung-Bi Shih <tzungbi@google.com>
> ---
> The patch bases on next-20211018.
>
> In Mark's tree[1], the commit f07c776f6d7e hasn't shown up.
> In Matthias's tree[2], mt8192-afe-pcm.yaml hasn't applied.
>
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/
> [2]: https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/
>
> Also, I am not sure if the commit hash "f07c776f6d7e" would change
> or not after it applies to mainline.
>
>  Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Not great as things will break again in the merge window temporarily.
You could just not use the 1 reset define in the example and avoid all
that, but this is fine:

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-19 13:47   ` Rob Herring
  0 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2021-10-19 13:47 UTC (permalink / raw)
  To: Tzung-Bi Shih
  Cc: Mark Brown, Matthias Brugger, Linux-ALSA, devicetree,
	Enric Balletbo i Serra, LINUX-WATCHDOG,
	moderated list:ARM/Mediatek SoC support

On Mon, Oct 18, 2021 at 5:16 AM Tzung-Bi Shih <tzungbi@google.com> wrote:
>
> Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
> constants into common location") moves the reset controller headers.
> However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.
>
> Renames the DT example to pass dt_binding_check.
>
> Fixes: f07c776f6d7e ("arm64: dts: mediatek: Move reset controller constants into common location")
> Signed-off-by: Tzung-Bi Shih <tzungbi@google.com>
> ---
> The patch bases on next-20211018.
>
> In Mark's tree[1], the commit f07c776f6d7e hasn't shown up.
> In Matthias's tree[2], mt8192-afe-pcm.yaml hasn't applied.
>
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git/
> [2]: https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/
>
> Also, I am not sure if the commit hash "f07c776f6d7e" would change
> or not after it applies to mainline.
>
>  Documentation/devicetree/bindings/sound/mt8192-afe-pcm.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Not great as things will break again in the merge window temporarily.
You could just not use the 1 reset define in the example and avoid all
that, but this is fine:

Acked-by: Rob Herring <robh@kernel.org>

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
  2021-10-18 10:16 ` Tzung-Bi Shih
  (?)
@ 2021-10-20 11:28   ` Mark Brown
  -1 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2021-10-20 11:28 UTC (permalink / raw)
  To: robh+dt, Tzung-Bi Shih, matthias.bgg
  Cc: Mark Brown, linux-mediatek, alsa-devel, linux-watchdog,
	devicetree, enric.balletbo

On Mon, 18 Oct 2021 18:16:08 +0800, Tzung-Bi Shih wrote:
> Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
> constants into common location") moves the reset controller headers.
> However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.
> 
> Renames the DT example to pass dt_binding_check.
> 
> 
> [...]

Applied to

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

Thanks!

[1/1] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
      commit: a2b5c48abd485b289d3502466a9e8e1785f5adf3

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-20 11:28   ` Mark Brown
  0 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2021-10-20 11:28 UTC (permalink / raw)
  To: robh+dt, Tzung-Bi Shih, matthias.bgg
  Cc: Mark Brown, linux-mediatek, alsa-devel, linux-watchdog,
	devicetree, enric.balletbo

On Mon, 18 Oct 2021 18:16:08 +0800, Tzung-Bi Shih wrote:
> Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
> constants into common location") moves the reset controller headers.
> However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.
> 
> Renames the DT example to pass dt_binding_check.
> 
> 
> [...]

Applied to

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

Thanks!

[1/1] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
      commit: a2b5c48abd485b289d3502466a9e8e1785f5adf3

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

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
@ 2021-10-20 11:28   ` Mark Brown
  0 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2021-10-20 11:28 UTC (permalink / raw)
  To: robh+dt, Tzung-Bi Shih, matthias.bgg
  Cc: devicetree, alsa-devel, linux-watchdog, Mark Brown,
	linux-mediatek, enric.balletbo

On Mon, 18 Oct 2021 18:16:08 +0800, Tzung-Bi Shih wrote:
> Commit f07c776f6d7e ("arm64: dts: mediatek: Move reset controller
> constants into common location") moves the reset controller headers.
> However, it forgot to rename the DT example in mt8192-afe-pcm.yaml.
> 
> Renames the DT example to pass dt_binding_check.
> 
> 
> [...]

Applied to

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

Thanks!

[1/1] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example
      commit: a2b5c48abd485b289d3502466a9e8e1785f5adf3

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2021-10-20 11:31 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-18 10:16 [PATCH] ASoC: dt-bindings: mediatek: rename reset controller headers in DT example Tzung-Bi Shih
2021-10-18 10:16 ` Tzung-Bi Shih
2021-10-18 10:16 ` Tzung-Bi Shih
2021-10-19 13:47 ` Rob Herring
2021-10-19 13:47   ` Rob Herring
2021-10-19 13:47   ` Rob Herring
2021-10-20 11:28 ` Mark Brown
2021-10-20 11:28   ` Mark Brown
2021-10-20 11:28   ` Mark Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.