From patchwork Tue May 20 06:03:11 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 891447 Received: from mail-ed1-f41.google.com (mail-ed1-f41.google.com [209.85.208.41]) (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 9532D255F55 for ; Tue, 20 May 2025 06:03:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.41 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747721003; cv=none; b=FRo00w52lFRBJHCjy+3qxDj56XGZQIA3mL+AkhGMjyIpAp1B44gh1LakLrBi2CNVdm9J6NKeKUPwBWFx2gT+Gnl/+jVenQgCtpHwGglR7oqtXx1aAsu+PfwtoTZZGMVNTCv3z3edeDGrZ0DMiLE31ABIgwop17VFSgyBKfRgRCI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747721003; c=relaxed/simple; bh=kzHaamZZFjKOYQvT00HR9118hSlIU006QwWA7TkYXTE=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=ljDKbFKy2J9rSjErnWBe76fPOFmlQZjTMG/Ba2LODwM2c883NRpudF+v6vKjqmJrdsx9gf+jUUHrIvXfJA/UyF+pwxI6dXiM+Ugq7ZJBNXcVVBUelDCw+6mff3tM+tDFENXHey52H0ZjJR6gz+ncQPpaRLy0yRPoFEf9LVvJhHM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=wMvhR67+; arc=none smtp.client-ip=209.85.208.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="wMvhR67+" Received: by mail-ed1-f41.google.com with SMTP id 4fb4d7f45d1cf-5f6f86580ecso1099087a12.1 for ; Mon, 19 May 2025 23:03:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1747721000; x=1748325800; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=h+3L+5Wk0I29VJqzwPhcySS+DGDSqL1wf3tzyzbHFwE=; b=wMvhR67+isk0Ux6wa9g8SDl5z0nxn/EkdQUTSVJbTe1RYgOjsNgA7OeVYGdn/oUcpG c4eq+WHFi5y8lpk7S/t14gAXr9SVkdm2cwFts1lhfTZWKA0gEH0epUhfvZ5q0lKOWmIp CKoItivgqG3wwJl6wjEUmGv+WNgQK2nulc57eDCZr2I2enpNkjNUOOJ9zSmKoL8iLwJy PNptv/vJXS6UIUMESSl2WcXULBTzikrvGrnuKvRFO1tctpnbAArYyJWl6RNQMavAultW Z/zp6IwRut04IQnWdpCafMFvpI7xKLshL9Bx5NfcUR/cBBibPhVK2hHYRgaxm04bgW22 4fXA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1747721000; x=1748325800; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=h+3L+5Wk0I29VJqzwPhcySS+DGDSqL1wf3tzyzbHFwE=; b=iC2fgNYxAUeCSVx+JpR1u09fNQfwQefY+1FAncbVU5mei9R13sihB4j81QKfzVJpxE bXzvw3wsBSkTBuAKtlys/9hO1FAynR5goEgKhsL1MAvvSd+7VeIjpOUbpzSEMN4G4/fX 0aKyFZP62Vxr4yufEUWXX6ywZ9rZtpzpkFVKm5VhI1w9vpZlygBxNxcKig4tvPgT7Cu7 rb3eOCZ9RFEijJRm2G5Kx8HKNXvfhsWQxpU+Nju3jTSPxi14lz8m7iuj7N9J8tAU40ZA U2FIIDEcSWB7kJueRx082I3mu2WKDmAbIq+U4yRjq4tLz4lvcYo1QF8Dp4ctqnTLJXBU 8KWQ== X-Forwarded-Encrypted: i=1; AJvYcCVZWP2xRgfbYCIxc6dcqCjwAw1yK1QnQiSIj/TYASArXu8IDTUTiobm33kPwJ1DbjmI3MJZi+2dPXo31w==@vger.kernel.org X-Gm-Message-State: AOJu0YxUlLxEKHz+JSmZMaghehn9KEVBn0gL92J8tBezowrZ9+S+ANIw qHKodrJADSH4rJaPHRahcdWfx1hCZqmcpKYpjlZWPN9votqwTovuyi7Wy0iujc+IM5w= X-Gm-Gg: ASbGnctHgsTfo6eTWdQlvbq5QVsv/bpo/yYwdO7jouo+q2RoF94WOaUbdcTkpc+b6Sp Ixs2xj48MC59HXb1+4FovwQIumqD8oQGCh1TfwUZPZ4HAWuhpoBblB1C5cULCjiaK74bp793PFg zUHAhY0iG4NGA5tEkyY8oq6n0zrxcN4CSyLciOKlJVKTMGrApRicAXyGlG3Vt1x/KRFMOsSgg8D 7IOTbiDriNu2HYbuB+lyYB/jFD3ypx9cIFZeCBkjIYkoewStmYZZDura1Gq/XBSY9XQCMbnp0nJ +8WZakVKbwTfBdWkt3H5rLM2TIqW8Urk2joOggUEF53vzJz1o/RfkG4Tgcqe2g== X-Google-Smtp-Source: AGHT+IFTQtDRhko0YqLPOd33vYaqapq33LlBdmjbtssnUwIAMa1VgmHJ3TaRN7JuOLNsHMIWUEATgA== X-Received: by 2002:a05:6402:134b:b0:601:acee:fd7c with SMTP id 4fb4d7f45d1cf-601acef0123mr2625931a12.10.1747720999462; Mon, 19 May 2025 23:03:19 -0700 (PDT) Received: from kuoka.. ([178.197.223.125]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6005ac33a88sm6698710a12.51.2025.05.19.23.03.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 May 2025 23:03:18 -0700 (PDT) From: Krzysztof Kozlowski To: Robert Foss , Todor Tomov , "Bryan O'Donoghue" , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Krzysztof Kozlowski Subject: [PATCH v2 1/3] media: qcom: camss: vfe: Stop spamming logs with version Date: Tue, 20 May 2025 08:03:11 +0200 Message-ID: <20250520060310.7543-4-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.45.2 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=7752; i=krzysztof.kozlowski@linaro.org; h=from:subject; bh=kzHaamZZFjKOYQvT00HR9118hSlIU006QwWA7TkYXTE=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBoLBseIePo2FAZyDudhFWmnw+pWByEYl/L26o59 FLopTYLESmJAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaCwbHgAKCRDBN2bmhouD 1/tnD/9k7aSGFPMl/PfpwSoAneaOE7E3061Om8hqrfT3lRkx8UA/wGi7qzgy1wRRgEGoGHJgA+/ I+j1t/G2W9kJlIP0H9TqigzzU9bkkhj4YwrHZ0KtQT3QCFRMmhfJwGcy6Zq+Brdc1l5QazeIfG+ eh//4cZlFHIHyHpWGlEIVoJ/pH5fYl7K+F0iZpMicFfY9apsveUe43U1cF3dn9qBwYjnnmclllE JOn1v7I+6BZdvTOp5kTOm6Hh0pM5T1HuUxLX1S1pyue1infu3eeoxXKgss6bUs0BOu9lEmegETB Ix9SZ005X4XJpLfQxsCv5n48ztKlavfIb6QnT/ebVkrYmTl4ec3OzZDxFKF+Ns5cHs+685Nles0 bnGMEUgBGBhg9CUgRkF5eSCLFgH0b+y9nb0Twhl33k+LEiVSfnVCjAQ70lv0+ejIPq+mdlTdY8i joHnenKBlnXbYfXeNpWhzH13qVHRfKq/I+lELKBye4sPwy6dmXHHhqQsNM2dMVrTgRbVsGIKpTy MCB2R4nfWUryT1g0NwjWlLokaicqZaB0wRyQT2QrCLLiJyV81DvEMNnD1021m2igom7CfwqGIsT tD2Kns0g8+6Xb8uhWsk2oYHCmFEZHw906ZLwfInYtPYhhlmUINkusEY2Tr+wjZ0Xd12vtHauRoO SKsQvWst8GiOS8Q== X-Developer-Key: i=krzysztof.kozlowski@linaro.org; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B Camss drivers spam kernel dmesg with 64 useless messages during boot: qcom-camss acb7000.isp: VFE:1 HW Version = 3.0.2 qcom-camss acb7000.isp: VFE:2 HW Version = 2.4.0 All of these messages are the same, so it makes no sense to print same information 32 times. The driver does not use read version at all, so if it was needed for any real debugging purpose it would be provided via debugfs interface. However even then printing this is pointless, because version of hardware block is deducible from the compatible. Fix the code to adhere to Linux kernel coding style: being silent on success. For the same reasons this should not be even dbg message (see driver development debug guide: "In almost all cases the debug statements shouldn't be upstreamed"). Signed-off-by: Krzysztof Kozlowski --- Changes in v2: 1. Extend commit msg, because apparently coding style and debug guide has to be reminded in commit msg, otherwise rules don't apply. --- .../media/platform/qcom/camss/camss-vfe-17x.c | 1 - .../media/platform/qcom/camss/camss-vfe-4-1.c | 1 - .../media/platform/qcom/camss/camss-vfe-4-7.c | 1 - .../media/platform/qcom/camss/camss-vfe-4-8.c | 1 - .../media/platform/qcom/camss/camss-vfe-480.c | 1 - .../media/platform/qcom/camss/camss-vfe-680.c | 1 - .../media/platform/qcom/camss/camss-vfe-780.c | 1 - drivers/media/platform/qcom/camss/camss-vfe.c | 22 ------------------- drivers/media/platform/qcom/camss/camss-vfe.h | 8 ------- 9 files changed, 37 deletions(-) diff --git a/drivers/media/platform/qcom/camss/camss-vfe-17x.c b/drivers/media/platform/qcom/camss/camss-vfe-17x.c index e5ee7e717b3b..e0d12c3f6015 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-17x.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-17x.c @@ -577,7 +577,6 @@ static void vfe_subdev_init(struct device *dev, struct vfe_device *vfe) const struct vfe_hw_ops vfe_ops_170 = { .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr_read = vfe_isr_read, .isr = vfe_isr, .pm_domain_off = vfe_pm_domain_off, diff --git a/drivers/media/platform/qcom/camss/camss-vfe-4-1.c b/drivers/media/platform/qcom/camss/camss-vfe-4-1.c index 901677293d97..7620ce42b49b 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-4-1.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-4-1.c @@ -993,7 +993,6 @@ static void vfe_subdev_init(struct device *dev, struct vfe_device *vfe) const struct vfe_hw_ops vfe_ops_4_1 = { .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr_read = vfe_isr_read, .isr = vfe_isr, .pm_domain_off = vfe_4_1_pm_domain_off, diff --git a/drivers/media/platform/qcom/camss/camss-vfe-4-7.c b/drivers/media/platform/qcom/camss/camss-vfe-4-7.c index 76729607db02..b3b6ccb4748e 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-4-7.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-4-7.c @@ -1145,7 +1145,6 @@ static void vfe_subdev_init(struct device *dev, struct vfe_device *vfe) const struct vfe_hw_ops vfe_ops_4_7 = { .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr_read = vfe_isr_read, .isr = vfe_isr, .pm_domain_off = vfe_pm_domain_off, diff --git a/drivers/media/platform/qcom/camss/camss-vfe-4-8.c b/drivers/media/platform/qcom/camss/camss-vfe-4-8.c index b2f7d855d8dd..5a4b4f486aca 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-4-8.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-4-8.c @@ -1135,7 +1135,6 @@ static void vfe_subdev_init(struct device *dev, struct vfe_device *vfe) const struct vfe_hw_ops vfe_ops_4_8 = { .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr_read = vfe_isr_read, .isr = vfe_isr, .pm_domain_off = vfe_pm_domain_off, diff --git a/drivers/media/platform/qcom/camss/camss-vfe-480.c b/drivers/media/platform/qcom/camss/camss-vfe-480.c index 4feea590a47b..edd92308af62 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-480.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-480.c @@ -278,7 +278,6 @@ static void vfe_buf_done_480(struct vfe_device *vfe, int port_id) const struct vfe_hw_ops vfe_ops_480 = { .enable_irq = vfe_enable_irq, .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr = vfe_isr, .isr_read = vfe_isr_read, .reg_update = vfe_reg_update, diff --git a/drivers/media/platform/qcom/camss/camss-vfe-680.c b/drivers/media/platform/qcom/camss/camss-vfe-680.c index 99036e7c1e76..96a927acc6bb 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-680.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-680.c @@ -227,7 +227,6 @@ static inline void vfe_reg_update_clear(struct vfe_device *vfe, const struct vfe_hw_ops vfe_ops_680 = { .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr = vfe_isr, .pm_domain_off = vfe_pm_domain_off, .pm_domain_on = vfe_pm_domain_on, diff --git a/drivers/media/platform/qcom/camss/camss-vfe-780.c b/drivers/media/platform/qcom/camss/camss-vfe-780.c index b9812d70f91b..e5023eb7ad60 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe-780.c +++ b/drivers/media/platform/qcom/camss/camss-vfe-780.c @@ -142,7 +142,6 @@ static int vfe_halt(struct vfe_device *vfe) const struct vfe_hw_ops vfe_ops_780 = { .global_reset = vfe_global_reset, - .hw_version = vfe_hw_version, .isr = vfe_isr, .pm_domain_off = vfe_pm_domain_off, .pm_domain_on = vfe_pm_domain_on, diff --git a/drivers/media/platform/qcom/camss/camss-vfe.c b/drivers/media/platform/qcom/camss/camss-vfe.c index 4bca6c3abaff..1ae523219525 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe.c +++ b/drivers/media/platform/qcom/camss/camss-vfe.c @@ -415,26 +415,6 @@ static u32 vfe_src_pad_code(struct vfe_line *line, u32 sink_code, return 0; } -/* - * vfe_hw_version - Process write master done interrupt - * @vfe: VFE Device - * - * Return vfe hw version - */ -u32 vfe_hw_version(struct vfe_device *vfe) -{ - u32 hw_version = readl_relaxed(vfe->base + VFE_HW_VERSION); - - u32 gen = (hw_version >> HW_VERSION_GENERATION) & 0xF; - u32 rev = (hw_version >> HW_VERSION_REVISION) & 0xFFF; - u32 step = (hw_version >> HW_VERSION_STEPPING) & 0xFFFF; - - dev_dbg(vfe->camss->dev, "VFE:%d HW Version = %u.%u.%u\n", - vfe->id, gen, rev, step); - - return hw_version; -} - /* * vfe_buf_done - Process write master done interrupt * @vfe: VFE Device @@ -1088,8 +1068,6 @@ int vfe_get(struct vfe_device *vfe) vfe_reset_output_maps(vfe); vfe_init_outputs(vfe); - - vfe->res->hw_ops->hw_version(vfe); } else { ret = vfe_check_clock_rates(vfe); if (ret < 0) diff --git a/drivers/media/platform/qcom/camss/camss-vfe.h b/drivers/media/platform/qcom/camss/camss-vfe.h index a23f666be753..1553ca89bd86 100644 --- a/drivers/media/platform/qcom/camss/camss-vfe.h +++ b/drivers/media/platform/qcom/camss/camss-vfe.h @@ -101,7 +101,6 @@ struct vfe_device; struct vfe_hw_ops { void (*enable_irq)(struct vfe_device *vfe); void (*global_reset)(struct vfe_device *vfe); - u32 (*hw_version)(struct vfe_device *vfe); irqreturn_t (*isr)(int irq, void *dev); void (*isr_read)(struct vfe_device *vfe, u32 *value0, u32 *value1); void (*pm_domain_off)(struct vfe_device *vfe); @@ -259,13 +258,6 @@ void vfe_put(struct vfe_device *vfe); */ bool vfe_is_lite(struct vfe_device *vfe); -/* - * vfe_hw_version - Process write master done interrupt - * @vfe: VFE Device - * - * Return vfe hw version - */ -u32 vfe_hw_version(struct vfe_device *vfe); /* * vfe_enable - Enable streaming on VFE line * @line: VFE line From patchwork Tue May 20 06:03:13 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 891446 Received: from mail-ed1-f46.google.com (mail-ed1-f46.google.com [209.85.208.46]) (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 8A02D25C805 for ; Tue, 20 May 2025 06:03:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747721006; cv=none; b=VdAYpqzsrx/lG6K0B0Adk93BmZHUuYYowVJcmkSEzdl0Jot6Tc4UxH5bjARJE0YycsXbfEa2tvWB3ZZ8KfHYPN1uZxDz3Pw9AVO0Q3AJQI9R+Egbd0M6GUxdNqKFPTWFzUvgmiezruC92UApjgro7ZWo3IACPNXTrk/Zto2jh/0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747721006; c=relaxed/simple; bh=s5ZFlWa2MpdvYNmAn19sW48vqvwbzszHzHufPZ8IlvE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=KeXdK2I2SZ6po/jknem0doNoM3AVt/924KDAlo/rCv34ylUp2drv+4g4DOK6sPW/6JHu2kud7gUm+vkQp6cua8GSl+v29EKtv3nEtu2pjwiOMluhQPNrgPuyNSJqmz4gdjmoRdmQ4uPSGnnxHKpMfj86ye3gB4GLVK2Qupf3lAs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Y9je46LL; arc=none smtp.client-ip=209.85.208.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Y9je46LL" Received: by mail-ed1-f46.google.com with SMTP id 4fb4d7f45d1cf-5faaa717cfbso898445a12.0 for ; Mon, 19 May 2025 23:03:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1747721003; x=1748325803; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=GunpgoETVGf3K3HF59YPy781/KUyyc4hFczZi9N9xRw=; b=Y9je46LLNnQXdbG7GCBaSwOXxuGl6ZS/bymKriIOQL3TflNf0zT73JntNhOoc8Qgo5 3GTGG8MIppdiPWRDTEFdE/Bbe9RINQZFywUu6ZDGSuoyWGNwPoep+nKZQLd6qfr4N8eT TONcy13rfRk9QobZNqZtndAB8vupBiNnsTJoXTBxcuO5oDuKnwQRKFTlZzq67hIPsgN2 FxoMSaGswDEwJLV4eWycEzcOMDNYJf354UJ3FEf73LJCbjerD5FvENo+mW/9sj13C0TR k0kPpKY0RGR6Qnti9bh/nu4PrQLrPXnq/vUaOQSzn0YTetNDkuyJ0iwDvIOKTTP3olQW C1Mw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1747721003; x=1748325803; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=GunpgoETVGf3K3HF59YPy781/KUyyc4hFczZi9N9xRw=; b=caxeUbWxuXUN09PqjR5OUN4HedUWyKgDCLkM02WE/DVWoylamHGeGDxTd9K52E7XBV dbOfcdOK3xqPCi25IfrjWzfcrbKf9zVl18GwXbKQPWCzHKoTUXc5qsc6izcdNxvP2wVd DbU3jMeVvJu7/BFsW9Df4bA/qZvwYu6cydevDMYlRtsPz/O2aRce82FzKSzPHv9LA3fI p8cej27LJlBWs2gcRfV3ogOJg4yX0LHtgyN+AirOUWlGgOgcgYw9L6ZzmQSwYni1qeK7 QDYq7YzwD2VEwBIHYVVyX+KOXtzljGPZHsOJeXUCaxJu/ePIdcIYpgGVw1Gy9Qejusaw jB+Q== X-Forwarded-Encrypted: i=1; AJvYcCUxYC6WEdMJBXf0aID3AD1RDdjpeHYJA9hMuBY/ZFYLcaCfpofm3uUoVjjkEYa+YpQgWM1OFTP9coLPug==@vger.kernel.org X-Gm-Message-State: AOJu0Yx0HTbu9aQYPY+so+9KQ9WeSoli7MDfvYHzXf9/16kAowS9I40G ay9+KVLvnHGWfOj40YIaTQ8dmR8fD43vRkMyPqNzpMIq57wgZLLBp9AqpNWLzycphe8= X-Gm-Gg: ASbGnctvm1MhmGHkkIlQgK5NaGE/JMCADGp28vsAi6FXG+DvjJZmL7YmBBQBN7gBYRW 6f+VHgrjQqziva+W0f5fnrbRaMrWUE4XCrfr1wcUNTm+eIlWGpXb7aHRMS2Agan7vemLjAOC1uc Wjna0/q3GrfHxrEQHwaw48oepRugd0qg8C1ZLvRALiqxCOyTS8SzCnw7WSDOgCTic7+hmH45I68 SaQapIRPEAVVeKNWpLxABELIqsgXq7mAzYqkboeUTTLgGZH7mRUYinZBfMFQkNFIIDkuR04pQxT ou/fh9KWvRvVKFpm3kA+Ef7bBnoUmX+wl4/QRo+mkyoaFsxhKbLXxKdO22+cOA== X-Google-Smtp-Source: AGHT+IHxcxUmDMY6HrZgZSsQpx9fvPtvfK/+uzfpHOMfp00cUxC0UaWFC598+1/y+YpvBHLB6GAKmA== X-Received: by 2002:a05:6402:254b:b0:5fe:c344:5ed1 with SMTP id 4fb4d7f45d1cf-6008a3a25aemr5183705a12.1.1747721002546; Mon, 19 May 2025 23:03:22 -0700 (PDT) Received: from kuoka.. ([178.197.223.125]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6005ac33a88sm6698710a12.51.2025.05.19.23.03.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 May 2025 23:03:21 -0700 (PDT) From: Krzysztof Kozlowski To: Robert Foss , Todor Tomov , "Bryan O'Donoghue" , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Krzysztof Kozlowski Subject: [PATCH v2 3/3] media: qcom: camss: csiphy: Stop spamming logs with version Date: Tue, 20 May 2025 08:03:13 +0200 Message-ID: <20250520060310.7543-6-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.45.2 In-Reply-To: <20250520060310.7543-4-krzysztof.kozlowski@linaro.org> References: <20250520060310.7543-4-krzysztof.kozlowski@linaro.org> Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=4539; i=krzysztof.kozlowski@linaro.org; h=from:subject; bh=s5ZFlWa2MpdvYNmAn19sW48vqvwbzszHzHufPZ8IlvE=; b=owEBbQKS/ZANAwAKAcE3ZuaGi4PXAcsmYgBoLBsjSlBwNYTLmWtVGQ9R3zgaEbN1PnfIYX9vt xKiK0S8372JAjMEAAEKAB0WIQTd0mIoPREbIztuuKjBN2bmhouD1wUCaCwbIwAKCRDBN2bmhouD 1wnSD/92zfJ7ofRfnAh3nDOWREUebPj6OL6stRO4TTMWC+ve0bwjvTL2xJclxfPP/vSNOweYBJR Qj4tn/LR1At9cViTF5wlCsYntFeAA0GK3CkTUDjhiR8BEgXX2W69+4kk0zyHvpaczvgfSh0+XJd K6s3c3e++0Lhvxl09Z/uW44CSY6i43yxsr5UKeESV99YBMR431H0ejF1C9PeVtFflUFMI7M1H09 TJ3KvM5A58DCLAAyvVNcJDewWD8SNG1kOJYwJFJl/LfZ6O+JtYLPjZJ1qPqUm+a61XBot0YyvU1 BpmPRdQqqZytWC8vU2aqzHApJmvgO6L8S9Ei8J84ClZiZiUF9SNW8ksCCRMGJhMrry9dmUXGU1s hVawP8+q6gngjmuvvSXo7kjdozH76XnDS4uiMPvmo0c6roBywe4YhGTZZgxjnEbHY6te93tyuCl dD3tMa/1w8V6mYlkOSjfgXgMhVlwmkkJKRH6Dc9cJGpGBpCnJ0+PRWUC+OStUN7tkE8ocHlsFmZ w9fN10FvAjaVLlLe8cYvJYSZ03u+knWXhZj1mVYw+0/yP5hu3hY5rdRI1DV1HmfJDaubNUtUh0P LLPdYFpdXCgnYdc8eJ02wZb8/mCOWHQwaRIMJAVuxqZ9Dl2pPJra3sMnjaB1jNsmSna9XM5eTHL 1xpRnqEA9GityVQ== X-Developer-Key: i=krzysztof.kozlowski@linaro.org; a=openpgp; fpr=9BD07E0E0C51F8D59677B7541B93437D3B41629B Camss drivers spam kernel dmesg with 64 useless messages during boot: qcom-camss acb7000.isp: VFE:1 HW Version = 3.0.2 qcom-camss acb7000.isp: VFE:2 HW Version = 2.4.0 All of these messages are the same, so it makes no sense to print same information 32 times. The driver does not use read version at all, so if it was needed for any real debugging purpose it would be provided via debugfs interface. However even then printing this is pointless, because version of hardware block is deducible from the compatible. Fix the code to adhere to Linux kernel coding style: being silent on success. For the same reasons this should not be even dbg message (see driver development debug guide: "In almost all cases the debug statements shouldn't be upstreamed"). Signed-off-by: Krzysztof Kozlowski --- Changes in v2: 1. Extend commit msg, because apparently coding style and debug guide has to be reminded in commit msg, otherwise rules don't apply. --- .../qcom/camss/camss-csiphy-2ph-1-0.c | 10 --------- .../qcom/camss/camss-csiphy-3ph-1-0.c | 22 ------------------- .../media/platform/qcom/camss/camss-csiphy.c | 2 -- 3 files changed, 34 deletions(-) diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-2ph-1-0.c b/drivers/media/platform/qcom/camss/camss-csiphy-2ph-1-0.c index 9d67e7fa6366..09d3b21e222f 100644 --- a/drivers/media/platform/qcom/camss/camss-csiphy-2ph-1-0.c +++ b/drivers/media/platform/qcom/camss/camss-csiphy-2ph-1-0.c @@ -40,15 +40,6 @@ static u8 csiphy_get_lane_mask(struct csiphy_lanes_cfg *lane_cfg) return lane_mask; } -static void csiphy_hw_version_read(struct csiphy_device *csiphy, - struct device *dev) -{ - u8 hw_version = readl_relaxed(csiphy->base + - CAMSS_CSI_PHY_HW_VERSION); - - dev_dbg(dev, "CSIPHY HW Version = 0x%02x\n", hw_version); -} - /* * csiphy_reset - Perform software reset on CSIPHY module * @csiphy: CSIPHY device @@ -187,7 +178,6 @@ static int csiphy_init(struct csiphy_device *csiphy) const struct csiphy_hw_ops csiphy_ops_2ph_1_0 = { .get_lane_mask = csiphy_get_lane_mask, - .hw_version_read = csiphy_hw_version_read, .reset = csiphy_reset, .lanes_enable = csiphy_lanes_enable, .lanes_disable = csiphy_lanes_disable, diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c index f732a76de93e..bc605931278b 100644 --- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c +++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c @@ -541,27 +541,6 @@ csiphy_lane_regs lane_regs_x1e80100[] = { {0x0C64, 0x7F, 0x00, CSIPHY_SKEW_CAL}, }; -static void csiphy_hw_version_read(struct csiphy_device *csiphy, - struct device *dev) -{ - struct csiphy_device_regs *regs = csiphy->regs; - u32 hw_version; - - writel(CSIPHY_3PH_CMN_CSI_COMMON_CTRL6_SHOW_REV_ID, csiphy->base + - CSIPHY_3PH_CMN_CSI_COMMON_CTRLn(regs->offset, 6)); - - hw_version = readl_relaxed(csiphy->base + - CSIPHY_3PH_CMN_CSI_COMMON_STATUSn(regs->offset, 12)); - hw_version |= readl_relaxed(csiphy->base + - CSIPHY_3PH_CMN_CSI_COMMON_STATUSn(regs->offset, 13)) << 8; - hw_version |= readl_relaxed(csiphy->base + - CSIPHY_3PH_CMN_CSI_COMMON_STATUSn(regs->offset, 14)) << 16; - hw_version |= readl_relaxed(csiphy->base + - CSIPHY_3PH_CMN_CSI_COMMON_STATUSn(regs->offset, 15)) << 24; - - dev_dbg(dev, "CSIPHY 3PH HW Version = 0x%08x\n", hw_version); -} - /* * csiphy_reset - Perform software reset on CSIPHY module * @csiphy: CSIPHY device @@ -858,7 +837,6 @@ static int csiphy_init(struct csiphy_device *csiphy) const struct csiphy_hw_ops csiphy_ops_3ph_1_0 = { .get_lane_mask = csiphy_get_lane_mask, - .hw_version_read = csiphy_hw_version_read, .reset = csiphy_reset, .lanes_enable = csiphy_lanes_enable, .lanes_disable = csiphy_lanes_disable, diff --git a/drivers/media/platform/qcom/camss/camss-csiphy.c b/drivers/media/platform/qcom/camss/camss-csiphy.c index c622efcc92ff..111c3a52a6d1 100644 --- a/drivers/media/platform/qcom/camss/camss-csiphy.c +++ b/drivers/media/platform/qcom/camss/camss-csiphy.c @@ -243,8 +243,6 @@ static int csiphy_set_power(struct v4l2_subdev *sd, int on) enable_irq(csiphy->irq); csiphy->res->hw_ops->reset(csiphy); - - csiphy->res->hw_ops->hw_version_read(csiphy, dev); } else { disable_irq(csiphy->irq);