Re: [RFC 7/8] fpga-region: add sysfs interface

From: Moritz Fischer <moritz.fischer@ettus.com>
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>
Subject: Re: [RFC 7/8] fpga-region: add sysfs interface
Date: Wed, 22 Feb 2017 09:57:02 -0800	[thread overview]
Message-ID: <CAAtXAHdK4JW+Qsa=U1Z_mA9NAJ-2i19g6Pp2S4pKXt3wDDZ3MQ@mail.gmail.com> (raw)
In-Reply-To: <20170222175410.GA16445@obsidianresearch.com>

On Wed, Feb 22, 2017 at 9:54 AM, Jason Gunthorpe
<jgunthorpe@obsidianresearch.com> wrote:
> On Wed, Feb 22, 2017 at 09:50:54AM -0800, Moritz Fischer wrote:
>
>> > 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.
>
> Really? You mean there is no sync word? That really surprises me..

Oh wait, that might have been because I didn't drop the header, now
that I think about it,
it probably still has the sync word. I thought it might have been
encrypted (which makes no sense
of course ... )

Ignore the noise :D

Moritz