mbox series

[v5,0/2] Remove power-domains property for devices with compatible ti,am62-sa3ul

Message ID 20230614-sa3ul-v5-0-29dd2366fba3@ti.com
Headers show
Series Remove power-domains property for devices with compatible ti,am62-sa3ul | expand

Message

Kamlesh Gurudasani July 14, 2023, 9:12 a.m. UTC
SYSFW don't allow access to power of devices with compatible ti,am62-sa3ul
from main domain.

Power-domains property, if present will try to access the power of the
device, which will result into failure in probing of driver for that
device.

Make power-domains property as false for devices with compatible
ti,am62-sa3ul.

Signed-off-by: Kamlesh Gurudasani <kamlesh@ti.com>
---
Changes in v5:
- Update recipient list
- Link to v4: https://lore.kernel.org/r/20230614-sa3ul-v4-0-7c969f626796@ti.com

Changes in v4:
- s/items/contains/ for dt-binding patch
- Added "fixes commit" for dt-binding patch
- Link to v3: https://lore.kernel.org/r/20230614-sa3ul-v3-0-2eeb810327a1@ti.com

Changes in v3:
- Add patch dt-bindings: crypto: ti,sa2ul: make power-domains conditional
- Link to v2: https://lore.kernel.org/r/20230614-sa3ul-v2-1-3a832b82339c@ti.com

Changes in v2:
- Make commit message clearer

---
Kamlesh Gurudasani (2):
      dt-bindings: crypto: ti,sa2ul: make power-domains conditional
      arm64: dts: ti: k3-am62-main: Remove power-domains from crypto node

 Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 14 +++++++++++++-
 arch/arm64/boot/dts/ti/k3-am62-main.dtsi               |  1 -
 2 files changed, 13 insertions(+), 2 deletions(-)
---
base-commit: 53ab6975c12d1ad86c599a8927e8c698b144d669
change-id: 20230614-sa3ul-cb4bd1bb4d08

Best regards,

Comments

Kamlesh Gurudasani July 14, 2023, 9:12 a.m. UTC | #1
Only SYSFW has control of SA3UL power.
Nishanth Menon July 25, 2023, 11:39 a.m. UTC | #2
Herbert,

I am going to assume that you are ok with me picking this series up as this
results in a few broken boots for various boards. Let me know if that is not
the case, and I will drop the series from my tree.

Hi Kamlesh Gurudasani,

On Fri, 14 Jul 2023 14:42:40 +0530, Kamlesh Gurudasani wrote:
> SYSFW don't allow access to power of devices with compatible ti,am62-sa3ul
> from main domain.
> 
> Power-domains property, if present will try to access the power of the
> device, which will result into failure in probing of driver for that
> device.
> 
> [...]

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/2] dt-bindings: crypto: ti,sa2ul: make power-domains conditional
      commit: e1f7d17a734c5c617d05c3d188939d5032d3d5a2
[2/2] arm64: dts: ti: k3-am62-main: Remove power-domains from crypto node
      commit: b573bf35ef3f113c1717fa22cefdfdfbb83aec70

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant 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.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
Herbert Xu July 26, 2023, 12:29 a.m. UTC | #3
On Tue, Jul 25, 2023 at 06:39:48AM -0500, Nishanth Menon wrote:
> Herbert,
> 
> I am going to assume that you are ok with me picking this series up as this
> results in a few broken boots for various boards. Let me know if that is not
> the case, and I will drop the series from my tree.

Sure, no problems.

Thanks!