From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S932978AbdBVRvH (ORCPT <rfc822;w@1wt.eu>);
        Wed, 22 Feb 2017 12:51:07 -0500
Received: from mail-lf0-f50.google.com ([209.85.215.50]:35707 "EHLO
        mail-lf0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S932867AbdBVRu5 (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 22 Feb 2017 12:50:57 -0500
MIME-Version: 1.0
In-Reply-To: <20170222164400.GA18745@obsidianresearch.com>
References: <CANk1AXTCi6ccimHmXrNPXUn_MJBAoserCmC2kmapQ3Y=zjM79w@mail.gmail.com>
 <CANk1AXTVuVJR6mAKPWgs-+fyRRU1akgmYidu2ubcGJXAwu0oBQ@mail.gmail.com>
 <CAAtXAHd5C4yz3s0WZBL_YKjhwEEntbowsvfg1GUdRDbnq+t9zg@mail.gmail.com>
 <CANk1AXQxpcbGm-MrjrE-7FCmnXAW=Cieg+8QV6ypc6OSCLHTQg@mail.gmail.com>
 <1CC272501B5BC543A05DB90AA509DED50B005A@fmsmsx122.amr.corp.intel.com>
 <CAAtXAHdBv4Q_WykwBS7yXkRH=DnyPBjCmv+V91Jexr32Pf_WOQ@mail.gmail.com>
 <20170222051201.GB29755@obsidianresearch.com> <CAAtXAHe-3L4E7sj9RJJwgMEASJCFmYcXyJdm6z7NtnXWGVGG=w@mail.gmail.com>
 <1CC272501B5BC543A05DB90AA509DED50B010C@fmsmsx122.amr.corp.intel.com>
 <CAAtXAHf0+DM+3uL5_dLxFZ_VmruYZzOpi5p0yu0jQwKniHA86A@mail.gmail.com> <20170222164400.GA18745@obsidianresearch.com>
From: Moritz Fischer <moritz.fischer@ettus.com>
Date: Wed, 22 Feb 2017 09:50:54 -0800
Message-ID: <CAAtXAHe4qTvEApJFUZt4DKYYVxLGO8jFZPri0o7J_3chsx8uxw@mail.gmail.com>
Subject: Re: [RFC 7/8] fpga-region: add sysfs interface
To: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
Cc: "Nadathur, Sundar" <sundar.nadathur@intel.com>,
        Alan Tull <delicious.quinoa@gmail.com>,
        Yves Vandervennet <yves.vandervennet@linux.intel.com>,
        "matthew.gerlach@linux.intel.com" <matthew.gerlach@linux.intel.com>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        "linux-fpga@vger.kernel.org" <linux-fpga@vger.kernel.org>,
        "Marek Va??ut" <marex@denx.de>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Jason,

On Wed, Feb 22, 2017 at 8:44 AM, Jason Gunthorpe
<jgunthorpe@obsidianresearch.com> wrote:
> On Tue, Feb 21, 2017 at 10:05:42PM -0800, Moritz Fischer wrote:
>
>> That being said older drivers / fpga-mgr  will not deal with newer features.
>> TLV / KV or whatever doesn't change this fact, or am I missing something?
>
> Often a scheme will have an OPTIONAL and REQUIRED flag for each
> value. If a REQUIRED value is present but the parser does not
> support it then the parse fails.

Oh, of course. I'm a dorque :) Obvioiusly better customer experience like that.

> For instance, we could mark Encrypted as required, and a zynq driver
> that does not support the Encrypted tag would not load the bitfile
> rather than try to load it wrongly.

Funny you'd mention that. Thanks to your patch the zynq driver won't ;-)
I ran into that while testing, hehe.

> This can be forced into any of the approaches with various levels of
> hackery.

Yeah doesn't seem to hard,

Cheers

Moritz