Re: [PATCH 1/2] pinctrl: sh-pfc: enable to indicate GPSR/IPSR/MOD_SEL for debug

2016-03-19 Thread Geert Uytterhoeven
Hi Morimoto-san, On Thu, Mar 17, 2016 at 1:30 AM, Kuninori Morimoto wrote: >> > From: Kuninori Morimoto >> > >> > Current sh_pfc can't indicate GPSR/IPSR/MOD_SEL name for debug. >> > Of course we can get it from indicated

Re: [PATCH 1/2] pinctrl: sh-pfc: enable to indicate GPSR/IPSR/MOD_SEL for debug

2016-03-19 Thread Kuninori Morimoto
Hi Geert > > From: Kuninori Morimoto > > > > Current sh_pfc can't indicate GPSR/IPSR/MOD_SEL name for debug. > > Of course we can get it from indicated register address, but it > > is not convenient. This patch enables to indicate these. > > > > Signed-off-by:

Re: [PATCH 1/2] pinctrl: sh-pfc: enable to indicate GPSR/IPSR/MOD_SEL for debug

2016-03-18 Thread Kuninori Morimoto
Hi Laurent, Geert > > struct pinmux_cfg_reg { > > + const char *name; > > This will increase the kernel size, I would remove the name field when > compiling the kernel in non-debug mode. You could use CONFIG_DYNAMIC_DEBUG or > CONFIG_DEBUG_PINCTRL. OK, [1/2] patch is not a big deal. I'm

Re: [PATCH 1/2] pinctrl: sh-pfc: enable to indicate GPSR/IPSR/MOD_SEL for debug

2016-03-16 Thread Geert Uytterhoeven
Hi Morimoto-san, On Wed, Mar 16, 2016 at 1:47 AM, Kuninori Morimoto wrote: > From: Kuninori Morimoto > > Current sh_pfc can't indicate GPSR/IPSR/MOD_SEL name for debug. > Of course we can get it from indicated register address,

[PATCH 1/2] pinctrl: sh-pfc: enable to indicate GPSR/IPSR/MOD_SEL for debug

2016-03-15 Thread Kuninori Morimoto
From: Kuninori Morimoto Current sh_pfc can't indicate GPSR/IPSR/MOD_SEL name for debug. Of course we can get it from indicated register address, but it is not convenient. This patch enables to indicate these. Signed-off-by: Kuninori Morimoto