From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (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 CD0B62185A3 for ; Thu, 5 Dec 2024 12:28:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733401711; cv=none; b=QR5l/CFdnbsmzBjep18FU6qDWtmVkQdvs7Ir0R4MKt4YLdtWdzBXYsUuyjSb1KdrkLahSLRoB2t6Rb3PhznAE+LYYbqj0v67ryWgbmsNKMm8OxrZWoo0BY/FJ54eEZw1zJ4ns6b7p+xYnDRt5ao36foYW8L6WX/EXg3eE4/5l0I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733401711; c=relaxed/simple; bh=18bpV5A/4haIrWRQJEDM74qxWf1ybq3kSy/rlI4CMR4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=HjFC35AMl6suNyf1gq83Fn+KmgJwJW0rP8FQ/Ws5/vjObsXeeO61ymNfjTCrpDJ6AZUlHIXdGVtg9Dtrmljm9wm8x0hsAFL/OF8ovDZDa7Fm9eGUv3Z2J/0VJr8n/GC2j1D+AApa4jbz93t6CrVOVBZJOYeRlwstI889Oz3jmnQ= 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=AstnWdgY; arc=none smtp.client-ip=209.85.128.44 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="AstnWdgY" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-43494a20379so8764415e9.0 for ; Thu, 05 Dec 2024 04:28:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1733401708; x=1734006508; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=0/z0Y/or6wVIG3eV60xxWRF95SZcxwisFQI5iGbciCs=; b=AstnWdgYcF9rIfZ3krtwW4N0xjeEtGWuj0/s1/pcO4jloTNUI9BNPUo1MsaDCkRDFZ x+kofT7OIEWH6OE4wtjfPDdwy6vaW1ahRVdpyITI0DDMFm3BUmUys67l0B1piUismqBD RAJ1+uh56N+YE6+NdIftuokYVEg8qK9sBGTTfaNcwjBaJ5QBs4iJGKuMLB91fiw+LJI7 o2PM4LWC4NB1XUrFEvcUX3JpeOVh8+se3FG+bHc1XYonNTfKOSnVPbbmwfWTRnzzZu/q OaYZOhmv9xrauN8o+6gRNJCxhNMQgJqHVejoGJp5Sgq1ZnksEA/AiMJ4QHdY6YaaQwQz pPFg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1733401708; x=1734006508; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=0/z0Y/or6wVIG3eV60xxWRF95SZcxwisFQI5iGbciCs=; b=D6Nn8ls+57J4dxpQ75/VJa8NDCYN3BE61shCxtCUxzKg3HSnJag7dUBv2oe2MO0gCv PHxSfQcwDxzEqoJ1QIxyVbyk6BnKvJtb/kftt0rCuNOWKvu+zaFMogyGMMKIgWBayBWA TKYlnlcwxPw4Tz68wrT0UOoV5O2U2qoB25PYxCZ3Qc2vnkR0CGLAG2jlw93jNB8OWk+v X8R8f2nXDuIvRPvMQsMI6q63WS8kOQvzJW4M18IlN6HBQav6WealwdCU1eHkTYw/EhPo IRlgjQcrZWTQfIfgFgMBr+Dp7D+qd+oq6gj5hY/8tq1EgFIUQSj+AyxKraIaF5ka70Of j5mA== X-Forwarded-Encrypted: i=1; AJvYcCWxamnplNLb0PpPsVFLb1jTrAQN3Ih8EhRxEpUIcVzt3Y5th9bLfzi7tiUiTMvKUf1FNNvQEDagxqjLNzc=@vger.kernel.org X-Gm-Message-State: AOJu0YwlO6wGmittUhEUze0fkJwNwYnAtlqqqnPujB5ED6ABtSn+skSt wNsjuUpUQvX0XqeeQnF0ZqTubhUI5ME8zna4rzkZa03CEWLFhVyNqm1lO5tjyvA= X-Gm-Gg: ASbGnctS0WFO3VeMCWT86Lxo3rGAap4jcjFDoQhbn1R6gzmMRUbUB47HBXjpY0F3SQI r4f7px2c7b7K8La44MzFDeZFjy19nrPc09ecfQU3BzzHN0faf62QJW25+Ohv8OVaXFYUPU9cvVM nraQ54n65T3dVWm5/2RuNMl/+FYwgZOe9TYp/r09HBEwzPRYkSW1WpH2xzWe657CJZxJ9X++HvW SdWX5++J1d3aRIRsv92H0RLi0OH4ZlFpJYBnjuqNKqKyxbNumfa2VWXxVwpCNQ= X-Google-Smtp-Source: AGHT+IFhSy7QZi+I3R64sZovAcTtAbg/HjBABy5kQ0+6tHCowzlGyLUtEVz4AzlfM38dgl84SsqQ3A== X-Received: by 2002:a05:600c:4f12:b0:434:a962:2a8c with SMTP id 5b1f17b1804b1-434d0a03abfmr84962955e9.22.1733401708213; Thu, 05 Dec 2024 04:28:28 -0800 (PST) Received: from [192.168.0.40] ([176.61.106.227]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-434da0d2698sm22267365e9.8.2024.12.05.04.28.27 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 05 Dec 2024 04:28:27 -0800 (PST) Message-ID: <2afca6ca-10eb-43b3-8730-386d6ca84b60@linaro.org> Date: Thu, 5 Dec 2024 12:28:26 +0000 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2] dmaengine: qcom: bam_dma: Avoid writing unavailable register To: Md Sadre Alam , vkoul@kernel.org, martin.petersen@oracle.com, kees@kernel.org, av2082000@gmail.com, fenghua.yu@intel.com, linux-arm-msm@vger.kernel.org, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org Cc: quic_varada@quicinc.com, quic_srichara@quicinc.com References: <20241205120016.948960-1-quic_mdalam@quicinc.com> Content-Language: en-US From: Bryan O'Donoghue In-Reply-To: <20241205120016.948960-1-quic_mdalam@quicinc.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 05/12/2024 12:00, Md Sadre Alam wrote: The commit log: > Avoid writing unavailable register in BAM-Lite mode. > BAM_DESC_CNT_TRSHLD register is unavailable in BAM-Lite > mode. Its only available in BAM-NDP mode. So avoid writing and the action taken in the code: > + if (bdev->bam_revision >= BAM_LITE && bdev->bam_revision < BAM_NDP) > + writel_relaxed(DEFAULT_CNT_THRSHLD, Really don't match up. You've said in your commit log BAM_DESC_CNT_TRSHLD is unavailable to the LITE module but, then you say if (bam_revision >= BAM_LITE...) How can checking if the revision == BAM_LITE match up with the stated objective in your commit log => _not_ writing to DEFAULT_CNT_THRSHLD in lite mode ... ? --- bod