From patchwork Mon Feb 19 23:11:18 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luca Weiss X-Patchwork-Id: 20626 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:693c:2685:b0:108:e6aa:91d0 with SMTP id mn5csp87180dyc; Mon, 19 Feb 2024 15:12:46 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCUE9UBvQ4DorPrMc0uIgJp7c3FQZrmvDbGfXJfAnI3JrZe2tgBkWScyk8HgjKtLV/D3uX/9y1ZGm/1RviSy+gdzu6k0Bw== X-Google-Smtp-Source: AGHT+IEj9Vl3yB0FnEcpZ4bgI0yBOWwc80u9hKYJwU8crUpFp9RMuH+iQRlixZGYnh+4pAjdc2/Q X-Received: by 2002:a19:f70c:0:b0:512:b89f:7db4 with SMTP id z12-20020a19f70c000000b00512b89f7db4mr2038802lfe.29.1708384366270; Mon, 19 Feb 2024 15:12:46 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708384366; cv=pass; d=google.com; s=arc-20160816; b=zdnQ38WdSetEjS8w0YTHYLTUITWT9zAUnrn+dkqXpIkVeLWxf1LVGR1AsLixyZJ61z 0ikr1C1+AoW1zdqRm0T+livPT5xXpoldlYgsFp0GBX4w3pJBN+JsD6JD191yZY5lkf+n PMM2g0p/Yy3GmCPWQq9mQgCTnZiQhiwLOYwlQpVT3HC0CZuSkpzKzmJZndBAvXGpVbls DlLTVIilIrHep3ut4SKZyhwB5AlwhNusCSYJ/9+xfviNEwFjvDow/XkBu+kqAX3dm3p8 RCd4pHeifjnTJj3wCwmeM3buDf+LaHBxJwar7au5vBElkhESI984obTRMeNUbDKXYmTe gWrg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:from :dkim-signature; bh=ewWK/I/S5xrV/DBnIdJRtcW/OXXmGi1pnliCgFAenko=; fh=v9kGMCGYwANI+V6qVbQ/zBduViEZBi/Y8v/CkAMAuHo=; b=GbVe9NG7LjXrJpcOx27h4BYTd4+15PI1sS/Cpk5nTQrqzjfI5srDRSRLXr0XF+Awm4 Gn0r4zwTZjGcScZxWkKWsGapnirgVLa0RyK7XLppmhRsH5/LeRojYo4kYqGCUP3qeKNu ThyqCT56DlvTJVQUY0G708/Rnyc7BkUuL+D0vSH/ueyHembn+Kh887wwTAYHAxBsLDds aTTAm5PW2OqXOnpUTuQOANOcZWYtjkdMjoikueLcyDA85JimjZ1h4pP6JyXKSHlyhi4+ cZXMRleuftdnVJ4UpTrAfPy2CRiqz62zRZXkHefT+yCkWx/ZQA5jzrYO1FSLHjmK00pA ZzeQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@z3ntu.xyz header.s=s1 header.b=ewnDJnty; arc=pass (i=1 spf=pass spfdomain=z3ntu.xyz dkim=pass dkdomain=z3ntu.xyz dmarc=pass fromdomain=z3ntu.xyz); spf=pass (google.com: domain of linux-kernel+bounces-72121-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-72121-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=z3ntu.xyz Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id ec18-20020a170906b6d200b00a3dfb75758dsi2791779ejb.92.2024.02.19.15.12.46 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Feb 2024 15:12:46 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-72121-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@z3ntu.xyz header.s=s1 header.b=ewnDJnty; arc=pass (i=1 spf=pass spfdomain=z3ntu.xyz dkim=pass dkdomain=z3ntu.xyz dmarc=pass fromdomain=z3ntu.xyz); spf=pass (google.com: domain of linux-kernel+bounces-72121-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-72121-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=z3ntu.xyz Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id DD9081F2282A for ; Mon, 19 Feb 2024 23:12:45 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id B79BC59160; Mon, 19 Feb 2024 23:11:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=z3ntu.xyz header.i=@z3ntu.xyz header.b="ewnDJnty" Received: from ahti.lucaweiss.eu (ahti.lucaweiss.eu [128.199.32.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 61EE736B18; Mon, 19 Feb 2024 23:11:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=128.199.32.197 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708384308; cv=none; b=czudIVZIDmnDse17QLdB5wlJJr4ieoUJo4kxTyXNdiB193hzovhUZoQk+5SwHojp/2flKnd9AAG5KHcCa2hII8wZmhbn4XfNg914DTWw9u42/mifjtCMaLplLQej+CSysjjxME/ibX3fqzTnu/ipuhmKimlSQ1ju4Gy5WdTz7GM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708384308; c=relaxed/simple; bh=NeEj8AGKphheuB6alfncyh7fBudqA8Is+Zqjc3OT8Z0=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=kv/yB036KKqxvkGMVRFJaYabIlBgqawQPVvdnS1CelhYBclgWoyAxkGmvY6qp4NEg9PfHkVGWTt0uvPBJKI2LbT7wvoRFq8sjPGS/MdeGSFlCgZtVT8LHKkb1RZLm/Vkjwr58rvwLmHo3LUSKmJSlSHf9O6wiyIcg+s4uM1bzOs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=z3ntu.xyz; spf=pass smtp.mailfrom=z3ntu.xyz; dkim=pass (1024-bit key) header.d=z3ntu.xyz header.i=@z3ntu.xyz header.b=ewnDJnty; arc=none smtp.client-ip=128.199.32.197 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=z3ntu.xyz Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=z3ntu.xyz DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=z3ntu.xyz; s=s1; t=1708384299; bh=NeEj8AGKphheuB6alfncyh7fBudqA8Is+Zqjc3OT8Z0=; h=From:Subject:Date:To:Cc; b=ewnDJntySR99hk7Y+C2rNPaU8W2c51BkPkv55TF5pT2E6EdT1qp6HJQ1xPcRMkkyk LmuXp92GmavpDBKO1VLay7QYD/GAZa2kCKSCO7ySm2ARtFg0BXngT9Pjre/TB6DiAN j3qsRZ/ugJAecPUpmxdnU7viEmuQ3y4voKYdLRkk= From: Luca Weiss Subject: [PATCH 0/4] Various fixes for the lm3630a backlight driver Date: Tue, 20 Feb 2024 00:11:18 +0100 Message-Id: <20240220-lm3630a-fixups-v1-0-9ca62f7e4a33@z3ntu.xyz> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIABbg02UC/x2MQQqAIBAAvxJ7TjDN0L4SHcS2WigTpQjEvycdh 2EmQ8JImGBsMkR8KNHlK3RtA263fkNGS2UQXPRcdIYdpxwkt2yl9w6JaWukMqisdgg1ChGr+Yf TXMoHaRKMAWAAAAA= To: ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org, Lee Jones , Daniel Thompson , Jingoo Han , Helge Deller , Andrew Morton , "G.Shark Jeong" , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Maximilian Weigand Cc: dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, Luca Weiss X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=1469; i=luca@z3ntu.xyz; h=from:subject:message-id; bh=NeEj8AGKphheuB6alfncyh7fBudqA8Is+Zqjc3OT8Z0=; b=owEBbQKS/ZANAwAIAXLYQ7idTddWAcsmYgBl0+AeE/jVURL67849OakgmKw1qtHVwyn5sId8p QY853PAXy+JAjMEAAEIAB0WIQQ5utIvCCzakboVj/py2EO4nU3XVgUCZdPgHgAKCRBy2EO4nU3X VpP0EACNUE3CLBdgEIlA++Y7oQGLuk9N6C1625ObzLjLTNrYp81coe1VlkNJkkTDq5QdY0I04Oh UbZx6wB/KAUNTjxvhEhZhOevm/PYQMArePXT8X01GwEMFAdTznVRBMmIDUnuQ0jFt3to3QaX9HS RL4RS4fuin414ZQVULOSy1wtrjPpE//ujlUWOTWeedw9EBQW0BKOH5W71TQ3/PBI6WIVHYEB+g2 zhjsUzpGLoQJfVky5XetACzWAx0juYnZhVliS6if/pVLnM9XqVcaFoy5/ezXb5WwJEpc2L7onI4 LqTySNxBqtYAMhpP/x0huUbbRNMrGBhOgXw5BKtpU+qfD4QHTixaGrOZ1VN24cYQ4pMrVRqzdIA QRS5dJigf4rKKMYC83ZAXiIof6XTsCknwdMoNsujaTUcPcmPjNvNUkkjeGeq3K0Jj+nr1t6pXxt UHgfCoCp/8Kt+nIXz6Fsur0tU5aN92KK5xyyLgQJ18yrzoemR0eOm+WIlHOjGY9xNTM8ZQaHYNL 5wfuZUF0W91ptmQaxnzjkxFLPKgASwaU8D9eM0ROAZs0/xfG1GdZJ/hqDUiw4sSdYYzfnU8maMh gNR1sgccozBpP8tDtAq2yJ+sOXhAtn06e2Xl9gLatMuC36JavLafUY8Q61+eJ/4vnFj1rFJoY/6 e0t+URshdDX8GWw== X-Developer-Key: i=luca@z3ntu.xyz; a=openpgp; fpr=BD04DA24C971B8D587B2B8D7FAF69CF6CD2D02CD X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1791370845386025960 X-GMAIL-MSGID: 1791370845386025960 On the MSM8974 Nexus 5 and OnePlus One phones (latter doesn't have display upstream) the display backlight was turning off whenever you would write a brightness to sysfs since a recent commit to the driver (kernel v6.5). backlight: lm3630a: Turn off both led strings when display is blank Turns out, backlight_is_blank() thought the display was blanked because the props variable is was checking was never actually initialized so it was just reading some value that was left before. The first commit in this series fixes this, and the others are some cleanups / fixes I noticed while working on this. As last commit, we can finally hook up the panel and backlight on the Nexus 5 so blanking the screen actually turns off the backlight. Signed-off-by: Luca Weiss --- Luca Weiss (4): backlight: lm3630a: Initialize backlight_properties on init backlight: lm3630a: Don't set bl->props.brightness in get_brightness backlight: lm3630a: Use backlight_get_brightness helper in update_status ARM: dts: qcom: msm8974-hammerhead: Hook up backlight .../qcom/qcom-msm8974-lge-nexus5-hammerhead.dts | 4 ++- drivers/video/backlight/lm3630a_bl.c | 29 ++++++++++------------ 2 files changed, 16 insertions(+), 17 deletions(-) --- base-commit: b401b621758e46812da61fa58a67c3fd8d91de0d change-id: 20240219-lm3630a-fixups-8a9359e5a8ce Best regards,