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;
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
To:     Caleb Connolly <caleb@connolly.tech>
Cc:     "Martin K . Petersen" <martin.petersen@oracle.com>,
        jaegeuk@kernel.org, alim.akhtar@samsung.com, beanhuo@micron.com,
        avri.altman@wdc.com, asutoshd@codeaurora.org, cang@codeaurora.org,
        ejb@linux.ibm.com, linux-kernel@vger.kernel.org,
        linux-scsi@vger.kernel.org, stanley.chu@mediatek.com
Subject: Re: v3: scsi: ufshcd: use a macro for UFS versions
Date:   Thu, 18 Mar 2021 23:46:33 -0400
Message-Id: <161612513552.25210.14724545372813795454.b4-ty@oracle.com>
In-Reply-To: <20210310153215.371227-1-caleb@connolly.tech>
References: <20210310153215.371227-1-caleb@connolly.tech>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from ca-mkp.mkp.ca.oracle.com (138.3.200.9) by SJ0PR05CA0182.namprd05.prod.outlook.com (2603:10b6:a03:330::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3977.9 via Frontend Transport; Fri, 19 Mar 2021 03:46:51 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 74d9c33a-0118-4239-8e7f-08d8ea89a1f7
X-MS-TrafficTypeDiagnostic: PH0PR10MB4616:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <PH0PR10MB46160F2F7B9C9EEF5D16DFC08E689@PH0PR10MB4616.namprd10.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:7219;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: a4fnXl3pzisA390hg6HSpSRnqihuTuFFMYpE8QB04LRctud9TVUPApBNIlq1BgSyLwHBjS7I4PPjxU4ifjm+IXvXsUCWEhbloohga3NtDymiP6dGvLnpZfQW6/uGt+1qhVB2WyZ0h6YG/wiwgZ7H4uMkZgZdqfGZQOWtjAcgthW6uA7CTZ5QLGCP/fRW1F+bAzZkB+ExwNsZm5Yf+ZJf+iagByVMnH9y/BPiMBLYiwmm01hOiPD4kEbiblQIqCFguVFSHVWtrDl+VO/4H2n1EU8D2uKH1dRBJ6M+9VuL+MBcbX7CUypTVphInycMb24XhlXb0LSD5L2/Dt0tMFDw4DLywbSzrDPLVhf74Ps7WGocaBeNq6OI3qhZ22P420Ehdp9n36AP46ILtJVcapeGb0JZQP2757BBLZBCTiXRCPu/6XiuhJAvUSvjB1hg3yb6S7RO6ZfftcpyeY6+dwjhiQ89Kdde31FDVltI8n1NXFshog31CqNLMR8RHlMTMBTdWRYXfRppymnKxH/XxUnshUtIiFnV/lmhRrpTA5OMz1gPGqH3nHr2df5blU6DTeAVybOEEtNGwryXPjdSQsmsTVW+sJTRJRD47/2w645Y9ZNiyq8jXow+pw3G7qqcNot5Tq2MbN+1+eIi0dSZtvmDU6d0sXSr2nZmwdBJYARg8MRmdUNmQngQTT7z9yVVu/jo+y/TshaLy45vsDdz4gosS78h//h+dF8SxaLTiD9/His=
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR10MB4759.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(376002)(136003)(346002)(39860400002)(366004)(396003)(966005)(316002)(4326008)(478600001)(4744005)(6666004)(66556008)(8676002)(103116003)(38100700001)(8936002)(186003)(6916009)(36756003)(6486002)(16526019)(7416002)(26005)(86362001)(5660300002)(956004)(2616005)(66946007)(52116002)(2906002)(66476007)(7696005);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?SGhaSUNFSFVaQS9WL3lFMDJqcWFxSnJZMFgzOUkrckdlb2tTRmFHVytHSGV0?=
 =?utf-8?B?bEZGUUdaR291bEZHdktjbDgvMGhGU2RqVWt3dWlGcFhZUHJQNHBGMkVCcUF5?=
 =?utf-8?B?Zi9GbnZXbllUZVJkV1ZTZDRLZEFRM1FCdWFIdXdlVkFhcWRuZTY1c0dPNHZ1?=
 =?utf-8?B?d09vQ2FYNzVoaGRzN1NtLzUyck1nQkJFaWJxQWc4aXVaZC9PVGhlemhkZXkv?=
 =?utf-8?B?R2xZTGlQVWxSajB1N2p2NlRJM3FuUEdJcGpRckNIL0JIOUNFZGNMZXRBdHla?=
 =?utf-8?B?THlPMHhES0c4YklXZ2NMV3hnamlGUStxTTdIYW5hRjFZU2FRcDJoYnNQNEw0?=
 =?utf-8?B?NUtOWmk4aUJ1K0hvYTg3ZWZDV2NBMUVGRzF1YjlKak1uaHJvVnp0SFR1TjRN?=
 =?utf-8?B?d2xORnZPK3puNXJRM0ZPV2ZBRXViT09EQWxZSW0zVkhZN3lBeHRRS1BEb2dk?=
 =?utf-8?B?cHZQWGZOdDA5ZEhBYXNEZnBmaGJJMkZFOU1RalRHc3NRT3NsM2h6Zy8xWmwz?=
 =?utf-8?B?VmNnR1AwazU4TDBWUTVJUktLbjNQQmVUcGRHalpNRzRhV0ZDU0xxSzMyQ0xM?=
 =?utf-8?B?dXpvTCs5MW9rcVdCekI5UlZpc3ZpdWg3OVlMTjlZM2NvV2RDRU10QlIxNHUy?=
 =?utf-8?B?aEM4TFpYbWNHWlladzVhTSt0QkU2VE9YNWtWTW9CTHI0TFZKRCtKbmp6Q1pJ?=
 =?utf-8?B?UzNyUlZTdTAzayttcGZDUlRjdmZ0VFZpazc4Y00rY3k0NHVBTWRiV1QraEVo?=
 =?utf-8?B?TWR1d2N2clZzZk04QTh2aytxUnJRUDJFZGlEbElORG1YVUQvb0FiTnl1RVNE?=
 =?utf-8?B?Sm1mc0xFSHNFT0ozeGVWTlJJVWJNc0dvNzJKZTcrWll0M1FFdUhweDFQTi9u?=
 =?utf-8?B?UEtkdjJ2ZUhvMTcrMnIzdU54Rkl6dVlkL2tlY0xWWHI3WFRlTEVhV1VCK3Qr?=
 =?utf-8?B?WUxZTjhsM21RUDZJWGRHcEFiUFFYdzI1S1JjUkRQWUxmY2xsU3Znb2VlZ0My?=
 =?utf-8?B?VUVjS2pOKzY5SmdRMHVoWWVXR3AxSVlEbGo3SU5qWEx3R1dERVlyQmtQbEZq?=
 =?utf-8?B?ay9sNll6cmxBRDNoTGdJMFVabnJjbnp1NUUwbmpWRk5ZZXdGL0swSEhNSkN0?=
 =?utf-8?B?YlZpY1dKS0tZM0VTc2w0THFmWlp3QjNSbDB3RlFRMUpkZ1JSY0Fqc2Fjdy9l?=
 =?utf-8?B?dnM5ZklBMUlCZkdTUTBENUFpLys2WEQvK3QxYmZTUjZQeXRxdTZUR0x2VDJh?=
 =?utf-8?B?bmN6cGFjckhjdHhLTksyRkZSN0JodmJyQlVUb3NjZmdOcG5KeGRpUnFrVlpH?=
 =?utf-8?B?NnVBb1FOb0QwelNQYlBOcTRMQVZZemdTOWYyUlJHaTRpNVkwL3Z3K3lQaDR6?=
 =?utf-8?B?K0hXR3Z2WHFXZS9GaENBRkR5YWdGUjNKQ1ZreTc5NUxYVDBNbktqTlVXS2Q0?=
 =?utf-8?B?eUoyb0w1ZTQzL1RzTnIvb2lBbDVtWURkUmppWUlzbkhhZUlkYVpsUm1DZm02?=
 =?utf-8?B?YzlHam9qS1F6TkFSM2svNDF0NFFXMitVZmpmUDBpS2YvNXJJK09XcllFN2hk?=
 =?utf-8?B?U29iRmN6cGhSR2hMbXhyZnNhWHozT0RtbDd6bVZmWWRkL1hKVU83eWVsRUs0?=
 =?utf-8?B?N2xZWXIwYnB2SVpLVmR2Z1N0S0tTcXFZREkzZk9CejhXMGw1RC9kZk4zK3RZ?=
 =?utf-8?B?cWE3S0pTVFF0RWtNTFo1YXNPTGNsaUtVOVNLSG84U2lQdmN5S0JZR3pyMWRk?=
 =?utf-8?Q?aw67s4kYM5byBgIwd1K2IEH9WQYaioR518BP+kg?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 74d9c33a-0118-4239-8e7f-08d8ea89a1f7
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Mar 2021 03:46:52.3312
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: j/egrF4g8VYYxgBMdz+WSNknSiTcxKTkRpeScdsbVk70BKxfbOR41Vu5+ZsM5O6EExatOL6dBoADEjbU9VNVsAa6T+xsn9KQofupckVABqg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB4616
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9927 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 adultscore=0 malwarescore=0
 spamscore=0 phishscore=0 mlxlogscore=811 suspectscore=0 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2103190023
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9927 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 malwarescore=0
 adultscore=0 mlxscore=0 mlxlogscore=983 lowpriorityscore=0 phishscore=0
 bulkscore=0 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2009150000 definitions=main-2103190023
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, 10 Mar 2021 15:33:30 +0000, Caleb Connolly wrote:

> When using a device with UFS > 2.1 the error "invalid UFS version" is
> misleadingly printed. There was a patch for this almost a year
> ago to which this solution was suggested.
> 
> This series replaces the use of the growing UFSHCI_VERSION_xy macros with
> an inline function to encode a major and minor version into the scheme
> used on devices, that being:
> 
> [...]

Applied to 5.13/scsi-queue, thanks!

[1/3] scsi: ufshcd: use a function to calculate versions
      https://git.kernel.org/mkp/scsi/c/514288180178
[2/3] scsi: ufs: qcom: use ufshci_version function
      https://git.kernel.org/mkp/scsi/c/f065aca20a26
[3/3] scsi: ufshcd: remove version check
      https://git.kernel.org/mkp/scsi/c/4f5e51c0ebf0

-- 
Martin K. Petersen	Oracle Linux Engineering