Hi!
> +static int atlas_setup_input(void)
> +{
> + int err;
> +
> + input_dev = input_allocate_device();
> + if (!input_dev) {
> + printk(KERN_ERR "atlas: insufficient mem to allocate input device\n");
> + return -ENOMEM;
> + }
> +
> + input_dev->name = "Atlas ACPI button driver";
> + input_dev->phys = "acpi/input0";
> + input_dev->id.bustype = BUS_HOST;
> + input_dev->evbit[LONG(EV_KEY)] = BIT(EV_KEY);
> + set_bit(KEY_F1 ,input_dev->keybit);
", " please... and are you sure you want it as a F1..F9 keys?
> +/* button handling code */
> +static acpi_status acpi_atlas_button_setup(acpi_handle region_handle,
> + u32 function, void *handler_context, void **return_context)
> +{
> + *return_context =
> + (function != ACPI_REGION_DEACTIVATE) ? handler_context : NULL;
Too many spaces after ? I'd say.
> +static struct acpi_driver atlas_acpi_driver = {
> + .name = ACPI_ATLAS_NAME,
> + .class = ACPI_ATLAS_CLASS,
> + .ids = ACPI_ATLAS_BUTTON_HID,
> + .ops = {
> + .add = atlas_acpi_button_add,
> + .remove = atlas_acpi_button_remove,
> + },
> +};
Watch that whitespace...
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
