| Message ID | 20230103013145.9570-1-hal.feng@starfivetech.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Temperature sensor support for StarFive JH7110 RISC-V SoC | expand |
On Tue, Jan 03, 2023 at 09:31:41AM +0800, Hal Feng wrote: > This patch series adds temperature sensor support for StarFive JH7110 SoC. > The last two patches depend on series [1]. > > [1]: https://lore.kernel.org/all/20221220011247.35560-1-hal.feng@starfivetech.com/ > > Emil Renner Berthing (4): > dt-bindings: hwmon: Add starfive,jh71x0-temp > hwmon: (sfctemp) Add StarFive JH71x0 temperature sensor > riscv: dts: starfive: jh7110: Add temperature sensor node > riscv: dts: starfive: visionfive-2: Add thermal-zones > The hardware monitoring driver is obviously either the same or derived from the previous series at https://patchwork.kernel.org/project/linux-hwmon/list/?series=&submitter=&state=*&q=starfive Why is this not submitted as v4 of the original series ? What has changed, and what is the rationale for (re-)submitting it as v1 ? Guenter > .../bindings/hwmon/starfive,jh71x0-temp.yaml | 75 ++++ > Documentation/hwmon/index.rst | 1 + > Documentation/hwmon/sfctemp.rst | 33 ++ > MAINTAINERS | 8 + > .../jh7110-starfive-visionfive-2.dtsi | 28 ++ > arch/riscv/boot/dts/starfive/jh7110.dtsi | 13 + > drivers/hwmon/Kconfig | 10 + > drivers/hwmon/Makefile | 1 + > drivers/hwmon/sfctemp.c | 350 ++++++++++++++++++ > 9 files changed, 519 insertions(+) > create mode 100644 Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml > create mode 100644 Documentation/hwmon/sfctemp.rst > create mode 100644 drivers/hwmon/sfctemp.c > > > base-commit: 830b3c68c1fb1e9176028d02ef86f3cf76aa2476 > prerequisite-patch-id: 4dc515731ce237184553c1606ffb3afaeb51c3d8 > prerequisite-patch-id: 09c98554df52d17ba5fd604125f8cdd62cbe80d1 > prerequisite-patch-id: a798370d170dc2bcc79ed86f741c21c1e6d87c78 > prerequisite-patch-id: bd9fd8b5cb2376dc7a5e08e1a1fbb969cf475926 > prerequisite-patch-id: c57ebb83bc43ccd2a8366ff166eb499da1e1d2cf > prerequisite-patch-id: a1673a9e9f19d6fab5a51abb721e54e36636f067 > prerequisite-patch-id: 94860423c7acc9025249d4bb36652a585bd0a797 > prerequisite-patch-id: b5084253283929d9a6d0e66c350400c7c85d034d > prerequisite-patch-id: 6e369dbe9dca2785e4ea7d0b80e525e227a90a6e > prerequisite-patch-id: e08806183c152714c563f3a21c6d7b2f539c4d6e > prerequisite-patch-id: 79db8036abdc48fd36da227652ec62627a6b548b > prerequisite-patch-id: 06971b8e6bddc0e87e63bfdb0ce8bfb653bd73aa > prerequisite-patch-id: 16309a0e23811a2c55d2e56886de3e8eccc51554 > prerequisite-patch-id: bf4f7ab0b6cfa90b6e49e66c7d75ed2eaaebbe78 > prerequisite-patch-id: 38468d532e87867990055d3320679f18c5f52278 > prerequisite-patch-id: 4710f2ac22dca0bdd9ff5d744d2c37cab3c74515 > prerequisite-patch-id: 6bb9a780c62af3bcc2368dfd20303c7b1bc91e23 > prerequisite-patch-id: 258ea5f9b8bf41b6981345dcc81795f25865d38f > prerequisite-patch-id: 8b6f2c9660c0ac0ee4e73e4c21aca8e6b75e81b9 > prerequisite-patch-id: e3b986b9c60b2b93b7812ec174c9e1b4cfb14c97 > prerequisite-patch-id: a2b3a9cff8a683422eb0ccf3a0850091401812d4 > prerequisite-patch-id: e0ba7af0f8d3d41844da9fbcba14b548cbc18f55 > prerequisite-patch-id: bc0176325c11a632c6abaa83e54e891cc92d1c74 > -- > 2.38.1 >
On Mon, 2 Jan 2023 17:56:01 -0800, Guenter Roeck wrote: > On Tue, Jan 03, 2023 at 09:31:41AM +0800, Hal Feng wrote: > > This patch series adds temperature sensor support for StarFive JH7110 SoC. > > The last two patches depend on series [1]. > > > > [1]: https://lore.kernel.org/all/20221220011247.35560-1-hal.feng@starfivetech.com/ > > > > Emil Renner Berthing (4): > > dt-bindings: hwmon: Add starfive,jh71x0-temp > > hwmon: (sfctemp) Add StarFive JH71x0 temperature sensor > > riscv: dts: starfive: jh7110: Add temperature sensor node > > riscv: dts: starfive: visionfive-2: Add thermal-zones > > > > The hardware monitoring driver is obviously either the same > or derived from the previous series at > https://patchwork.kernel.org/project/linux-hwmon/list/?series=&submitter=&state=*&q=starfive > > Why is this not submitted as v4 of the original series ? > What has changed, and what is the rationale for (re-)submitting > it as v1 ? Sorry for the late reply. I feel sorry to say that I didn't know the submitting history of this patch series and Emil forgot to tell me maybe. After comparing with the previous series, I find the changes between this one and the previous one can be concluded as below. Change log: - Added support for StarFive JH7110 SoC besides JH7100. - Added clock and reset support in dt-bindings and driver. - Added two patches for adding nodes in JH7110 and VisionFive 2 dts which were being reviewed. Thank you for your kindly reminding. I will resend this patch series as version 4 and add the change log. Best regards, Hal
On Fri, 6 Jan 2023 09:11:53 +0800, Hal Feng wrote: > On Mon, 2 Jan 2023 17:56:01 -0800, Guenter Roeck wrote: > > On Tue, Jan 03, 2023 at 09:31:41AM +0800, Hal Feng wrote: > > > This patch series adds temperature sensor support for StarFive JH7110 SoC. > > > The last two patches depend on series [1]. > > > > > > [1]: https://lore.kernel.org/all/20221220011247.35560-1-hal.feng@starfivetech.com/ > > > > > > Emil Renner Berthing (4): > > > dt-bindings: hwmon: Add starfive,jh71x0-temp > > > hwmon: (sfctemp) Add StarFive JH71x0 temperature sensor > > > riscv: dts: starfive: jh7110: Add temperature sensor node > > > riscv: dts: starfive: visionfive-2: Add thermal-zones > > > > > > > The hardware monitoring driver is obviously either the same > > or derived from the previous series at > > https://patchwork.kernel.org/project/linux-hwmon/list/?series=&submitter=&state=*&q=starfive > > > > Why is this not submitted as v4 of the original series ? > > What has changed, and what is the rationale for (re-)submitting > > it as v1 ? > > Sorry for the late reply. I feel sorry to say that I didn't know > the submitting history of this patch series and Emil forgot to > tell me maybe. After comparing with the previous series, I find > the changes between this one and the previous one can be concluded > as below. > > Change log: > - Added support for StarFive JH7110 SoC besides JH7100. > - Added clock and reset support in dt-bindings and driver. > - Added two patches for adding nodes in JH7110 and VisionFive 2 dts > which were being reviewed. > > Thank you for your kindly reminding. I will resend this patch > series as version 4 and add the change log. I have resent this series as v4 [1]. Even though there are some issues need to be discussed with Guenter, I wanna resend this series in advance so that the v4 can be mostly synchronized with the latest patches [2] [3] from Emil. The new change will be added in the v5. [1] https://lore.kernel.org/all/20230207072314.62040-1-hal.feng@starfivetech.com/ [2] https://github.com/esmil/linux/commit/8c7f2f00105384390ee02d745b6ccb6637e28d25 [3] https://github.com/esmil/linux/commit/d04731cf0dc0462bc76afbadf95366ff0edbe642 Best regards, Hal