Add MT8365 SMI driver support

[PATCH v2 0/3] Add MT8365 SMI driver support

Posted by Alexandre Mergnat 1 year, 5 months ago

This serie add MT8365 SMI common driver support, to be bisectable and
avoid issue with the DTS change. This driver change and the related
change in the DTS should be applied in the right order (driver first).

Here the related DTS change: [1]

Also add the SMI common and LARB bindings for MT8365 support,
I kept the Matthias Brugger reviewed-by from the previous series [3] [4].

Regards,
Alexandre

[1]: https://lore.kernel.org/all/20230207-iommu-support-v2-7-60d5fa00e4e5@baylibre.com/
[2]: https://lore.kernel.org/all/9ad8c606026586b682df5e0ce0c699e21a1e6fb7.camel@mediatek.com/
[3]: https://lore.kernel.org/all/d1dc242d-c381-3307-b939-7bc35c6ce55b@gmail.com/
[4]: https://lore.kernel.org/all/0e863367-5b24-9559-f782-4d3e5dc06961@gmail.com/

Signed-off-by: Alexandre Mergnat <amergnat@baylibre.com>
---
Changes in v2:
- Add SMI common and LARB bindings for MT8365 support.
  I kept trailers [3] [4]
- Forgot to mention, in the v1, I've addressed the Yong Wu's comment [2]
  by removing "has_gals=true" from the mtk_smi_common_mt8365 structure.
  Then I've put the reviewed-by.
- Link to v1: https://lore.kernel.org/r/20230407-smi-driver-v1-1-036d6d8e8993@baylibre.com

---
Alexandre Mergnat (3):
      memory: mtk-smi: mt8365: Add SMI Support
      dt-bindings: memory-controllers: mediatek,smi-common: add mt8365
      dt-bindings: memory-controllers: mediatek,smi-larb: add mt8365

 .../devicetree/bindings/memory-controllers/mediatek,smi-common.yaml | 2 ++
 .../devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml   | 4 ++++
 drivers/memory/mtk-smi.c                                            | 6 ++++++
 3 files changed, 12 insertions(+)
---
base-commit: ea2dcf9394947d5d8e24cb9d52144923f6645632
change-id: 20230407-smi-driver-782270b6da2e

Best regards,
-- 
Alexandre Mergnat <amergnat@baylibre.com>

Re: [PATCH v2 0/3] Add MT8365 SMI driver support

Posted by Krzysztof Kozlowski 1 year, 5 months ago

On Fri, 07 Apr 2023 11:13:50 +0200, Alexandre Mergnat wrote:
> This serie add MT8365 SMI common driver support, to be bisectable and
> avoid issue with the DTS change. This driver change and the related
> change in the DTS should be applied in the right order (driver first).
> 
> Here the related DTS change: [1]
> 
> Also add the SMI common and LARB bindings for MT8365 support,
> I kept the Matthias Brugger reviewed-by from the previous series [3] [4].
> 
> [...]

Applied, thanks! It's late, so it might miss current cycle.

With corrected order of patches (I don't know why you decided to reorder
bindings with driver. Bindings should be before its usage).


[1/3] memory: mtk-smi: mt8365: Add SMI Support
      https://git.kernel.org/krzk/linux-mem-ctrl/c/3ec0e1ea4770e40575bfb2bb4e9ebbbaa3c80d3f
[2/3] dt-bindings: memory-controllers: mediatek,smi-common: add mt8365
      https://git.kernel.org/krzk/linux-mem-ctrl/c/67025255ee8b1ec496b01f74e8c298d211bc7a82
[3/3] dt-bindings: memory-controllers: mediatek,smi-larb: add mt8365
      https://git.kernel.org/krzk/linux-mem-ctrl/c/4ad9a801a3597ccfd406c911638aca2c61e57bdf

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>