Hi Guenter, On Thu, Jul 21, 2016 at 6:30 PM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > On 07/21/2016 03:37 PM, Hoan Tran wrote: >> >> This patch set adds hardware temperature and power reading support for >> APM X-Gene SoC using the mailbox communication interface. >> For device tree, it is the standard DT mailbox. >> For ACPI, it is the PCC mailbox. >> >> For ACPI, this patch is built on top and depends on patch[1]: >> [1] http://www.spinics.net/lists/linux-acpi/msg66041.html >> - [PATCH v3] mailbox: pcc: Support HW-Reduced Communication Subspace >> type 2 >> > > One thought: You might consider taking ACPI support out of the driver for > now, > and add it back in after the dependent patch was accepted. > Thanks for your suggestion. Rafael agreed to apply this patch as the link below but he didn't do it yet. http://www.spinics.net/lists/linux-acpi/msg67100.html Let me check with him again. Thanks Hoan > Guenter > > >> v4 >> - Return 0 if driver registers successfully >> >> v3 >> - Order include files alphabetically >> - Use sign_extend32() to decode temperature >> - Use DEVICE_ATTR_RO() for temperature and power attributes >> - Temperature and power attributes start with index 1 >> - Use !!amsg->param2 >> - Fix checkpatch WARNING with --strict flag >> - Use hwmon_device_register_with_groups() >> - Check invalid sensor data >> >> v2 >> - Increase power reading accurateness by using 2 registers >> (a register for Watt, another for milli-Watt) >> - Remove power reading for SoC >> - Fix review comments from Guenter >> >> v1 >> - Initial >> >> Hoan Tran (3): >> Documentation: dtb: xgene: Add hwmon dts binding documentation >> hwmon: xgene: Add hwmon driver >> arm64: dts: apm: Add X-Gene SoC hwmon to device tree >> >> .../devicetree/bindings/hwmon/apm-xgene-hwmon.txt | 14 + >> Documentation/hwmon/xgene-hwmon | 30 + >> arch/arm64/boot/dts/apm/apm-shadowcat.dtsi | 5 + >> arch/arm64/boot/dts/apm/apm-storm.dtsi | 5 + >> drivers/hwmon/Kconfig | 7 + >> drivers/hwmon/Makefile | 1 + >> drivers/hwmon/xgene-hwmon.c | 755 >> +++++++++++++++++++++ >> 7 files changed, 817 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/hwmon/apm-xgene-hwmon.txt >> create mode 100644 Documentation/hwmon/xgene-hwmon >> create mode 100644 drivers/hwmon/xgene-hwmon.c >> > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html