Re: [PATCH v2 3/5] sh: mach-rsk: rsk7203: use static device properties for LEDs and GPIO buttons

From: Andy Shevchenko

Date: Tue Jun 02 2026 - 19:54:40 EST


On Wed, Jun 03, 2026 at 02:45:39AM +0300, Andy Shevchenko wrote:
> On Tue, Jun 02, 2026 at 03:06:14PM -0700, Dmitry Torokhov wrote:
> > On Wed, Jun 03, 2026 at 01:02:56AM +0300, Andy Shevchenko wrote:
> > > On Wed, May 20, 2026 at 10:13:19PM -0700, Dmitry Torokhov wrote:

...

> > > > + for (i = 0; i < ARRAY_SIZE(rsk7203_devices); i++) {
> > >
> > > for (unsigned int i = 0; i < ARRAY_SIZE(rsk7203_devices); i++) {
> > >
> > > ...and drop the above definition of i.
> >
> > Why? I do not see coding style suggesting this.
>
> Linus Torvalds.

https://lore.kernel.org/lkml/CAHk-=wgy8p4is8ApEQCT5NS7XFb+NXeo-TKz7jRRZVksLLBSrQ@xxxxxxxxxxxxxx/

And IIRC it was another discussion (more recent than that) where he again
mentioned nice of use for-loops with local iterators.

--
With Best Regards,
Andy Shevchenko