Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp3731289ybz; Mon, 27 Apr 2020 22:28:27 -0700 (PDT) X-Google-Smtp-Source: APiQypKR5X8jBK4OeOCDbsu0K5l9ayeviBzopk7R//x7d5FevQLPkM4SQr55v8vwLmP/NxE93rxU X-Received: by 2002:aa7:c5cf:: with SMTP id h15mr21889295eds.213.1588051707402; Mon, 27 Apr 2020 22:28:27 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1588051707; cv=pass; d=google.com; s=arc-20160816; b=XsCFF13czCYNpztpwvyf0P80VucUZ26CJvLlgDRC0d2H5k+++deM8THHcJsU4ji9X7 T8tmUGgCRZLUa+qnPXrMeghqiAXeSw1YTAbTrwEM92OkJSFbeS4hsYfI7BmAxRgZ0lX8 bA2ZH6hsldm/u5O08W5l/JtSl3Q3X9l6BcsYCWA1tqRF9yAk/aHWgtl/+yoJ5wSucCQb iH968qsZR514s3tYKdDkFxUGiwbrCCRZXTbZjopFgBPZrql1arXY26f/0nTuefwdDDs9 pcLCHZ8jXZ+Ur6Z+wQ4XcjxvDrsX796v4jH4Jmu1E/bIhAbdto8OOow+Y2eBPj9oW5Qu M7NA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :content-id:content-language:accept-language:in-reply-to:references :message-id:date:thread-index:thread-topic:subject:cc:to:from :dkim-signature:ironport-sdr:dkim-signature; bh=oVC18K9BtI/AKktrklZ3dHNsnTKEF2AEl7XyjguVkTY=; b=ym/G/wJ1xWss51GuVrYMNtxmhtbMKIre11UPAoxvJXhbqdzhI/tFIBgHytB5Aze8Va AITz5J3MVmQMcqGGhZoMV72dXStkC98TlTpGbO9qfx12hRCnetUHC+s7/Ju0PY9xFLJe 7nU0thD3b0K2mb4V9GHF3RaNlOSFZaAVt66SM9DXw3Jw4QUp7yKrLgALLnbZsQTzDvyj nm1EwFeSCIWTlohOYSDvC8PsFzSGtZYI0se3eqPU6na6gVh1/d1HYoBU6KTuFZzPFqR5 7sJ209gz319WXUj2PPMJELYnWEkX3vjCaVt5yOGJybFiKCL1F0QYXi4q1420hKTzgDxr 6thw== ARC-Authentication-Results: i=2; mx.google.com; dkim=fail header.i=@microchip.com header.s=mchp header.b=QCiTBeb9; dkim=pass header.i=@microchiptechnology.onmicrosoft.com header.s=selector2-microchiptechnology-onmicrosoft-com header.b=bExNrrj2; arc=pass (i=1 spf=pass spfdomain=microchip.com dkim=pass dkdomain=microchip.com dmarc=pass fromdomain=microchip.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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=microchip.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id d8si1015738edq.328.2020.04.27.22.28.03; Mon, 27 Apr 2020 22:28:27 -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; dkim=fail header.i=@microchip.com header.s=mchp header.b=QCiTBeb9; dkim=pass header.i=@microchiptechnology.onmicrosoft.com header.s=selector2-microchiptechnology-onmicrosoft-com header.b=bExNrrj2; arc=pass (i=1 spf=pass spfdomain=microchip.com dkim=pass dkdomain=microchip.com dmarc=pass fromdomain=microchip.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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=microchip.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726401AbgD1FZX (ORCPT + 99 others); Tue, 28 Apr 2020 01:25:23 -0400 Received: from esa4.microchip.iphmx.com ([68.232.154.123]:64884 "EHLO esa4.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726042AbgD1FZW (ORCPT ); Tue, 28 Apr 2020 01:25:22 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1588051522; x=1619587522; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=askazfUyw2e73qwx0VflbvWWRC8nIg8IV4RxJBVcztg=; b=QCiTBeb9GIIqGP4xrZpRkKX5f/7g2MUFw60VsVp2w2ovszjXXg8TxU7U KuKg74UEA2OG5Irn2wCa9K4WAz9B4UjS/B1/cmIFNskVAuVHrMNTP9uyK FCtIUAus13y1j733kILAnE6mkMZcWdvXVCqMnlZFreISV5cjvRcnTvYb/ yhQFAHV6oms+fG7s2wTYclkf4I4AaWRTcnXzrnEnnNJ29qq+g+TFSEGkF 7pNjEHfdBD53vvgthySVd1h0S9mx90BJneoigP5q+Vso+JRjAmDBBrz/L dK5eCoocIs7HoRt6hWbCIaX8D0sOcwPgk+9BeGEIC9h+q9FSnieiWtW+i w==; IronPort-SDR: b5j2Cbd28r94XAaeNHmYvq6Wwtx3bhlYV7VOOzr3N7yfzWb7Gg3MCPLwe2vXmpdWGZT93Vzls7 aSwAsFuouXFl6Ip4Sg5tJBfq65apnpnsY7+84sV2mxUZ4S3gIC7JXzRZCI2BgqjNGzHSPhq7tf +EOUlvZV2azAe3xjJ5IFKvdX5jLBZAxx6/dvNunL1hi+EPm4UUOHBrNK1WuofVgJ18slIyZAtB vd+cICm+sjZOIe3118StTq6i80reLXseSvDpH/XY320FOpLs3lDEXIoGHRXJ2AO4YOV5q/6CEq gYU= X-IronPort-AV: E=Sophos;i="5.73,326,1583218800"; d="scan'208";a="71726627" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa4.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 27 Apr 2020 22:25:21 -0700 Received: from chn-vm-ex02.mchp-main.com (10.10.87.72) by chn-vm-ex02.mchp-main.com (10.10.87.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Mon, 27 Apr 2020 22:25:19 -0700 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (10.10.215.89) by email.microchip.com (10.10.87.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5 via Frontend Transport; Mon, 27 Apr 2020 22:25:19 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SIwRds6AlAORSHX4oIBDesoqz2mKko2BDHKkGlesGhduZU3sF1w+mcFQfUZ0z1PFRl0NQ2Lk0GHAqphogN7X8KIcQp//hmN/3d3F3irng1OSJAd+Lr4Vpupus5CgcIum/ZuHdnuKcoaXopXRz5935Zd5Lf2xqkB+lntJXC5IAM0Vq6coIexND3g0D/8anY2zmeLBnRE50AELs4rdbPCiY1HQRTpgXnijrwsJAll27W9SUeX+mKh8pxFxh4jHfCVNdeQDyfZweQRbroswzHfrfAM7k3KVRrjvmdgwpH6s/YDObycwvkUjM4cxzx4Rl4LKRHyVPoRN3xvTsOVy+5IFpg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=oVC18K9BtI/AKktrklZ3dHNsnTKEF2AEl7XyjguVkTY=; b=Xjm+K6QHl3fTJmGNOsootW1fI9NsQPHQ7F3H3cjbOHiIAJuI+K9tMKJ0f2TY+uUezLX883vz9bcX8mc3F2fje9SphPOyL9VbpiViYiaqpNqUDP8HSKUwXCb+Wgn8szkKPk5wzHBOg30VU/IK/vt/1ZqnF29lLDRmaqxg1sSJPBfqXXrF6Cgw2uFdNJJxesWf85nM901Ol3ttoFioPlrvO82AQJn1UW4wxloEiQ1xyByWJ8Tr3Q4XdYU5dQXF5eH3FZ5AWmnCoVv3V93JRwon9feG69yCGMfFV0mzOx2GljYwHEP8igpicW6m0h7BCNt/oGRd81UA3hnbaQtOLj1gYg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microchip.com; dmarc=pass action=none header.from=microchip.com; dkim=pass header.d=microchip.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microchiptechnology.onmicrosoft.com; s=selector2-microchiptechnology-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=oVC18K9BtI/AKktrklZ3dHNsnTKEF2AEl7XyjguVkTY=; b=bExNrrj2RK7hYNSrgBXMlGl5hRV/7mY6+s59NsWlNdyoaphfUrsYRIgKPtHhad6mFst9aoWIb3y1GZYGlxxe8An9rxHZx4ERCUzHLvjuxIbM7H5HITwJT+O/UEngwSOGRnybAO10CniMcCQBvP92+dssap5g1xOrWcntjy98zDo= Received: from BY5PR11MB4419.namprd11.prod.outlook.com (2603:10b6:a03:1c8::13) by BY5PR11MB4388.namprd11.prod.outlook.com (2603:10b6:a03:1c9::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2937.22; Tue, 28 Apr 2020 05:25:17 +0000 Received: from BY5PR11MB4419.namprd11.prod.outlook.com ([fe80::d847:5d58:5325:c536]) by BY5PR11MB4419.namprd11.prod.outlook.com ([fe80::d847:5d58:5325:c536%7]) with mapi id 15.20.2937.023; Tue, 28 Apr 2020 05:25:17 +0000 From: To: CC: , , , , , , , , , , , , , Subject: Re: [PATCH v4 05/16] mtd: spi-nor: default to address width of 3 for configurable widths Thread-Topic: [PATCH v4 05/16] mtd: spi-nor: default to address width of 3 for configurable widths Thread-Index: AQHWHLkjDWuKSC8VwkCztQzj3fjN/6iNwPMAgABAZYA= Date: Tue, 28 Apr 2020 05:25:17 +0000 Message-ID: <2970660.P4alVuHkju@192.168.0.120> References: <20200424184410.8578-1-p.yadav@ti.com> <20200427172336.ihezwq3wn75m7k3l@yadavpratyush.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=Tudor.Ambarus@microchip.com; x-originating-ip: [94.177.32.156] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 095a901a-fb80-4d4a-3d5e-08d7eb34897b x-ms-traffictypediagnostic: BY5PR11MB4388: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-bypassexternaltag: True x-ms-oob-tlc-oobclassifiers: OLM:8882; x-forefront-prvs: 0387D64A71 x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BY5PR11MB4419.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(39860400002)(346002)(366004)(136003)(396003)(376002)(478600001)(6916009)(66476007)(7416002)(71200400001)(86362001)(66946007)(66556008)(66446008)(91956017)(64756008)(2906002)(8936002)(4326008)(8676002)(81156014)(54906003)(9686003)(76116006)(186003)(53546011)(6512007)(5660300002)(6506007)(6486002)(316002)(14286002)(26005)(39026012);DIR:OUT;SFP:1101; received-spf: None (protection.outlook.com: microchip.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: JO0JQRqy5u4tYzgOi569b/PRpYGR35Hf2hch0aNwWt0Xe7/unTfbELJBHM50TH1ZM4UkWb7Q0LFSICLuHhoyb0NdiAeoSEnve3gUct1iGueQ68s1vEgBzKsYhHEE3D5GjRQ9XGxnGkyvkTUTrcXBZpijpwcx29m8BUWdUuozUOVWETfh9NcG8GGjv1NJaDIDpUHgk3WwUlFpWMNhHtc0Clt0ppNaXbecOlSKwhoSuU2pbehMFMoWRz640oMaHCPmI0T4CgwNxnlznWoisPupubOtLngGeN290qi5C6EByM4CdPqlV2G2/iOD0WFSjE2dNvK3K5dSq4x/Zq43rSMzh935Kl1ExTkOfMGHz/4/MpEchbQ3o8JbrEUFr5P3qS8RLzKau6A9xFJ54qQSrEz25k5auQUPEjE1Xa0Bz7qUN7drq3Uad2b48ewDEdDju/6NPJv4uoLfzIfd5+PLX4UsZntAammObbny5n85SpjJcKc4XiHDW1Ov/CJBHFdoWQEj x-ms-exchange-antispam-messagedata: 9ETPCM5ft8La19ri868JeF8Vs3+D+IKrZWWBERiqEYVZzwCwnuJdQog7KGY3D1dI2nZ/or9aUiJUJJx8U4hqs2un2y9REkxMwYpl2hvyD3FWyhaI4QKEJIaNc8mEiVhrwnmZHr3xT5oGKFyz/Yp0WBt0Ctjgh1vcteq2T4XYO5HRmhkfovgKsUbHwG12Aypn55fG9oB9J/2aFldvZzK3YmRXhenndrAtGrdTfjhzPm1vPPtauXJr9jRqg+NmUESsWEmU89AlE8jckMYcVkWJSzgwgroUqIutApk2ghBbAMAYUPh51k+c+Ry+ZIHmSQ4bLKR2vNZ7XecYT9WAeS7wsNIKabZKPxPZ34MIj420eDT5u8NOi3KEgb5wdRDqfWnZKGsOT1Lij6X7NTfA010acrj6/I7IpWMBvXs5WCORDWt4qRTRQ5eLWpEb9jige8uBXJ6cjcKBi+HpdM+jZBMS6jBjt0Ehu5SdjJquSc+YfO57UuDQIAgyBRzILVGQGIC7FDin4zfePpW+BvC1eoUnuLr+Z+lSszeE+0EV7ZnWyH2BYW4C2eyLMBA3781u6YIF3KyMj4OhNk47ki1Bf2lt0cXHey+KNQT1Z4Mg1z4kt7WHWfAf6AwERlnqtZHm4RO0VOVa8Hb6YTIDYA4E5aOT0s3y4WKupvBz6/HiPGilXj5OJm5ebXXy4nbXu5N4wzOTlGK9iyWe4yyRCkvRyAeOCnNbaUn8ltxh37DT4Hd3te3a51zcWU5TJe/NBCBhgXJcTat05t36/EMZLZR6207hGt5DgGPuy/52AuaM62lxHkI= Content-Type: text/plain; charset="us-ascii" Content-ID: Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: 095a901a-fb80-4d4a-3d5e-08d7eb34897b X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Apr 2020 05:25:17.2177 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: is1jLxdD6o3cL5bz14IJTy4tikIHjTBNkM+MFjmX0CYSOOChgkwqdmDA3LeF/0H8L0kr6eF/CafXIbpsARFH5qD8YvioZ596LEsvRyzTqZc= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB4388 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday, April 28, 2020 4:34:46 AM EEST Yicong Yang wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know th= e > content is safe >=20 > Hi Pratyush, >=20 > On 2020/4/28 1:23, Pratyush Yadav wrote: > > Hi Yicong, > >=20 > > On 26/04/20 11:53AM, Yicong Yang wrote: > >> On 2020/4/25 2:43, Pratyush Yadav wrote: > >>> JESD216D.01 says that when the address width can be 3 or 4, it defaul= ts > >>> to 3 and enters 4-byte mode when given the appropriate command. So, w= hen > >>> we see a configurable width, default to 3 and let flash that default = to > >>> 4 change it in a post-bfpt fixup. > >>>=20 > >>> This fixes SMPT parsing for flashes with configurable address width. = If > >>> the SMPT descriptor advertises variable address width, we use > >>> nor->addr_width as the address width. But since it was not set to any > >>> value from the SFDP table, the read command uses an address width of = 0, > >>> resulting in an incorrect read being issued. > >>>=20 > >>> Signed-off-by: Pratyush Yadav > >>> --- > >>>=20 > >>> drivers/mtd/spi-nor/sfdp.c | 1 + > >>> 1 file changed, 1 insertion(+) > >>>=20 > >>> diff --git a/drivers/mtd/spi-nor/sfdp.c b/drivers/mtd/spi-nor/sfdp.c > >>> index f917631c8110..5cecc4ba2141 100644 > >>> --- a/drivers/mtd/spi-nor/sfdp.c > >>> +++ b/drivers/mtd/spi-nor/sfdp.c > >>> @@ -460,6 +460,7 @@ static int spi_nor_parse_bfpt(struct spi_nor *nor= , > >>>=20 > >>> /* Number of address bytes. */ > >>> switch (bfpt.dwords[BFPT_DWORD(1)] & BFPT_DWORD1_ADDRESS_BYTES_MA= SK) > >>> { > >>>=20 > >>> case BFPT_DWORD1_ADDRESS_BYTES_3_ONLY: > >>> + case BFPT_DWORD1_ADDRESS_BYTES_3_OR_4: > >>> nor->addr_width =3D 3; > >>> break; > >>=20 > >> Should we also assign address width to 3 in default condition. At leas= t > >> we should not leave it uninitialized here. > >=20 > > The default condition would be taken when this field is 3. The value 3 > > is reserved, and so no current device should use this value. That said, > > I don't see any downsides of doing so. If the value 3 means something > > else in later revisions of the standard, this code would need to change > > anyway. If not, we would use a relatively sane default for devices with > > a faulty BFPT. >=20 > The purpose is to set one possible value which may be used later in parsi= ng > smpt. In current driver, if we do nothing with the post-bfpt fixup, then > the width will be unset. Otherwise, maybe the width can also be set in > spi_nor_smpt_addr_width() >=20 > default: > + if (!nor->addr_width) > + nor->addr_width =3D 3; > return nor->addr_width; >=20 > But set when parsing bfpt seems more reasonable. Please don't. Any deviations from the standard should be addressed with fix= up=20 hooks. >=20 > > I haven't received any comments on my series so far. If end up having t= o Thanks for the patience, they are in my todo list, I will get soon to them. Cheers, ta