Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp3084882pxb; Mon, 16 Nov 2020 05:32:04 -0800 (PST) X-Google-Smtp-Source: ABdhPJzNrl1fADjDwHTdun6GQ+1YydebQgh52BFJAke42Vq/Ha1/rmwfSq/BPLyvrlS3Q+SZmzSZ X-Received: by 2002:a50:99d6:: with SMTP id n22mr15975223edb.261.1605533524511; Mon, 16 Nov 2020 05:32:04 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1605533524; cv=none; d=google.com; s=arc-20160816; b=ZPbo0VCEDxPkRqrf5V4JZy3N+ArlmSfWdibeyZKZyNa/dAj2sP2gtlX2JTxjgDK2Gp LtWb2y4k6FJMlp1N0TrNniYRHvJYkBjbMLx/gH2RISBB41YlFl9oXz6Wak2TdGHW4BbI YUdsbVxHGvlrK8QnUJsZtbLp6/2NYn6KHHiSUgolONo4KcoqBtwI8zDOfNIGAccw8J0D yPsUtm4hX/Dcvt3LK69Jw8T91tef64amwroEC0Wl4KUCoEPWTVi8/xebOKOUr4hAw3ep f8wnAP0AzEouXoLq91twppTPYKHGWb/YKuH968gLpkFXAPNdZ8KHXjJYLe59o8opCOHE f9vg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:ironport-sdr:ironport-sdr; bh=9YD+i/IpZHj4xSBhSPNz5Ym47Ns411YwIpXBJaeq+7s=; b=ZUPK/wxdMP1zmW11/OKUUCSI21iuO8kQFRb7v/NGPsPateXEs5MyUCIYl1gQ8kwMWA 5WRFPgtiHjZZ5EkKhLPKLw7ns9BZFcwHvk987owIEm2+7KuS8VUkn6L2NQ5wy+vIpn3H c0xKPTqvjU2Nexf6R+LWlOlX8lEtsXWDyneNC2kHAEconxK3qBBmHcVx5Z3pxBcqZ9UZ gQpplfEmFc6ts9hrGjDVCMmlKOAB3xQjEgyCAhtpyiDHnIvK2vgvuAP/vMzHa8T8FPBi yIU2PAYKCcYFh/WG0QiTiALId0gpHw4eJEBrBzJIoIj21ma106amWtDnrmzeOaqNfBeQ A3gQ== ARC-Authentication-Results: i=1; mx.google.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=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ga12si12784801ejc.14.2020.11.16.05.31.40; Mon, 16 Nov 2020 05:32:04 -0800 (PST) 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; 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=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728308AbgKPN3J (ORCPT + 99 others); Mon, 16 Nov 2020 08:29:09 -0500 Received: from mga11.intel.com ([192.55.52.93]:62361 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727210AbgKPN3I (ORCPT ); Mon, 16 Nov 2020 08:29:08 -0500 IronPort-SDR: DuVvzoQ6Any6ujegBFuLLkqu/UDGAdeSTEEYm8u1Tb8kWDULwbp02MNisOlUYmYPmvXZnCfojI oEZF/87s1gHA== X-IronPort-AV: E=McAfee;i="6000,8403,9806"; a="167232510" X-IronPort-AV: E=Sophos;i="5.77,482,1596524400"; d="scan'208";a="167232510" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Nov 2020 05:29:06 -0800 IronPort-SDR: fryqlkjKT0wx4qZEbae7wf0xveDIySmbyBErit51MXeBrBc/ptD9NSIz7JSI4qwD4BdKaNZJp0 qLewOPF3EOBA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,482,1596524400"; d="scan'208";a="531846785" Received: from marshy.an.intel.com (HELO [10.122.105.143]) ([10.122.105.143]) by fmsmga006.fm.intel.com with ESMTP; 16 Nov 2020 05:29:05 -0800 Subject: Re: [PATCHv1 3/4] dt-bindings: fpga: add authenticate-fpga-config property To: Tom Rix , mdf@kernel.org, linux-fpga@vger.kernel.org, linux-kernel@vger.kernel.org Cc: dinguyen@kernel.org, sridhar.rajagopal@intel.com, Richard Gong References: <1605204403-6663-1-git-send-email-richard.gong@linux.intel.com> <1605204403-6663-4-git-send-email-richard.gong@linux.intel.com> <94ccfc8d-ecf9-7782-9044-d3c0cb862118@redhat.com> From: Richard Gong Message-ID: <50204b8d-91dd-e1d4-45f0-9b3e21d7309b@linux.intel.com> Date: Mon, 16 Nov 2020 07:50:07 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Tom, On 11/14/20 9:59 AM, Tom Rix wrote: > > On 11/14/20 6:52 AM, Richard Gong wrote: >> Hi Tom, >> >>>>       prior to OS boot up. >>>>   - encrypted-fpga-config : boolean, set if the bitstream is encrypted >>>> +- authenticate-fpga-config : boolean, set if do bitstream authentication >>> >>> The list is mostly in alphabetical order so the new 'authenticate-... ' should go at the top. >>> >> >> The original list is not in alphabetical order. The order of partial-fpga-config, external-fpga-config and encrypted-fpga-config here follows the implementation in the of-fpga-region.c file. >> >> So authenticate-fpga-config should follow the way, correct? >> > This is why i say 'mostly' .. > > In general when listing options for a user to read, you should make it easy for them to find > > the option they are looking for.  Ordering them alphabetically is an obvious but not the only > > way.  I am not asking for you to fix the whole table, just what you are adding. If there is > > a better way to organize them please propose the method. > How about put authenticate-fpga-config above partial-fpga-config? I would like to group all xxx-fpga-config flags together. Regards, Richard > Tom > >>> Improve what you mean by 'authentication' similar to my comment in the first patch. >>> >> >> Will do in the version 2 submission. >> >> Regards, >> Richard >> >>> Tom >>> >>>>   - region-unfreeze-timeout-us : The maximum time in microseconds to wait for >>>>       bridges to successfully become enabled after the region has been >>>>       programmed. >>> >> >