Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
On Tue, 13 Oct 2020 19:17:37 -0700, Florian Fainelli wrote: > When more than a single SCMI device are present in the system, the > creation of the notification workqueue with the WQ_SYSFS flag will lead > to the following sysfs duplicate node warning: > > [9.259990] sysfs: cannot create

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Florian Fainelli
On 10/14/20 10:39 AM, Sudeep Holla wrote: > On Wed, Oct 14, 2020 at 10:13:04AM -0700, Florian Fainelli wrote: >> On 10/14/20 9:18 AM, Sudeep Holla wrote: >>> On Wed, Oct 14, 2020 at 02:48:19PM +0100, Cristian Marussi wrote: >>> >>> [...] >>> > > I have pushed a version with above change

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
On Wed, Oct 14, 2020 at 10:13:04AM -0700, Florian Fainelli wrote: > On 10/14/20 9:18 AM, Sudeep Holla wrote: > > On Wed, Oct 14, 2020 at 02:48:19PM +0100, Cristian Marussi wrote: > > > > [...] > > > >>> > >>> I have pushed a version with above change [1], please check if you are > >>> happy with

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
On Wed, Oct 14, 2020 at 06:32:42PM +0100, Sudeep Holla wrote: > On Wed, Oct 14, 2020 at 10:08:32AM -0700, Florian Fainelli wrote: > > On 10/14/20 2:18 AM, Sudeep Holla wrote: > > > Hi Florian, > > > > > > Thanks for the patch, it shows someone else is also using this and > > > testing . > > > > >

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
On Wed, Oct 14, 2020 at 10:08:32AM -0700, Florian Fainelli wrote: > On 10/14/20 2:18 AM, Sudeep Holla wrote: > > Hi Florian, > > > > Thanks for the patch, it shows someone else is also using this and > > testing . > > > > On Tue, Oct 13, 2020 at 07:17:37PM -0700, Florian Fainelli wrote: > >> When

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Florian Fainelli
On 10/14/20 9:18 AM, Sudeep Holla wrote: > On Wed, Oct 14, 2020 at 02:48:19PM +0100, Cristian Marussi wrote: > > [...] > >>> >>> I have pushed a version with above change [1], please check if you are >>> happy with that ? >>> >>> [1] https://git.kernel.org/sudeep.holla/linux/c/b2cd15549b >> >> I

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Florian Fainelli
On 10/14/20 2:18 AM, Sudeep Holla wrote: > Hi Florian, > > Thanks for the patch, it shows someone else is also using this and > testing . > > On Tue, Oct 13, 2020 at 07:17:37PM -0700, Florian Fainelli wrote: >> When more than a single SCMI device are present in the system, the >> creation of

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
On Wed, Oct 14, 2020 at 02:48:19PM +0100, Cristian Marussi wrote: [...] > > > > I have pushed a version with above change [1], please check if you are > > happy with that ? > > > > [1] https://git.kernel.org/sudeep.holla/linux/c/b2cd15549b > > I agree with the need to retain _notify name, but

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Cristian Marussi
Hi On Wed, Oct 14, 2020 at 02:05:24PM +0100, Sudeep Holla wrote: > On Wed, Oct 14, 2020 at 10:18:06AM +0100, Sudeep Holla wrote: > > Hi Florian, > > > > Thanks for the patch, it shows someone else is also using this and > > testing . > > > > On Tue, Oct 13, 2020 at 07:17:37PM -0700, Florian

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
On Wed, Oct 14, 2020 at 10:18:06AM +0100, Sudeep Holla wrote: > Hi Florian, > > Thanks for the patch, it shows someone else is also using this and > testing . > > On Tue, Oct 13, 2020 at 07:17:37PM -0700, Florian Fainelli wrote: > > When more than a single SCMI device are present in the system,

[PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Florian Fainelli
When more than a single SCMI device are present in the system, the creation of the notification workqueue with the WQ_SYSFS flag will lead to the following sysfs duplicate node warning: [9.259990] sysfs: cannot create duplicate filename '/devices/virtual/workqueue/scmi_notify' [9.260024]

Re: [PATCH] firmware: arm_scmi: Fix duplicate workqueue name

2020-10-14 Thread Sudeep Holla
Hi Florian, Thanks for the patch, it shows someone else is also using this and testing . On Tue, Oct 13, 2020 at 07:17:37PM -0700, Florian Fainelli wrote: > When more than a single SCMI device are present in the system, the > creation of the notification workqueue with the WQ_SYSFS flag will