From: "H. Peter Anvin" <hpa@zytor.com>
To: Pavel Machek <pavel@denx.de>,
Pantelis Antoniou <pantelis.antoniou@konsulko.com>
Cc: Alan Tull <atull@altera.com>,
linux-kernel@vger.kernel.org, Michal Simek <monstr@monstr.eu>,
Michal Simek <michal.simek@xilinx.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jason Gunthorpe <jgunthorpe@obsidianresearch.com>,
Mark Brown <broonie@kernel.org>,
Philip Balister <philip@balister.org>,
Alessandro Rubini <rubini@gnudd.com>,
Steffen Trumtrar <s.trumtrar@pengutronix.de>,
Jason Cooper <jason@lakedaemon.net>,
Kyle Teske <kyle.teske@ni.com>,
Josh Cartwright <joshc@eso.teric.us>,
Nicolas Pitre <nico@linaro.org>, Felipe Balbi <balbi@ti.com>,
Mauro Carvalho Chehab <m.chehab@samsung.com>,
David Brown <davidb@codeaurora.org>,
Rob Landley <rob@landley.net>,
"David S. Miller" <davem@davemloft.net>,
Joe Perches <joe@perches.com>,
Cesar Eduardo Barros <cesarb@cesarb.net>,
Samuel Ortiz <sameo@linux.intel.com>,
Andrew Morton <akpm@linux-foundation.org>,
Alan Tull <delicious.quinoa@gmail.com>,
Dinh Nguyen <dinguyen@altera.com>,
Yves Vandervennet <rocket.yvanderv@gmail.com>
Subject: Re: [RFC PATCH v1] fpga manager framework with configfs interface
Date: Tue, 13 May 2014 15:52:16 -0700 [thread overview]
Message-ID: <5372A220.6080301@zytor.com> (raw)
In-Reply-To: <20140513205641.GA4114@amd.pavel.ucw.cz>
On 05/13/2014 01:56 PM, Pavel Machek wrote:
> Hi!
>
>>> Echoing filenames into files in configfs is strange at the very
>>> least. Think chroot, and what about permissions...?
>>>
>>> It would be much better to just cat bitstream into configfs...
>>
>> No binary attributes supported for configfs. Apparently this is done on purpose.
>>
>> In fact it was my first idea about the configfs interface for DT overlays but decided
>> to go with firmware file names instead.
>>
>> If this is valid use case (which seems to be) would patches adding binary attributes
>> to configfs be accepted?
>
> I'd say this is very valid use, but I'm not configfs maintainer. It is
> certainly better then echoing filenames there.
>
Almost anything is better...
-hpa
next parent reply other threads:[~2014-05-13 22:53 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1399411381-10902-1-git-send-email-atull@altera.com>
[not found] ` <1399411381-10902-2-git-send-email-atull@altera.com>
[not found] ` <20140513192330.GA6561@amd.pavel.ucw.cz>
[not found] ` <4190CF2D-6114-4652-90BA-899B5003ACCC@konsulko.com>
[not found] ` <20140513205641.GA4114@amd.pavel.ucw.cz>
2014-05-13 22:52 ` H. Peter Anvin [this message]
[not found] ` <20140625125449.GV30852@ZenIV.linux.org.uk>
[not found] ` <DF9393AB-E2A0-494A-A007-CB4A19A64E2A@konsulko.com>
[not found] ` <1cacd7d5-826b-40e0-8b5c-f3463fe7a809@email.android.com>
[not found] ` <D4CC4816-F293-427B-8D52-1D4F93635E2D@konsulko.com>
[not found] ` <CANk1AXQcqzu16OaT3LziWaroeyyuUoXU1CQy4e6BoGKLOk6fSg@mail.gmail.com>
[not found] ` <20140625175440.GC13460@obsidianresearch.com>
[not found] ` <CANk1AXS+JZCv7CMJW956PvnJ_YvebyyLVm_TkCPZr5TX1Wb4Yw@mail.gmail.com>
[not found] ` <53AB1842.5020703@zytor.com>
2014-06-25 18:54 ` [RFC PATCH v1] fpga manager framework with configfs interface Philip Balister
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5372A220.6080301@zytor.com \
--to=hpa@zytor.com \
--cc=akpm@linux-foundation.org \
--cc=atull@altera.com \
--cc=balbi@ti.com \
--cc=broonie@kernel.org \
--cc=cesarb@cesarb.net \
--cc=davem@davemloft.net \
--cc=davidb@codeaurora.org \
--cc=delicious.quinoa@gmail.com \
--cc=dinguyen@altera.com \
--cc=gregkh@linuxfoundation.org \
--cc=jason@lakedaemon.net \
--cc=jgunthorpe@obsidianresearch.com \
--cc=joe@perches.com \
--cc=joshc@eso.teric.us \
--cc=kyle.teske@ni.com \
--cc=linux-kernel@vger.kernel.org \
--cc=m.chehab@samsung.com \
--cc=michal.simek@xilinx.com \
--cc=monstr@monstr.eu \
--cc=nico@linaro.org \
--cc=pantelis.antoniou@konsulko.com \
--cc=pavel@denx.de \
--cc=philip@balister.org \
--cc=rob@landley.net \
--cc=rocket.yvanderv@gmail.com \
--cc=rubini@gnudd.com \
--cc=s.trumtrar@pengutronix.de \
--cc=sameo@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox