From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933038AbcHYJwM (ORCPT ); Thu, 25 Aug 2016 05:52:12 -0400 Received: from atlantic540.startdedicated.de ([188.138.9.77]:60307 "EHLO atlantic540.startdedicated.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932228AbcHYJwL (ORCPT ); Thu, 25 Aug 2016 05:52:11 -0400 From: Daniel Wagner To: linux-kernel@vger.kernel.org Cc: Daniel Wagner , Ming Lei , "Luis R . Rodriguez" , Greg Kroah-Hartman Subject: [PATCH v3 0/3] firmware_class: encapsulate firmware loading status Date: Thu, 25 Aug 2016 11:52:00 +0200 Message-Id: <1472118723-22762-1-git-send-email-wagi@monom.org> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Daniel Wagner This version should address all comments from Luis. In the last patch the fw_lock dependency is dropped. This only works if we garantee not to race between the reader and the writer side in combination in going to sleep and waking up. So here that should now be a good argument for swait :) cheers, daniel changes since v2: - more splitting out - first patch factors out all the bit ops into fw_status - second patch gets rid of the bit ops - third get rid of fw_lock by using swait changes since v1: - moved swait change into its own patch - added ifdef section for FW_LOADER_USER_HELPER_FALLBACK - updated commit message highlighting the mutex usage drop a bit https://lkml.org/lkml/2016/8/4/239 Cc: Ming Lei Cc: Luis R. Rodriguez Cc: Greg Kroah-Hartman Daniel Wagner (3): firmware_class: encapsulate firmware loading status firmware_class: Drop bit ops in favor of simple state machine firmware_class: Do not use fw_lock for fw_status protection drivers/base/firmware_class.c | 153 ++++++++++++++++++++++++++---------------- 1 file changed, 94 insertions(+), 59 deletions(-) -- 2.7.4