On Wed, May 31, 2017 at 11:09 PM, Eduardo Valentin <[email protected]> wrote:
> On Wed, May 31, 2017 at 03:55:44PM -0400, Jon Mason wrote:
>> ARCH_BCM_IPROC includes support for many SoCs, not all of which have the
>> same thermal hardware interface as the Northstar/Northstar Plus SoCs.
>> This is not a major issue, as this driver will only be probed if the
>> relevant device tree entry is present. However, it will result in a
>> slightly larger than necessary kernel. By limiting the driver to only
>> work on the NS/NSP SoCs, we can avoid the issue.
>>
>> Suggested-by: Scott Branden <[email protected]>
>> Signed-off-by: Jon Mason <[email protected]>
>
> Should this commit message include a Fixes: tag?
>
> Maybe
> Fixes: a94cb7e ("thermal: broadcom: add Northstar thermal driver")
Technically, it would be 2 patches
Fixes: a94cb7eeecc4 ("thermal: broadcom: add Northstar thermal driver")
Fixes: a54c51863ed1 ("thermal: broadcom: ns-thermal: default on iProc SoCs")
We could probably get away with only doing the latter one, as it is
the one that makes it enabled by default
>> ---
>> drivers/thermal/broadcom/Kconfig | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/thermal/broadcom/Kconfig
>> b/drivers/thermal/broadcom/Kconfig
>> index 42c098e86f84..c729dad2348c 100644
>> --- a/drivers/thermal/broadcom/Kconfig
>> +++ b/drivers/thermal/broadcom/Kconfig
>> @@ -8,8 +8,8 @@ config BCM2835_THERMAL
>>
>> config BCM_NS_THERMAL
>> tristate "Northstar thermal driver"
>> - depends on ARCH_BCM_IPROC || COMPILE_TEST
>> - default y if ARCH_BCM_IPROC
>> + depends on ARCH_BCM_NSP || ARCH_BCM_5301X || COMPILE_TEST
>> + default y if ARCH_BCM_NSP || ARCH_BCM_5301X
>> help
>> Support for the Northstar and Northstar Plus family of SoCs (e.g.
>> BCM4708, BCM4709, BCM5301x, BCM95852X, etc). It contains DMU (Device
>> --
>> 2.7.4
>>
