From: Pavel Machek <pavel@denx.de> To: Michal Simek <michal.simek@xilinx.com> Cc: atull <atull@opensource.altera.com>, Pantelis Antoniou <pantelis.antoniou@konsulko.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, jgunthorpe@obsidianresearch.com, hpa@zytor.com, Michal Simek <monstr@monstr.eu>, rdunlap@infradead.org, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, devicetree@vger.kernel.org, robh+dt@kernel.org, Grant Likely <grant.likely@linaro.org>, iws@ovro.caltech.edu, linux-doc@vger.kernel.org, Mark Brown <broonie@kernel.org>, philip@balister.org, rubini@gnudd.com, Steffen Trumtrar <s.trumtrar@pengutronix.de>, jason@lakedaemon.net, kyle.teske@ni.com, nico@linaro.org, Felipe Balbi <balbi@ti.com>, m.chehab@samsung.com, davidb@codeaurora.org, Rob Landley <rob@landley.net>, davem@davemloft.net, cesarb@cesarb.net, sameo@linux.intel.com, akpm@linux-foundation.org, Linus Walleij <linus.walleij@linaro.org>, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, devel@driverdev.osuosl.org, Alan Tull <delicious.quinoa@gmail.com>, dinguyen@opensource.altera.com, yvanderv@opensource.altera.com Subject: Re: [PATCH v8 2/4] fpga manager: add sysfs interface document Date: Tue, 13 Jan 2015 08:28:57 +0100 [thread overview] Message-ID: <20150113072857.GA11760@amd> (raw) In-Reply-To: <bed1d7ce1b984c79acc75a47495cb959@BL2FFO11FD043.protection.gbl> Hi! > >>>>>>>> +What: /sys/class/fpga_manager/<fpga>/firmware > >>>>>>>> +Date: October 2014 > >>>>>>>> +KernelVersion: 3.18 > >>>>>>>> +Contact: Alan Tull <atull@opensource.altera.com> > >>>>>>>> +Description: Name of the FPGA image file to load using firmware > >>>>>>>> class. > >>>>>>> > >>>>>>> This one is ugly: it unneccessarily passes firmware name through the > >>>>>>> kernel. Just make interface and code simpler by always passing > >>>>>>> "socfpga-fpga-image" or something like that. ... > > What is cumbersome about symlink? Why is "fake" symlink in sysfs better? > > > >> Previous uses of the firmware layer has been to use it to load once after > >> bootup; this is different since some use cases will want to switch out > >> the FPGA image. If someone wants there to be only one FPGA image on > >> the FGPA forever, they will probably not be using this framework; their > >> FPGA will probably be loaded before Linux boots up. > > > > Why? I have just one image on the fpga, and would prefer to load it > > from Linux. > > Pavel: These patches target staging and sysfs interface doesn't need to be stable > at this time. I would prefer to add these patches to staging for 3.20 > and feel free to send the patch which fix this. Interesting way to address patch review. "We'll merge it, and you can fix it up later". > With your code will be exactly clear how you want to use it and we can > talk about it. I'm pretty sure Alan knows what I want at this point, he just does not want to do it. For the record, I want to drop "firmware" file, use fixed firmware name, and deal with multiple firmwares in userspace (using symlink or udev magic). I believe this is simplest solution, should be adequate, and is certainly less ugly than implementing fake symlink in /sys/.../firmware. And I have yet to hear what is wrong with that suggestion. Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
WARNING: multiple messages have this Message-ID (diff)
From: Pavel Machek <pavel@denx.de> To: Michal Simek <michal.simek@xilinx.com> Cc: mark.rutland@arm.com, linux-doc@vger.kernel.org, rubini@gnudd.com, Pantelis Antoniou <pantelis.antoniou@konsulko.com>, hpa@zytor.com, Steffen Trumtrar <s.trumtrar@pengutronix.de>, devel@driverdev.osuosl.org, sameo@linux.intel.com, nico@linaro.org, ijc+devicetree@hellion.org.uk, kyle.teske@ni.com, jgunthorpe@obsidianresearch.com, Grant Likely <grant.likely@linaro.org>, davidb@codeaurora.org, Linus Walleij <linus.walleij@linaro.org>, cesarb@cesarb.net, devicetree@vger.kernel.org, jason@lakedaemon.net, pawel.moll@arm.com, iws@ovro.caltech.edu, atull <atull@opensource.altera.com>, Mark Brown <broonie@kernel.org>, philip@balister.org, dinguyen@opensource.altera.com, Michal Simek <monstr@monstr.eu>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, yvanderv@opensource.altera.com, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Felipe Balbi <balbi@ti.com> Subject: Re: [PATCH v8 2/4] fpga manager: add sysfs interface document Date: Tue, 13 Jan 2015 08:28:57 +0100 [thread overview] Message-ID: <20150113072857.GA11760@amd> (raw) In-Reply-To: <bed1d7ce1b984c79acc75a47495cb959@BL2FFO11FD043.protection.gbl> Hi! > >>>>>>>> +What: /sys/class/fpga_manager/<fpga>/firmware > >>>>>>>> +Date: October 2014 > >>>>>>>> +KernelVersion: 3.18 > >>>>>>>> +Contact: Alan Tull <atull@opensource.altera.com> > >>>>>>>> +Description: Name of the FPGA image file to load using firmware > >>>>>>>> class. > >>>>>>> > >>>>>>> This one is ugly: it unneccessarily passes firmware name through the > >>>>>>> kernel. Just make interface and code simpler by always passing > >>>>>>> "socfpga-fpga-image" or something like that. ... > > What is cumbersome about symlink? Why is "fake" symlink in sysfs better? > > > >> Previous uses of the firmware layer has been to use it to load once after > >> bootup; this is different since some use cases will want to switch out > >> the FPGA image. If someone wants there to be only one FPGA image on > >> the FGPA forever, they will probably not be using this framework; their > >> FPGA will probably be loaded before Linux boots up. > > > > Why? I have just one image on the fpga, and would prefer to load it > > from Linux. > > Pavel: These patches target staging and sysfs interface doesn't need to be stable > at this time. I would prefer to add these patches to staging for 3.20 > and feel free to send the patch which fix this. Interesting way to address patch review. "We'll merge it, and you can fix it up later". > With your code will be exactly clear how you want to use it and we can > talk about it. I'm pretty sure Alan knows what I want at this point, he just does not want to do it. For the record, I want to drop "firmware" file, use fixed firmware name, and deal with multiple firmwares in userspace (using symlink or udev magic). I believe this is simplest solution, should be adequate, and is certainly less ugly than implementing fake symlink in /sys/.../firmware. And I have yet to hear what is wrong with that suggestion. Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2015-01-13 7:29 UTC|newest] Thread overview: 128+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-01-06 20:13 [PATCH v8 0/4] FPGA Manager Framework atull 2015-01-06 20:13 ` atull 2015-01-06 20:13 ` [PATCH v8 1/4] doc: add bindings document for altera fpga manager atull 2015-01-06 20:13 ` atull 2015-01-06 22:05 ` Rob Herring 2015-01-06 22:05 ` Rob Herring 2015-01-06 22:34 ` atull 2015-01-06 22:34 ` atull 2015-01-09 15:50 ` Rob Herring 2015-01-09 15:50 ` Rob Herring 2015-01-09 18:58 ` atull 2015-01-09 18:58 ` atull 2015-01-06 20:13 ` [PATCH v8 2/4] fpga manager: add sysfs interface document atull 2015-01-06 20:13 ` atull 2015-01-07 8:48 ` Pavel Machek 2015-01-07 8:48 ` Pavel Machek 2015-01-09 19:14 ` atull 2015-01-09 19:14 ` atull 2015-01-09 20:56 ` Pavel Machek 2015-01-09 20:56 ` Pavel Machek 2015-01-10 8:10 ` Pantelis Antoniou 2015-01-10 8:10 ` Pantelis Antoniou 2015-01-10 15:11 ` Pavel Machek 2015-01-10 15:11 ` Pavel Machek 2015-01-11 16:29 ` atull 2015-01-11 16:29 ` atull 2015-01-12 8:45 ` Pavel Machek 2015-01-12 8:45 ` Pavel Machek 2015-01-12 13:48 ` Michal Simek 2015-01-12 13:48 ` Michal Simek 2015-01-13 7:28 ` Pavel Machek [this message] 2015-01-13 7:28 ` Pavel Machek 2015-01-13 7:40 ` Pantelis Antoniou 2015-01-13 7:40 ` Pantelis Antoniou 2015-01-13 7:56 ` Pavel Machek 2015-01-13 7:56 ` Pavel Machek 2015-01-13 17:27 ` atull 2015-01-13 17:27 ` atull 2015-01-12 16:05 ` Rob Herring 2015-01-12 16:05 ` Rob Herring 2015-01-12 16:26 ` Mark Brown 2015-01-12 16:26 ` Mark Brown 2015-01-12 18:06 ` Jason Gunthorpe 2015-01-12 18:06 ` Jason Gunthorpe 2015-01-13 16:21 ` One Thousand Gnomes 2015-01-13 16:21 ` One Thousand Gnomes 2015-01-15 21:52 ` Pavel Machek 2015-01-15 21:52 ` Pavel Machek 2015-01-12 21:01 ` One Thousand Gnomes 2015-01-12 21:01 ` One Thousand Gnomes 2015-01-12 21:43 ` Jason Gunthorpe 2015-01-12 21:43 ` Jason Gunthorpe 2015-01-13 16:28 ` One Thousand Gnomes 2015-01-13 16:28 ` One Thousand Gnomes 2015-01-13 17:26 ` Pantelis Antoniou 2015-01-13 17:26 ` Pantelis Antoniou 2015-01-13 19:44 ` atull 2015-01-13 19:44 ` atull 2015-01-14 15:58 ` One Thousand Gnomes 2015-01-14 15:58 ` One Thousand Gnomes 2015-01-13 20:00 ` Jason Gunthorpe 2015-01-13 20:00 ` Jason Gunthorpe 2015-01-13 21:37 ` atull 2015-01-13 21:37 ` atull 2015-01-13 22:24 ` Jason Gunthorpe 2015-01-13 22:24 ` Jason Gunthorpe 2015-01-14 16:06 ` One Thousand Gnomes 2015-01-14 16:06 ` One Thousand Gnomes 2015-01-14 18:12 ` Jason Gunthorpe 2015-01-14 18:12 ` Jason Gunthorpe 2015-01-14 19:01 ` Pantelis Antoniou 2015-01-14 19:01 ` Pantelis Antoniou 2015-01-15 11:36 ` One Thousand Gnomes 2015-01-15 11:36 ` One Thousand Gnomes 2015-01-15 11:44 ` Mark Brown 2015-01-15 11:44 ` Mark Brown 2015-01-15 16:34 ` atull 2015-01-15 16:34 ` atull 2015-01-15 18:47 ` Jason Gunthorpe 2015-01-15 18:47 ` Jason Gunthorpe 2015-01-15 20:45 ` One Thousand Gnomes 2015-01-15 20:45 ` One Thousand Gnomes 2015-01-15 20:54 ` Pantelis Antoniou 2015-01-15 20:54 ` Pantelis Antoniou 2015-01-21 16:01 ` One Thousand Gnomes 2015-01-21 16:01 ` One Thousand Gnomes 2015-01-21 16:33 ` Pantelis Antoniou 2015-01-21 16:33 ` Pantelis Antoniou 2015-01-21 20:27 ` Jason Gunthorpe 2015-01-21 20:27 ` Jason Gunthorpe 2015-01-21 20:32 ` Pantelis Antoniou 2015-01-21 20:32 ` Pantelis Antoniou 2015-02-15 22:40 ` Pavel Machek 2015-02-15 22:40 ` Pavel Machek 2015-02-17 17:07 ` Rob Landley 2015-02-17 17:07 ` Rob Landley 2015-02-17 19:17 ` Pavel Machek 2015-02-17 19:17 ` Pavel Machek 2015-02-19 12:46 ` Michal Simek 2015-02-19 12:46 ` Michal Simek 2015-02-21 6:31 ` atull 2015-02-21 6:31 ` atull 2015-02-17 18:12 ` Jason Gunthorpe 2015-02-17 18:12 ` Jason Gunthorpe 2015-01-15 21:42 ` Jason Gunthorpe 2015-01-15 21:42 ` Jason Gunthorpe 2015-01-17 21:11 ` Pavel Machek 2015-01-17 21:11 ` Pavel Machek 2015-01-06 20:13 ` [PATCH v8 3/4] staging: fpga manager: framework core atull 2015-01-06 20:13 ` atull 2015-01-06 20:13 ` [PATCH v8 4/4] staging: fpga manager: add driver for socfpga fpga manager atull 2015-01-06 20:13 ` atull 2015-01-10 18:11 ` [PATCH v8 0/4] FPGA Manager Framework Konrad Zapalowicz 2015-01-10 18:11 ` Konrad Zapalowicz 2015-01-11 16:08 ` atull 2015-01-11 16:08 ` atull 2015-01-11 16:24 ` Konrad Zapalowicz 2015-01-11 16:24 ` Konrad Zapalowicz 2015-01-11 19:52 ` Pavel Machek 2015-01-11 19:52 ` Pavel Machek 2015-01-11 20:58 ` Konrad Zapalowicz 2015-01-11 20:58 ` Konrad Zapalowicz 2015-01-11 21:31 ` Pavel Machek 2015-01-11 21:31 ` Pavel Machek 2015-01-12 13:50 ` Michal Simek 2015-01-12 13:50 ` Michal Simek 2015-01-12 14:06 ` Dan Carpenter 2015-01-12 14:06 ` Dan Carpenter
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=20150113072857.GA11760@amd \ --to=pavel@denx.de \ --cc=akpm@linux-foundation.org \ --cc=atull@opensource.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=devel@driverdev.osuosl.org \ --cc=devicetree@vger.kernel.org \ --cc=dinguyen@opensource.altera.com \ --cc=galak@codeaurora.org \ --cc=grant.likely@linaro.org \ --cc=gregkh@linuxfoundation.org \ --cc=hpa@zytor.com \ --cc=ijc+devicetree@hellion.org.uk \ --cc=iws@ovro.caltech.edu \ --cc=jason@lakedaemon.net \ --cc=jgunthorpe@obsidianresearch.com \ --cc=kyle.teske@ni.com \ --cc=linus.walleij@linaro.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=m.chehab@samsung.com \ --cc=mark.rutland@arm.com \ --cc=michal.simek@xilinx.com \ --cc=monstr@monstr.eu \ --cc=nico@linaro.org \ --cc=pantelis.antoniou@konsulko.com \ --cc=pawel.moll@arm.com \ --cc=philip@balister.org \ --cc=rdunlap@infradead.org \ --cc=rob@landley.net \ --cc=robh+dt@kernel.org \ --cc=rubini@gnudd.com \ --cc=s.trumtrar@pengutronix.de \ --cc=sameo@linux.intel.com \ --cc=yvanderv@opensource.altera.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: linkBe 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.