On 03/07/2018 06:20 PM, Mark Brown wrote:
On Tue, Mar 06, 2018 at 10:58:26PM +0800, JeffyChen wrote:
even this is already fixed by a430ab205d29 ("regmap: debugfs: Disambiguate
dummy debugfs file name")
but maybe we can still have this for a better debugfs name?
That's why the facility is there.
Please don't top post, reply in line with needed context. This allows
readers to readily follow the flow of conversation and understand what
you are talking about and also helps ensure that everything in the
discussion is being addressed.
ok, sorry for that :)
recently we are now allowing to register debugfs for syscon regmap, then
i hit a lot of:
(NULL device *): Failed to create debugfs directory
so i sent this patch to fix that by using the of_node name to create
but later i found we have:
Author: Fabio Estevam <fabio.este...@nxp.com>
Date: Mon Mar 5 15:52:09 2018 -0300
regmap: debugfs: Disambiguate dummy debugfs file name
which would fix that in another way by naming dummy0, dummy1, dummy2.
even if the issue is fixed, i wonder could we still have this patch to
make it more readable:
@@ -109,6 +109,7 @@ static struct syscon *of_syscon_register(struct
syscon_config.reg_stride = reg_io_width;
syscon_config.val_bits = reg_io_width * 8;
syscon_config.max_register = resource_size(&res) - reg_io_width;
+ syscon_config.name = of_node_full_name(np);
and i've sent 2 more patches to fix the memleak there:
and sorry again, i forgot to add a cover letter, and i should CC you the