Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp4324870pxk; Tue, 29 Sep 2020 23:07:09 -0700 (PDT) X-Google-Smtp-Source: ABdhPJym+wNdCxPAnl6rf8CaIjMEZa1KnjRgVz7DqKBNsc2GknzP6qXDbV9G42CV6Lva0qNiJMGO X-Received: by 2002:a17:906:1b15:: with SMTP id o21mr1165873ejg.377.1601446029755; Tue, 29 Sep 2020 23:07:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1601446029; cv=none; d=google.com; s=arc-20160816; b=rE1ab90+E5H35CB3GycN5u+hB3l+3ewqa48NX/Ewa8xF8QjP9VCPHGSHP3W6vo/JPK Bn4ae+lzm5pwetfjkyTtfVWNk6kpLG+86OtjMZrP/7e8SkCw3QM1CZWMi1mlermH2BfK ES0OYYOA/szhLMosEBb5C2AWg/kWlkKSk53KVOX03LeJ9ucrlmnv2ikEbxZ7XP3mSWDM LFvVmeQykQmCastlEqjYWJo8KMJRlKIowG/FRzacm4BC6riZ0PgxBZZ2gOlyski4TVmu Of3RIduzHZedPdvFFW5Rl3dGn1IDe9AaGS5AtvLjUZS5qstuyhhEaBSJiR1WWmMK7zSp 4iwg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from; bh=wMdvqjOthuEtAdz6/8CSm79vaHeQ2WmL9tk1SI96qO4=; b=Rw12Bfkb0RsoDgGn0ZilSM42vV5W18nFKmjWdluS3AE0Ppdh97CIpZdxPQoWVs9KAf jL8lWFV/eS2yL7S+XRE3Kp6By2/pk0PczAmLHaGWHpfPesRHPnMEX5VbYRpVKswCZNdK Eh3MGQH6uI8JU4ASCRL5J26Z+jP7sGMION27EMmGbYFKVKPbslz9RbJmBjz6vUjK1t6g xTXH7ebiTqWAnuGgnVFWag4k9UBP10pUCGk/3e9V8WwauGFq/AKnpAPMlsCo1OmOKguT qIW8p8RLd/sUbTt9VA04tZtzDwuv9kwGhuzJGU9OSYcv1Z7VQR/5IiOSbejwNTTqxfWc cztg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p7si403836edy.43.2020.09.29.23.06.46; Tue, 29 Sep 2020 23:07:09 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725860AbgI3GFp (ORCPT + 99 others); Wed, 30 Sep 2020 02:05:45 -0400 Received: from alexa-out.qualcomm.com ([129.46.98.28]:4091 "EHLO alexa-out.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725306AbgI3GFp (ORCPT ); Wed, 30 Sep 2020 02:05:45 -0400 Received: from ironmsg08-lv.qualcomm.com ([10.47.202.152]) by alexa-out.qualcomm.com with ESMTP; 29 Sep 2020 23:05:44 -0700 X-QCInternal: smtphost Received: from ironmsg01-blr.qualcomm.com ([10.86.208.130]) by ironmsg08-lv.qualcomm.com with ESMTP/TLS/AES256-SHA; 29 Sep 2020 23:05:42 -0700 X-QCInternal: smtphost Received: from parashar-linux.qualcomm.com ([10.206.13.63]) by ironmsg01-blr.qualcomm.com with ESMTP; 30 Sep 2020 11:35:29 +0530 Received: by parashar-linux.qualcomm.com (Postfix, from userid 2363307) id 03CE1213D0; Wed, 30 Sep 2020 11:35:27 +0530 (IST) From: Paras Sharma To: Greg Kroah-Hartman Cc: Jiri Slaby , linux-arm-msm@vger.kernel.org, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, akashast@codeaurora.org, Paras Sharma Subject: [PATCH V6] serial: qcom_geni_serial: To correct QUP Version detection logic Date: Wed, 30 Sep 2020 11:35:26 +0530 Message-Id: <1601445926-23673-1-git-send-email-parashar@codeaurora.org> X-Mailer: git-send-email 2.7.4 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org For QUP IP versions 2.5 and above the oversampling rate is halved from 32 to 16. Commit ce734600545f ("tty: serial: qcom_geni_serial: Update the oversampling rate") is pushed to handle this scenario. But the existing logic is failing to classify QUP Version 3.0 into the correct group ( 2.5 and above). As result Serial Engine clocks are not configured properly for baud rate and garbage data is sampled to FIFOs from the line. So, fix the logic to detect QUP with versions 2.5 and above. Fixes: ce734600545f ("tty: serial: qcom_geni_serial: Update the oversampling rate") Signed-off-by: Paras Sharma --- Changes in V6: Removed newline unneeded change Changes in V5: Moved QUP_SE_VERSION_2_5 to common header file qcom-geni-se.h Changes in V4: Created a new #define QUP_SE_VERSION_2_5 for Qup serial engine having version 2.5 Changes in V3: Replaced the condition for detecting Qup version(2.5 or greater) with value 0x20050000 Changes in V2: Changed subject line and logic for checking Qup version drivers/tty/serial/qcom_geni_serial.c | 2 +- include/linux/qcom-geni-se.h | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/tty/serial/qcom_geni_serial.c b/drivers/tty/serial/qcom_geni_serial.c index f0b1b47..464ffc9 100644 --- a/drivers/tty/serial/qcom_geni_serial.c +++ b/drivers/tty/serial/qcom_geni_serial.c @@ -1000,7 +1000,7 @@ static void qcom_geni_serial_set_termios(struct uart_port *uport, sampling_rate = UART_OVERSAMPLING; /* Sampling rate is halved for IP versions >= 2.5 */ ver = geni_se_get_qup_hw_version(&port->se); - if (GENI_SE_VERSION_MAJOR(ver) >= 2 && GENI_SE_VERSION_MINOR(ver) >= 5) + if (ver >= QUP_SE_VERSION_2_5) sampling_rate /= 2; clk_rate = get_clk_div_rate(baud, sampling_rate, &clk_div); diff --git a/include/linux/qcom-geni-se.h b/include/linux/qcom-geni-se.h index 8f385fb..1c31f26 100644 --- a/include/linux/qcom-geni-se.h +++ b/include/linux/qcom-geni-se.h @@ -248,6 +248,9 @@ struct geni_se { #define GENI_SE_VERSION_MINOR(ver) ((ver & HW_VER_MINOR_MASK) >> HW_VER_MINOR_SHFT) #define GENI_SE_VERSION_STEP(ver) (ver & HW_VER_STEP_MASK) +/* QUP SE VERSION value for major number 2 and minor number 5 */ +#define QUP_SE_VERSION_2_5 0x20050000 + /* * Define bandwidth thresholds that cause the underlying Core 2X interconnect * clock to run at the named frequency. These baseline values are recommended -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation