From nobody Sun Jul 13 03:56:26 2025 Delivered-To: importer2@patchew.org Received-SPF: pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; envelope-from=linux-kernel-owner@vger.kernel.org; helo=vger.kernel.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail(p=none dis=none) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mx.zohomail.com with SMTP id 1643740683925328.3361855148945; Tue, 1 Feb 2022 10:38:03 -0800 (PST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242009AbiBASiB (ORCPT ); Tue, 1 Feb 2022 13:38:01 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55278 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240143AbiBASiA (ORCPT ); Tue, 1 Feb 2022 13:38:00 -0500 Received: from mail-wm1-x32b.google.com (mail-wm1-x32b.google.com [IPv6:2a00:1450:4864:20::32b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C5DA0C061714; Tue, 1 Feb 2022 10:37:59 -0800 (PST) Received: by mail-wm1-x32b.google.com with SMTP id j5-20020a05600c1c0500b0034d2e956aadso2238497wms.4; Tue, 01 Feb 2022 10:37:59 -0800 (PST) Received: from X1C7EK5.wind3.hub ([151.40.130.120]) by smtp.gmail.com with ESMTPSA id j19sm3097477wmq.17.2022.02.01.10.37.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 01 Feb 2022 10:37:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=ve8tBclI5IMrrMI15B03jvD+eu/iK6m1qrk9Pz//fBI=; b=lFfSrnIIiNiQ9HzJ7/WifUaPADKRkkv+F3hqEDj0Jn9XW84XpBgFQbIMqpTr7/urD+ YO6bYyWaiAOVo0Mun2QQ3wXa/eJFfmhy+GjW0FNeD5y8ZrTQC4PWkJhxHRmoLWIn8gfI sxewSoTkQHlDS36c6E8dYKP8E+0noOceVjOoUu5CxKvs6bO9GYmjWijoO9FvJKzrb7At CdrlfBmKwnxraAm2WQX5r+N+4hNai/jUk4G+NHJ1jXejeeuJxZNjdg8/OnCqZYa1/u2E 85I/KW9EX0/SfGgISkhYMMANYaoxEgsLhmMLMC1B0B7OppI62gL83WZOreOXz3ulLF1K IuAg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=ve8tBclI5IMrrMI15B03jvD+eu/iK6m1qrk9Pz//fBI=; b=daRtCnFYkgNMw4fM/7nQLoTcZb+mJYzd02P+uNNtVDmBzlUgGkGS9pCIhYmEQm42JY 1CwZmRL8SsmleLF0fXZJt2fiCYo3oQOjgDkXJFaMwATzS7VJCiHQRgZHYD5VX5Ew8VL8 PWXJothBBCJ/RwV5zphrmZZe68Og97cywd0qHZnXBiDhLjwrSvMjP9An3Fn//ThKxqgl gl6nolmDnaURPF8iYs3fNpr3hchM1q0ZwanZfDKRQmyk9mOtF6zPswcOSfA3X5BMu/jz BEhaUHMRq4EGX6YmirjOS9pyLrEicuSrkV9Lk/RTVmbUfNXEpYPQrcIDSZ7WOgjxSrTe cT+w== X-Gm-Message-State: AOAM5336lsovmmw2MjlxK26wASeUrphZ7mQXZkG9FmuopswvVzMD/+Sh s/y34+yPLAuMtOoHtBA56zPahvy+efrpdw== X-Google-Smtp-Source: ABdhPJwI6E3HElHpzSsZnZmMM8bQrF62l2ToUjMY8b8KAp0cybrgUDH9guSk/WKKQnqpcg75zMvrzQ== X-Received: by 2002:a05:600c:3505:: with SMTP id h5mr2978052wmq.122.1643740678276; Tue, 01 Feb 2022 10:37:58 -0800 (PST) From: ektor5 Cc: Hans Verkuil , Mauro Carvalho Chehab , Ettore Chimenti , linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linuxfancy@googlegroups.com Subject: [PATCH 2/2] media: cec: seco: add newlines in debug messages Date: Tue, 1 Feb 2022 19:37:34 +0100 Message-Id: <20220201183734.224756-3-ek5.chimenti@gmail.com> X-Mailer: git-send-email 2.35.1 In-Reply-To: <20220201183734.224756-1-ek5.chimenti@gmail.com> References: <20220201183734.224756-1-ek5.chimenti@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZM-MESSAGEID: 1643740686282100001 Content-Type: text/plain; charset="utf-8" From: Ettore Chimenti Newlines were missing in almost all regular and debug printk. Signed-off-by: Ettore Chimenti --- drivers/media/cec/platform/seco/seco-cec.c | 54 +++++++++++----------- 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/drivers/media/cec/platform/seco/seco-cec.c b/drivers/media/cec= /platform/seco/seco-cec.c index ae138cc253fd..78856a55e637 100644 --- a/drivers/media/cec/platform/seco/seco-cec.c +++ b/drivers/media/cec/platform/seco/seco-cec.c @@ -129,7 +129,7 @@ static int secocec_adap_enable(struct cec_adapter *adap= , bool enable) if (status) goto err; =20 - dev_dbg(dev, "Device enabled"); + dev_dbg(dev, "Device enabled\n"); } else { /* Clear the status register */ status =3D smb_rd16(SECOCEC_STATUS_REG_1, &val); @@ -141,7 +141,7 @@ static int secocec_adap_enable(struct cec_adapter *adap= , bool enable) ~SECOCEC_ENABLE_REG_1_CEC & ~SECOCEC_ENABLE_REG_1_IR); =20 - dev_dbg(dev, "Device disabled"); + dev_dbg(dev, "Device disabled\n"); } =20 return 0; @@ -264,12 +264,12 @@ static void secocec_rx_done(struct cec_adapter *adap,= u16 status_val) =20 if (status_val & SECOCEC_STATUS_RX_OVERFLOW_MASK) { /* NOTE: Untested, it also might not be necessary */ - dev_warn(dev, "Received more than 16 bytes. Discarding"); + dev_warn(dev, "Received more than 16 bytes. Discarding\n"); flag_overflow =3D true; } =20 if (status_val & SECOCEC_STATUS_RX_ERROR_MASK) { - dev_warn(dev, "Message received with errors. Discarding"); + dev_warn(dev, "Message received with errors. Discarding\n"); status =3D -EIO; goto rxerr; } @@ -390,12 +390,12 @@ static int secocec_ir_probe(void *priv) if (status !=3D 0) goto err; =20 - dev_dbg(dev, "IR enabled"); + dev_dbg(dev, "IR enabled\n"); =20 status =3D devm_rc_register_device(dev, cec->ir); =20 if (status) { - dev_err(dev, "Failed to prepare input device"); + dev_err(dev, "Failed to prepare input device\n"); cec->ir =3D NULL; goto err; } @@ -408,7 +408,7 @@ static int secocec_ir_probe(void *priv) smb_wr16(SECOCEC_ENABLE_REG_1, val & ~SECOCEC_ENABLE_REG_1_IR); =20 - dev_dbg(dev, "IR disabled"); + dev_dbg(dev, "IR disabled\n"); return status; } =20 @@ -431,13 +431,13 @@ static int secocec_ir_rx(struct secocec_data *priv) =20 rc_keydown(cec->ir, RC_PROTO_RC5, RC_SCANCODE_RC5(addr, key), toggle); =20 - dev_dbg(dev, "IR key pressed: 0x%02x addr 0x%02x toggle 0x%02x", key, + dev_dbg(dev, "IR key pressed: 0x%02x addr 0x%02x toggle 0x%02x\n", key, addr, toggle); =20 return 0; =20 err: - dev_err(dev, "IR Receive message failed (%d)", status); + dev_err(dev, "IR Receive message failed (%d)\n", status); return -EIO; } #else @@ -497,7 +497,7 @@ static irqreturn_t secocec_irq_handler(int irq, void *p= riv) return IRQ_HANDLED; =20 err: - dev_err_once(dev, "IRQ: R/W SMBus operation failed (%d)", status); + dev_err_once(dev, "IRQ: R/W SMBus operation failed %d\n", status); =20 /* Reset status register */ val =3D SECOCEC_STATUS_REG_1_CEC | SECOCEC_STATUS_REG_1_IR; @@ -553,16 +553,16 @@ static int secocec_acpi_probe(struct secocec_data *sd= ev) =20 gpio =3D devm_gpiod_get(dev, NULL, GPIOF_IN); if (IS_ERR(gpio)) { - dev_err(dev, "Cannot request interrupt gpio"); + dev_err(dev, "Cannot request interrupt gpio\n"); return PTR_ERR(gpio); } =20 irq =3D gpiod_to_irq(gpio); if (irq < 0) { - dev_err(dev, "Cannot find valid irq"); + dev_err(dev, "Cannot find valid irq\n"); return -ENODEV; } - dev_dbg(dev, "irq-gpio is bound to IRQ %d", irq); + dev_dbg(dev, "irq-gpio is bound to IRQ %d\n", irq); =20 sdev->irq =3D irq; =20 @@ -590,7 +590,7 @@ static int secocec_probe(struct platform_device *pdev) =20 /* Request SMBus regions */ if (!request_muxed_region(BRA_SMB_BASE_ADDR, 7, "CEC00001")) { - dev_err(dev, "Request memory region failed"); + dev_err(dev, "Request memory region failed\n"); return -ENXIO; } =20 @@ -598,14 +598,14 @@ static int secocec_probe(struct platform_device *pdev) secocec->dev =3D dev; =20 if (!has_acpi_companion(dev)) { - dev_dbg(dev, "Cannot find any ACPI companion"); + dev_dbg(dev, "Cannot find any ACPI companion\n"); ret =3D -ENODEV; goto err; } =20 ret =3D secocec_acpi_probe(secocec); if (ret) { - dev_err(dev, "Cannot assign gpio to IRQ"); + dev_err(dev, "Cannot assign gpio to IRQ\n"); ret =3D -ENODEV; goto err; } @@ -613,11 +613,11 @@ static int secocec_probe(struct platform_device *pdev) /* Firmware version check */ ret =3D smb_rd16(SECOCEC_VERSION, &val); if (ret) { - dev_err(dev, "Cannot check fw version"); + dev_err(dev, "Cannot check fw version\n"); goto err; } if (val < SECOCEC_LATEST_FW) { - dev_err(dev, "CEC Firmware not supported (v.%04x). Use ver > v.%04x", + dev_err(dev, "CEC Firmware not supported (v.%04x). Use ver > v.%04x\n", val, SECOCEC_LATEST_FW); ret =3D -EINVAL; goto err; @@ -631,7 +631,7 @@ static int secocec_probe(struct platform_device *pdev) dev_name(&pdev->dev), secocec); =20 if (ret) { - dev_err(dev, "Cannot request IRQ %d", secocec->irq); + dev_err(dev, "Cannot request IRQ %d\n", secocec->irq); ret =3D -EIO; goto err; } @@ -666,7 +666,7 @@ static int secocec_probe(struct platform_device *pdev) =20 platform_set_drvdata(pdev, secocec); =20 - dev_dbg(dev, "Device registered"); + dev_dbg(dev, "Device registered\n"); =20 return ret; =20 @@ -691,14 +691,14 @@ static int secocec_remove(struct platform_device *pde= v) =20 smb_wr16(SECOCEC_ENABLE_REG_1, val & ~SECOCEC_ENABLE_REG_1_IR); =20 - dev_dbg(&pdev->dev, "IR disabled"); + dev_dbg(&pdev->dev, "IR disabled\n"); } cec_notifier_cec_adap_unregister(secocec->notifier, secocec->cec_adap); cec_unregister_adapter(secocec->cec_adap); =20 release_region(BRA_SMB_BASE_ADDR, 7); =20 - dev_dbg(&pdev->dev, "CEC device removed"); + dev_dbg(&pdev->dev, "CEC device removed\n"); =20 return 0; } @@ -709,7 +709,7 @@ static int secocec_suspend(struct device *dev) int status; u16 val; =20 - dev_dbg(dev, "Device going to suspend, disabling"); + dev_dbg(dev, "Device going to suspend, disabling\n"); =20 /* Clear the status register */ status =3D smb_rd16(SECOCEC_STATUS_REG_1, &val); @@ -733,7 +733,7 @@ static int secocec_suspend(struct device *dev) return 0; =20 err: - dev_err(dev, "Suspend failed (err: %d)", status); + dev_err(dev, "Suspend failed: %d\n", status); return status; } =20 @@ -742,7 +742,7 @@ static int secocec_resume(struct device *dev) int status; u16 val; =20 - dev_dbg(dev, "Resuming device from suspend"); + dev_dbg(dev, "Resuming device from suspend\n"); =20 /* Clear the status register */ status =3D smb_rd16(SECOCEC_STATUS_REG_1, &val); @@ -762,12 +762,12 @@ static int secocec_resume(struct device *dev) if (status) goto err; =20 - dev_dbg(dev, "Device resumed from suspend"); + dev_dbg(dev, "Device resumed from suspend\n"); =20 return 0; =20 err: - dev_err(dev, "Resume failed (err: %d)", status); + dev_err(dev, "Resume failed: %d\n", status); return status; } =20 --=20 2.17.1