[RFC] pinctrl: pinconf-generic: move ..dt_node_to_map_pinmux() to amlogic-am4 driver

[Conor Dooley]

On Wed, Feb 04, 2026 at 04:26:29PM +0200, Andy Shevchenko wrote:
> On Wed, Feb 04, 2026 at 04:22:47PM +0200, Andy Shevchenko wrote:
> > On Wed, Feb 04, 2026 at 02:15:10PM +0000, Conor Dooley wrote:
> > > On Wed, Feb 04, 2026 at 09:05:34AM +0100, Andy Shevchenko wrote:
> > > > On Wed, Feb 04, 2026 at 12:34:36AM +0100, Linus Walleij wrote:
> > > > > On Tue, Feb 3, 2026 at 5:17 PM Conor Dooley <conor at kernel.org> wrote:
> 
> ...
> 
> > > > Note, please, remove extra '.' (dot) in the Subject.o
> > > 
> > > fwiw, the .. was intentional cos I was truncating the pinconf_generic
> > > from the function since the subject was really long, not referring to
> > > a member of an ops struct.
> > 
> > Yes, and that's how we refer to the callbacks — with a single dot and parentheses:
> > 
> > 	.my_cool_cb()
> > 
> > Alternatively
> > 
> > 	->my_cool_cb()
> > 
> > but it one character longer and TBH it slightly less readable (I personally
> > used the latter and then switched to the former in the recent years).
> 
> Hmm... My memory tricked me, it seems I switched to ->cb() notation, at least
> there are patches with that from October last year. Whatever, choose one and
> use it :-)

I think you missed my point, I was /not/ trying to refer to an ops struct
member. For those I follow the first of the two notations you listed.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20260204/f36babe5/attachment.sig>