From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760744AbcALDhQ (ORCPT ); Mon, 11 Jan 2016 22:37:16 -0500 Received: from arroyo.ext.ti.com ([192.94.94.40]:39069 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760002AbcALDhM (ORCPT ); Mon, 11 Jan 2016 22:37:12 -0500 Subject: Re: [PATCH v2 6/9] mfd: add TI LMU hardware fault monitoring driver To: Lee Jones References: <1448521025-2796-1-git-send-email-milo.kim@ti.com> <1448521025-2796-7-git-send-email-milo.kim@ti.com> <20160111102117.GU14104@x1> CC: , , , , , From: Milo Kim Message-ID: <569474DB.3090408@ti.com> Date: Tue, 12 Jan 2016 12:36:59 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <20160111102117.GU14104@x1> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 >> Cc: Jacek Anaszewski >> Cc: Mark Brown >> Cc: Rob Herring >> Cc: devicetree@vger.kernel.org >> Cc: linux-leds@vger.kernel.org >> Cc: linux-kernel@vger.kernel.org >> Signed-off-by: Milo Kim >> --- >> .../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