lkml.org 
[lkml]   [2016]   [Jan]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 6/9] mfd: add TI LMU hardware fault monitoring driver
From
Date
On 01/11/2016 07:21 PM, Lee Jones wrote:
> On Thu, 26 Nov 2015, Milo Kim wrote:
>
>> LM3633 and LM3697 are TI LMU MFD device.
>> Those devices have hardware monitoring feature which detects open or
>> short circuit case.
>>
>> Debugfs
>> -------
>> Two files are created.
>> open_fault: check light output channel is open or not.
>> short_fault: check light output channel is shorted or not.
>>
>> The driver checks the status of backlight output channels.
>> LM3633 and LM3697 have same sequence to check channels, so common
>> functions are used.
>> ABI/testing document is also included.
>>
>> Operations
>> ----------
>> Two devices have common control flow but register addresses are different.
>> The structure, 'ti_lmu_reg' is used for register configuration.
>>
>> Event notifier
>> --------------
>> After fault monitoring is done, LMU device is reset. So backlight and
>> LED device should be reinitialized. It notifies an event as soon as
>> the monitoring is done. Then, LM3633 and LM3697 backlight and LED drivers
>> handle this event.
>>
>> Cc: Lee Jones <lee.jones@linaro.org>
>> Cc: Jacek Anaszewski <j.anaszewski@samsung.com>
>> Cc: Mark Brown <broonie@kernel.org>
>> Cc: Rob Herring <robh+dt@kernel.org>
>> Cc: devicetree@vger.kernel.org
>> Cc: linux-leds@vger.kernel.org
>> Cc: linux-kernel@vger.kernel.org
>> Signed-off-by: Milo Kim <milo.kim@ti.com>
>> ---
>> .../ABI/testing/debugfs-ti-lmu-fault-monitor | 32 ++
>> drivers/mfd/Kconfig | 10 +
>> drivers/mfd/Makefile | 1 +
>> drivers/mfd/ti-lmu-fault-monitor.c | 405 +++++++++++++++++++++
>
> I think this device is part of the MFD, rather than an MFD itself.
> Please relocate it to somewhere more appropriate.

Actually, initial patch driver was created under 'drivers/hwmon/' but
HWMON maintainer pointed this driver doesn't include HW sensor feature.
So I moved here.

This driver doesn't need any subsystem infrastructure, 'drivers/misc/'
could be OK. Does it make sense?

Best regards,
Milo

\
 
 \ /
  Last update: 2016-01-12 05:21    [W:0.130 / U:1.996 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site