Re: [PATCH 0/5] soc: ti: add k3 platforms chipid module driver

[santosh . shilimkar]

On 5/1/20 1:55 PM, Grygorii Strashko wrote:
> Hi Santosh, Tero
>
> On 23/04/2020 21:05, Grygorii Strashko wrote:
> > Hi All,
> >
> > This series introduces TI K3 Multicore SoC platforms chipid module driver
> > which provides identification support of the TI K3 SoCs (family, 
> > revision)
> > and register this information with the SoC bus. It is available under
> > /sys/devices/soc0/ for user space, and can be checked, where needed,
> > in Kernel using soc_device_match().
> > It is also required for introducing support for new revisions of
> > K3 AM65x/J721E SoCs.
> >
> > Example J721E:
> > ÂÂ # cat /sys/devices/soc0/{machine,family,revision}
> > ÂÂ Texas Instruments K3 J721E SoC
> > ÂÂ J721E
> > ÂÂ SR1.0
> >
> > Example AM65x:
> > ÂÂ # cat /sys/devices/soc0/{machine,family,revision}
> > ÂÂ Texas Instruments AM654 Base Board
> > ÂÂ AM65X
> > ÂÂ SR1.0
> >
> > Grygorii Strashko (5):
> > ÂÂ dt-bindings: soc: ti: add binding for k3 platforms chipid module
> > ÂÂ soc: ti: add k3 platforms chipid module driver
> > ÂÂ arm64: arch_k3: enable chipid driver
> > ÂÂ arm64: dts: ti: k3-am65-wakeup: add k3 platforms chipid module node
> > ÂÂ arm64: dts: ti: k3-j721e-mcu-wakeup: add k3 platforms chipid module
> > ÂÂÂÂ node
> >
> > Â .../bindings/soc/ti/k3-socinfo.yamlÂÂÂÂÂÂÂÂÂÂ |Â 40 ++++++
> > Â arch/arm64/Kconfig.platformsÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 1 +
> > Â arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsiÂÂÂ |ÂÂ 5 +
> > Â .../boot/dts/ti/k3-j721e-mcu-wakeup.dtsiÂÂÂÂÂ |ÂÂ 5 +
> > Â drivers/soc/ti/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |Â 10 ++
> > Â drivers/soc/ti/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂ 1 +
> > Â drivers/soc/ti/k3-socinfo.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ | 135 ++++++++++++++++++
> > Â 7 files changed, 197 insertions(+)
> > Â create mode 100644 
> > Documentation/devicetree/bindings/soc/ti/k3-socinfo.yaml
> > Â create mode 100644 drivers/soc/ti/k3-socinfo.c
>
> Any more comments? I'm going resend it.
If you have acks from DT maintainers, then I suggest you to split this 
series and post platform and drivers patches separately.

Regards,
Santosh