[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b169157e-3477-3bc3-b4dd-92b3a69798b7@kontron.de>
Date: Mon, 30 Jan 2023 10:07:06 +0100
From: Frieder Schrempf <frieder.schrempf@...tron.de>
To: Manuel Traut <manuel.traut@...com>, linux-kernel@...r.kernel.org
Cc: Dmitry Torokhov <dmitry.torokhov@...il.com>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
linux-input@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH v8 5/5] input: pwm-beeper: handle module unloading
properly
Hi Manuel,
On 26.01.23 10:18, Manuel Traut wrote:
> 'input: pwm-beeper: add feature to set volume via sysfs' adds device
> attributes without removing them on error or if the module is unloaded.
>
> If the module will be unloaded and loaded again it fails:
> [ 1007.918180] sysfs: cannot create duplicate filename '/devices/platform/buzzer/volume'
>
> Therefore remove device attributes on module unloading and in case
> registration at the input subsystem fails.
>
> Signed-off-by: Manuel Traut <manuel.traut@...com>
Thanks for picking up these old patches!
I think you need to merge this fixup patch 5/5 into patch 2/5 of this
series.
Thanks
Frieder
Powered by blists - more mailing lists