* [PATCH] Staging: nvec: Fixed various style issues.
@ 2011-08-17 20:36 Akshay Joshi
2011-08-17 21:17 ` Greg KH
2011-08-18 6:58 ` Marc Dietrich
0 siblings, 2 replies; 7+ messages in thread
From: Akshay Joshi @ 2011-08-17 20:36 UTC (permalink / raw)
To: Greg Kroah-Hartman, Marc Dietrich, Jiri Kosina, Vitaliy Ivanov,
devel, linux-kernel
Cc: Akshay Joshi
Removed trailing spaces, brace/parenthesis issues and other coding style
problems, along with commented out code.
Signed-off-by: Akshay Joshi <me@akshayjoshi.com>
---
drivers/staging/nvec/Kconfig | 2 +-
drivers/staging/nvec/nvec-keytable.h | 58 ++++----
drivers/staging/nvec/nvec.c | 84 ++++------
drivers/staging/nvec/nvec.h | 8 +-
drivers/staging/nvec/nvec_kbd.c | 22 ++--
drivers/staging/nvec/nvec_power.c | 288 ++++++++++++++++------------------
drivers/staging/nvec/nvec_ps2.c | 50 +++---
7 files changed, 241 insertions(+), 271 deletions(-)
diff --git a/drivers/staging/nvec/Kconfig b/drivers/staging/nvec/Kconfig
index 987ad48..9a7e4e4 100644
--- a/drivers/staging/nvec/Kconfig
+++ b/drivers/staging/nvec/Kconfig
@@ -9,7 +9,7 @@ config KEYBOARD_NVEC
bool "Keyboard on nVidia compliant EC"
depends on MFD_NVEC
help
- Say Y here to enable support for a keyboard connected to
+ Say Y here to enable support for a keyboard connected to
a nVidia compliant embedded controller.
config SERIO_NVEC_PS2
diff --git a/drivers/staging/nvec/nvec-keytable.h b/drivers/staging/nvec/nvec-keytable.h
index 6a1c4f7..acd3929 100644
--- a/drivers/staging/nvec/nvec-keytable.h
+++ b/drivers/staging/nvec/nvec-keytable.h
@@ -111,42 +111,42 @@ static unsigned short code_tab_102us[] = {
KEY_102ND, //VK_OEM_102 henry+ 0x2B (43) BACKSLASH have been used,change to use 0X56 (86)
KEY_F11, //VK_F11
KEY_F12, //VK_F12
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0, // 60
0,
0,
- KEY_SEARCH, // add search key map
- 0,
0,
0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0, // 70
+ 0,
+ 0,
+ 0,
+ 0, // 60
+ 0,
+ 0,
+ KEY_SEARCH, // add search key map
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0, // 70
0,
0,
KEY_KP5, //73 for JP keyboard '\' key, report 0x4c
- 0,
0,
0,
- 0,
- 0,
- 0,
- 0,
- 0,
- 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
+ 0,
KEY_KP9, //7d for JP keyboard '|' key, report 0x49
};
@@ -246,7 +246,7 @@ static unsigned short extcode_tab_us102[] = {
0, //VK_RWIN
KEY_ESC, //VK_APPS
KEY_KPMINUS, //for power button workaround
- 0,
+ 0,
0,
0,
0,
@@ -263,4 +263,4 @@ static unsigned short extcode_tab_us102[] = {
0, //VK_LAUNCH_MEDIA_SELECT
};
-static unsigned short* code_tabs[] = {code_tab_102us, extcode_tab_us102 };
+static unsigned short *code_tabs[] = {code_tab_102us, extcode_tab_us102 };
diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
index 72258e8..b822cda 100644
--- a/drivers/staging/nvec/nvec.c
+++ b/drivers/staging/nvec/nvec.c
@@ -24,9 +24,9 @@
#include <linux/platform_device.h>
#include "nvec.h"
-static unsigned char EC_DISABLE_EVENT_REPORTING[] = {'\x04','\x00','\x00'};
-static unsigned char EC_ENABLE_EVENT_REPORTING[] = {'\x04','\x00','\x01'};
-static unsigned char EC_GET_FIRMWARE_VERSION[] = {'\x07','\x15'};
+static unsigned char EC_DISABLE_EVENT_REPORTING[] = {'\x04', '\x00', '\x00'};
+static unsigned char EC_ENABLE_EVENT_REPORTING[] = {'\x04', '\x00', '\x01'};
+static unsigned char EC_GET_FIRMWARE_VERSION[] = {'\x07', '\x15'};
static struct nvec_chip *nvec_power_handle;
@@ -43,7 +43,7 @@ static int nvec_status_notifier(struct notifier_block *nb, unsigned long event_t
unsigned char *msg = (unsigned char *)data;
int i;
- if(event_type != NVEC_CNTL)
+ if (event_type != NVEC_CNTL)
return NOTIFY_DONE;
printk("unhandled msg type %ld, payload: ", event_type);
@@ -75,25 +75,23 @@ static void nvec_request_master(struct work_struct *work)
{
struct nvec_chip *nvec = container_of(work, struct nvec_chip, tx_work);
- if(!list_empty(&nvec->tx_data)) {
+ if (!list_empty(&nvec->tx_data))
gpio_set_value(nvec->gpio, 0);
- }
}
static int parse_msg(struct nvec_chip *nvec, struct nvec_msg *msg)
{
int i;
- if((msg->data[0] & 1<<7) == 0 && msg->data[3]) {
+ if ((msg->data[0] & 1<<7) == 0 && msg->data[3]) {
dev_err(nvec->dev, "ec responded %02x %02x %02x %02x\n", msg->data[0],
msg->data[1], msg->data[2], msg->data[3]);
return -EINVAL;
}
- if ((msg->data[0] >> 7 ) == 1 && (msg->data[0] & 0x0f) == 5)
- {
+ if ((msg->data[0] >> 7) == 1 && (msg->data[0] & 0x0f) == 5) {
dev_warn(nvec->dev, "ec system event ");
- for (i=0; i < msg->data[1]; i++)
+ for (i = 0; i < msg->data[1]; i++)
dev_warn(nvec->dev, "%02x ", msg->data[2+i]);
dev_warn(nvec->dev, "\n");
}
@@ -125,20 +123,18 @@ static void nvec_dispatch(struct work_struct *work)
struct nvec_chip *nvec = container_of(work, struct nvec_chip, rx_work);
struct nvec_msg *msg;
- while(!list_empty(&nvec->rx_data))
- {
+ while (!list_empty(&nvec->rx_data)) {
msg = list_first_entry(&nvec->rx_data, struct nvec_msg, node);
list_del_init(&msg->node);
- if(nvec->sync_write_pending == (msg->data[2] << 8) + msg->data[0])
- {
+ if (nvec->sync_write_pending == (msg->data[2] << 8) + msg->data[0]) {
dev_dbg(nvec->dev, "sync write completed!\n");
nvec->sync_write_pending = 0;
nvec->last_sync_msg = msg;
complete(&nvec->sync_write);
} else {
parse_msg(nvec, msg);
- if((!msg) || (!msg->data))
+ if ((!msg) || (!msg->data))
dev_warn(nvec->dev, "attempt access zero pointer");
else {
kfree(msg->data);
@@ -159,20 +155,17 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
status = readl(i2c_regs + I2C_SL_STATUS);
- if(!(status & I2C_SL_IRQ))
- {
+ if (!(status & I2C_SL_IRQ)) {
dev_warn(nvec->dev, "nvec Spurious IRQ\n");
- //Yup, handled. ahum.
+ /* Yup, handled. ahum. */
goto handled;
}
- if(status & END_TRANS && !(status & RCVD))
- {
- //Reenable IRQ only when even has been sent
+ if (status & END_TRANS && !(status & RCVD)) {
+ /* Reenable IRQ only when even has been sent */
//printk("Write sequence ended !\n");
- //parse_msg(nvec);
+ //parse_msg(nvec);
nvec->state = NVEC_WAIT;
- if(nvec->rx->size > 1)
- {
+ if (nvec->rx->size > 1) {
list_add_tail(&nvec->rx->node, &nvec->rx_data);
schedule_work(&nvec->rx_work);
} else {
@@ -180,15 +173,13 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
kfree(nvec->rx);
}
return IRQ_HANDLED;
- } else if(status & RNW)
- {
+ } else if (status & RNW) {
// Work around for AP20 New Slave Hw Bug. Give 1us extra.
// nvec/smbus/nvec_i2c_transport.c in NV`s crap for reference
- if(status & RCVD)
+ if (status & RCVD)
udelay(3);
- if(status & RCVD)
- {
+ if (status & RCVD) {
nvec->state = NVEC_WRITE;
//Master wants something from us. New communication
// dev_dbg(nvec->dev, "New read comm!\n");
@@ -197,15 +188,14 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
// dev_dbg(nvec->dev, "Read comm cont !\n");
}
//if(msg_pos<msg_size) {
- if(list_empty(&nvec->tx_data))
- {
+ if (list_empty(&nvec->tx_data)) {
dev_err(nvec->dev, "nvec empty tx - sending no-op\n");
to_send = 0x8a;
nvec_write_async(nvec, "\x07\x02", 2);
// to_send = 0x01;
} else {
msg = list_first_entry(&nvec->tx_data, struct nvec_msg, node);
- if(msg->pos < msg->size) {
+ if (msg->pos < msg->size) {
to_send = msg->data[msg->pos];
msg->pos++;
} else {
@@ -213,8 +203,7 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
to_send = 0x01;
}
- if(msg->pos >= msg->size)
- {
+ if (msg->pos >= msg->size) {
list_del_init(&msg->node);
kfree(msg->data);
kfree(msg);
@@ -232,13 +221,12 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
} else {
received = readl(i2c_regs + I2C_SL_RCVD);
//Workaround?
- if(status & RCVD) {
+ if (status & RCVD) {
writel(0, i2c_regs + I2C_SL_RCVD);
goto handled;
}
- if (nvec->state == NVEC_WAIT)
- {
+ if (nvec->state == NVEC_WAIT) {
nvec->state = NVEC_READ;
msg = kzalloc(sizeof(struct nvec_msg), GFP_NOWAIT);
msg->data = kzalloc(32, GFP_NOWAIT);
@@ -307,7 +295,7 @@ static int __devinit tegra_nvec_probe(struct platform_device *pdev)
unsigned char *i2c_regs;
nvec = kzalloc(sizeof(struct nvec_chip), GFP_KERNEL);
- if(nvec == NULL) {
+ if (nvec == NULL) {
dev_err(&pdev->dev, "failed to reserve memory\n");
return -ENOMEM;
}
@@ -324,7 +312,7 @@ static int __devinit tegra_nvec_probe(struct platform_device *pdev)
clk_enable(i2c_clk);
*/
i2c_regs = ioremap(pdata->base, pdata->size);
- if(!i2c_regs) {
+ if (!i2c_regs) {
dev_err(nvec->dev, "failed to ioremap registers\n");
goto failed;
}
@@ -332,7 +320,7 @@ static int __devinit tegra_nvec_probe(struct platform_device *pdev)
nvec->i2c_regs = i2c_regs;
i2c_clk = clk_get_sys(pdata->clock, NULL);
- if(IS_ERR_OR_NULL(i2c_clk)) {
+ if (IS_ERR_OR_NULL(i2c_clk)) {
dev_err(nvec->dev, "failed to get clock tegra-i2c.2\n");
goto failed;
}
@@ -340,7 +328,7 @@ static int __devinit tegra_nvec_probe(struct platform_device *pdev)
tegra_init_i2c_slave(pdata, i2c_regs, i2c_clk);
err = request_irq(nvec->irq, i2c_interrupt, IRQF_DISABLED, "nvec", nvec);
- if(err) {
+ if (err) {
dev_err(nvec->dev, "couldn't request irq");
goto failed;
}
@@ -350,7 +338,7 @@ static int __devinit tegra_nvec_probe(struct platform_device *pdev)
/* Set the gpio to low when we've got something to say */
err = gpio_request(nvec->gpio, "nvec gpio");
- if(err < 0)
+ if (err < 0)
dev_err(nvec->dev, "couldn't request gpio\n");
tegra_gpio_enable(nvec->gpio);
@@ -375,10 +363,9 @@ static int __devinit tegra_nvec_probe(struct platform_device *pdev)
nvec_ps2(nvec);
#endif
- /* setup subdevs */
- for (i = 0; i < pdata->num_subdevs; i++) {
+ /* setup subdevs */
+ for (i = 0; i < pdata->num_subdevs; i++)
ret = nvec_add_subdev(nvec, &pdata->subdevs[i]);
- }
nvec->nvec_status_notifier.notifier_call = nvec_status_notifier;
nvec_register_notifier(nvec, &nvec->nvec_status_notifier, 0);
@@ -431,8 +418,8 @@ static int tegra_nvec_suspend(struct platform_device *pdev, pm_message_t state)
return 0;
}
-static int tegra_nvec_resume(struct platform_device *pdev) {
-
+static int tegra_nvec_resume(struct platform_device *pdev)
+{
struct nvec_chip *nvec = platform_get_drvdata(pdev);
dev_dbg(nvec->dev, "resuming\n");
@@ -446,8 +433,7 @@ static int tegra_nvec_resume(struct platform_device *pdev) {
#define tegra_nvec_resume NULL
#endif
-static struct platform_driver nvec_device_driver =
-{
+static struct platform_driver nvec_device_driver = {
.probe = tegra_nvec_probe,
.remove = __devexit_p(tegra_nvec_remove),
.suspend = tegra_nvec_suspend,
diff --git a/drivers/staging/nvec/nvec.h b/drivers/staging/nvec/nvec.h
index a2d82dc..d1f083b 100644
--- a/drivers/staging/nvec/nvec.h
+++ b/drivers/staging/nvec/nvec.h
@@ -16,7 +16,7 @@ typedef enum {
} how_care;
typedef enum {
- NVEC_SYS=1,
+ NVEC_SYS = 1,
NVEC_BAT,
NVEC_KBD = 5,
NVEC_PS2,
@@ -26,9 +26,9 @@ typedef enum {
} nvec_event;
typedef enum {
- NVEC_WAIT,
- NVEC_READ,
- NVEC_WRITE
+ NVEC_WAIT,
+ NVEC_READ,
+ NVEC_WRITE
} nvec_state;
struct nvec_msg {
diff --git a/drivers/staging/nvec/nvec_kbd.c b/drivers/staging/nvec/nvec_kbd.c
index 9a98507..04bd285 100644
--- a/drivers/staging/nvec/nvec_kbd.c
+++ b/drivers/staging/nvec/nvec_kbd.c
@@ -4,7 +4,7 @@
#include "nvec-keytable.h"
#include "nvec.h"
-#define ACK_KBD_EVENT {'\x05','\xed','\x01'}
+#define ACK_KBD_EVENT {'\x05', '\xed', '\x01'}
static unsigned char keycodes[ARRAY_SIZE(code_tab_102us)
+ ARRAY_SIZE(extcode_tab_us102)];
@@ -27,10 +27,10 @@ static int nvec_keys_notifier(struct notifier_block *nb,
nvec_size _size = (msg[0] & (3 << 5)) >> 5;
/* power on/off button */
- if(_size == NVEC_VAR_SIZE)
+ if (_size == NVEC_VAR_SIZE)
return NOTIFY_STOP;
- if(_size == NVEC_3BYTES)
+ if (_size == NVEC_3BYTES)
msg++;
code = msg[1] & 0x7f;
@@ -51,13 +51,13 @@ static int nvec_kbd_event(struct input_dev *dev, unsigned int type,
unsigned char buf[] = ACK_KBD_EVENT;
struct nvec_chip *nvec = keys_dev.nvec;
- if(type==EV_REP)
+ if (type == EV_REP)
return 0;
- if(type!=EV_LED)
+ if (type != EV_LED)
return -1;
- if(code!=LED_CAPSL)
+ if (code != LED_CAPSL)
return -1;
buf[2] = !!value;
@@ -73,11 +73,11 @@ int __init nvec_kbd_init(struct nvec_chip *nvec)
j = 0;
- for(i = 0; i < ARRAY_SIZE(code_tab_102us); ++i)
+ for (i = 0; i < ARRAY_SIZE(code_tab_102us); ++i)
keycodes[j++] = code_tab_102us[i];
- for(i = 0; i < ARRAY_SIZE(extcode_tab_us102); ++i)
- keycodes[j++]=extcode_tab_us102[i];
+ for (i = 0; i < ARRAY_SIZE(extcode_tab_us102); ++i)
+ keycodes[j++] = extcode_tab_us102[i];
idev = input_allocate_device();
idev->name = "Tegra nvec keyboard";
@@ -89,12 +89,12 @@ int __init nvec_kbd_init(struct nvec_chip *nvec)
idev->keycodesize = sizeof(unsigned char);
idev->keycodemax = ARRAY_SIZE(keycodes);
- for( i = 0; i < ARRAY_SIZE(keycodes); ++i)
+ for (i = 0; i < ARRAY_SIZE(keycodes); ++i)
set_bit(keycodes[i], idev->keybit);
clear_bit(0, idev->keybit);
err = input_register_device(idev);
- if(err)
+ if (err)
goto fail;
keys_dev.input = idev;
diff --git a/drivers/staging/nvec/nvec_power.c b/drivers/staging/nvec/nvec_power.c
index df164ad..d82dd97 100644
--- a/drivers/staging/nvec/nvec_power.c
+++ b/drivers/staging/nvec/nvec_power.c
@@ -7,8 +7,7 @@
#include <linux/delay.h>
#include "nvec.h"
-struct nvec_power
-{
+struct nvec_power {
struct notifier_block notifier;
struct delayed_work poller;
struct nvec_chip *nvec;
@@ -77,10 +76,8 @@ static int nvec_power_notifier(struct notifier_block *nb,
if (event_type != NVEC_SYS)
return NOTIFY_DONE;
- if(res->sub_type == 0)
- {
- if (power->on != res->plu)
- {
+ if (res->sub_type == 0) {
+ if (power->on != res->plu) {
power->on = res->plu;
power_supply_changed(&nvec_psy);
}
@@ -89,8 +86,7 @@ static int nvec_power_notifier(struct notifier_block *nb,
return NOTIFY_OK;
}
-static const int bat_init[] =
-{
+static const int bat_init[] = {
LAST_FULL_CHARGE_CAPACITY, DESIGN_CAPACITY, CRITICAL_CAPACITY,
MANUFACTURER, MODEL, TYPE,
};
@@ -100,8 +96,7 @@ static void get_bat_mfg_data(struct nvec_power *power)
int i;
char buf[] = { '\x02', '\x00' };
- for (i = 0; i < ARRAY_SIZE(bat_init); i++)
- {
+ for (i = 0; i < ARRAY_SIZE(bat_init); i++) {
buf[1] = bat_init[i];
nvec_write_async(power->nvec, buf, 2);
}
@@ -117,91 +112,87 @@ static int nvec_power_bat_notifier(struct notifier_block *nb,
if (event_type != NVEC_BAT)
return NOTIFY_DONE;
- switch(res->sub_type)
- {
- case SLOT_STATUS:
- if (res->plc[0] & 1)
- {
- if (power->bat_present == 0)
- {
- status_changed = 1;
- get_bat_mfg_data(power);
- }
-
- power->bat_present = 1;
-
- switch ((res->plc[0] >> 1) & 3)
- {
- case 0:
- power->bat_status = POWER_SUPPLY_STATUS_NOT_CHARGING;
- break;
- case 1:
- power->bat_status = POWER_SUPPLY_STATUS_CHARGING;
- break;
- case 2:
- power->bat_status = POWER_SUPPLY_STATUS_DISCHARGING;
- break;
- default:
- power->bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
- }
- } else {
- if (power->bat_present == 1)
- status_changed = 1;
-
- power->bat_present = 0;
+ switch (res->sub_type) {
+ case SLOT_STATUS:
+ if (res->plc[0] & 1) {
+ if (power->bat_present == 0) {
+ status_changed = 1;
+ get_bat_mfg_data(power);
+ }
+
+ power->bat_present = 1;
+
+ switch ((res->plc[0] >> 1) & 3) {
+ case 0:
+ power->bat_status = POWER_SUPPLY_STATUS_NOT_CHARGING;
+ break;
+ case 1:
+ power->bat_status = POWER_SUPPLY_STATUS_CHARGING;
+ break;
+ case 2:
+ power->bat_status = POWER_SUPPLY_STATUS_DISCHARGING;
+ break;
+ default:
power->bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
}
- power->bat_cap = res->plc[1];
- if (status_changed)
- power_supply_changed(&nvec_bat_psy);
- break;
- case VOLTAGE:
- power->bat_voltage_now = res->plu * 1000;
- break;
- case TIME_REMAINING:
- power->time_remain = res->plu * 3600;
- break;
- case CURRENT:
- power->bat_current_now = res->pls * 1000;
- break;
- case AVERAGE_CURRENT:
- power->bat_current_avg = res->pls * 1000;
- break;
- case CAPACITY_REMAINING:
- power->capacity_remain = res->plu * 1000;
- break;
- case LAST_FULL_CHARGE_CAPACITY:
- power->charge_last_full = res->plu * 1000;
- break;
- case DESIGN_CAPACITY:
- power->charge_full_design = res->plu * 1000;
- break;
- case CRITICAL_CAPACITY:
- power->critical_capacity = res->plu * 1000;
- break;
- case TEMPERATURE:
- power->bat_temperature = res->plu - 2732;
- break;
- case MANUFACTURER:
- memcpy(power->bat_manu, &res->plc, res->length-2);
- power->bat_model[res->length-2] = '\0';
- break;
- case MODEL:
- memcpy(power->bat_model, &res->plc, res->length-2);
- power->bat_model[res->length-2] = '\0';
- break;
- case TYPE:
- memcpy(power->bat_type, &res->plc, res->length-2);
- power->bat_type[res->length-2] = '\0';
- /* this differs a little from the spec
- fill in more if you find some */
- if (!strncmp(power->bat_type, "Li", 30))
- power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_LION;
- else
- power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_UNKNOWN;
- break;
- default:
- return NOTIFY_STOP;
+ } else {
+ if (power->bat_present == 1)
+ status_changed = 1;
+
+ power->bat_present = 0;
+ power->bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
+ }
+ power->bat_cap = res->plc[1];
+ if (status_changed)
+ power_supply_changed(&nvec_bat_psy);
+ break;
+ case VOLTAGE:
+ power->bat_voltage_now = res->plu * 1000;
+ break;
+ case TIME_REMAINING:
+ power->time_remain = res->plu * 3600;
+ break;
+ case CURRENT:
+ power->bat_current_now = res->pls * 1000;
+ break;
+ case AVERAGE_CURRENT:
+ power->bat_current_avg = res->pls * 1000;
+ break;
+ case CAPACITY_REMAINING:
+ power->capacity_remain = res->plu * 1000;
+ break;
+ case LAST_FULL_CHARGE_CAPACITY:
+ power->charge_last_full = res->plu * 1000;
+ break;
+ case DESIGN_CAPACITY:
+ power->charge_full_design = res->plu * 1000;
+ break;
+ case CRITICAL_CAPACITY:
+ power->critical_capacity = res->plu * 1000;
+ break;
+ case TEMPERATURE:
+ power->bat_temperature = res->plu - 2732;
+ break;
+ case MANUFACTURER:
+ memcpy(power->bat_manu, &res->plc, res->length-2);
+ power->bat_model[res->length-2] = '\0';
+ break;
+ case MODEL:
+ memcpy(power->bat_model, &res->plc, res->length-2);
+ power->bat_model[res->length-2] = '\0';
+ break;
+ case TYPE:
+ memcpy(power->bat_type, &res->plc, res->length-2);
+ power->bat_type[res->length-2] = '\0';
+ /* this differs a little from the spec
+ fill in more if you find some */
+ if (!strncmp(power->bat_type, "Li", 30))
+ power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_LION;
+ else
+ power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_UNKNOWN;
+ break;
+ default:
+ return NOTIFY_STOP;
}
return NOTIFY_STOP;
@@ -228,56 +219,55 @@ static int nvec_battery_get_property(struct power_supply *psy,
{
struct nvec_power *power = dev_get_drvdata(psy->dev->parent);
- switch(psp)
- {
- case POWER_SUPPLY_PROP_STATUS:
- val->intval = power->bat_status;
- break;
- case POWER_SUPPLY_PROP_CAPACITY:
- val->intval = power->bat_cap;
- break;
- case POWER_SUPPLY_PROP_PRESENT:
- val->intval = power->bat_present;
- break;
- case POWER_SUPPLY_PROP_VOLTAGE_NOW:
- val->intval = power->bat_voltage_now;
- break;
- case POWER_SUPPLY_PROP_CURRENT_NOW:
- val->intval = power->bat_current_now;
- break;
- case POWER_SUPPLY_PROP_CURRENT_AVG:
- val->intval = power->bat_current_avg;
- break;
- case POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW:
- val->intval = power->time_remain;
- break;
- case POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN:
- val->intval = power->charge_full_design;
- break;
- case POWER_SUPPLY_PROP_CHARGE_FULL:
- val->intval = power->charge_last_full;
- break;
- case POWER_SUPPLY_PROP_CHARGE_EMPTY:
- val->intval = power->critical_capacity;
- break;
- case POWER_SUPPLY_PROP_CHARGE_NOW:
- val->intval = power->capacity_remain;
- break;
- case POWER_SUPPLY_PROP_TEMP:
- val->intval = power->bat_temperature;
- break;
- case POWER_SUPPLY_PROP_MANUFACTURER:
- val->strval = power->bat_manu;
- break;
- case POWER_SUPPLY_PROP_MODEL_NAME:
- val->strval = power->bat_model;
- break;
- case POWER_SUPPLY_PROP_TECHNOLOGY:
- val->intval = power->bat_type_enum;
- break;
- default:
- return -EINVAL;
- }
+ switch (psp) {
+ case POWER_SUPPLY_PROP_STATUS:
+ val->intval = power->bat_status;
+ break;
+ case POWER_SUPPLY_PROP_CAPACITY:
+ val->intval = power->bat_cap;
+ break;
+ case POWER_SUPPLY_PROP_PRESENT:
+ val->intval = power->bat_present;
+ break;
+ case POWER_SUPPLY_PROP_VOLTAGE_NOW:
+ val->intval = power->bat_voltage_now;
+ break;
+ case POWER_SUPPLY_PROP_CURRENT_NOW:
+ val->intval = power->bat_current_now;
+ break;
+ case POWER_SUPPLY_PROP_CURRENT_AVG:
+ val->intval = power->bat_current_avg;
+ break;
+ case POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW:
+ val->intval = power->time_remain;
+ break;
+ case POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN:
+ val->intval = power->charge_full_design;
+ break;
+ case POWER_SUPPLY_PROP_CHARGE_FULL:
+ val->intval = power->charge_last_full;
+ break;
+ case POWER_SUPPLY_PROP_CHARGE_EMPTY:
+ val->intval = power->critical_capacity;
+ break;
+ case POWER_SUPPLY_PROP_CHARGE_NOW:
+ val->intval = power->capacity_remain;
+ break;
+ case POWER_SUPPLY_PROP_TEMP:
+ val->intval = power->bat_temperature;
+ break;
+ case POWER_SUPPLY_PROP_MANUFACTURER:
+ val->strval = power->bat_manu;
+ break;
+ case POWER_SUPPLY_PROP_MODEL_NAME:
+ val->strval = power->bat_model;
+ break;
+ case POWER_SUPPLY_PROP_TECHNOLOGY:
+ val->intval = power->bat_type_enum;
+ break;
+ default:
+ return -EINVAL;
+ }
return 0;
}
@@ -328,8 +318,7 @@ static struct power_supply nvec_psy = {
};
static int counter = 0;
-static int const bat_iter[] =
-{
+static int const bat_iter[] = {
SLOT_STATUS, VOLTAGE, CURRENT, CAPACITY_REMAINING,
#ifdef EC_FULL_DIAG
AVERAGE_CURRENT, TEMPERATURE, TIME_REMAINING,
@@ -352,11 +341,9 @@ static void nvec_power_poll(struct work_struct *work)
/* select a battery request function via round robin
doing it all at once seems to overload the power supply */
buf[0] = '\x02'; /* battery */
- buf[1] = bat_iter[counter++];
+ buf[1] = bat_iter[counter++];
nvec_write_async(power->nvec, buf, 2);
-// printk("%02x %02x\n", buf[0], buf[1]);
-
schedule_delayed_work(to_delayed_work(work), msecs_to_jiffies(5000));
};
@@ -381,7 +368,7 @@ static int __devinit nvec_power_probe(struct platform_device *pdev)
case BAT:
psy = &nvec_bat_psy;
- power->notifier.notifier_call = nvec_power_bat_notifier;
+ power->notifier.notifier_call = nvec_power_bat_notifier;
break;
default:
kfree(power);
@@ -398,14 +385,13 @@ static int __devinit nvec_power_probe(struct platform_device *pdev)
static struct platform_driver nvec_power_driver = {
.probe = nvec_power_probe,
-// .remove = __devexit_p(nvec_power_remove),
.driver = {
.name = "nvec-power",
.owner = THIS_MODULE,
}
};
-static int __init nvec_power_init(void)
+static int __init nvec_power_init(void)
{
return platform_driver_register(&nvec_power_driver);
}
diff --git a/drivers/staging/nvec/nvec_ps2.c b/drivers/staging/nvec/nvec_ps2.c
index 6bb9430..d8e9fd0 100644
--- a/drivers/staging/nvec/nvec_ps2.c
+++ b/drivers/staging/nvec/nvec_ps2.c
@@ -3,12 +3,11 @@
#include <linux/delay.h>
#include "nvec.h"
-#define START_STREAMING {'\x06','\x03','\x01'}
-#define STOP_STREAMING {'\x06','\x04'}
-#define SEND_COMMAND {'\x06','\x01','\xf4','\x01'}
+#define START_STREAMING {'\x06', '\x03', '\x01'}
+#define STOP_STREAMING {'\x06', '\x04'}
+#define SEND_COMMAND {'\x06', '\x01', '\xf4', '\x01'}
-struct nvec_ps2
-{
+struct nvec_ps2 {
struct serio *ser_dev;
struct notifier_block notifier;
struct nvec_chip *nvec;
@@ -54,23 +53,22 @@ static int nvec_ps2_notifier(struct notifier_block *nb,
unsigned char *msg = (unsigned char *)data;
switch (event_type) {
- case NVEC_PS2_EVT:
- serio_interrupt(ps2_dev.ser_dev, msg[2], 0);
- return NOTIFY_STOP;
-
- case NVEC_PS2:
- if (msg[2] == 1)
- for(i = 0; i < (msg[1] - 2); i++)
- serio_interrupt(ps2_dev.ser_dev, msg[i+4], 0);
- else if (msg[1] != 2) /* !ack */
- {
- printk("nvec_ps2: unhandled mouse event ");
- for(i = 0; i <= (msg[1]+1); i++)
- printk("%02x ", msg[i]);
- printk(".\n");
- }
-
- return NOTIFY_STOP;
+ case NVEC_PS2_EVT:
+ serio_interrupt(ps2_dev.ser_dev, msg[2], 0);
+ return NOTIFY_STOP;
+
+ case NVEC_PS2:
+ if (msg[2] == 1)
+ for (i = 0; i < (msg[1] - 2); i++)
+ serio_interrupt(ps2_dev.ser_dev, msg[i+4], 0);
+ else if (msg[1] != 2) { /* !ack */
+ printk("nvec_ps2: unhandled mouse event ");
+ for (i = 0; i <= (msg[1]+1); i++)
+ printk("%02x ", msg[i]);
+ printk(".\n");
+ }
+
+ return NOTIFY_STOP;
}
return NOTIFY_DONE;
@@ -81,10 +79,10 @@ int __init nvec_ps2(struct nvec_chip *nvec)
{
struct serio *ser_dev = kzalloc(sizeof(struct serio), GFP_KERNEL);
- ser_dev->id.type=SERIO_8042;
- ser_dev->write=ps2_sendcommand;
- ser_dev->open=ps2_startstreaming;
- ser_dev->close=ps2_stopstreaming;
+ ser_dev->id.type = SERIO_8042;
+ ser_dev->write = ps2_sendcommand;
+ ser_dev->open = ps2_startstreaming;
+ ser_dev->close = ps2_stopstreaming;
strlcpy(ser_dev->name, "NVEC PS2", sizeof(ser_dev->name));
strlcpy(ser_dev->phys, "NVEC I2C slave", sizeof(ser_dev->phys));
--
1.7.5.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] Staging: nvec: Fixed various style issues.
2011-08-17 20:36 [PATCH] Staging: nvec: Fixed various style issues Akshay Joshi
@ 2011-08-17 21:17 ` Greg KH
2011-08-17 21:23 ` Akshay Joshi
2011-08-18 6:58 ` Marc Dietrich
1 sibling, 1 reply; 7+ messages in thread
From: Greg KH @ 2011-08-17 21:17 UTC (permalink / raw)
To: Akshay Joshi
Cc: Marc Dietrich, Jiri Kosina, Vitaliy Ivanov, devel, linux-kernel
On Wed, Aug 17, 2011 at 04:36:32PM -0400, Akshay Joshi wrote:
> Removed trailing spaces, brace/parenthesis issues and other coding style
> problems, along with commented out code.
Please break this into individual patches, one per logical change, so we
can properly review it and apply it easier.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Staging: nvec: Fixed various style issues.
2011-08-17 21:17 ` Greg KH
@ 2011-08-17 21:23 ` Akshay Joshi
2011-08-18 18:28 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Akshay Joshi @ 2011-08-17 21:23 UTC (permalink / raw)
To: Greg KH; +Cc: Marc Dietrich, Jiri Kosina, Vitaliy Ivanov, devel, linux-kernel
Can I split by file or should I redo it by splitting the changes into
different commits?
Akshay
On Wed, Aug 17, 2011 at 5:17 PM, Greg KH <gregkh@suse.de> wrote:
> On Wed, Aug 17, 2011 at 04:36:32PM -0400, Akshay Joshi wrote:
>> Removed trailing spaces, brace/parenthesis issues and other coding style
>> problems, along with commented out code.
>
> Please break this into individual patches, one per logical change, so we
> can properly review it and apply it easier.
>
> thanks,
>
> greg k-h
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Staging: nvec: Fixed various style issues.
2011-08-17 20:36 [PATCH] Staging: nvec: Fixed various style issues Akshay Joshi
2011-08-17 21:17 ` Greg KH
@ 2011-08-18 6:58 ` Marc Dietrich
2011-08-18 7:30 ` Joe Perches
1 sibling, 1 reply; 7+ messages in thread
From: Marc Dietrich @ 2011-08-18 6:58 UTC (permalink / raw)
To: Akshay Joshi
Cc: Greg Kroah-Hartman, Jiri Kosina, Vitaliy Ivanov, devel,
linux-kernel, Joe Perches
Hi Akshay,
On Wednesday 17 August 2011 22:36:32 Akshay Joshi wrote:
> Removed trailing spaces, brace/parenthesis issues and other coding style
> problems, along with commented out code.
>
> Signed-off-by: Akshay Joshi <me@akshayjoshi.com>
> ---
> drivers/staging/nvec/Kconfig | 2 +-
> drivers/staging/nvec/nvec-keytable.h | 58 ++++----
> drivers/staging/nvec/nvec.c | 84 ++++------
> drivers/staging/nvec/nvec.h | 8 +-
> drivers/staging/nvec/nvec_kbd.c | 22 ++--
> drivers/staging/nvec/nvec_power.c | 288
> ++++++++++++++++------------------ drivers/staging/nvec/nvec_ps2.c |
> 50 +++---
> 7 files changed, 241 insertions(+), 271 deletions(-)
thanks for looking. I posted an updated patch a while ago which is pending
review/merge (see: http://driverdev.linuxdriverproject.org/pipermail/devel/2011-
August/018959.html). It has most of the coding style fixes already included.
If you like, you may take a look at these patches and check again for coding
style or other problems. I appreciate your help.
Greg, I haven't yet heard of Joe (a reviewer, see:
http://driverdev.linuxdriverproject.org/pipermail/devel/2011-August/019217.html)
for some days now. So if you don't mind, can you please apply my changes (listed
above), so people can work with the current code base? I also have some patches
ontop of it and I like to get them into staging before the merge window opens
again.
Thanks
Marc
>
> diff --git a/drivers/staging/nvec/Kconfig b/drivers/staging/nvec/Kconfig
> index 987ad48..9a7e4e4 100644
> --- a/drivers/staging/nvec/Kconfig
> +++ b/drivers/staging/nvec/Kconfig
> @@ -9,7 +9,7 @@ config KEYBOARD_NVEC
> bool "Keyboard on nVidia compliant EC"
> depends on MFD_NVEC
> help
> - Say Y here to enable support for a keyboard connected to
> + Say Y here to enable support for a keyboard connected to
> a nVidia compliant embedded controller.
>
> config SERIO_NVEC_PS2
> diff --git a/drivers/staging/nvec/nvec-keytable.h
> b/drivers/staging/nvec/nvec-keytable.h index 6a1c4f7..acd3929 100644
> --- a/drivers/staging/nvec/nvec-keytable.h
> +++ b/drivers/staging/nvec/nvec-keytable.h
> @@ -111,42 +111,42 @@ static unsigned short code_tab_102us[] = {
> KEY_102ND, //VK_OEM_102 henry+ 0x2B (43) BACKSLASH have been
> used,change to use 0X56 (86) KEY_F11, //VK_F11
> KEY_F12, //VK_F12
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0, // 60
> 0,
> 0,
> - KEY_SEARCH, // add search key map
> - 0,
> 0,
> 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0, // 70
> + 0,
> + 0,
> + 0,
> + 0, // 60
> + 0,
> + 0,
> + KEY_SEARCH, // add search key map
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0, // 70
> 0,
> 0,
> KEY_KP5, //73 for JP keyboard '\' key, report 0x4c
> - 0,
> 0,
> 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> - 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> + 0,
> KEY_KP9, //7d for JP keyboard '|' key, report 0x49
> };
>
> @@ -246,7 +246,7 @@ static unsigned short extcode_tab_us102[] = {
> 0, //VK_RWIN
> KEY_ESC, //VK_APPS
> KEY_KPMINUS, //for power button workaround
> - 0,
> + 0,
> 0,
> 0,
> 0,
> @@ -263,4 +263,4 @@ static unsigned short extcode_tab_us102[] = {
> 0, //VK_LAUNCH_MEDIA_SELECT
> };
>
> -static unsigned short* code_tabs[] = {code_tab_102us, extcode_tab_us102 };
> +static unsigned short *code_tabs[] = {code_tab_102us, extcode_tab_us102 };
> diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
> index 72258e8..b822cda 100644
> --- a/drivers/staging/nvec/nvec.c
> +++ b/drivers/staging/nvec/nvec.c
> @@ -24,9 +24,9 @@
> #include <linux/platform_device.h>
> #include "nvec.h"
>
> -static unsigned char EC_DISABLE_EVENT_REPORTING[]
> = {'\x04','\x00','\x00'}; -static unsigned char
> EC_ENABLE_EVENT_REPORTING[] = {'\x04','\x00','\x01'}; -static unsigned
> char EC_GET_FIRMWARE_VERSION[] = {'\x07','\x15'}; +static unsigned char
> EC_DISABLE_EVENT_REPORTING[] = {'\x04', '\x00', '\x00'}; +static
> unsigned char EC_ENABLE_EVENT_REPORTING[] = {'\x04', '\x00', '\x01'};
> +static unsigned char EC_GET_FIRMWARE_VERSION[] = {'\x07', '\x15'};
>
> static struct nvec_chip *nvec_power_handle;
>
> @@ -43,7 +43,7 @@ static int nvec_status_notifier(struct notifier_block
> *nb, unsigned long event_t unsigned char *msg = (unsigned char *)data;
> int i;
>
> - if(event_type != NVEC_CNTL)
> + if (event_type != NVEC_CNTL)
> return NOTIFY_DONE;
>
> printk("unhandled msg type %ld, payload: ", event_type);
> @@ -75,25 +75,23 @@ static void nvec_request_master(struct work_struct
> *work) {
> struct nvec_chip *nvec = container_of(work, struct nvec_chip, tx_work);
>
> - if(!list_empty(&nvec->tx_data)) {
> + if (!list_empty(&nvec->tx_data))
> gpio_set_value(nvec->gpio, 0);
> - }
> }
>
> static int parse_msg(struct nvec_chip *nvec, struct nvec_msg *msg)
> {
> int i;
>
> - if((msg->data[0] & 1<<7) == 0 && msg->data[3]) {
> + if ((msg->data[0] & 1<<7) == 0 && msg->data[3]) {
> dev_err(nvec->dev, "ec responded %02x %02x %02x %02x\n", msg->data[0],
> msg->data[1], msg->data[2], msg->data[3]);
> return -EINVAL;
> }
>
> - if ((msg->data[0] >> 7 ) == 1 && (msg->data[0] & 0x0f) == 5)
> - {
> + if ((msg->data[0] >> 7) == 1 && (msg->data[0] & 0x0f) == 5) {
> dev_warn(nvec->dev, "ec system event ");
> - for (i=0; i < msg->data[1]; i++)
> + for (i = 0; i < msg->data[1]; i++)
> dev_warn(nvec->dev, "%02x ", msg->data[2+i]);
> dev_warn(nvec->dev, "\n");
> }
> @@ -125,20 +123,18 @@ static void nvec_dispatch(struct work_struct *work)
> struct nvec_chip *nvec = container_of(work, struct nvec_chip, rx_work);
> struct nvec_msg *msg;
>
> - while(!list_empty(&nvec->rx_data))
> - {
> + while (!list_empty(&nvec->rx_data)) {
> msg = list_first_entry(&nvec->rx_data, struct nvec_msg, node);
> list_del_init(&msg->node);
>
> - if(nvec->sync_write_pending == (msg->data[2] << 8) + msg->data[0])
> - {
> + if (nvec->sync_write_pending == (msg->data[2] << 8) + msg->data[0]) {
> dev_dbg(nvec->dev, "sync write completed!\n");
> nvec->sync_write_pending = 0;
> nvec->last_sync_msg = msg;
> complete(&nvec->sync_write);
> } else {
> parse_msg(nvec, msg);
> - if((!msg) || (!msg->data))
> + if ((!msg) || (!msg->data))
> dev_warn(nvec->dev, "attempt access zero pointer");
> else {
> kfree(msg->data);
> @@ -159,20 +155,17 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
>
> status = readl(i2c_regs + I2C_SL_STATUS);
>
> - if(!(status & I2C_SL_IRQ))
> - {
> + if (!(status & I2C_SL_IRQ)) {
> dev_warn(nvec->dev, "nvec Spurious IRQ\n");
> - //Yup, handled. ahum.
> + /* Yup, handled. ahum. */
> goto handled;
> }
> - if(status & END_TRANS && !(status & RCVD))
> - {
> - //Reenable IRQ only when even has been sent
> + if (status & END_TRANS && !(status & RCVD)) {
> + /* Reenable IRQ only when even has been sent */
> //printk("Write sequence ended !\n");
> - //parse_msg(nvec);
> + //parse_msg(nvec);
> nvec->state = NVEC_WAIT;
> - if(nvec->rx->size > 1)
> - {
> + if (nvec->rx->size > 1) {
> list_add_tail(&nvec->rx->node, &nvec->rx_data);
> schedule_work(&nvec->rx_work);
> } else {
> @@ -180,15 +173,13 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
> kfree(nvec->rx);
> }
> return IRQ_HANDLED;
> - } else if(status & RNW)
> - {
> + } else if (status & RNW) {
> // Work around for AP20 New Slave Hw Bug. Give 1us extra.
> // nvec/smbus/nvec_i2c_transport.c in NV`s crap for reference
> - if(status & RCVD)
> + if (status & RCVD)
> udelay(3);
>
> - if(status & RCVD)
> - {
> + if (status & RCVD) {
> nvec->state = NVEC_WRITE;
> //Master wants something from us. New communication
> // dev_dbg(nvec->dev, "New read comm!\n");
> @@ -197,15 +188,14 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
> // dev_dbg(nvec->dev, "Read comm cont !\n");
> }
> //if(msg_pos<msg_size) {
> - if(list_empty(&nvec->tx_data))
> - {
> + if (list_empty(&nvec->tx_data)) {
> dev_err(nvec->dev, "nvec empty tx - sending no-op\n");
> to_send = 0x8a;
> nvec_write_async(nvec, "\x07\x02", 2);
> // to_send = 0x01;
> } else {
> msg = list_first_entry(&nvec->tx_data, struct nvec_msg, node);
> - if(msg->pos < msg->size) {
> + if (msg->pos < msg->size) {
> to_send = msg->data[msg->pos];
> msg->pos++;
> } else {
> @@ -213,8 +203,7 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
> to_send = 0x01;
> }
>
> - if(msg->pos >= msg->size)
> - {
> + if (msg->pos >= msg->size) {
> list_del_init(&msg->node);
> kfree(msg->data);
> kfree(msg);
> @@ -232,13 +221,12 @@ static irqreturn_t i2c_interrupt(int irq, void *dev)
> } else {
> received = readl(i2c_regs + I2C_SL_RCVD);
> //Workaround?
> - if(status & RCVD) {
> + if (status & RCVD) {
> writel(0, i2c_regs + I2C_SL_RCVD);
> goto handled;
> }
>
> - if (nvec->state == NVEC_WAIT)
> - {
> + if (nvec->state == NVEC_WAIT) {
> nvec->state = NVEC_READ;
> msg = kzalloc(sizeof(struct nvec_msg), GFP_NOWAIT);
> msg->data = kzalloc(32, GFP_NOWAIT);
> @@ -307,7 +295,7 @@ static int __devinit tegra_nvec_probe(struct
> platform_device *pdev) unsigned char *i2c_regs;
>
> nvec = kzalloc(sizeof(struct nvec_chip), GFP_KERNEL);
> - if(nvec == NULL) {
> + if (nvec == NULL) {
> dev_err(&pdev->dev, "failed to reserve memory\n");
> return -ENOMEM;
> }
> @@ -324,7 +312,7 @@ static int __devinit tegra_nvec_probe(struct
> platform_device *pdev) clk_enable(i2c_clk);
> */
> i2c_regs = ioremap(pdata->base, pdata->size);
> - if(!i2c_regs) {
> + if (!i2c_regs) {
> dev_err(nvec->dev, "failed to ioremap registers\n");
> goto failed;
> }
> @@ -332,7 +320,7 @@ static int __devinit tegra_nvec_probe(struct
> platform_device *pdev) nvec->i2c_regs = i2c_regs;
>
> i2c_clk = clk_get_sys(pdata->clock, NULL);
> - if(IS_ERR_OR_NULL(i2c_clk)) {
> + if (IS_ERR_OR_NULL(i2c_clk)) {
> dev_err(nvec->dev, "failed to get clock tegra-i2c.2\n");
> goto failed;
> }
> @@ -340,7 +328,7 @@ static int __devinit tegra_nvec_probe(struct
> platform_device *pdev) tegra_init_i2c_slave(pdata, i2c_regs, i2c_clk);
>
> err = request_irq(nvec->irq, i2c_interrupt, IRQF_DISABLED, "nvec", nvec);
> - if(err) {
> + if (err) {
> dev_err(nvec->dev, "couldn't request irq");
> goto failed;
> }
> @@ -350,7 +338,7 @@ static int __devinit tegra_nvec_probe(struct
> platform_device *pdev)
>
> /* Set the gpio to low when we've got something to say */
> err = gpio_request(nvec->gpio, "nvec gpio");
> - if(err < 0)
> + if (err < 0)
> dev_err(nvec->dev, "couldn't request gpio\n");
>
> tegra_gpio_enable(nvec->gpio);
> @@ -375,10 +363,9 @@ static int __devinit tegra_nvec_probe(struct
> platform_device *pdev) nvec_ps2(nvec);
> #endif
>
> - /* setup subdevs */
> - for (i = 0; i < pdata->num_subdevs; i++) {
> + /* setup subdevs */
> + for (i = 0; i < pdata->num_subdevs; i++)
> ret = nvec_add_subdev(nvec, &pdata->subdevs[i]);
> - }
>
> nvec->nvec_status_notifier.notifier_call = nvec_status_notifier;
> nvec_register_notifier(nvec, &nvec->nvec_status_notifier, 0);
> @@ -431,8 +418,8 @@ static int tegra_nvec_suspend(struct platform_device
> *pdev, pm_message_t state) return 0;
> }
>
> -static int tegra_nvec_resume(struct platform_device *pdev) {
> -
> +static int tegra_nvec_resume(struct platform_device *pdev)
> +{
> struct nvec_chip *nvec = platform_get_drvdata(pdev);
>
> dev_dbg(nvec->dev, "resuming\n");
> @@ -446,8 +433,7 @@ static int tegra_nvec_resume(struct platform_device
> *pdev) { #define tegra_nvec_resume NULL
> #endif
>
> -static struct platform_driver nvec_device_driver =
> -{
> +static struct platform_driver nvec_device_driver = {
> .probe = tegra_nvec_probe,
> .remove = __devexit_p(tegra_nvec_remove),
> .suspend = tegra_nvec_suspend,
> diff --git a/drivers/staging/nvec/nvec.h b/drivers/staging/nvec/nvec.h
> index a2d82dc..d1f083b 100644
> --- a/drivers/staging/nvec/nvec.h
> +++ b/drivers/staging/nvec/nvec.h
> @@ -16,7 +16,7 @@ typedef enum {
> } how_care;
>
> typedef enum {
> - NVEC_SYS=1,
> + NVEC_SYS = 1,
> NVEC_BAT,
> NVEC_KBD = 5,
> NVEC_PS2,
> @@ -26,9 +26,9 @@ typedef enum {
> } nvec_event;
>
> typedef enum {
> - NVEC_WAIT,
> - NVEC_READ,
> - NVEC_WRITE
> + NVEC_WAIT,
> + NVEC_READ,
> + NVEC_WRITE
> } nvec_state;
>
> struct nvec_msg {
> diff --git a/drivers/staging/nvec/nvec_kbd.c
> b/drivers/staging/nvec/nvec_kbd.c index 9a98507..04bd285 100644
> --- a/drivers/staging/nvec/nvec_kbd.c
> +++ b/drivers/staging/nvec/nvec_kbd.c
> @@ -4,7 +4,7 @@
> #include "nvec-keytable.h"
> #include "nvec.h"
>
> -#define ACK_KBD_EVENT {'\x05','\xed','\x01'}
> +#define ACK_KBD_EVENT {'\x05', '\xed', '\x01'}
>
> static unsigned char keycodes[ARRAY_SIZE(code_tab_102us)
> + ARRAY_SIZE(extcode_tab_us102)];
> @@ -27,10 +27,10 @@ static int nvec_keys_notifier(struct notifier_block
> *nb, nvec_size _size = (msg[0] & (3 << 5)) >> 5;
>
> /* power on/off button */
> - if(_size == NVEC_VAR_SIZE)
> + if (_size == NVEC_VAR_SIZE)
> return NOTIFY_STOP;
>
> - if(_size == NVEC_3BYTES)
> + if (_size == NVEC_3BYTES)
> msg++;
>
> code = msg[1] & 0x7f;
> @@ -51,13 +51,13 @@ static int nvec_kbd_event(struct input_dev *dev,
> unsigned int type, unsigned char buf[] = ACK_KBD_EVENT;
> struct nvec_chip *nvec = keys_dev.nvec;
>
> - if(type==EV_REP)
> + if (type == EV_REP)
> return 0;
>
> - if(type!=EV_LED)
> + if (type != EV_LED)
> return -1;
>
> - if(code!=LED_CAPSL)
> + if (code != LED_CAPSL)
> return -1;
>
> buf[2] = !!value;
> @@ -73,11 +73,11 @@ int __init nvec_kbd_init(struct nvec_chip *nvec)
>
> j = 0;
>
> - for(i = 0; i < ARRAY_SIZE(code_tab_102us); ++i)
> + for (i = 0; i < ARRAY_SIZE(code_tab_102us); ++i)
> keycodes[j++] = code_tab_102us[i];
>
> - for(i = 0; i < ARRAY_SIZE(extcode_tab_us102); ++i)
> - keycodes[j++]=extcode_tab_us102[i];
> + for (i = 0; i < ARRAY_SIZE(extcode_tab_us102); ++i)
> + keycodes[j++] = extcode_tab_us102[i];
>
> idev = input_allocate_device();
> idev->name = "Tegra nvec keyboard";
> @@ -89,12 +89,12 @@ int __init nvec_kbd_init(struct nvec_chip *nvec)
> idev->keycodesize = sizeof(unsigned char);
> idev->keycodemax = ARRAY_SIZE(keycodes);
>
> - for( i = 0; i < ARRAY_SIZE(keycodes); ++i)
> + for (i = 0; i < ARRAY_SIZE(keycodes); ++i)
> set_bit(keycodes[i], idev->keybit);
>
> clear_bit(0, idev->keybit);
> err = input_register_device(idev);
> - if(err)
> + if (err)
> goto fail;
>
> keys_dev.input = idev;
> diff --git a/drivers/staging/nvec/nvec_power.c
> b/drivers/staging/nvec/nvec_power.c index df164ad..d82dd97 100644
> --- a/drivers/staging/nvec/nvec_power.c
> +++ b/drivers/staging/nvec/nvec_power.c
> @@ -7,8 +7,7 @@
> #include <linux/delay.h>
> #include "nvec.h"
>
> -struct nvec_power
> -{
> +struct nvec_power {
> struct notifier_block notifier;
> struct delayed_work poller;
> struct nvec_chip *nvec;
> @@ -77,10 +76,8 @@ static int nvec_power_notifier(struct notifier_block
> *nb, if (event_type != NVEC_SYS)
> return NOTIFY_DONE;
>
> - if(res->sub_type == 0)
> - {
> - if (power->on != res->plu)
> - {
> + if (res->sub_type == 0) {
> + if (power->on != res->plu) {
> power->on = res->plu;
> power_supply_changed(&nvec_psy);
> }
> @@ -89,8 +86,7 @@ static int nvec_power_notifier(struct notifier_block *nb,
> return NOTIFY_OK;
> }
>
> -static const int bat_init[] =
> -{
> +static const int bat_init[] = {
> LAST_FULL_CHARGE_CAPACITY, DESIGN_CAPACITY, CRITICAL_CAPACITY,
> MANUFACTURER, MODEL, TYPE,
> };
> @@ -100,8 +96,7 @@ static void get_bat_mfg_data(struct nvec_power *power)
> int i;
> char buf[] = { '\x02', '\x00' };
>
> - for (i = 0; i < ARRAY_SIZE(bat_init); i++)
> - {
> + for (i = 0; i < ARRAY_SIZE(bat_init); i++) {
> buf[1] = bat_init[i];
> nvec_write_async(power->nvec, buf, 2);
> }
> @@ -117,91 +112,87 @@ static int nvec_power_bat_notifier(struct
> notifier_block *nb, if (event_type != NVEC_BAT)
> return NOTIFY_DONE;
>
> - switch(res->sub_type)
> - {
> - case SLOT_STATUS:
> - if (res->plc[0] & 1)
> - {
> - if (power->bat_present == 0)
> - {
> - status_changed = 1;
> - get_bat_mfg_data(power);
> - }
> -
> - power->bat_present = 1;
> -
> - switch ((res->plc[0] >> 1) & 3)
> - {
> - case 0:
> - power->bat_status = POWER_SUPPLY_STATUS_NOT_CHARGING;
> - break;
> - case 1:
> - power->bat_status = POWER_SUPPLY_STATUS_CHARGING;
> - break;
> - case 2:
> - power->bat_status = POWER_SUPPLY_STATUS_DISCHARGING;
> - break;
> - default:
> - power->bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
> - }
> - } else {
> - if (power->bat_present == 1)
> - status_changed = 1;
> -
> - power->bat_present = 0;
> + switch (res->sub_type) {
> + case SLOT_STATUS:
> + if (res->plc[0] & 1) {
> + if (power->bat_present == 0) {
> + status_changed = 1;
> + get_bat_mfg_data(power);
> + }
> +
> + power->bat_present = 1;
> +
> + switch ((res->plc[0] >> 1) & 3) {
> + case 0:
> + power->bat_status = POWER_SUPPLY_STATUS_NOT_CHARGING;
> + break;
> + case 1:
> + power->bat_status = POWER_SUPPLY_STATUS_CHARGING;
> + break;
> + case 2:
> + power->bat_status = POWER_SUPPLY_STATUS_DISCHARGING;
> + break;
> + default:
> power->bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
> }
> - power->bat_cap = res->plc[1];
> - if (status_changed)
> - power_supply_changed(&nvec_bat_psy);
> - break;
> - case VOLTAGE:
> - power->bat_voltage_now = res->plu * 1000;
> - break;
> - case TIME_REMAINING:
> - power->time_remain = res->plu * 3600;
> - break;
> - case CURRENT:
> - power->bat_current_now = res->pls * 1000;
> - break;
> - case AVERAGE_CURRENT:
> - power->bat_current_avg = res->pls * 1000;
> - break;
> - case CAPACITY_REMAINING:
> - power->capacity_remain = res->plu * 1000;
> - break;
> - case LAST_FULL_CHARGE_CAPACITY:
> - power->charge_last_full = res->plu * 1000;
> - break;
> - case DESIGN_CAPACITY:
> - power->charge_full_design = res->plu * 1000;
> - break;
> - case CRITICAL_CAPACITY:
> - power->critical_capacity = res->plu * 1000;
> - break;
> - case TEMPERATURE:
> - power->bat_temperature = res->plu - 2732;
> - break;
> - case MANUFACTURER:
> - memcpy(power->bat_manu, &res->plc, res->length-2);
> - power->bat_model[res->length-2] = '\0';
> - break;
> - case MODEL:
> - memcpy(power->bat_model, &res->plc, res->length-2);
> - power->bat_model[res->length-2] = '\0';
> - break;
> - case TYPE:
> - memcpy(power->bat_type, &res->plc, res->length-2);
> - power->bat_type[res->length-2] = '\0';
> - /* this differs a little from the spec
> - fill in more if you find some */
> - if (!strncmp(power->bat_type, "Li", 30))
> - power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_LION;
> - else
> - power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_UNKNOWN;
> - break;
> - default:
> - return NOTIFY_STOP;
> + } else {
> + if (power->bat_present == 1)
> + status_changed = 1;
> +
> + power->bat_present = 0;
> + power->bat_status = POWER_SUPPLY_STATUS_UNKNOWN;
> + }
> + power->bat_cap = res->plc[1];
> + if (status_changed)
> + power_supply_changed(&nvec_bat_psy);
> + break;
> + case VOLTAGE:
> + power->bat_voltage_now = res->plu * 1000;
> + break;
> + case TIME_REMAINING:
> + power->time_remain = res->plu * 3600;
> + break;
> + case CURRENT:
> + power->bat_current_now = res->pls * 1000;
> + break;
> + case AVERAGE_CURRENT:
> + power->bat_current_avg = res->pls * 1000;
> + break;
> + case CAPACITY_REMAINING:
> + power->capacity_remain = res->plu * 1000;
> + break;
> + case LAST_FULL_CHARGE_CAPACITY:
> + power->charge_last_full = res->plu * 1000;
> + break;
> + case DESIGN_CAPACITY:
> + power->charge_full_design = res->plu * 1000;
> + break;
> + case CRITICAL_CAPACITY:
> + power->critical_capacity = res->plu * 1000;
> + break;
> + case TEMPERATURE:
> + power->bat_temperature = res->plu - 2732;
> + break;
> + case MANUFACTURER:
> + memcpy(power->bat_manu, &res->plc, res->length-2);
> + power->bat_model[res->length-2] = '\0';
> + break;
> + case MODEL:
> + memcpy(power->bat_model, &res->plc, res->length-2);
> + power->bat_model[res->length-2] = '\0';
> + break;
> + case TYPE:
> + memcpy(power->bat_type, &res->plc, res->length-2);
> + power->bat_type[res->length-2] = '\0';
> + /* this differs a little from the spec
> + fill in more if you find some */
> + if (!strncmp(power->bat_type, "Li", 30))
> + power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_LION;
> + else
> + power->bat_type_enum = POWER_SUPPLY_TECHNOLOGY_UNKNOWN;
> + break;
> + default:
> + return NOTIFY_STOP;
> }
>
> return NOTIFY_STOP;
> @@ -228,56 +219,55 @@ static int nvec_battery_get_property(struct
> power_supply *psy, {
> struct nvec_power *power = dev_get_drvdata(psy->dev->parent);
>
> - switch(psp)
> - {
> - case POWER_SUPPLY_PROP_STATUS:
> - val->intval = power->bat_status;
> - break;
> - case POWER_SUPPLY_PROP_CAPACITY:
> - val->intval = power->bat_cap;
> - break;
> - case POWER_SUPPLY_PROP_PRESENT:
> - val->intval = power->bat_present;
> - break;
> - case POWER_SUPPLY_PROP_VOLTAGE_NOW:
> - val->intval = power->bat_voltage_now;
> - break;
> - case POWER_SUPPLY_PROP_CURRENT_NOW:
> - val->intval = power->bat_current_now;
> - break;
> - case POWER_SUPPLY_PROP_CURRENT_AVG:
> - val->intval = power->bat_current_avg;
> - break;
> - case POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW:
> - val->intval = power->time_remain;
> - break;
> - case POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN:
> - val->intval = power->charge_full_design;
> - break;
> - case POWER_SUPPLY_PROP_CHARGE_FULL:
> - val->intval = power->charge_last_full;
> - break;
> - case POWER_SUPPLY_PROP_CHARGE_EMPTY:
> - val->intval = power->critical_capacity;
> - break;
> - case POWER_SUPPLY_PROP_CHARGE_NOW:
> - val->intval = power->capacity_remain;
> - break;
> - case POWER_SUPPLY_PROP_TEMP:
> - val->intval = power->bat_temperature;
> - break;
> - case POWER_SUPPLY_PROP_MANUFACTURER:
> - val->strval = power->bat_manu;
> - break;
> - case POWER_SUPPLY_PROP_MODEL_NAME:
> - val->strval = power->bat_model;
> - break;
> - case POWER_SUPPLY_PROP_TECHNOLOGY:
> - val->intval = power->bat_type_enum;
> - break;
> - default:
> - return -EINVAL;
> - }
> + switch (psp) {
> + case POWER_SUPPLY_PROP_STATUS:
> + val->intval = power->bat_status;
> + break;
> + case POWER_SUPPLY_PROP_CAPACITY:
> + val->intval = power->bat_cap;
> + break;
> + case POWER_SUPPLY_PROP_PRESENT:
> + val->intval = power->bat_present;
> + break;
> + case POWER_SUPPLY_PROP_VOLTAGE_NOW:
> + val->intval = power->bat_voltage_now;
> + break;
> + case POWER_SUPPLY_PROP_CURRENT_NOW:
> + val->intval = power->bat_current_now;
> + break;
> + case POWER_SUPPLY_PROP_CURRENT_AVG:
> + val->intval = power->bat_current_avg;
> + break;
> + case POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW:
> + val->intval = power->time_remain;
> + break;
> + case POWER_SUPPLY_PROP_CHARGE_FULL_DESIGN:
> + val->intval = power->charge_full_design;
> + break;
> + case POWER_SUPPLY_PROP_CHARGE_FULL:
> + val->intval = power->charge_last_full;
> + break;
> + case POWER_SUPPLY_PROP_CHARGE_EMPTY:
> + val->intval = power->critical_capacity;
> + break;
> + case POWER_SUPPLY_PROP_CHARGE_NOW:
> + val->intval = power->capacity_remain;
> + break;
> + case POWER_SUPPLY_PROP_TEMP:
> + val->intval = power->bat_temperature;
> + break;
> + case POWER_SUPPLY_PROP_MANUFACTURER:
> + val->strval = power->bat_manu;
> + break;
> + case POWER_SUPPLY_PROP_MODEL_NAME:
> + val->strval = power->bat_model;
> + break;
> + case POWER_SUPPLY_PROP_TECHNOLOGY:
> + val->intval = power->bat_type_enum;
> + break;
> + default:
> + return -EINVAL;
> + }
> return 0;
> }
>
> @@ -328,8 +318,7 @@ static struct power_supply nvec_psy = {
> };
>
> static int counter = 0;
> -static int const bat_iter[] =
> -{
> +static int const bat_iter[] = {
> SLOT_STATUS, VOLTAGE, CURRENT, CAPACITY_REMAINING,
> #ifdef EC_FULL_DIAG
> AVERAGE_CURRENT, TEMPERATURE, TIME_REMAINING,
> @@ -352,11 +341,9 @@ static void nvec_power_poll(struct work_struct *work)
> /* select a battery request function via round robin
> doing it all at once seems to overload the power supply */
> buf[0] = '\x02'; /* battery */
> - buf[1] = bat_iter[counter++];
> + buf[1] = bat_iter[counter++];
> nvec_write_async(power->nvec, buf, 2);
>
> -// printk("%02x %02x\n", buf[0], buf[1]);
> -
> schedule_delayed_work(to_delayed_work(work), msecs_to_jiffies(5000));
> };
>
> @@ -381,7 +368,7 @@ static int __devinit nvec_power_probe(struct
> platform_device *pdev) case BAT:
> psy = &nvec_bat_psy;
>
> - power->notifier.notifier_call = nvec_power_bat_notifier;
> + power->notifier.notifier_call = nvec_power_bat_notifier;
> break;
> default:
> kfree(power);
> @@ -398,14 +385,13 @@ static int __devinit nvec_power_probe(struct
> platform_device *pdev)
>
> static struct platform_driver nvec_power_driver = {
> .probe = nvec_power_probe,
> -// .remove = __devexit_p(nvec_power_remove),
> .driver = {
> .name = "nvec-power",
> .owner = THIS_MODULE,
> }
> };
>
> -static int __init nvec_power_init(void)
> +static int __init nvec_power_init(void)
> {
> return platform_driver_register(&nvec_power_driver);
> }
> diff --git a/drivers/staging/nvec/nvec_ps2.c
> b/drivers/staging/nvec/nvec_ps2.c index 6bb9430..d8e9fd0 100644
> --- a/drivers/staging/nvec/nvec_ps2.c
> +++ b/drivers/staging/nvec/nvec_ps2.c
> @@ -3,12 +3,11 @@
> #include <linux/delay.h>
> #include "nvec.h"
>
> -#define START_STREAMING {'\x06','\x03','\x01'}
> -#define STOP_STREAMING {'\x06','\x04'}
> -#define SEND_COMMAND {'\x06','\x01','\xf4','\x01'}
> +#define START_STREAMING {'\x06', '\x03', '\x01'}
> +#define STOP_STREAMING {'\x06', '\x04'}
> +#define SEND_COMMAND {'\x06', '\x01', '\xf4', '\x01'}
>
> -struct nvec_ps2
> -{
> +struct nvec_ps2 {
> struct serio *ser_dev;
> struct notifier_block notifier;
> struct nvec_chip *nvec;
> @@ -54,23 +53,22 @@ static int nvec_ps2_notifier(struct notifier_block *nb,
> unsigned char *msg = (unsigned char *)data;
>
> switch (event_type) {
> - case NVEC_PS2_EVT:
> - serio_interrupt(ps2_dev.ser_dev, msg[2], 0);
> - return NOTIFY_STOP;
> -
> - case NVEC_PS2:
> - if (msg[2] == 1)
> - for(i = 0; i < (msg[1] - 2); i++)
> - serio_interrupt(ps2_dev.ser_dev, msg[i+4], 0);
> - else if (msg[1] != 2) /* !ack */
> - {
> - printk("nvec_ps2: unhandled mouse event ");
> - for(i = 0; i <= (msg[1]+1); i++)
> - printk("%02x ", msg[i]);
> - printk(".\n");
> - }
> -
> - return NOTIFY_STOP;
> + case NVEC_PS2_EVT:
> + serio_interrupt(ps2_dev.ser_dev, msg[2], 0);
> + return NOTIFY_STOP;
> +
> + case NVEC_PS2:
> + if (msg[2] == 1)
> + for (i = 0; i < (msg[1] - 2); i++)
> + serio_interrupt(ps2_dev.ser_dev, msg[i+4], 0);
> + else if (msg[1] != 2) { /* !ack */
> + printk("nvec_ps2: unhandled mouse event ");
> + for (i = 0; i <= (msg[1]+1); i++)
> + printk("%02x ", msg[i]);
> + printk(".\n");
> + }
> +
> + return NOTIFY_STOP;
> }
>
> return NOTIFY_DONE;
> @@ -81,10 +79,10 @@ int __init nvec_ps2(struct nvec_chip *nvec)
> {
> struct serio *ser_dev = kzalloc(sizeof(struct serio), GFP_KERNEL);
>
> - ser_dev->id.type=SERIO_8042;
> - ser_dev->write=ps2_sendcommand;
> - ser_dev->open=ps2_startstreaming;
> - ser_dev->close=ps2_stopstreaming;
> + ser_dev->id.type = SERIO_8042;
> + ser_dev->write = ps2_sendcommand;
> + ser_dev->open = ps2_startstreaming;
> + ser_dev->close = ps2_stopstreaming;
>
> strlcpy(ser_dev->name, "NVEC PS2", sizeof(ser_dev->name));
> strlcpy(ser_dev->phys, "NVEC I2C slave", sizeof(ser_dev->phys));
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Staging: nvec: Fixed various style issues.
2011-08-18 6:58 ` Marc Dietrich
@ 2011-08-18 7:30 ` Joe Perches
2011-08-18 8:05 ` Marc Dietich
0 siblings, 1 reply; 7+ messages in thread
From: Joe Perches @ 2011-08-18 7:30 UTC (permalink / raw)
To: Marc Dietrich
Cc: Akshay Joshi, Greg Kroah-Hartman, Jiri Kosina, Vitaliy Ivanov,
devel, linux-kernel
On Thu, 2011-08-18 at 08:58 +0200, Marc Dietrich wrote:
> Greg, I haven't yet heard of Joe (a reviewer, see:
> http://driverdev.linuxdriverproject.org/pipermail/devel/2011-August/019217.html)
> for some days now.
Hey Marc.
I thought I was pretty clear in my reviewvand
didn't need to reply further.
I think mixing index # and field comment in the
same style isn't very readable.
It's your code, do what you think best.
cheers, Joe
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Staging: nvec: Fixed various style issues.
2011-08-18 7:30 ` Joe Perches
@ 2011-08-18 8:05 ` Marc Dietich
0 siblings, 0 replies; 7+ messages in thread
From: Marc Dietich @ 2011-08-18 8:05 UTC (permalink / raw)
To: Joe Perches
Cc: Akshay Joshi, Greg Kroah-Hartman, Jiri Kosina, Vitaliy Ivanov,
devel, linux-kernel
Hey Joe,
> > On Thu, 2011-08-18 at 08:58 +0200, Marc Dietrich wrote:
> > Greg, I haven't yet heard of Joe (a reviewer, see:
> > http://driverdev.linuxdriverproject.org/pipermail/devel/2011-August/01921
> > 7.html) for some days now.
>
> I thought I was pretty clear in my reviewvand
> didn't need to reply further.
>
> I think mixing index # and field comment in the
> same style isn't very readable.
>
> It's your code, do what you think best.
this file certainly needs another cleanup (or maybe be moved into nvec_kbd.c). I
just want to make "small" steps.
Thanks
Marc
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Staging: nvec: Fixed various style issues.
2011-08-17 21:23 ` Akshay Joshi
@ 2011-08-18 18:28 ` Greg KH
0 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2011-08-18 18:28 UTC (permalink / raw)
To: Akshay Joshi
Cc: Marc Dietrich, Jiri Kosina, Vitaliy Ivanov, devel, linux-kernel
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Wed, Aug 17, 2011 at 05:23:16PM -0400, Akshay Joshi wrote:
> Can I split by file or should I redo it by splitting the changes into
> different commits?
Take a look at my original request here, which answers this exact
question...
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-08-18 22:50 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-17 20:36 [PATCH] Staging: nvec: Fixed various style issues Akshay Joshi
2011-08-17 21:17 ` Greg KH
2011-08-17 21:23 ` Akshay Joshi
2011-08-18 18:28 ` Greg KH
2011-08-18 6:58 ` Marc Dietrich
2011-08-18 7:30 ` Joe Perches
2011-08-18 8:05 ` Marc Dietich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox