From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 624B5C433F5 for ; Thu, 12 May 2022 16:06:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:CC:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kV4bKvn6vn6daLJYSz8BnItTNFgYCbA2FdlczvvtzEs=; b=aAauhTvgNiOwHN p08RRp6E3TGT3DchfjpuHRsWThwO/16gz+GGtJ1WsNmHLqFO0SIGHaNoOe9xPxKv+loVGJJ0CwCud vZZopRfCBkcd8BcfvuPxO9PMgnkxyI6d1B7jYI8uTXFzfnPU9ziWvbODONN4ZL+y2x3xVBbdUC+qh IHGHZoODQTo/pHWAYwlIZS0fGPCtSNkiHJdPfeULkYydb7Ny0fQanEcOR6dBoZwo627JE6ItEkcRS k+d0WDsTJCCj35t9E5urV/0KhX/kTDYdPUrNenPK5s91ZHA30hFQrQCu8c17UA/TOC4j62rj2I14f L7PlZRPjgSmlqlFkhAeA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1npBJF-00ClQn-CN; Thu, 12 May 2022 16:05:01 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1npBJB-00ClPk-K1 for linux-arm-kernel@lists.infradead.org; Thu, 12 May 2022 16:04:59 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1652371498; x=1683907498; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=w42JLM9Wyf6y8519BjvIA3XZsoPn1vSQOXp5i9tMXzc=; b=Jp5RRtLww5N64nd6w2sjIAma1yapQIWOKAtVyS26+UTa5wt2FByeBQ3E r+NOT2/8BPVxMOckD1hapvYPEazSwZv7zSlTyalYCk849mrQ3zvhrR5IB T0Aoq47hpLlRzFDyde51TY4/f6eV1xWW8ati3d/84yEzw4lzC70qf21a6 5rGUtUQAt/iMnBZYlFa6O2BMimwtNhB0ho5CB0ThDQMaX/qYxsZrXPmfY 97evf5I7xZLJTAHZygKRkLlykyXBN3ZaK7bnv/K4sL406IlZ8okC+iAXc 7Bi/RESE3eF8qF14KRTRq84k6OMe+IaM7dmD/2jIxG9w1GN1uCuPXM9Gv w==; X-IronPort-AV: E=Sophos;i="5.91,220,1647327600"; d="scan'208";a="155817031" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa4.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 12 May 2022 09:04:54 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.85.143) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Thu, 12 May 2022 09:04:52 -0700 Received: from NAM12-DM6-obe.outbound.protection.outlook.com (10.10.215.89) by email.microchip.com (10.10.87.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17 via Frontend Transport; Thu, 12 May 2022 09:04:52 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ihYcoG2AG/bhjDflgUYmBqk0V9n/Jyxy5VpZUJD/PjdbIGYu8ZfMYOBK4hkSA/Z/XHr8D87ViYRPHi7n89kbaIZvi07PohyRZa/prUoFNXOKxj+tmzTzhqfkTcI/L2mJM9f8qih9qAYKFZBW8G08E4nE+7YDtLv7f2oygtVKjAe2YmE6vh8efk7Eh/lh8vUzP6k9r4Z29/Nf14w639JzJRzBJhaVHaczz0untTfirZ6jaGYdtaecqX9uJhOvGLNcQYJMtn2e9OXLfbQqOqGEyOP+UmYQ83I1EX8iMEA52a+WY/R35FIzqvVexeb97ucXRvm4/C+EAkOORXlEdoI0rA== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=w42JLM9Wyf6y8519BjvIA3XZsoPn1vSQOXp5i9tMXzc=; b=bJbU/psDdC9FyqFjT77tb772cQkNBApnX85s73EifAxYCRD1dEd6TM+U+xsDyHX20VhPuXBCoQl8xu40wX3pulKLMuGboqLAyV3p7a7ctZf/ZNZ8Gij8s30CVSoGmNfgLMjJmPQClavEtFVG2/syvDlbrk8S5t8PxoU7T9/lK5MfTz4HBsQ8L9WGkAcWa3L/FKCorpAPQUXBTngFsqyjoPJexFbEtc7QlBhjd1WSN8olULVpUpJn7InE6QRqGUW92vyt2BsRMC3qpAmaPpKkev5jU/Zcilm4YBT2jcE7QznsGgUXi9Ztg8Ow0VDARj6SKs7wAqTbh5fX4nS2Ncpr+g== 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=w42JLM9Wyf6y8519BjvIA3XZsoPn1vSQOXp5i9tMXzc=; b=cIcFIdAHJi90uxcT3WgN6zJAjhTJE+DzJ1g1u/rFtXCYtG85+zoCNmjyc5/1YAUFlmzuDw7wblI196PXVW3k08YBKss+TKGUwstox0A2i3gOS8QwZLk3PADO8Zx6P2Sw9FKXLjfVFow8mEThkMJ7hRUOxGsZdfag7p+oPiBLMCE= Received: from CO1PR11MB4769.namprd11.prod.outlook.com (2603:10b6:303:95::10) by DM6PR11MB2635.namprd11.prod.outlook.com (2603:10b6:5:c5::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5227.21; Thu, 12 May 2022 16:04:48 +0000 Received: from CO1PR11MB4769.namprd11.prod.outlook.com ([fe80::e906:3e8d:4741:c0f0]) by CO1PR11MB4769.namprd11.prod.outlook.com ([fe80::e906:3e8d:4741:c0f0%6]) with mapi id 15.20.5250.014; Thu, 12 May 2022 16:04:48 +0000 From: To: , , , CC: , , Subject: Re: [PATCH 1/2] dt-bindings: microchip-otpc: document Microchip OTPC Thread-Topic: [PATCH 1/2] dt-bindings: microchip-otpc: document Microchip OTPC Thread-Index: AQHYZdBf287PvcEoQUmQslcb1o7LPQ== Date: Thu, 12 May 2022 16:04:48 +0000 Message-ID: <706eb7b4-07ce-ffd2-3366-5656ddba21d6@microchip.com> References: <20220510094457.4070764-1-claudiu.beznea@microchip.com> <20220510094457.4070764-2-claudiu.beznea@microchip.com> <75ce6291-77c7-c932-e8bb-a8bbae02431d@linaro.org> <59b47f0b-6201-3814-e17f-1435ce5c2dad@microchip.com> <6ce3c0b1-3f6a-24f6-bcf2-4d4125a97604@linaro.org> In-Reply-To: <6ce3c0b1-3f6a-24f6-bcf2-4d4125a97604@linaro.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.0 authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microchip.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 8963af8a-14db-41dd-3aa1-08da34312397 x-ms-traffictypediagnostic: DM6PR11MB2635:EE_ x-microsoft-antispam-prvs: x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 9DwqwwD56wA9iOPE1SxHBqimZE2cGMJh/umMouup0QJTQjPMP41CkUItjW2JAJXLbpxml+vQvgIVeELhVJmMHtirdKBNGVodevJDqY+SMu8BSN8M8mlbBeMz5cTQn1+S14ignELTxFp6hAcrZ/v3qSp2ubySj4sISnOLo5dlklFVO5ebQ3CXO5PNFUA3CbgyhLZ5HAyhJGOc63WqzPlVCa2xWihFCQjtVf9NLkz1MAALoknNKQtPO0plQgbdT22+WvLkgc4bIcHJSJleUDmpqxvLYNznJ1jQpA17tkQW1QDQ4Bvx8j1PTNnUdaiPgIomdr8h/RsLbUiyLryvhUq7mgTW7ndHnsbyCVqQp+NwuMMbzl4DxWb8OkBFTOkHmpYAXmqw17+lpIKF8vUMA8kCfjKsjgAIIovNfYdDi0VegGXoaubPyERAdXy4aXFfwL/cQqmDhqlku6GcAqS+kTnEiYTXediTEGZNKSGyk9dxp/WdY5AhtvrancACbVn640PbxicfeRExpvrwZ1vP0Bf4GQ7bvxsSnhOI968cQFsuna4rfsqJAwfr9y270JqLaD9m0U75yShtuCjZsQ6S8/bJvBU94ZieMPZi8YOap73IAmvMjuLP0bOrKyP2egxgiUlNlu82TpakDEeKDmezwuWyVQ13lzViRmWtql2rU6F2vGZ8LPS7d68VlxZXNFqn+NL8pMIua5z286ntWC+UhLlzLweFu/a1MWFldnv2D4ckWyw2J6iVqAMUwFBoSKIudFmp5stvvMAS5g5japitzLJ6DCO26SV0KCQEuIEOhBp4W+ZQ/ARrEN//ix5tNnqvdzNSQNC98J7Td+C0GVZQM1XmRIJXnWrlCPqqGI00O+qI2Sw= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CO1PR11MB4769.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(366004)(966005)(38100700002)(186003)(83380400001)(5660300002)(36756003)(31686004)(2616005)(8676002)(86362001)(8936002)(31696002)(4326008)(54906003)(110136005)(66446008)(64756008)(66946007)(76116006)(66556008)(66476007)(316002)(26005)(6506007)(508600001)(71200400001)(53546011)(122000001)(6512007)(2906002)(38070700005)(6486002)(45980500001)(43740500002); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?dkp6emFia3NBL2ZydzNldEpnTE8xSmFyMHUxSFpHR3R1MlJhZ255T1NmUDl2?= =?utf-8?B?eXRkT01aUEdLYUhiOXNsdUU2dWhhUHVUNE9NUUtUZzZYOFhMejZUa0cvUkc0?= =?utf-8?B?cVhtZmlrMFJWNzRGVkFTcUVoQmxnZUE3QVRyY0UwajUyQ2xEM1ZPL0FBMWgw?= =?utf-8?B?dGNjYjE5KzJhc2VLekU5NXhUN0dWdFUzUlJ6SjBQSlhycmdsR3VZNmhRTWNK?= =?utf-8?B?VWFtV0dwMFRYdlpOK21iWnU4c3RwSzIrRTJ3blk1RUVCVTlTVlo4ZDBPSi80?= =?utf-8?B?Sy9mYm5wTStmQzlwa2NqUXBEUGQ2b2FCZW1GeDlBUzZId3BJeFVWMkxMb3Rk?= =?utf-8?B?UGE4TzBnWk4xc0RtclY2M2E0QUNsVlpnazAxQmhUQ1cxbTF3MTJGaHFEcENj?= =?utf-8?B?cTFSOXNieDVzVXdrYzZqRGZGUW5lMVJUODgySmxRSjJGaWk1UjhjVEtCSjY0?= =?utf-8?B?K2RxQUI3UlpDL1p0TkkxVk5QQU1aelhjODV0ZlR2UHA4N2tDZll1THN6aFlD?= =?utf-8?B?TzVOUE8xYjA1OVVjQUlVMDMrNHRncW5BWHFoSzJmUk5NcDhUZDY5N3dzLzFm?= =?utf-8?B?K21NMW8wUU0wVFhRZC9hV2w1U2hnU3I5MG1vaHY1bUZ3enBhQkFtdndKc3ll?= =?utf-8?B?SFlsejFWckJkWStVcXhIRC90Rk56ZnlXL01lS3oyb2lzemtNWDZBN0dCekUr?= =?utf-8?B?SkpWT2lKTElUZlc5UUxMRGFmZkV0QXBXSjgxUjRueEhrbjhhR0RrMGJDVjdT?= =?utf-8?B?bS9INE1hYjlpY1NQWm56Nlgra0wvZXhXZmJsMnYxQUhmMG5jczd5VEdtaXFC?= =?utf-8?B?QnlobDYrRTMxOWRlb1MrN2ZRc25mTXIxS081eC9FNTkxb1lJSERyeGZjc0Jl?= =?utf-8?B?TmVodXd4SXZXZ3FDR0hkNzhxeUFabHV2WE90ekFVRnVRZmdCR1dGdzBOY3JQ?= =?utf-8?B?YnlPQ2hUbGhNR0c4U2g5MHQ5YUhoR2RLRlQwaVBKWFA3UU11VGFKTWp2WkEz?= =?utf-8?B?Q0QzaVBUYUV0dFJFOEJRQytQUm4yRmJ4Vkp6TUNvaGFYbE1MUEtXZTFLejdu?= =?utf-8?B?c3lhWmVHNEdFMG9wWUgrMmpRSVoyU01CZVBtWTdTNmwxZ01JUVBEa1BRSXFv?= =?utf-8?B?UkM5VERSVUpGM1B5MjJxaithSmZ5cEg1YkRBNkt4VzZJS3RhVm9XSWdiNWZZ?= =?utf-8?B?eFgrR1UyTzVLNERUNUp5b1RySjdYUUp5eWVMU3pNazYyZzZCRzF3UGhVUGpj?= =?utf-8?B?YmhLb25DMzRQM2orVFltZEtNcGE2MVhqR0ZmZWxLVG9VS1g5SExQeWhxczNR?= =?utf-8?B?Ny9TZG1OVlU3TGdLdWVoYXJKNkdRV1JpYjVyZGUrU21XTjNHZjhBUjY2NExv?= =?utf-8?B?ZFFRTWxtdDE5NlVTeFc5ZGFhbVZxMG9BeDQ4YnFDbGMyNENZaG05bU1nR01J?= =?utf-8?B?WTdRUXhkZ1B2WU9FQUJ4RGRFRzRxTkp2M2xCYjcxOVZUZXYwN0VFOFhUVFN4?= =?utf-8?B?eVJYRnhuZ3NOZGFtRXRQeUYwYUhqTUR1Tk9yc2oyQm1sRk5KSGlyUlVqeDhZ?= =?utf-8?B?elA1VUhsNUFRZExDNHAyR1IvTnEzWjhEeCtOS3pVdDBUZy9vV0ZoQjl2VVZY?= =?utf-8?B?ZDYrZTdOV2luNGVlcWZnSEUrczdkMTkreUtHSzlQeko4cWdLWmU5YVdhV0dj?= =?utf-8?B?Y3dCdTZhOW5aeVhHMVVQZDBPUFYvbm1nZFdYNFJyRThXUzlGYUUyYXFtcEVF?= =?utf-8?B?TzYxbHpPTG1jVWdRckx6S1VCTkZ6eGIyY1hmQ0RMbHBZUzA3YmV0NDJ2RXF4?= =?utf-8?B?THVXSWY0WkxlcVFSYjlHMzcwNWcydEtrUnRZVjRzamZVcEtWRkVVVGh6dzNJ?= =?utf-8?B?akYyTFNhVVQ1SW0wNUdLcGVpR3hnRXJ1elBTSWF1NnpZMkNSd2ZDblE2N0RC?= =?utf-8?B?U3ZwcTJ1KzM5SlFCcGNYbG5wakpUYm5HdTZLZWpEWll6cHhTeFNVOTRtaWlj?= =?utf-8?B?SEdocnV3M284aVBEdkxUOURmaTc0RjZ5UXg1TER5WkdnS1ZtcnFicytLaHZp?= =?utf-8?B?RDdsd2g5UlFmR0xrNkpFZUhIVXNiYWdwT1pDYTM3V3NQOFRsekF0ekg4a1oz?= =?utf-8?B?QTNKU1JHRmk3amRjQ09BQ3lxV1pUV04zejVncG1aaloyMEI1dXR3R3NvaHNl?= =?utf-8?B?U0xqaWpPSzhZd3lHdFA0ZG1XWmJNSjhPM2hPYlhrWGllSkVpclhxd0Z5WjFi?= =?utf-8?B?dWpkSnNWZzFJNkplYjFXK0FsclVVVGl3RzkzSlRsWUVmVUpTdHZaeTY1SDdm?= =?utf-8?B?QTNaL0R6NWJHZC9idE40UThpT0VaVndCeExRcmprenY3TDlrODRXZlF6SXIv?= =?utf-8?Q?ttn1zEDHcfb9noJM=3D?= Content-ID: <4E22DE5BF696BA4A8011B8E5176F39E6@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4769.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8963af8a-14db-41dd-3aa1-08da34312397 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 May 2022 16:04:48.0876 (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: yo9qlfdcLoY6PNVjXetXpNhRIXvxQBfoBJ8uWyT2t1mnZq0ekuQcYh7JTtqiDfZ+7UNEu/vzo1WaxKhLUmFbfFaen608WaT8/1BU8YpCYfA= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB2635 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220512_090457_787613_45970708 X-CRM114-Status: GOOD ( 25.83 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 12.05.2022 18:35, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 12/05/2022 17:31, Claudiu.Beznea@microchip.com wrote: > >>> >>> Macro is a nice idea if it can be stable. I understood that length of >>> packets depends on hardware, so this part could be stable. But what >>> about number of packets, so the OTP_PKT_SAMA7G5_TEMP_CALIB_LEN below? >> >> The OTP_PKT_SAMA7G5_TEMP_CALIB_LEN here is the length of thermal >> calibration packet. This length is fixed and will not be changed. >> >> After these 2 packets (provided by Microchip) user may further flash any >> number of packets and use them as they wish. >> >> Driver is in charge of scanning the NVMEM for the available packets and >> prepare a list with their IDs and their starting offsets in NVMEM memory >> such that when it receives a read request it will be able to decode the >> packet offset based on packet identifier. >> >> In case different number of packets are available in NVMEM for different >> kind of setups (boards) these could also be referenced in board specific DT >> using OTP_PKT() macro and with proper length (which will depend on what >> user flashed). >> >>> You wrote "Boot configuration packet may vary in length", so it could be >>> changed by Microchip? >> >> Yes, between chip revisions its length could be changed. > > Chip revisions like different board compatibles thus different > bindings/macro values? Not necessarily. It may happen that only ROM code to be updated (1st stage bootloader) end everything else on Linux side to be able to run as is. Or to just fix some bugs in different IPs. Things that will not necessarily need adding new compatibles for the new chip. And it may happen that new chip revisions to be populated on previous board revisions. > Chip revisions like different board compatibles thus different > *macro* values? If you're referring to the OTP_PKT_SAMA7G5_TEMP_CALIB_LEN macro, this is established that will remain fixed b/w revisions. This is the length of the 2nd packet in NVMEM (that is of interest for thermal management). Only the length of the 1st packet may change. And addressing the NVMEM with packet id based index should take care of temperature calibration NVMEM DT binding to work all the time. > If not, then maybe better skip the length out of > bindings and just provide the first macro. As far as I know the length is part of the way the NVMEM cells are described in DT: it needs the offset in memory (for the data to be retrieved) and its length. Thank you, Claudiu Beznea > > > > Best regards, > Krzysztof > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel