Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760027Ab3EXDvi (ORCPT ); Thu, 23 May 2013 23:51:38 -0400 Received: from mail-ie0-f177.google.com ([209.85.223.177]:33780 "EHLO mail-ie0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759950Ab3EXDvg convert rfc822-to-8bit (ORCPT ); Thu, 23 May 2013 23:51:36 -0400 Date: Thu, 23 May 2013 22:51:32 -0500 From: Rob Landley Subject: Re: [PATCH 03/07] Documentation patch of ams AS3722 PMIC against linux_3.8.8 To: Florian Lobmaier Cc: "sameo@linux.intel.com" , "linux-kernel@vger.kernel.org" In-Reply-To: (from Florian.Lobmaier@ams.com on Thu May 23 07:07:37 2013) X-Mailer: Balsa 2.4.11 Message-Id: <1369367492.2776.12@driftwood> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; DelSp=Yes; Format=Flowed Content-Disposition: inline Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 6781 Lines: 213 On 05/23/2013 07:07:37 AM, Florian Lobmaier wrote: > From: Florian Lobmaier > > Added multi-function device driver support for ams AS3722 PMIC > Includes modules gpio, regulator, rtc, and watchdog > > Signed-off-by: Florian Lobmaier This seems like company-internal documentation that's got large irrelevant sections once merged into Linus's tree. > --- > diff -uprN -X Documentation/dontdiff > ../kernel_3.8.8/linux-kernel/Documentation/mfd-as3722.txt > ./Documentation/mfd-as3722.txt > --- ../kernel_3.8.8/linux-kernel/Documentation/mfd-as3722.txt > 1970-01-01 01:00:00.000000000 +0100 > +++ ./Documentation/mfd-as3722.txt 2013-05-23 13:12:35.000000000 > +0200 Is there a better place for this than the top level Documentation directory? > @@ -0,0 +1,133 @@ > +AS3722 PMIC Driver documentation > +-------------------------------- > +Author: Florian Lobmaier > +Last update: 2013-04-12 This is information you use git to find. > +This document describes how to use the multi-function device driver > for > +AS3722 PMICs. The AS3722 PMICs are accessed via i2c. > + > +Version info: > +============= > +Linux kernel: 3.4.38 > +Patch created with: $ git diff > f3b5af9a6e2a873110bb8546b42ae7c51f2213b3 > as3722_driver_v0.0.1.patch > +Apply patch with: $ patch -p0 < as3722_driver_v0.0.1.patch Again: if this goes into the tree with git, you're copying the git history into Documentation. And "Apply patch with" means what after merging...? (You are submitting this for inclusion, right?) $ git show f3b5af9a6e2a fatal: ambiguous argument 'f3b5af9a6e2a': unknown revision or path not in the working tree. > +Description of files: > +===================== > + > +as3722_platform-data.c: > +Platform data and as3722 regulator initialisation support. > + > +drivers/mfd/as3722-core.c: > +AS3722 communication and device management > + > +drivers/regulator/as3722-regulator.c: > +Adds the LDO's DCDC's and current regulators to /sys/class/regulator. > + > +drivers/gpio/gpio-as3722.c: > +Adds the GPIOs of AS3722 to /sys/class/gpio and to GPIO framework. > + > +drivers/rtc/rtc-as3722.c: > +Adds the RTC functionality of AS3722 to RTC framework. > + > +drivers/mfd/as3722-test.c: > +Regulator test framework. Exports some tests to userspace (see below) > + > +include/linux/mfd/as3722-reg.h: > +Register definitions > + > +include/linux/mfd/as3722-plat.h: > +platform data struct definitions Putting a one line comment at the top of each file would be more appropriate. > +How do I configure it? > +====================== > + > +1. Configure it in config by selecting > + > +CONFIG_MFD_CORE=y > +CONFIG_MFD_AS3722=y > +CONFIG_REGULATOR=y > +CONFIG_REGULATOR_AS3722=y > +CONFIG_GPIO_AS3722=y > +CONFIG_RTC_DRV_AS3722=y Menuconfig help exists, and forward slash search "AS3722" within menuconfig also exists. There's bound to be a better way to phrase this... > +2. Configure the driver according to the schematic by filling out > the > +as3722_platform_data. Additionally regulator initial data and > initial register > +setup can be done here. Add it to the list of attached devices when > +registering the proper i2c bus. Ok, this looks like it might belong in Documentation. Except it doesn't say how to do any of that. Also, this does _not_ support device tree? (Given the mention of gpio I'd generally assumed this was arm, but you haven't mentioned any actual use cases for it yet...) The next section seems actually useful: > +Testing Regulators: > +=================== > + > +# insmod as3722-core.ko > +# insmod as3722-regulators.ko > + > +Regulators using the linux kernel regulator framework can be found at > +/sys/class/regulator. Via sysfs its possible to get some status > information: > + > +# cd /sys/class/regulator/regulator.9 > +# ls > +# ls > +device microvolts num_users type > +max_microamps min_microamps power uevent > +max_microvolts min_microvolts state > +microamps name subsystem > + > +We can perform the testing by using the as3722-test.ko modules which > provides > +a testframework to userspace: > + > +Mount debugfs > +# mount -t debugfs none /sys/kernel/debug/ > + > +Load test modules (needed for regulator tests) > +# insmod as3722-test.ko > + > +Use the provided test-scripts. Transfer the scripts and the > respective > +command text files to the rowboat board via adb: What's a rowboat board? Where are the provided test-scripts? > +(win cygwin) $ adb shell mkdir -p /testing/ && adb push regtest.sh > /testing/regtest.sh > +(win cygwin) $ adb push test-dig-ldo.txt /testing/test-dig-ldo.txt > + > +Run the script on the beagle-board testing the ldo3: > +# sh /testing/regtest.sh as3722-ldo3 /testing/test-dig-ldo.txt > + > +Example testing the sd1 (using test-sd.txt): > +# sh /testing/regtest.sh as3722-sd1 /testing/test-sd.txt > + > +Testing gpio: > +============= > +For testing the gpio framework the sysFS functions are used. Of > course > +all gpio framework functionality exists as well. > +# cd /sys/class/gpio/ > + > +Now "export" and "unexport" files should be available. Use export > function > +to create a gpio211 node for GPIO #211 if that is not requested by > kernel > +code yet. (GPIO #211 is used as an example number, please substitute > with > +a valid AS3722 gpio number): > +# echo 211 > export > + > +The gpio signal for gpio #211 should now be available at: > +# ls /sys/class/gpio/gpio211/ > +direction edge > +value active_low > + > +This gpio framework files may now be used to setup and test the gpio. > +Please refer to gpio.txt documentation on how to use the gpio > framework > +attributes. > + > +Testing rtc: > +============ > +The sysfs interface under /sys/class/rtc/rtcN provides access to > various > +rtc attributes without requiring the use of ioctls. All dates and > times > +are in the RTC's timezone, rather than in system time. > +# cd /sys/class/rtc/rtc0/ > +date max_user_freq since_epoch wakealarm > +hctosys name time > + > +In order to set the current time or alarm time, IOCTL interface > should > +be used. The ioctl calls are supported by /dev/rtc and by the RTC > class > +framework. > + > +To test the rtc via IOCTL interface calls, the file rtctest.c copied > from > +kernel/Documentation/rtc.txt can be used. I don't object to it going in as part of a series, but I'm not going to ack it because it needs cleanup. Rob-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/