1. 28 7月, 2014 3 次提交
  2. 25 7月, 2014 14 次提交
    • L
      mfd: max77686: Ensure device type IDs are architecture agnostic · ec8bd566
      Lee Jones 提交于
      Extinguishes:
      
      ../drivers/mfd/max77686.c: In function ‘max77686_i2c_probe’:
      ../drivers/mfd/max77686.c:254:20:
      	warning: cast from pointer to integer of different size
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      ec8bd566
    • J
      mfd: max77686: Add Maxim 77802 PMIC support · a259f389
      Javier Martinez Canillas 提交于
      Maxim MAX77802 is a power management chip that contains 10 high
      efficiency Buck regulators, 32 Low-dropout (LDO) regulators used
      to power up application processors and peripherals, a 2-channel
      32kHz clock outputs, a Real-Time-Clock (RTC) and a I2C interface
      to program the individual regulators, clocks outputs and the RTC.
      
      This patch adds support for MAX77802 to the MAX77686 driver and is
      based on a driver added to the Chrome OS kernel 3.8 by Simon Glass.
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Reviewed-by: NKrzysztof Kozlowski <k.kozlowski@samsung.com>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      a259f389
    • H
      mfd: tps6105x: Use managed resources when allocating memory · ad83533a
      Himangi Saraogi 提交于
      This patch introduces the use of devm_kzalloc instead of the
      corresponding unmanaged version and does away with the kfrees in the
      probe and remove functions. Also, a label is done away with.
      Signed-off-by: NHimangi Saraogi <himangi774@gmail.com>
      Acked-by: NJulia Lawall <julia.lawall@lip6.fr>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      ad83533a
    • L
      mfd: wm8997-tables: Suppress 'line over 80 chars' warnings · 23213218
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: line over 80 characters
      +       { 0x00000068, 0x01FF },    /* R104   - Always On Triggers Sequence Select 3 */
      
      WARNING: line over 80 characters
      +       { 0x00000069, 0x01FF },    /* R105   - Always On Triggers Sequence Select 4 */
      
      WARNING: line over 80 characters
      +       { 0x0000006A, 0x01FF },    /* R106   - Always On Triggers Sequence Select 5 */
      
      WARNING: line over 80 characters
      +       { 0x0000006B, 0x01FF },    /* R107   - Always On Triggers Sequence Select 6 */
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      23213218
    • L
      mfd: kempld-core: Correct a variety of checkpatch warnings · 9ef4e935
      Lee Jones 提交于
      WARNING: line over 80 characters
      +module_param_string(force_device_id, force_device_id, sizeof(force_device_id), 0);
      
      WARNING: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt
      +               msleep(1);
      
      WARNING: __initdata should be placed after kempld_dmi_table[]
      +static struct dmi_system_id __initdata kempld_dmi_table[] = {
      
      WARNING: line over 80 characters
      +               for (id = kempld_dmi_table; id->matches[0].slot != DMI_NONE; id++)
      
      total: 0 errors, 4 warnings, 771 lines checked
      
      This is part of an effort to clean-up the MFD subsystem.
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      9ef4e935
    • L
      mfd: ipaq-micro: Fix coding style errors/warnings reported by checkpatch · 9336fe91
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      ERROR: space required before the open parenthesis '('
      +                       if(!micro->msg)
      
      WARNING: unnecessary whitespace before a quoted newline
      +                       dev_dbg(micro->dev, "key message ignored, no handle \n");
      
      WARNING: unnecessary whitespace before a quoted newline
      +                       dev_dbg(micro->dev, "touchscreen message ignored, no handle \n");
      
      WARNING: space prohibited before semicolon
      +               rx->id = (ch & 0xf0) >> 4 ;
      
      total: 1 errors, 3 warnings, 482 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      9336fe91
    • L
      mfd: si476x-cmd: Remedy checkpatch style complains · d69d4212
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: line over 80 characters
      +                        struct si476x_rsq_status_args *rsqargs,
      
      WARNING: line over 80 characters
      +                        struct si476x_rsq_status_report *report)
      
      WARNING: Unnecessary space before function pointer arguments
      +       int (*power_up) (struct si476x_core *,
      
      WARNING: Unnecessary space before function pointer arguments
      +       int (*power_down) (struct si476x_core *,
      
      total: 0 errors, 4 warnings, 1555 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      d69d4212
    • L
      mfd: tc6387xb: Remove unnecessary braces and correct style warnings · d5ce79ff
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: braces {} are not necessary for single statement blocks
      +       if (!iomem) {
      +               return -EINVAL;
      +       }
      
      WARNING: sizeof *tc6387xb should be sizeof(*tc6387xb)
      +       tc6387xb = kzalloc(sizeof *tc6387xb, GFP_KERNEL);
      
      WARNING: Prefer [subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(...  to printk(KERN_INFO ...
      +       printk(KERN_INFO "Toshiba tc6387xb initialised\n");
      
      total: 0 errors, 3 warnings, 242 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      d5ce79ff
    • L
      mfd: twl6030-irq: Extinguish coding style warnings · 3103de8c
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: please, no space before tabs
      +^IRSV_INTR_OFFSET,  ^I/* Bit 12^IReserved^I^I*/$
      
      WARNING: Missing a blank line after declarations
      +       u8 unmask_value;
      +       ret = twl_i2c_read_u8(TWL_MODULE_PIH, &unmask_value,
      
      WARNING: Missing a blank line after declarations
      +       u8 mask_value;
      +       ret = twl_i2c_read_u8(TWL_MODULE_PIH, &mask_value,
      
      total: 0 errors, 3 warnings, 484 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      3103de8c
    • L
      mfd: wm8350-i2c: Replace spaces with tabs · 6db1c9ba
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: please, no spaces at the start of a line
      +       { "wm8350", 0 },$
      
      WARNING: please, no spaces at the start of a line
      +       { "wm8351", 0 },$
      
      WARNING: please, no spaces at the start of a line
      +       { "wm8352", 0 },$
      
      WARNING: please, no spaces at the start of a line
      +       { }$
      
      total: 0 errors, 4 warnings, 93 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      6db1c9ba
    • L
      mfd: max8925-core: Fix 'missing blank line after declarations' warning · a5c5accf
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: Missing a blank line after declarations
      +       struct max8925_chip *chip = irq_data_get_irq_chip_data(data);
      +       max8925_irqs[data->irq - chip->irq_base].enable
      
      WARNING: Missing a blank line after declarations
      +       struct max8925_chip *chip = irq_data_get_irq_chip_data(data);
      +       max8925_irqs[data->irq - chip->irq_base].enable = 0;
      
      total: 0 errors, 2 warnings, 927 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      a5c5accf
    • L
      mfd: twl4030-irq: Fix style warnings relating to pr_warn() and sizeof() · 04aa4438
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: Prefer pr_warn(... to pr_warning(...
      +               pr_warning("twl4030: I2C error %d reading PIH ISR\n", ret);
      
      WARNING: sizeof buf should be sizeof(buf)
      +       memset(buf, 0xff, sizeof buf);
      
      WARNING: sizeof *agent should be sizeof(*agent)
      +       agent = kzalloc(sizeof *agent, GFP_KERNEL);
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      04aa4438
    • L
      mfd: lp8788-irq: Fix 'missing blank line after declarations' warning · 52149ad2
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: Missing a blank line after declarations
      +       struct lp8788_irq_data *irqd = irq_data_get_irq_chip_data(data);
      +       irqd->enabled[data->hwirq] = 1;
      
      WARNING: Missing a blank line after declarations
      +       struct lp8788_irq_data *irqd = irq_data_get_irq_chip_data(data);
      +       irqd->enabled[data->hwirq] = 0;
      
      total: 0 errors, 2 warnings, 198 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      52149ad2
    • L
      mfd: ezx-pcap: Repair coding style errors picked up with checkpatch · 0309528a
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: sizeof t should be sizeof(t)
      +       memset(&t, 0, sizeof t);
      
      WARNING: void function return statements are not generally useful
      +       return;
      +}
      
      total: 0 errors, 2 warnings, 542 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      0309528a
  3. 22 7月, 2014 5 次提交
  4. 21 7月, 2014 11 次提交
    • L
      mfd: max8925-i2c: Fix 'blank line after declarations' warning · 4ed8f718
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      WARNING: Missing a blank line after declarations
      +       int ret;
      +       ret = i2c_add_driver(&max8925_driver);
      
      total: 0 errors, 1 warnings, 275 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      4ed8f718
    • L
      mfd: max77686: Fix 'line over 80 chars' warning · a31b0fa6
      Lee Jones 提交于
      This is part of an effort to clean-up the MFD subsystem.
      
      +WARNING: line over 80 characters
      +                                      &max77686_rtc_regmap_config);
      
      total: 0 errors, 1 warnings, 299 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      a31b0fa6
    • J
      mfd: max77686: Remove unneeded OOM error message · 8a789b64
      Javier Martinez Canillas 提交于
      There is no need to print out-of-memory errors since this is already
      done by the memory management subsystem which even calls dump_stack().
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      8a789b64
    • J
      mfd: max77686: Make error checking consistent · c0e0fcda
      Javier Martinez Canillas 提交于
      Error checking across the driver is mostly consistent besides
      a few exceptions, so change these exceptions for consistency.
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Reviewed-by: NKrzysztof Kozlowski <k.kozlowski@samsung.com>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      c0e0fcda
    • J
      mfd: max77686: Return correct error when pdata isn't found · b452d7b6
      Javier Martinez Canillas 提交于
      When platform data is not found an -EIO (I/O error) code is returned.
      This doesn't seem to be the correct error so better return -EINVAL
      (Invalid argument) which is what most drivers do in this case.
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      b452d7b6
    • J
      mfd: max77686: Make platform data over-rule DT · ede04c61
      Javier Martinez Canillas 提交于
      The function max77802_i2c_parse_dt_pdata() should only be called
      if there isn't already platform data for the device.
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      ede04c61
    • J
      mfd: max77686: Don't define dummy function if OF isn't enabled · 154409e4
      Javier Martinez Canillas 提交于
      When the CONFIG_OF option was not enabled, a dummy function
      max77686_i2c_parse_dt_pdata() was defined since this is called
      unconditionally on probe(). Just always define the real function
      and conditionally call it if CONFIG_OF is enabled instead.
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      154409e4
    • J
      mfd: max77686: Add power management support · 2b52b5d5
      Javier Martinez Canillas 提交于
      The driver doesn't have PM operations defined so add a suspend
      and resume function handlers to allow the PMIC IRQ to wakeup
      the system when it is put into a sleep state.
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Reviewed-by: NKrzysztof Kozlowski <k.kozlowski@samsung.com>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      2b52b5d5
    • J
      mfd: max77686: Convert to use regmap_irq · 6f1c1e71
      Javier Martinez Canillas 提交于
      By using the generic IRQ support in the Register map API, it
      is possible to get rid max77686-irq.c and simplify the code.
      Suggested-by: NKrzysztof Kozlowski <k.kozlowski@samsung.com>
      Signed-off-by: NJavier Martinez Canillas <javier.martinez@collabora.co.uk>
      Reviewed-by: NDoug Anderson <dianders@chromium.org>
      Tested-by: NDoug Anderson <dianders@chromium.org>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      6f1c1e71
    • C
      mfd: sun6i-prcm: Add support for Allwinner A23 PRCM · 4eb9560b
      Chen-Yu Tsai 提交于
      The Allwinner A23 SoC has a PRCM unit like the previous A31 SoC.
      The differences are the AR100 clock can no longer be modified,
      the APB0 clock has different divisors, and some clock gates are
      gone.
      
      This patch adds a compatible with a modified subdevice list for
      the A23.
      Signed-off-by: NChen-Yu Tsai <wens@csie.org>
      Acked-by: NMaxime Ripard <maxime.ripard@free-electrons.com>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      4eb9560b
    • L
      mfd: ab8500-debugfs: BIG clean-up · 4362175d
      Lee Jones 提交于
      When checkpatch is run on ab8500-debugfs.c it screamed blue murder!
      
      This patch fixes up all of the errors/warnings reported:
      
      WARNING: line over 80 characters
      +		err = seq_printf(s, "  [0x%02X/0x%02X]: 0x%02X\n",
      
      WARNING: Prefer [subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(...  to printk(KERN_INFO ...
      +		printk(KERN_INFO" [0x%02X/0x%02X]: 0x%02X\n",
      
      WARNING: Prefer seq_puts to seq_printf
      +	seq_printf(s, AB8500_NAME_STRING " register values:\n");
      
      WARNING: Prefer seq_puts to seq_printf
      +	seq_printf(s, AB8500_NAME_STRING " register values:\n");
      
      WARNING: Prefer [subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(...  to printk(KERN_INFO ...
      +	printk(KERN_INFO"ab8500 register values:\n");
      
      WARNING: Prefer [subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(...  to printk(KERN_INFO ...
      +		printk(KERN_INFO" bank 0x%02X:\n", i);
      
      WARNING: externs should be avoided in .c files
      +extern int prcmu_abb_read(u8 slave, u8 reg, u8 *value, u8 size);
      
      WARNING: quoted string split across lines
      +	pr_info("Saving all ABB registers at \"ab8500_complete_register_dump\" "
      +		"for crash analyze.\n");
      
      WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
      +		printk(KERN_ERR "abx500_set_reg failed %d, %d", err, __LINE__);
      
      WARNING: Prefer seq_puts to seq_printf
      +	seq_printf(s, "name: number:  number of: wake:\n");
      
      WARNING: line over 80 characters
      +	return single_open(file, ab8500_print_modem_registers, inode->i_private);
      
      WARNING: line over 80 characters
      +	return single_open(file, ab8500_gpadc_btemp_ball_print, inode->i_private);
      
      WARNING: line over 80 characters
      +	return single_open(file, ab8500_gpadc_main_bat_v_print, inode->i_private);
      
      WARNING: line over 80 characters
      +	vbat_true_meas_convert = ab8500_gpadc_ad_to_voltage(gpadc, VBAT_TRUE_MEAS,
      
      WARNING: line over 80 characters
      +static int ab8540_gpadc_vbat_true_meas_and_ibat_print(struct seq_file *s, void *p)
      
      WARNING: line over 80 characters
      +static const struct file_operations ab8540_gpadc_vbat_true_meas_and_ibat_fops = {
      
      WARNING: line over 80 characters
      +		vmain_l, vmain_h, btemp_l, btemp_h, vbat_l, vbat_h, ibat_l, ibat_h);
      
      WARNING: quoted string split across lines
      +		dev_err(dev, "debugfs error input: "
      +			"should be egal to 1, 4, 8 or 16\n");
      
      WARNING: Missing a blank line after declarations
      +	char *s = b;
      +	if ((*s == '0') && ((*(s+1) == 'x') || (*(s+1) == 'X'))) {
      
      WARNING: simple_strtoul is obsolete, use kstrtoul instead
      +			loc.mask = simple_strtoul(b, &b, 0);
      
      WARNING: simple_strtol is obsolete, use kstrtol instead
      +			loc.shift = simple_strtol(b, &b, 0);
      
      WARNING: simple_strtoul is obsolete, use kstrtoul instead
      +	loc.bank = simple_strtoul(b, &b, 0);
      
      WARNING: simple_strtoul is obsolete, use kstrtoul instead
      +	loc.addr = simple_strtoul(b, &b, 0);
      
      WARNING: simple_strtoul is obsolete, use kstrtoul instead
      +		val = simple_strtoul(b, &b, 0);
      
      WARNING: quoted string split across lines
      +	pr_warn("HWREG request: %s, %s, addr=0x%08X, mask=0x%X, shift=%d"
      +			"value=0x%X\n", (write) ? "write" : "read",
      
      WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
      +		printk(KERN_ERR "sysfs_create_file failed %d\n", err);
      
      WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
      +		printk(KERN_ERR "request_threaded_irq failed %d, %lu\n",
      
      ERROR: code indent should use tabs where possible
      +                       err, user_val);$
      
      WARNING: please, no spaces at the start of a line
      +                       err, user_val);$
      
      WARNING: Missing a blank line after declarations
      +	struct resource *res;
      +	debug_bank = AB8500_MISC;
      
      ERROR: space required after that ',' (ctx:VxV)
      +		sizeof(*dev_attr)*num_irqs,GFP_KERNEL);
       		                          ^
      
      WARNING: return of an errno should typically be -ve (return -ENXIO)
      +		return ENXIO;
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("register-bank", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("register-address", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("register-value", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("irq-subscribe", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("irq-unsubscribe", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("all-modem-registers", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("main_charger_v", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("main_charger_c", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +	file = debugfs_create_file("usb_charger_c", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +		file = debugfs_create_file("xtal_temp", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +			ab8500_gpadc_dir, &plf->dev, &ab8540_gpadc_xtal_temp_fops);
      
      WARNING: line over 80 characters
      +		file = debugfs_create_file("vbattruemeas", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +		file = debugfs_create_file("otp_calib", (S_IRUGO | S_IWUSR | S_IWGRP),
      
      WARNING: line over 80 characters
      +			ab8500_gpadc_dir, &plf->dev, &ab8540_gpadc_otp_calib_fops);
      
      total: 2 errors, 44 warnings, 3230 lines checked
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      4362175d
  5. 11 7月, 2014 2 次提交
  6. 09 7月, 2014 5 次提交
    • L
      Doc: mfd: as3722: Add details of optional missing property · 384d0f0e
      Laxman Dewangan 提交于
      Add details of following properties which are used on driver but
      not documented on DT binding document.
      - ams,enable-internal-int-pullup
      - ams,enable-internal-i2c-pullup
      Reported-by: NLee Jones <lee.jones@linaro.org>
      Signed-off-by: NLaxman Dewangan <ldewangan@nvidia.com>
      Acked-by: NLinus Walleij <linus.walleij@linaro.org>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      384d0f0e
    • D
      mfd: cros_ec: Use the proper size when looking at the cros_ec_i2c result · d6c15ed2
      Doug Anderson 提交于
      We know how many bytes the EC should be sending us (which is also the
      number of bytes transferred) and also how many bytes the EC actually
      wanted to send to us.  When computing the checksum and copying back
      data let's make sure we take the lesser of the two of those.  We'll
      also complain if the EC tried to send us too many bytes.  The EC
      sending us too few bytes is legit for when we send the EC an invalid
      command.
      
      This is based on similar code in cros_ec_spi.
      Signed-off-by: NDoug Anderson <dianders@chromium.org>
      Reviewed-by: NSimon Glass <sjg@chromium.org>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      d6c15ed2
    • L
      mfd: ab8500-core: Resolve code style issues · 7ccf40b1
      Lee Jones 提交于
      Soothes the following checkpatch warnings:
      
          WARNING: line over 80 characters
          #151: FILE: drivers/mfd/ab8500-core.c:151:
          +	0, 1, 2, 3, 4, -1, -1, -1, -1, 11, 18, 19, 20, 21, 12, 13, 24, 5, 22, 23,
      
          ERROR: spaces required around that '=' (ctx:VxW)
          #325: FILE: drivers/mfd/ab8500-core.c:325:
          +	ret= mask_and_set_register_interruptible(ab8500, bank, reg,
           	   ^
      
          WARNING: line over 80 characters
          #418: FILE: drivers/mfd/ab8500-core.c:418:
          +		else if (offset >= AB9540_INT_GPIO50R && offset <= AB9540_INT_GPIO54R)
      
          WARNING: line over 80 characters
          #420: FILE: drivers/mfd/ab8500-core.c:420:
          +		else if (offset == AB8540_INT_GPIO43R || offset == AB8540_INT_GPIO44R)
      
          ERROR: spaces required around that '==' (ctx:VxV)
          #454: FILE: drivers/mfd/ab8500-core.c:454:
          +	if ((i==3) && (*offset >= 24))
           	      ^
      
          ERROR: code indent should use tabs where possible
          #576: FILE: drivers/mfd/ab8500-core.c:576:
          +        .map    = ab8500_irq_map,$
      
          WARNING: please, no spaces at the start of a line
          #576: FILE: drivers/mfd/ab8500-core.c:576:
          +        .map    = ab8500_irq_map,$
      
          ERROR: code indent should use tabs where possible
          #577: FILE: drivers/mfd/ab8500-core.c:577:
          +        .xlate  = irq_domain_xlate_twocell,$
      
          WARNING: please, no spaces at the start of a line
          #577: FILE: drivers/mfd/ab8500-core.c:577:
          +        .xlate  = irq_domain_xlate_twocell,$
      
          WARNING: char * array declaration might be better as static const
          #1554: FILE: drivers/mfd/ab8500-core.c:1554:
          +	static char *switch_off_status[] = {
      
          WARNING: char * array declaration might be better as static const
          #1563: FILE: drivers/mfd/ab8500-core.c:1563:
          +	static char *turn_on_status[] = {
      
          WARNING: sizeof *ab8500 should be sizeof(*ab8500)
          #1582: FILE: drivers/mfd/ab8500-core.c:1582:
          +	ab8500 = devm_kzalloc(&pdev->dev, sizeof *ab8500, GFP_KERNEL);
      
          ERROR: space required after that close brace '}'
          #1639: FILE: drivers/mfd/ab8500-core.c:1639:
          +	}/* Configure AB8500 or AB9540 IRQ */
      
          WARNING: line over 80 characters
          #1652: FILE: drivers/mfd/ab8500-core.c:1652:
          +	ab8500->oldmask = devm_kzalloc(&pdev->dev, ab8500->mask_size, GFP_KERNEL);
      
          WARNING: Prefer [subsystem eg: netdev]_cont([subsystem]dev, ... then dev_cont(dev, ... then pr_cont(...  to printk(KERN_CONT ...
          #1677: FILE: drivers/mfd/ab8500-core.c:1677:
          +				printk(KERN_CONT " \"%s\"",
      
          WARNING: Prefer [subsystem eg: netdev]_cont([subsystem]dev, ... then dev_cont(dev, ... then pr_cont(...  to printk(KERN_CONT ...
          #1682: FILE: drivers/mfd/ab8500-core.c:1682:
          +		printk(KERN_CONT "\n");
      
          WARNING: Prefer [subsystem eg: netdev]_cont([subsystem]dev, ... then dev_cont(dev, ... then pr_cont(...  to printk(KERN_CONT ...
          #1684: FILE: drivers/mfd/ab8500-core.c:1684:
          +		printk(KERN_CONT " None\n");
      
          WARNING: printk() should include KERN_ facility level
          #1695: FILE: drivers/mfd/ab8500-core.c:1695:
          +				printk("\"%s\" ", turn_on_status[i]);
      
          WARNING: printk() should include KERN_ facility level
          #1700: FILE: drivers/mfd/ab8500-core.c:1700:
          +		printk("None\n");
      
          total: 5 errors, 14 warnings, 1869 lines checked
      Reviewed-by: NLinus Walleij <linus.walleij@linaro.org>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      7ccf40b1
    • L
      mfd: ab3100-core: Correct code sytle problems · cd63a894
      Lee Jones 提交于
      Corrects the following checkpatch gripes:
      
          WARNING: quoted string split across lines
          #95: FILE: drivers/mfd/ab3100-core.c:95:
          +			"write error (write register) "
          +			"%d bytes transferred (expected 2)\n",
      
          WARNING: quoted string split across lines
          #139: FILE: drivers/mfd/ab3100-core.c:139:
          +			"write error (write test register) "
          +			"%d bytes transferred (expected 2)\n",
      
          WARNING: quoted string split across lines
          #175: FILE: drivers/mfd/ab3100-core.c:175:
          +			"write error (send register address) "
          +			"%d bytes transferred (expected 1)\n",
      
          WARNING: quoted string split across lines
          #193: FILE: drivers/mfd/ab3100-core.c:193:
          +			"write error (read register) "
          +			"%d bytes transferred (expected 1)\n",
      
          WARNING: quoted string split across lines
          #241: FILE: drivers/mfd/ab3100-core.c:241:
          +			"write error (send first register address) "
          +			"%d bytes transferred (expected 1)\n",
      
          WARNING: quoted string split across lines
          #256: FILE: drivers/mfd/ab3100-core.c:256:
          +			"write error (read register page) "
          +			"%d bytes transferred (expected %d)\n",
      
          WARNING: quoted string split across lines
          #299: FILE: drivers/mfd/ab3100-core.c:299:
          +			"write error (maskset send address) "
          +			"%d bytes transferred (expected 1)\n",
      
          WARNING: quoted string split across lines
          #314: FILE: drivers/mfd/ab3100-core.c:314:
          +			"write error (maskset read register) "
          +			"%d bytes transferred (expected 1)\n",
      
          WARNING: quoted string split across lines
          #334: FILE: drivers/mfd/ab3100-core.c:334:
          +			"write error (write register) "
          +			"%d bytes transferred (expected 2)\n",
      
          WARNING: please, no spaces at the start of a line
          #374: FILE: drivers/mfd/ab3100-core.c:374:
          +  return blocking_notifier_chain_unregister(&ab3100->event_subscribers,$
      
          WARNING: Prefer seq_puts to seq_printf
          #458: FILE: drivers/mfd/ab3100-core.c:458:
          +	seq_printf(s, "AB3100 registers:\n");
      
          WARNING: quoted string split across lines
          #564: FILE: drivers/mfd/ab3100-core.c:564:
          +			 "debug write reg[0x%02x] with 0x%02x, "
          +			 "after readback: 0x%02x\n",
      
          WARNING: quoted string split across lines
          #723: FILE: drivers/mfd/ab3100-core.c:723:
          +			 "AB3100 P1E variant detected, "
          +			 "forcing chip to 32KHz\n");
      
          WARNING: quoted string split across lines
          #882: FILE: drivers/mfd/ab3100-core.c:882:
          +			"could not communicate with the AB3100 analog "
          +			"baseband chip\n");
      
          WARNING: quoted string split across lines
          #906: FILE: drivers/mfd/ab3100-core.c:906:
          +		dev_err(&client->dev, "accepting it anyway. Please update "
          +			"the driver.\n");
      
          total: 0 errors, 15 warnings, 999 lines checked
      Reviewed-by: NLinus Walleij <linus.walleij@linaro.org>
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      cd63a894
    • L
      mfd: aat2870-core: Stop using obsolte simple_strtoul() · 0ebc1c25
      Lee Jones 提交于
      Soothes checkpatch warning:
      
          WARNING: simple_strtoul is obsolete, use kstrtoul instead
          #306: FILE: drivers/mfd/aat2870-core.c:306:
          +	addr = simple_strtoul(start, &start, 16);
      Signed-off-by: NLee Jones <lee.jones@linaro.org>
      0ebc1c25