[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-fpga
Subject:    [PATCH 0/2] fpga: add attribute groups
From:       Alan Tull <atull () kernel ! org>
Date:       2017-08-22 15:53:53
Message-ID: 20170822155355.3896-1-atull () kernel ! org
[Download RAW message or body]

Make it easy to add device attribute groups when registering an FPGA
manager, bridge, or region.  This way, the sysfs attributes are added
in device_add instead of afterwards which can lead to race
conditions [1].

Patch 1: add groups to the structs that are used when a manager, bridge,
or region is registered.

Patch 2: changes to altera cvp FPGA manager to use this method of
registering.

I've tested this method on platform devices.  Patch 2 builds but is
untested.

This patchset is dependent on my "non-dt support" patchset currently
under review.

[1] http://www.kroah.com/log/blog/2013/06/26/how-to-create-a-sysfs-file-correctly/

Alan Tull (2):
  fpga: add attribute groups
  fpga: altera-cvp: register attribute groups using ops

 drivers/fpga/altera-cvp.c        | 39 +++++++++++++++++++--------------------
 drivers/fpga/fpga-bridge.c       |  1 +
 drivers/fpga/fpga-mgr.c          |  1 +
 drivers/fpga/fpga-region.c       |  1 +
 include/linux/fpga/fpga-bridge.h |  2 ++
 include/linux/fpga/fpga-mgr.h    |  2 ++
 include/linux/fpga/fpga-region.h |  2 ++
 7 files changed, 28 insertions(+), 20 deletions(-)

-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-fpga" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic