From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Date: Thu, 30 Jul 2015 11:13:10 +0000 Subject: Re: [PATCH] bus: subsys: update return type of ->remove_dev() to void Message-Id: <20150730110110.GA24546@linux> List-Id: References: <02c0569741812b655edde5124c3e2937bb79f9f8.1438248445.git.viresh.kumar@linaro.org> <20150730105917.GA25460@nazgul.tnic> In-Reply-To: <20150730105917.GA25460@nazgul.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: Borislav Petkov Cc: Greg Kroah-Hartman , Rafael Wysocki , Chris Metcalf , Dmitry Torokhov , "H. Peter Anvin" , Ingo Molnar , Joe Perches , Joerg Roedel , open list , "open list:CPU FREQUENCY DRIVERS" , "open list:SUPERH" , "Luis R. Rodriguez" , "open list:NETWORKING DRIVERS" , Nicolas Iooss , Paul Gortmaker , Thomas Gleixner , "maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)" On 30-07-15, 12:59, Borislav Petkov wrote: > On Thu, Jul 30, 2015 at 03:04:01PM +0530, Viresh Kumar wrote: > > Its return value is not used by the subsys core and nothing meaningful > > can be done with it, even if we want to use it. The subsys device is > > anyway getting removed. > > I'm assuming sysfs paths covered by it, will remain intact... ? For the > microcode loader case that's this hierarchy here: > > $ tree /sys/devices/system/cpu/cpu?/microcode/ > /sys/devices/system/cpu/cpu0/microcode/ > ├── processor_flags > └── version > /sys/devices/system/cpu/cpu1/microcode/ > ├── processor_flags > └── version Yeah, Its not touching the sysfs path at all. -- viresh