Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Colin Foster <colin.foster@in-advantage.com>
CC:     "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Russell King <linux@armlinux.org.uk>,
        Paolo Abeni <pabeni@redhat.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        "UNGLinuxDriver@microchip.com" <UNGLinuxDriver@microchip.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Lee Jones <lee@kernel.org>
Subject: Re: [RFC v1 net-next 7/8] mfd: ocelot: add regmaps for ocelot_ext
Thread-Topic: [RFC v1 net-next 7/8] mfd: ocelot: add regmaps for ocelot_ext
Thread-Index: AQHYxhmBZYjSTVdDzkG5f4rICTpo5q3cShSAgAAN+gA=
Date:   Mon, 12 Sep 2022 21:53:45 +0000
Message-ID: <20220912215344.attgqsxf5oph3cyu@skbuf>
References: <20220912170808.y4l4u2el7dozpx4j@skbuf>
 <20220911200244.549029-1-colin.foster@in-advantage.com>
 <20220911200244.549029-1-colin.foster@in-advantage.com>
 <20220911200244.549029-8-colin.foster@in-advantage.com>
 <20220911200244.549029-8-colin.foster@in-advantage.com>
 <20220912170808.y4l4u2el7dozpx4j@skbuf> <Yx+CzUCbNgAjDK5l@colin-ia-desktop>
 <Yx+CzUCbNgAjDK5l@colin-ia-desktop> <20220912202321.5yqmmf2j7gcljg4j@skbuf>
 <Yx+er8l1CzutF8jo@colin-ia-desktop>
In-Reply-To: <Yx+er8l1CzutF8jo@colin-ia-desktop>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?GCWlSvqH/ekDG3xfbKlDyFuzXIPElijiGXYcALndxi5XxmwKaGgPIX2qluMb?=
 =?us-ascii?Q?oyjxJuO/erRMJVq+bcNLRLOMfZ/krmZAvOnG5jXPuHk2KTkLHHMb0qUGjFb5?=
 =?us-ascii?Q?QMuUU8HlbkNKIEIbWtr0Owgh9Pnv44DNb7iUPQJcYeFaI9QbAfSk3t6xGewU?=
 =?us-ascii?Q?2cxreDW0O5gs80clCkxPJej9qazzQIPujh11eVzpUrqy/Oo5RomNfK6hs9Tq?=
 =?us-ascii?Q?nFEoncxu3jKuGaSClG7S8OifW+IDCU2FT/eFbmnrV4FxuTPmbb30mDj8TODG?=
 =?us-ascii?Q?6+h6nO6aut5eyaFjLmtfdVrXdKCIK1sOYzhB9ExOFAhLV+2DEsNY0aAHFYXs?=
 =?us-ascii?Q?/uC3Du463Oflb/ToPUn9wPSXncx0HyViaZ6As0yZaHwU1714DbYVzv80Pebb?=
 =?us-ascii?Q?KzA/ELodS0byOyPb8bLnA2fzNX0od1mruHEaaD/Xn0NMBd9P4szMCx6igX7c?=
 =?us-ascii?Q?1MmeOy55r0IY4WiTJXJ9LgQA2sDutnpc9ahdfYGF6s4jxKekDnlwHW087gdi?=
 =?us-ascii?Q?zWc0xaOKTy4Z4aY/2q0/jjLdpi86j96lNsggdTLF9pgtCGg+Dfk+Y5Darw6C?=
 =?us-ascii?Q?DR5iW96Ntieo8ZTz67BpdP2n+lrTU9U/bqiHbuDXEDHOdofaegjSwBpigFHw?=
 =?us-ascii?Q?t9bRMK9+FcWp5FXGcuKzQJHc267t/nw1Qmqr3kgoYxbSpN+Vq3xh+CttK2Vv?=
 =?us-ascii?Q?dV4iSOgE52SzW5CPixyK5yFB5Rl6DP8gYqb5NrxQ3QjWTlzbHTS8Zer13lE8?=
 =?us-ascii?Q?6mU75urxdUsU4HjzAVWY1KJft6yPvenO3Jce3hiK9rilG5rjVUvgP5Pmb/kk?=
 =?us-ascii?Q?XvO1fxsUkzAuTd8Z8QxrYJIv1TueAmHZ1zxw8VZyi7xLjethuK6ho/rfnrwy?=
 =?us-ascii?Q?DWFrEYWwYolGJkUBKJbVBso6ArEkI50seVD8TMhATqtzn+ygIBSSUokV70+K?=
 =?us-ascii?Q?8aPN2J1xhMj628rcLV97dx5Q7wpcZKOc9DbheLw6bqVdAT1iGriyzcNB8/2U?=
 =?us-ascii?Q?vwCdPyYGtTT6OltPUJ0qHcl8J0Th1IhjUBUAhavbekOx7XfQWL51xlQtLmh9?=
 =?us-ascii?Q?OvO0+D/Ji5LlRamdBEDCuNKMTapDOt2HdjANQcPd+V1ogrBQ8TDDx/C5kKgJ?=
 =?us-ascii?Q?o/gHSCCCmtiaZrYN2W1ouINZuc4f1Yu82T1B1rDEi5ee1RAAw0tRy7aWZPzB?=
 =?us-ascii?Q?z+i2w0H7H87/1EqBU0OuUAlqMzZoAggmYjHzKdww3FPFstXpAwThhhEEymvO?=
 =?us-ascii?Q?LDHuoYKIqUq7K6E9bT+HkKUW1/mrbkOsub2iluhjgMtd2UQjYjapwVzNOlFF?=
 =?us-ascii?Q?ffzo+i8YlY/9mlGks9ec97ckyGja/RLD2RDwQsRSk8NkFekJeucaw2RA2mu1?=
 =?us-ascii?Q?bCY7PN9fxxJ4X/dbXoAEVCTfZTRrz5S6OlDeaOOyrx41+mLBDsbYZdDDCikt?=
 =?us-ascii?Q?6oJB1cqNOxBwvWVBxgq+CJNCsDmJsbLja0LDB26rU9CwoqJLn/g5nT+7jQak?=
 =?us-ascii?Q?9iMgBqHt2Dw5PKxgHmR/fx+X8jYdskGR5SDYfYqUaNFzSaUsm/FYTUUwHrYp?=
 =?us-ascii?Q?JeFJbeAEKdKBS36QiKaWXCuoWY/iw6OFse/PqcPONu5JCo0xD6yZHG9gK3+Z?=
 =?us-ascii?Q?uQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <501C08EB7A71174F9B1843000AE56B16@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 43bea7db-a28f-4a3d-9da8-08da950943ed
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Sep 2022 21:53:45.3080
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: nOzQ/TF/b35IX2RDduBY/BebK4XoHnODHn3b3Cy3Nympf1tLkViAWwsfuKejrU8qSLuNJV6Va60Wb5EUU9x1nQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB6906
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Sep 12, 2022 at 02:03:43PM -0700, Colin Foster wrote:
> Yep, that's what my example above shows. Not my favorite.

Yeah, sorry, I was on mobile and I had exactly one pass at reading your
response. You did point this out with the "two loops" comment.

> This sounds reasonable. So long as it doesn't muddy up felix / seville
> too much - I'll take a look. It seems like it would just be moving
> a lot of the "resource configuration" code from felix_init_structs() into
> the felix->info->init_regmap(), or similar.

Possibly so, yes. I don't have any other comments for this series, btw.
Just make sure to grab the attention of one of the maintainers somehow
to get Lee's branch pulled, before you send the next version. Not exactly
sure how; LPC has just started and most people have their attention
focused there, I guess.=