Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-09-03 Thread Marc Zyngier
On 2020-09-03 14:25, Suman Anna wrote: On 9/3/20 3:26 AM, Marc Zyngier wrote: [...] Have we reached a conclusion here? Can we get a fix in mainline? Marc, we can go with your patch based on Mark's response. Patch resent[1]. M. [1] https://lore.kernel.org/r/20200903160237.932818-

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-09-03 Thread Suman Anna
On 9/3/20 3:26 AM, Marc Zyngier wrote: > On 2020-08-27 21:32, Suman Anna wrote: >> Hi Marc, >> >> + Mark Brown >> >> On 8/27/20 3:06 PM, Marc Zyngier wrote: >>> Hi Suman, >>> >>> On 2020-08-27 19:28, Suman Anna wrote: Hi Marc, On 8/27/20 9:46 AM, Marc Zyngier wrote: > > [...] > >>>

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-09-03 Thread Marc Zyngier
On 2020-08-27 21:32, Suman Anna wrote: Hi Marc, + Mark Brown On 8/27/20 3:06 PM, Marc Zyngier wrote: Hi Suman, On 2020-08-27 19:28, Suman Anna wrote: Hi Marc, On 8/27/20 9:46 AM, Marc Zyngier wrote: [...] This patch triggers some illegal memory accesses when debugfs is enabled, as regma

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-08-28 Thread Mark Brown
On Thu, Aug 27, 2020 at 03:32:05PM -0500, Suman Anna wrote: > Can you clarify the lifecycle expectations on the config->name and do you have > any suggestions here? The regmap name is expected to be managed by the caller and should be live as long as the regmap is live, it is almost always static

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-08-27 Thread Suman Anna
Hi Marc, + Mark Brown On 8/27/20 3:06 PM, Marc Zyngier wrote: > Hi Suman, > > On 2020-08-27 19:28, Suman Anna wrote: >> Hi Marc, >> >> On 8/27/20 9:46 AM, Marc Zyngier wrote: >>> Hi all, >>> >>> On 2020-07-27 22:10, Suman Anna wrote: The DT node full name is currently being used in regmap_c

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-08-27 Thread Marc Zyngier
Hi Suman, On 2020-08-27 19:28, Suman Anna wrote: Hi Marc, On 8/27/20 9:46 AM, Marc Zyngier wrote: Hi all, On 2020-07-27 22:10, Suman Anna wrote: The DT node full name is currently being used in regmap_config which in turn is used to create the regmap debugfs directories. This name however is

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-08-27 Thread Suman Anna
Hi Marc, On 8/27/20 9:46 AM, Marc Zyngier wrote: > Hi all, > > On 2020-07-27 22:10, Suman Anna wrote: >> The DT node full name is currently being used in regmap_config >> which in turn is used to create the regmap debugfs directories. >> This name however is not guaranteed to be unique and the re

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-08-27 Thread Marc Zyngier
Hi all, On 2020-07-27 22:10, Suman Anna wrote: The DT node full name is currently being used in regmap_config which in turn is used to create the regmap debugfs directories. This name however is not guaranteed to be unique and the regmap debugfs registration can fail in the cases where the sysco

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-07-28 Thread Suman Anna
On 7/28/20 2:44 AM, Arnd Bergmann wrote: On Mon, Jul 27, 2020 at 11:10 PM Suman Anna wrote: The DT node full name is currently being used in regmap_config which in turn is used to create the regmap debugfs directories. This name however is not guaranteed to be unique and the regmap debugfs reg

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-07-28 Thread Lee Jones
On Mon, 27 Jul 2020, Suman Anna wrote: > The DT node full name is currently being used in regmap_config > which in turn is used to create the regmap debugfs directories. > This name however is not guaranteed to be unique and the regmap > debugfs registration can fail in the cases where the syscon

Re: [RESEND PATCH v2] mfd: syscon: Use a unique name with regmap_config

2020-07-28 Thread Arnd Bergmann
On Mon, Jul 27, 2020 at 11:10 PM Suman Anna wrote: > > The DT node full name is currently being used in regmap_config > which in turn is used to create the regmap debugfs directories. > This name however is not guaranteed to be unique and the regmap > debugfs registration can fail in the cases whe