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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.