Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp878178ybi; Fri, 24 May 2019 13:00:46 -0700 (PDT) X-Google-Smtp-Source: APXvYqzlhw+URz/9iQa3R8mKIcOR8B1rZb8D5yB8MFCZmVZn8Zv3tFt74TaO33I0wpH6Fbon4hf+ X-Received: by 2002:a65:62c6:: with SMTP id m6mr13414307pgv.306.1558728046567; Fri, 24 May 2019 13:00:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558728046; cv=none; d=google.com; s=arc-20160816; b=0OGgfEjH6mNXvWGtpevu2GFOoZ4pXWdovKOgCbSeYgwUW77wEoHuLhVSVUfo4i7ac3 yjBN770P1wHFJZcw3kvc2pWmj5uwrdrj8+s2MnQgyIZLKmrZJGuahFALJ2WJEqrxkXyi jDfE5r0mWNvkcbZs7W1h3vJ8DEtcNzzKMhaUErR290HNT9+ou1sILB0NOy0QrBaKLYK7 JYJwNRT5RBi5yoBIOFixg2HstwQxtpXwZsZolUXoYQ0Kh4FGgGGnq5fAgdEA4qHAhLnC p95FRL4BvWOp7kr4bBaqOE7SPmCU934IVN57QkwsChpqKF6bn2ieILcrXhnDG4fvKi+/ oocw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=RfJ7Wmu+isnYJEI0NneC1wcTrVYA9eunxNNo2wKm6wc=; b=RrhCK30QO/rGTLGX+aBAoOY1UOcVENrSYl5WK+i23+tRgLAW+21iQGhu2h1XIIfYUp Y06RAljThDOeHhGFLqcrI+9C+LlwB5ifBrk4VChCHaOz9NWUZkmDtqT2pjxOQt3b8Fbc a6BTlb+SQT5ibgDf76UOBBJadzsbqII9xWy2IRPj8hEjfiLoMUpQOfuZypSkRULvEKoK u3X30LhLYWp+2zB3/WNIf8RnOJAOo6QWAtqnBpYMEA5qTrMLwIqlTnp8vh1LWjGycTiE NPH1eHQOOK8j4UVFnDnBvRcRIYpzxsT9HRNgBf2RNt6qWZ+CmxirDXZu0EgGbNPPdub2 gMKQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@endlessm-com.20150623.gappssmtp.com header.s=20150623 header.b=WkfMoLBX; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id bf1si5261187plb.49.2019.05.24.13.00.30; Fri, 24 May 2019 13:00:46 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@endlessm-com.20150623.gappssmtp.com header.s=20150623 header.b=WkfMoLBX; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2391739AbfEXT6C (ORCPT + 99 others); Fri, 24 May 2019 15:58:02 -0400 Received: from mail-qk1-f195.google.com ([209.85.222.195]:37956 "EHLO mail-qk1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2391612AbfEXT6C (ORCPT ); Fri, 24 May 2019 15:58:02 -0400 Received: by mail-qk1-f195.google.com with SMTP id p26so6721341qkj.5 for ; Fri, 24 May 2019 12:58:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=endlessm-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=RfJ7Wmu+isnYJEI0NneC1wcTrVYA9eunxNNo2wKm6wc=; b=WkfMoLBXlXaD8fN+Z7zVQLMt0cNkLBAr6nPCu0BzVrAL6KLcuTRZxhzhj3Rw0r9aPe 6scipkf0Hh2C0ICehiVOm5O1NExP1/GNZCJjUMAhD3eAsBxl0MoohZ+wTpPxYPng6VkU 9ZgOLZbTGSv4gT320oeYwbGJP6w1WPYs8PE4Kqv1dnlB/yf3UtAVXSHn7OAYsVx7cmxp rT6io7BGKgFgGtZIR+pKfJuQ2/N7hmMBqFHYLwxTK+ZK0mIRk4YrctauDi2RFZXIvRls mV6lR+WDOOGQ9LDe+vr2xq4ONJ7TgNRpzvJS2+7+x+H8XUpN11C7rzzK4ElDPHM5V2tX hPhg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=RfJ7Wmu+isnYJEI0NneC1wcTrVYA9eunxNNo2wKm6wc=; b=Yju9i+hm/UyU4U9mFNbTjuJqe2j/WpH1eJiu+ZGIufptvjxAk/W4nNoEqigfaQXkGL UFndDINZRbUUb3YPOui+eJDXq1ZSaGyhd4nkFnLTEV8Ku5b92s0Mm5gqIlL1yrVS+Ima OjF6N++wRlrSJRyEyh3hBnllaX4qcAiQtwyOBLu9bBmBdgTnXWltJwM4pbAPs7LA7Ag1 37s7Fski+6bq69Kxy8UlMg+A0kKHa7kRr4s1P5axQJCo3EU/+5WxycE6BptcD3JAy2ou HS89IQB5qDFZAcEoODEKJdauJkMam8ig02Mn0EowHDpyySz2UxfmWLhGWP1+9mR8e/0r ZxHQ== X-Gm-Message-State: APjAAAUql74q69JQHMdpXI/DmREpKyXEerpZcYBgqsC2ZB152Sa6tyrt DXK/DtxyJVs6uzIahvkpEqjnCWDAINQ7m1AxhqAERQ== X-Received: by 2002:a37:9ac7:: with SMTP id c190mr70006657qke.300.1558727881655; Fri, 24 May 2019 12:58:01 -0700 (PDT) MIME-Version: 1.0 References: <40403d07-293a-62a1-e0e9-ad6a2ba84844@gmail.com> In-Reply-To: <40403d07-293a-62a1-e0e9-ad6a2ba84844@gmail.com> From: Daniel Drake Date: Fri, 24 May 2019 13:57:50 -0600 Message-ID: Subject: Re: [PATCH v4 02/13] platform/x86: asus-wmi: Fix preserving keyboard backlight intensity on load To: Yurii Pavlovskyi Cc: Corentin Chary , Darren Hart , Andy Shevchenko , Chris Chiu , acpi4asus-user , Platform Driver , Linux Kernel Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 14, 2019 at 12:51 PM Yurii Pavlovskyi wrote: > > The error code and return value are mixed up. The intensity is always set > to 0 on load as kbd_led_read returns either 0 or negative value. To > reproduce set backlight to maximum, reload driver and try to increase it > using keyboard hotkey, the intensity will drop as a result. Correct the > implementation. > > Signed-off-by: Yurii Pavlovskyi Reviewed-by: Daniel Drake