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 7DEE1C3271E for ; Mon, 8 Jul 2024 09:43:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=pPFgBOg0BwvjqhaIzOQzuUDIJVPwIY9J0BPpaCd2cV8=; b=Dgad6kX2XdGT2zQ3pxsYqF+mZt fS+IFQxzWa5BSSwufxa/NZ5/VhLPy+lzCKn1t2xQhokqCKriq+XsqKqcEEFIJBWsw7IPXEwX+NNCD gtTmTHtvWTeir4eDVEOklnCJei5KJW1jzddycJUg6EruXl2b+w2rITql2YJg7aiDBxfsyxNWPOZue zcZF/1lFginrDrtD2n+2yfQrfYPBCNCj9+dNRGigOulqKhuw0rcqRE0vK4QpOx2akWpSyFxGqiq1A ZIK0LxccO/7VEE55bWDCbKYa6idKNIitKrveFAdGSjeLF86ajg4axR7YMb2gLf0Tdez4MW31kRcG2 +qArGUIg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQktb-00000003LdW-3Vyp; Mon, 08 Jul 2024 09:42:55 +0000 Received: from mail-ed1-x52c.google.com ([2a00:1450:4864:20::52c]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQktL-00000003LXU-11ty; Mon, 08 Jul 2024 09:42:40 +0000 Received: by mail-ed1-x52c.google.com with SMTP id 4fb4d7f45d1cf-58bac81f3f9so4710510a12.2; Mon, 08 Jul 2024 02:42:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1720431757; x=1721036557; darn=lists.infradead.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=pPFgBOg0BwvjqhaIzOQzuUDIJVPwIY9J0BPpaCd2cV8=; b=dCBibxMf7kax7NoG6OQibkK6UaRfP0z/HQ4JXNma1Qmo3QItK8/aWn7T9BR/eT/i+D OdU8VRs0Q/bZx0DBpFNNwkCTcV+zyP7R6Z4kFQiP+JXIASw5OzX54gRi5l1QJ5o3zMLT RBs39fNXbIzSFSN62TMgKCji9T72QZ1O0G/ADkWdae4uEVLxtjoIab8gOQgWfvlr5Vjv 1Rbp4Z58ExksRW6/WkOzV3tgCHxVf2a5dScdS2qMeRrBhQZGstQs/BlbcE6cKd/G5cAI F61IiUyPDDMmVZ4ZsO+ZB3Sqhw5ZG4qU9A7rnZrIjhlOT3ePaaKgu4bJU9WeZY4PIqz9 jfhQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720431757; x=1721036557; 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=pPFgBOg0BwvjqhaIzOQzuUDIJVPwIY9J0BPpaCd2cV8=; b=MPCu6pBIHuaDkwYsUQqgfAsc/w8IuNnSJNx55/5GuOgIQOfSGquOHcQmBeFOg0TAPu afCT7cNgZ8gAaYP/ZbFT10+v0dT01t8eaZVhfCJ6ozW+lKCHjJMGFzob5pwQfVy+DYz7 TkoCEyVIPgIuu0o29BAEzcZyji51kB3QHxXGUz6NZmpwIMl2YA7mipIDkyvGcGdXTsHb 31bxKW3AvVZCVHfPbnY/v2NDwWe1OY/aLvvCdJ/YZLBv3PU92OzL18dQvGz296g9/a/+ t/m6+YJVXsSzU2Qic4FWHEPzTfyWbwI2f4s08NQcbIZpNCY0MCqN9TmPGOOdXBuFoy/i tPWw== X-Forwarded-Encrypted: i=1; AJvYcCUgS/OIH9WZALSeo7HZyGsi+dhdYANiIX4azRvszjPza4LWy9FVIkVajKrdpFGRdHBc2XOVqTZdXmUtAmY+6sXL9+nuequIQI5if1iKNRBStJDb3cEp2rsWc5tW8wHZLM4wwtUAA4T01leUBQfT+AT2d3DD X-Gm-Message-State: AOJu0YwRmthaTItes+gdvBXRfFuyadgfreRLL32t4WTA0GqhLOosTs5N 9viqS1IIbz42P7DsNVbHoTbSzbYiIJ9a74Rjwne/yPoTEJ8pJtqE X-Google-Smtp-Source: AGHT+IGJTVwfxRPLlqsy1KIoSb8jqnFfw/igQQIrxqMOb9EfhE4RP8yDb4xRHk0zohcC0359UuhRFw== X-Received: by 2002:a05:6402:2349:b0:57c:aac9:cd8 with SMTP id 4fb4d7f45d1cf-58e5a7e8b66mr7953030a12.8.1720431756473; Mon, 08 Jul 2024 02:42:36 -0700 (PDT) Received: from [192.168.26.149] (031011218106.poznan.vectranet.pl. [31.11.218.106]) by smtp.googlemail.com with ESMTPSA id 4fb4d7f45d1cf-5925eb01124sm1539068a12.32.2024.07.08.02.42.34 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 08 Jul 2024 02:42:35 -0700 (PDT) Message-ID: <329fefa1-5274-4f74-8d7a-aaf8d2a65189@gmail.com> Date: Mon, 8 Jul 2024 11:42:33 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] dt-bindings: nvmem: convert U-Boot env to a layout To: Miquel Raynal Cc: Srinivas Kandagatla , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Greg Kroah-Hartman , Michael Walle , devicetree@vger.kernel.org, linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, u-boot@lists.denx.de, linux-kernel@vger.kernel.org, =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= References: <20240705225821.13196-1-zajec5@gmail.com> <20240708083208.322f4418@xps-13> Content-Language: en-US From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= In-Reply-To: <20240708083208.322f4418@xps-13> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240708_024239_304796_7EF80C7B X-CRM114-Status: GOOD ( 18.95 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 8.07.2024 08:32, Miquel Raynal wrote: > zajec5@gmail.com wrote on Sat, 6 Jul 2024 00:58:21 +0200: > >> From: Rafał Miłecki >> >> U-Boot environment variables can be stored in various data sources. MTD >> is just one of available options. Refactor DT binding into a layout so >> it can be used with UBI volumes and other NVMEM devices. >> >> Link: https://lore.kernel.org/all/20231221173421.13737-1-zajec5@gmail.com/ >> Signed-off-by: Rafał Miłecki >> --- >> I'm sending this PATCH without Linux changes to see if this is the >> right approach - for developers and (DT) maintainers to review it first. >> >> My previous attempt (see above Link) turned out in refusal so I'm just >> trying to save some time in case this one goes wrong as well. >> >> Hopefully the included example (which I really think we should add) >> explains well how I think this binding should be used with layouts. >> >> If I get some positive feedback I'll work on V2 with actual Linux >> changes. >> > > I am still in favor of this change. I know from a binding perspective > it might be questionable; but from an OS perspective it makes total > sense to deprecated the old U-Boot env "device" driver in favor of this. Please note that at this point I didn't decide to deprecate the old binding (using "u-boot,env" for flash device partition). That would require mixing "nvmem-layout" with "partitions" which I believe Rob objected to. My PATCH in its current form just allows reusing this binding with "nvmem-layout"s which is needed for cases where we agreed to allow "nvmem-layout" (UBI volumes and pure - non-MTD - NVMEM devices). If we want to refactor this binding in MTD partitions context I hope we can handle that at different time if we agree on some solution. I hope and I don't think this should be a blocked for this PATCH.