Subject: [PATCH v2 0/3] mtk-socinfo driver implementation

This purpose of the patches is for soc-related information retrival.
Such information includes manufacturer information, SoC name, SoC
segment name, and SoC marketing name.

Based on tag: next-20230915, linux-next/master

Changes in v2:
- Remove mtk-socinfo.h
- Consolidate different compatibles into mediatek,socinfo
- Move socinfo node out of MMIO bus
- Move mtk-socinfo.yaml to hwinfo
- Fix reviewer's comments

William-tw Lin (3):
arm64: dts: Add node for chip info driver
soc: mediatek: mtk-socinfo: Add driver for getting chip information
dt-bindings: hwinfo: Add mtk-socinfo driver

.../bindings/hwinfo/mtk-socinfo.yaml | 48 +++++
arch/arm64/boot/dts/mediatek/mt8173.dtsi | 15 ++
arch/arm64/boot/dts/mediatek/mt8183.dtsi | 15 ++
arch/arm64/boot/dts/mediatek/mt8186.dtsi | 10 ++
arch/arm64/boot/dts/mediatek/mt8192.dtsi | 14 ++
arch/arm64/boot/dts/mediatek/mt8195.dtsi | 9 +
drivers/soc/mediatek/Kconfig | 9 +
drivers/soc/mediatek/Makefile | 1 +
drivers/soc/mediatek/mtk-socinfo.c | 166 ++++++++++++++++++
9 files changed, 287 insertions(+)
create mode 100644 Documentation/devicetree/bindings/hwinfo/mtk-socinfo.yaml
create mode 100644 drivers/soc/mediatek/mtk-socinfo.c

--
2.18.0


2023-09-17 11:48:15

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v2 0/3] mtk-socinfo driver implementation

On 15/09/2023 17:26, William-tw Lin wrote:
> This purpose of the patches is for soc-related information retrival.
> Such information includes manufacturer information, SoC name, SoC
> segment name, and SoC marketing name.
>
> Based on tag: next-20230915, linux-next/master
>
> Changes in v2:
> - Remove mtk-socinfo.h
> - Consolidate different compatibles into mediatek,socinfo
> - Move socinfo node out of MMIO bus
> - Move mtk-socinfo.yaml to hwinfo
> - Fix reviewer's comments

This is way to vague, especially that you did not fix them. You need to
list all of them. Then we will nicely see what you ignored.


Best regards,
Krzysztof