On Mon, Jan 22, 2018 at 2:42 PM, Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > >> + status = acpi_evaluate_object(handle, "VGBS", NULL, &vgbs_output); >> + /* VGBS being present and returning something means >> + * we have a tablet mode switch >> + */ >> + if (ACPI_SUCCESS(status)) { >> + union acpi_object *obj = vgbs_output.pointer; >> + >> + input_set_capability(priv->input_dev, EV_SW, SW_TABLET_MODE); >> + >> + if (obj && obj->type == ACPI_TYPE_INTEGER) { >> + if (obj->integer.value & TABLET_MODE_FLAG) >> + input_report_switch(priv->input_dev, > >> + SW_TABLET_MODE, >> + 0); > > One line? checkpatch.pl was complaining about lines too long. -- Marco Martin