mbox series

[v2,0/9] nvmem: sfp: binding updates and additions

Message ID 20220422145147.2210587-1-sean.anderson@seco.com
Headers show
Series nvmem: sfp: binding updates and additions | expand

Message

Sean Anderson April 22, 2022, 2:51 p.m. UTC 
This adds several properties to the Layerscape Security Fuse Processor
(SFP) necessary for writing. Although the Linux driver does not use
these bindings, I plan to use them in U-Boot [1]. It also adds a new
compatibles for Trust Architecture (TA) 2.1 devices. In addition, it
also adds an SFP binding for all TA 2.1 and 3.0 devices.

[1] https://lore.kernel.org/u-boot/7c8e206a-cd40-2a77-6282-7f4bead2b13a@seco.com/T/#m591f8425b6f096ab3d54e6f7bd258e41cfa4c43b

Changes in v2:
- Mention "regulator" in the description for ta-prog-sfp-supply
- Convert sfp driver to use regmap
- Fix various typos in commit messages

Sean Anderson (9):
  dt-bindings: nvmem: sfp: Fix typo
  dt-bindings: nvmem: sfp: Add clock properties
  dt-bindings: nvmem: sfp: Add TA_PROG_SFP supply
  dt-bindings: nvmem: sfp: Add compatible binding for TA 2.1 SFPs
  arm64: dts: ls1028a: Update SFP binding to include clock
  ARM: dts: layerscape: Add SFP binding for TA 2.1 devices
  ARM: dts: Add SFP binding for TA 3.0 devices
  nvmem: sfp: Use regmap
  nvmem: sfp: Add support for TA 2.1 devices

 .../bindings/nvmem/fsl,layerscape-sfp.yaml    | 30 ++++++++++++++--
 arch/arm/boot/dts/ls1021a.dtsi                |  7 ++++
 .../arm64/boot/dts/freescale/fsl-ls1012a.dtsi |  8 +++++
 .../arm64/boot/dts/freescale/fsl-ls1028a.dtsi |  5 ++-
 .../arm64/boot/dts/freescale/fsl-ls1043a.dtsi |  8 +++++
 .../arm64/boot/dts/freescale/fsl-ls1046a.dtsi |  8 +++++
 .../arm64/boot/dts/freescale/fsl-ls1088a.dtsi |  8 +++++
 .../arm64/boot/dts/freescale/fsl-ls208xa.dtsi |  8 +++++
 .../arm64/boot/dts/freescale/fsl-lx2160a.dtsi |  8 +++++
 drivers/nvmem/Kconfig                         |  1 +
 drivers/nvmem/layerscape-sfp.c                | 36 ++++++++++++++-----
 11 files changed, 115 insertions(+), 12 deletions(-)

Comments

Sean Anderson April 22, 2022, 2:54 p.m. UTC | #1 
On 4/22/22 10:51 AM, Sean Anderson wrote:
> This adds several properties to the Layerscape Security Fuse Processor
> (SFP) necessary for writing. Although the Linux driver does not use
> these bindings, I plan to use them in U-Boot [1]. It also adds a new
> compatibles for Trust Architecture (TA) 2.1 devices. In addition, it
> also adds an SFP binding for all TA 2.1 and 3.0 devices.
> 
> [1] https://lore.kernel.org/u-boot/7c8e206a-cd40-2a77-6282-7f4bead2b13a@seco.com/T/#m591f8425b6f096ab3d54e6f7bd258e41cfa4c43b
> 
> Changes in v2:
> - Mention "regulator" in the description for ta-prog-sfp-supply
> - Convert sfp driver to use regmap
> - Fix various typos in commit messages
> 
> Sean Anderson (9):
>   dt-bindings: nvmem: sfp: Fix typo
>   dt-bindings: nvmem: sfp: Add clock properties
>   dt-bindings: nvmem: sfp: Add TA_PROG_SFP supply
>   dt-bindings: nvmem: sfp: Add compatible binding for TA 2.1 SFPs
>   arm64: dts: ls1028a: Update SFP binding to include clock
>   ARM: dts: layerscape: Add SFP binding for TA 2.1 devices
>   ARM: dts: Add SFP binding for TA 3.0 devices
>   nvmem: sfp: Use regmap
>   nvmem: sfp: Add support for TA 2.1 devices
> 
>  .../bindings/nvmem/fsl,layerscape-sfp.yaml    | 30 ++++++++++++++--
>  arch/arm/boot/dts/ls1021a.dtsi                |  7 ++++
>  .../arm64/boot/dts/freescale/fsl-ls1012a.dtsi |  8 +++++
>  .../arm64/boot/dts/freescale/fsl-ls1028a.dtsi |  5 ++-
>  .../arm64/boot/dts/freescale/fsl-ls1043a.dtsi |  8 +++++
>  .../arm64/boot/dts/freescale/fsl-ls1046a.dtsi |  8 +++++
>  .../arm64/boot/dts/freescale/fsl-ls1088a.dtsi |  8 +++++
>  .../arm64/boot/dts/freescale/fsl-ls208xa.dtsi |  8 +++++
>  .../arm64/boot/dts/freescale/fsl-lx2160a.dtsi |  8 +++++
>  drivers/nvmem/Kconfig                         |  1 +
>  drivers/nvmem/layerscape-sfp.c                | 36 ++++++++++++++-----
>  11 files changed, 115 insertions(+), 12 deletions(-)
> 

I forgot to pick up Reviewed-bys from v1. I can resend this or add them for v3.

--Sean
Andrew Lunn April 22, 2022, 3:01 p.m. UTC | #2 
> I forgot to pick up Reviewed-bys from v1. I can resend this or add
> them for v3.

I suggest you wait a few days and see if you get any more comments,
which require a v3.

      Andrew