Date:   Thu, 18 Nov 2021 18:14:37 -0800
From:   Colin Foster <colin.foster@in-advantage.com>
To:     Linus Walleij <linus.walleij@linaro.org>
Cc:     linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com, Andrew Lunn <andrew@lunn.ch>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Ioana Ciornei <ioana.ciornei@nxp.com>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        Lars Povlsen <lars.povlsen@microchip.com>,
        Steen Hegelund <Steen.Hegelund@microchip.com>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        Heikki Krogerus <heikki.krogerus@linux.intel.com>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Subject: Re: [RFC PATCH v4 net-next 00/23] add support for VSC75XX control
 over SPI
Message-ID: <20211119021437.GA1155@DESKTOP-LAINLKC.localdomain>
References: <20211116062328.1949151-1-colin.foster@in-advantage.com>
 <CACRpkdaFmFFpYrreFsD6XRPAoivDPK1nSfAVKacNG8bWUR7rHQ@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CACRpkdaFmFFpYrreFsD6XRPAoivDPK1nSfAVKacNG8bWUR7rHQ@mail.gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cTc4WXBwaFhTUmZJWGQxeEMrb3ZhNlY5WmFPVVErVnhQMTZrMHYzV1U3aGNS?=
 =?utf-8?B?UUI1ZHZoVXFRcWEzekswY3NKak4wckROb0VWMGZ2VmkxamV0eEduV29HY0RR?=
 =?utf-8?B?S1o2ZHBzcllRT2FhUFlnZkU4NlJkcFkwYnZYdk9zbW8wYzVGNXhJSEF6amh5?=
 =?utf-8?B?SWhGam4ybmxoamQ3dlZlditRUlg5aVlSTVhsY3BtczlORUdUczkzck4ydmNX?=
 =?utf-8?B?K1UwWWkvVmY1YVpzVWpIQ1F0UXlRejlIL2ZMdjZ3WlRXdDBlVi9wckRhYllJ?=
 =?utf-8?B?S0hvMEg3dnJpRTdiUHhzUUtSKzZCM3YvbzJpL3l5SjR0RzFSbUhaaFdSNXdp?=
 =?utf-8?B?SzhDWVIvcThGSHJYSTdmRk1MTVoya0xoVitYMUhGN08vNUVneGVTRjRxaW1H?=
 =?utf-8?B?UTJkOExLRk4yWHhVaFIyRjVMRE11bWhVeFlQQkFpTHNDRTAwNmRGVkJCSkF6?=
 =?utf-8?B?ZG80UUVmZkNEbmJGOVVQbStjNmYwOUU1eDRDTXkxUlYzVEhuRFNXMVRST0Y5?=
 =?utf-8?B?RDdGdVNmclBKQmlEV2Z1bVRhYkNyclVrUldQSkxrVGkwQUFoY1V0R2FhK1Q5?=
 =?utf-8?B?aWd4cUVndlhRd1Y0UHl6cEw0SCtISzJ2RzdSYUNZdjh4Q2wzVG52UDlRNG8x?=
 =?utf-8?B?VjlpczZSMGdpOXJRWlQ3dysvVWFTWlJTYkRWWEJuK20ySXNKbnJaem1pRGNo?=
 =?utf-8?B?RUY1Q1M0NCtJUURxaHRUUGFWbzIwL0xycmthUExpcG5UZFhVWHQ4MmtCZVAr?=
 =?utf-8?B?WXp0a3hkSkY3Y1JUblpPV0xhUWVpdWhTWnU5VUdSa0tCRXdhUjZaRFBPNVJx?=
 =?utf-8?B?RHpoWFBtbmg5MTJPWXUvdmx4SnEwWWhiZmVKYjg0V01JZVlXL2U3NnhReWsv?=
 =?utf-8?B?Z1hYRXJYYmNaMmU4UnV0MUZXQVRYU1lKYUJuTSsrWjdqZ2N4anNueUJtMzRn?=
 =?utf-8?B?Uk1hRkpuZ0lmQUpXUCtCdytMQ0NiU3FpU0UrNWtVZjNHQzgxYlcyRXJYRHM5?=
 =?utf-8?B?bkI5ZUl3UThLa2x6VW9TVU00SjFzQmlMV014cWZ5NUppa3p0U2JaZDhIUG5x?=
 =?utf-8?B?RzB1QVhoUHdySitRS3NacCtXaGVhWGVlY3F1Ky8rOHBHVnp3bG5ycTR4TlRz?=
 =?utf-8?B?Uk50ckNtQmhOU1VwSWlCVUFkc28xeVhCTytYS1BCTzFORUlqODRnWDZORkdS?=
 =?utf-8?B?eHdYS3VxOWFpNktGYlY4VXFLUjcwVjIyZml0SVN5M3dRbEQ2cUFVZjhOdlMz?=
 =?utf-8?B?SGhud29HUUt1YUZFc2V4QUNYWndvN3RjRUZKbFNTbFZlSVltVkVuVktObUxz?=
 =?utf-8?B?UmRKaldLN3NNQUNlT2RyYXhQNkkydnhNMlk2T0syOVIyYStaUm14bTJ5ZlJs?=
 =?utf-8?B?NzhWY0NhNGd5UWRZR3BMZk1CUXJXMTZVdDk3aGNBdkgrQVo4QkVCSVg0T0ZO?=
 =?utf-8?B?UUZORGkwTTJJUXl6VGdPT0RKTDFPU2dTZWl6QzRacEF5TTdpc09qbTYrclcy?=
 =?utf-8?B?SWdHTy9wNzFmdmd5OThFWnhzL0hUMTZRUFhVemxhakd1WG1YREowRWltdEFT?=
 =?utf-8?B?Ti8xUjR4NlN5OEIxVnc0VnBWbTVvRGFjYjhsTHdiMU9qZ2owRWRlb2oxSGdZ?=
 =?utf-8?B?V2ttdnRZdW53Qk1OeTZYUXp5VWZxYmFzendTckI2TWQxRDZ1Z1o5VThrelJN?=
 =?utf-8?B?ZFgvY3BrWkZJSkh1OUxPeTUzTnlCNTUzT2gvY09meGFxUHpRUkErZS9ETXlK?=
 =?utf-8?B?ZDV2VXN2OXpnMFpwVUltUXhSVzMvZVRCeTNpNytTalpSbENDYldHSlB2TFo5?=
 =?utf-8?B?NDZELzZYcUZLWmR4a3FZdmJrWFozSERnRXJEYXZoS2pob0tCbGlULzc4QnBw?=
 =?utf-8?B?QzF4aFF6ald0QkZuRU5Fa2FQUDJmcTR2SURTaVF6cllxc2NyV0RReDVZTVJy?=
 =?utf-8?B?ZmZYNGNHYkRpaWdVWG1pdGNlS1lZUEdpTmZkVmVqRzNodnl4TUtVc0hCU0R4?=
 =?utf-8?B?QmlkVUhzdStRdTVWaERVUVVYY3RvWDhwRTZhNHVzZTdaMnlWSlE0MEowNXBj?=
 =?utf-8?B?aXM3ekc3L1owWWl3cjFxeDIzUVZkVEQwTU9mWDdmWis0MDNkajBleHZiZEtC?=
 =?utf-8?B?dDJhaG1DUjg3NWRDQ3o5c1hUVWxpOTFnamo0dVJxV0p0Q3dZOU5IU3pVNWJF?=
 =?utf-8?B?MWxpeDJubDd4UDNMaDFyV0FpSTFuR05BUk5sN2ZPQXdXaEd5SlhrUDFuWG1Z?=
 =?utf-8?B?UC94QUllTVh3SFNWeW1Zc3Z1UmR3PT0=?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2ccc7744-2d60-4b11-f09b-08d9ab02597c
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 02:14:43.4007
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 3zpqeQF/FJWwSJhwu6xGJltgkQiwP3twYc2TGTX0I/O7XDxNB4d7uXnxid+qJts5tKC9dybjWGwmZRFP4eonvRRZiYKy5P4buMR60OLzzJQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR10MB5805
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Linus,

On Fri, Nov 19, 2021 at 02:58:47AM +0100, Linus Walleij wrote:
> Hi Colin,
> 
> nice work!

Thanks :)

> 
> On Tue, Nov 16, 2021 at 7:23 AM Colin Foster
> <colin.foster@in-advantage.com> wrote:
> 
> >   pinctrl: ocelot: combine get resource and ioremap into single call
> >   pinctrl: ocelot: update pinctrl to automatic base address
> >   pinctrl: ocelot: convert pinctrl to regmap
> >   pinctrl: ocelot: expose ocelot_pinctrl_core_probe interface
> >   pinctrl: microchip-sgpio: update to support regmap
> >   device property: add helper function fwnode_get_child_node_count
> >   pinctrl: microchip-sgpio: change device tree matches to use nodes
> >     instead of device
> >   pinctrl: microchip-sgpio: expose microchip_sgpio_core_probe interface
> 
> Can these patches be broken out to its own series and handled
> separately from the DSA stuff or is there build-time dependencies?

These should all be able to be a separate series if I did my job right. 
Everything should have no functional change except for this:

> >   pinctrl: ocelot: update pinctrl to automatic base address

Fortunately this was tested by Clément and didn't seem to have any
ill-fated side effects. 

I assume this isn't something I wouldn't want to submit to net-next...
is there a different place (tree? board? list?) where those should be
submitted?

> 
> Yours,
> Linus Walleij