On Mon, Jun 15, 2015 at 12:29:14PM -0300, Ramiro Morales wrote: > Implement change in logging during screen brightness capabilities > detection as suggested in > https://www.mail-archive.com/ibm-acpi-devel%40lists.sourceforge.net/msg03484.html > Summarize the rationale here, the change log needs to stand alone. You can reference a link, but the justification cannot depend on it. > Signed-off-by: Ramiro Morales <cramm0@xxxxxxxxx> > --- > drivers/platform/x86/thinkpad_acpi.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c > index 28f3281..b48efc9 100644 > --- a/drivers/platform/x86/thinkpad_acpi.c > +++ b/drivers/platform/x86/thinkpad_acpi.c > @@ -6450,19 +6450,16 @@ static void __init tpacpi_detect_brightness_capabilities(void) > switch (b) { > case 16: > bright_maxlvl = 15; > - pr_info("detected a 16-level brightness capable ThinkPad\n"); > break; > case 8: > case 0: > bright_maxlvl = 7; > - pr_info("detected a 8-level brightness capable ThinkPad\n"); > break; > default: > - pr_err("Unsupported brightness interface, " > - "please contact %s\n", TPACPI_MAIL); > tp_features.bright_unkfw = 1; > bright_maxlvl = b - 1; > } > + pr_debug("firmware reports %d brightness levels\n", bright_maxlvl); > } > > static int __init brightness_init(struct ibm_init_struct *iibm) > @@ -6482,6 +6479,9 @@ static int __init brightness_init(struct ibm_init_struct *iibm) > > /* if it is unknown, we don't handle it: it wouldn't be safe */ > if (tp_features.bright_unkfw) > + dbg_printk(TPACPI_DBG_INIT | TPACPI_DBG_BRGHT, > + "Unsupported brightness mode, use acpi video " > + "or gpu drivers"); Please run checkpatch on your patch (don't break quoted strings - it makes it hard to find them with grep). > return 1; > > if (!brightness_enable) { > -- > 1.9.1 > > -- Darren Hart Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html