From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751660AbdHVPyM (ORCPT ); Tue, 22 Aug 2017 11:54:12 -0400 Received: from mail.kernel.org ([198.145.29.99]:37338 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751038AbdHVPyK (ORCPT ); Tue, 22 Aug 2017 11:54:10 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org AFA7C20C48 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=atull@kernel.org From: Alan Tull To: Moritz Fischer , Anatolij Gustschin Cc: Alan Tull , linux-kernel@vger.kernel.org, linux-fpga@vger.kernel.org Subject: [PATCH 0/2] fpga: add attribute groups Date: Tue, 22 Aug 2017 10:53:53 -0500 Message-Id: <20170822155355.3896-1-atull@kernel.org> X-Mailer: git-send-email 2.11.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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