[PATCH V4 2/2] thermal: broadcom: add Northstar thermal driver
Rafał Miłecki
rafal at milecki.pl
Sat Apr 1 14:41:08 PDT 2017
On 04/01/2017 09:54 PM, Eduardo Valentin wrote:
> On Fri, Mar 31, 2017 at 10:11:24PM +0200, Rafał Miłecki wrote:
>> From: Rafał Miłecki <rafal at milecki.pl>
>>
>> Northstar is a SoC family commonly used in home routers. This commit
>> adds a driver for checking CPU temperature. As Northstar Plus seems to
>> also have this IP block this new symbol gets ARCH_BCM_IPROC dependency.
>>
>> Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
>> Signed-off-by: Jon Mason <jon.mason at broadcom.com>
>> ---
>
> This driver looks fine from what concerns the of thermal usage.
> I had only one request on the DT bindings example. I believe better to
> get the example fixed so bad DTs does not get copied.
Thanks! I'll comment on this in a reply to patch 1/2.
>> V2: Make it iProc specific as NSP can also use this driver
>> Select proper symbols in config ARCH_BCM_IPROC
>> Define PVTMON register bits
>> Update code selecting temperature monitor mode
>> Thank you Jon!
>> V3: More details in help message for BCM_NS_THERMAL
>> Use slope & offset
>> Drop arch code change (I'll be submitted using a proper tree)
>> Thank you Eduardo!
>> V4: Comment operations on PVTMON_CONTROL0 register
>> ---
>> drivers/thermal/Kconfig | 5 ++
>> drivers/thermal/Makefile | 1 +
>> drivers/thermal/broadcom/Kconfig | 8 +++
>> drivers/thermal/broadcom/Makefile | 1 +
>> drivers/thermal/broadcom/ns-thermal.c | 105 ++++++++++++++++++++++++++++++++++
>
> Also, I have just merged a BRCM driver. Does it make sense to move it
> here too?
>
> It does not need to be a blocking request for this driver though.
Yes, I plan to send a patch for that after getting this one accepted.
More information about the linux-arm-kernel
mailing list