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;
Date:   Tue, 14 Jun 2022 14:27:35 -0700
From:   Colin Foster <colin.foster@in-advantage.com>
To:     Rob Herring <robh@kernel.org>
Cc:     devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-gpio@vger.kernel.org,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Lee Jones <lee.jones@linaro.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Lars Povlsen <lars.povlsen@microchip.com>,
        Steen Hegelund <Steen.Hegelund@microchip.com>,
        UNGLinuxDriver@microchip.com,
        Linus Walleij <linus.walleij@linaro.org>,
        Wolfram Sang <wsa@kernel.org>,
        Terry Bowman <terry.bowman@amd.com>,
        Andy Shevchenko <andy.shevchenko@gmail.com>
Subject: Re: [PATCH v10 net-next 6/7] dt-bindings: mfd: ocelot: add bindings
 for VSC7512
Message-ID: <20220614212735.GA592415@colin-ia-desktop>
References: <20220610202330.799510-1-colin.foster@in-advantage.com>
 <20220610202330.799510-7-colin.foster@in-advantage.com>
 <20220614204345.GA2419690-robh@kernel.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220614204345.GA2419690-robh@kernel.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?lCEX1/uCFl7t2X8SASUx5iXBUjQ8jHIgmoDbbUdtJ21OEyZrnHQ/F6EzaGGY?=
 =?us-ascii?Q?LVRZerh6iNAX8TzJV3L9XkvN21jWVxw92lUFVVsqLAicLWHMd0L/JvJFbMuT?=
 =?us-ascii?Q?pqsK2BMFrFJUTL9d4weUafKrNayOHxn5WRDsgkWwEOjGUSQrWBFmN8zubVTs?=
 =?us-ascii?Q?1oYDVyWWA21XX780yJoXU0eRvVQWzrPkWjYnfMygAHulK1nm3BLinfQKn1Cv?=
 =?us-ascii?Q?8F9rIbM5vdFmO+93Ihct2ybI1cfdn3DK5Pper7Yr+Imtw9vLuOJklx/FlPRX?=
 =?us-ascii?Q?ubkPB1rBQIfZJth4/SIkw/5PvwXeAR2yzOwVLdApf67YCqC8jc2apuJH7PLP?=
 =?us-ascii?Q?UolOQa7y0Sope/s/xVmnQdyxLhMU94rbTx7DaqBMvqNcMKAwdqkmKhYj5pDZ?=
 =?us-ascii?Q?p60Lr2I0SMmFYSQqmsZrp0pl+jdXFAhR7iuHIlQzk5TyFnn1yMn2gCavhxMo?=
 =?us-ascii?Q?FInuZMepaJX8ycqj4rOofYWE2PgRJGLYizaibDOEf8yFzh7Q1EWiEGyf2Xg4?=
 =?us-ascii?Q?azBtY0DQuhhpWerXgPoYp+P+ujQt+ELoNiH2EY1ExxvQw7n3ZWtjr3vBRlcX?=
 =?us-ascii?Q?TD7xQ8lNRKN8b1t3tNvlAQoDNvGcbCvW4LgrHIJsUaYAsBC9dA3W5gSNyxtD?=
 =?us-ascii?Q?KmyQCduaSBXaAzAvutqUxwDeuKha4V3XfKkv5nNl+gb47ArTBuUtTxrXqQXy?=
 =?us-ascii?Q?lRq2MLnC+nxsqWB6zHm+06q2QUYiint2lqkUhyH/iwgsbE9mPhXk4GDsxiCs?=
 =?us-ascii?Q?NjloHmMUZU+0V8L/mJqa4OQHV6geoH6g2MygGaMlS81GR3B2/mEUKqGJtmZY?=
 =?us-ascii?Q?5ETzrzf6hcXJF2EwKv0nQ4yadKpKrGGc1WsKOfOAxJIeSL8IajW10sWIFyP8?=
 =?us-ascii?Q?RUea/MOT347zgKD8m4hoFWGH8XfPNWj3ZB5ysSbjXv7YCf6/G8aPWOkExDyZ?=
 =?us-ascii?Q?jhCu7eZ9VNopK1eXxbIgM9Wc9dpEMUL6uY73neRLXNw7lO3BvMVxqfgcsLEi?=
 =?us-ascii?Q?2VgMgW4FxYvYXyoAvlZxNDWL08Z1cs/EJctlEFG3swUvnVnPVy6VODBRusvT?=
 =?us-ascii?Q?+kbUMol7n7kDDZTuVubXIz6m6PwApLxPqtD4bA1Yx4sGJ0NcYrHxfcpOjd92?=
 =?us-ascii?Q?zfLIPD/S2EOtq1QAAOc6eEagUPqK3yyJgN3IqC0IfVepp0wui9vNk87tPoIj?=
 =?us-ascii?Q?nl/XzoVThMWVbW6HHGocVH6hi6wp4qklFUTU3wU+haUqUoel3r4QVGdn3STl?=
 =?us-ascii?Q?wI4k1c8xLCOXiYBoECxR729Z8pc1GxQbvbvkJljPRFFzefUm/LgDYyDu9KMo?=
 =?us-ascii?Q?bCCjXfzi6qlFoZexihF/VvKKp0SxUClNj4dGxlLt0k2FOOcbYaqCDhqVkHa+?=
 =?us-ascii?Q?AMJdBshD1l/WvlavMMikLep+V2eVekGYEb0e7jzsZHYzpKCNGMsc8tA2H8UQ?=
 =?us-ascii?Q?mL1Xvt3/+Ej1fiabZiN2oy/GwRlmEiBu3Ovs3Vp7BcsQDeBqpSXn+zu0Mlrc?=
 =?us-ascii?Q?d9qQwoXHMdUGntOcAsNW8FN+6MOtIblXKpxltnZCy8oof1roRej0dZAb2LOb?=
 =?us-ascii?Q?nw+2MugIcN1NvQSDhigZ2Iy7IfNwNSCbyit+Fpbwd9maPY/uoq8X8AGj0PgR?=
 =?us-ascii?Q?19vD37IN0ikxdJaE4hwVBDmt9bb3jXnXzTkDtdEsY/5EDmy0f9VzfEhw81I9?=
 =?us-ascii?Q?Swcq2gKEpVu9Jw1jHeckQ8rJuFhMo6crT6M7ONMybN39b2nAkBd1V4WInzf8?=
 =?us-ascii?Q?tmFFjj8KUeIndv/pkYf9rh4YlmVTNK0=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2c6fd3cb-4bde-43e8-3605-08da4e4cbba7
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jun 2022 21:27:50.8187
 (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: E9nmWjCweeEmBvYjftqzRQPbfKubMLXw+JxWy/FKjja+OIkjvdHzTTPjuqARxWnpt+7JqvXGxGnXqy4l/QrCim68By9lTUYyLgTp2M0Kdfo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR10MB5920
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,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

Hi Rob,

Thanks for the feedback! I'll include these changes in my next set.

On Tue, Jun 14, 2022 at 02:43:45PM -0600, Rob Herring wrote:
> On Fri, Jun 10, 2022 at 01:23:29PM -0700, Colin Foster wrote:
...
> > +properties:
> > +  compatible:
> > +    enum:
> > +      - mscc,vsc7512-spi
> 
> '-spi' is redundant as we know what bus this is on looking at the 
> parent.
> 

Ahh... I see this now. A spi driver and an i2c driver (or otherwise)
can share the same compatible string, and the device tree sorts that
out for me. Thanks!

> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  "#address-cells":
> > +    const: 1
> > +
> > +  "#size-cells":
> > +    const: 0
> 
> No size? That's odd given the child nodes are the same as memory mapped 
> peripherals which expect a size.

This one has gone back and forth a couple times. The base addresses were
hard-coded in the driver and kept out of the device tree. But I couldn't
explicitly differentiate between mfd children "mdio0" and "mdio1" below,
so I recently added the address cells back in. This way mfd_cell->of_reg
and mfd_cell->use_of_reg can be used.

Unless suggested otherwise, I'll add size in here. As it is right now,
that size will be essentially ignored though. The resources (base and
size) are all defined in drivers/mfd/ocelot-core.c during the last patch
of this series.