lkml.org 
[lkml]   [2015]   [Jan]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v7 2/4] fpga manager: add sysfs interface document
From
Date
Hi Pavel,

> On Jan 2, 2015, at 07:12 , Pavel Machek <pavel@denx.de> wrote:
>
>
>> +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 is wrong interface, right? The only use of this string is to pass
> it to udev. We should simply pass "fpga-0" as a firware name, and let
> udev figure it out, no need to to prepare name in separate component,
> then pass it to kernel, which passes it back to udev.
>

No. No dependencies on udev please. This should hook directly to the firmware
loader interface which does not rely on userspace at all.

>> +What: /sys/class/fpga_manager/<fpga>/reset
>> +Date: October 2014
>> +KernelVersion: 3.18
>> +Contact: Alan Tull <atull@opensource.altera.com>
>> +Description: Write 1 to reset the FPGA
>
> Does 0 need to be written there to pull the fpga out of reset? Is this
> a suitable interface?
>
> Pavel
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Regards

— Pantelis



\
 
 \ /
  Last update: 2015-01-02 12:01    [W:0.110 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site