Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Russell King - ARM Linux admin <linux@armlinux.org.uk>
CC:     Leon Romanovsky <leon@kernel.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        Jakub Kicinski <kuba@kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Vladimir Oltean <olteanv@gmail.com>,
        Vadym Kochan <vkochan@marvell.com>,
        Taras Chornyi <tchornyi@marvell.com>,
        Saeed Mahameed <saeedm@nvidia.com>,
        Jiri Pirko <jiri@nvidia.com>, Ido Schimmel <idosch@nvidia.com>,
        Lars Povlsen <lars.povlsen@microchip.com>,
        Steen Hegelund <Steen.Hegelund@microchip.com>,
        "UNGLinuxDriver@microchip.com" <UNGLinuxDriver@microchip.com>,
        Grygorii Strashko <grygorii.strashko@ti.com>,
        Julian Wiedmann <jwi@linux.ibm.com>,
        Karsten Graul <kgraul@linux.ibm.com>,
        Heiko Carstens <hca@linux.ibm.com>,
        Vasily Gorbik <gor@linux.ibm.com>,
        Christian Borntraeger <borntraeger@de.ibm.com>,
        Jianbo Liu <jianbol@nvidia.com>,
        Vlad Buslov <vladbu@nvidia.com>,
        Bjarni Jonasson <bjarni.jonasson@microchip.com>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        Tobias Waldekranz <tobias@waldekranz.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
        "linux-s390@vger.kernel.org" <linux-s390@vger.kernel.org>,
        Ido Schimmel <idosch@idosch.org>
Subject: Re: [PATCH net] net: switchdev: zero-initialize struct
 switchdev_notifier_fdb_info emitted by drivers towards the bridge
Thread-Topic: [PATCH net] net: switchdev: zero-initialize struct
 switchdev_notifier_fdb_info emitted by drivers towards the bridge
Thread-Index: AQHXjSAoM5FylMZBM0itAyPu8B/6WKtsTdGAgAAX6QCAAA/AAIAABjsA
Date:   Tue, 10 Aug 2021 09:34:57 +0000
Message-ID: <20210810093456.3ojqgvhe6ivo7sdr@skbuf>
References: <20210809131152.509092-1-vladimir.oltean@nxp.com>
 <YRIhwQ3ji8eqPQOQ@unreal> <20210810081616.wjx6jnlh3qxqsfm2@skbuf>
 <20210810091238.GB1343@shell.armlinux.org.uk>
In-Reply-To: <20210810091238.GB1343@shell.armlinux.org.uk>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?AdFSMIBUQ/cYhDRjaF/PJSR9ehCeLT+pN9T+UYnTxAqjKOHlGJRcBKRF5FV7?=
 =?us-ascii?Q?3dm343Dr6AlURBdKwIn1K1UY1VcWY6HpOvP/lvs/0EWL44FfcdVGSpq13N+8?=
 =?us-ascii?Q?VgYYyJHSe4xhXyknBx0F5RaEfUy+Nox8JV8jPbDKguQdJACNAHxphw7ri7Z3?=
 =?us-ascii?Q?vzYiojlLNaN+ZKWZAKKO+F6TYO0D2MNdWeEtmjnN0M4/+zb7JCvGGQuLJCPZ?=
 =?us-ascii?Q?wh4yHz+bDENkDgek6HWWqLJgIscsn+KJnb9OLtZ/PNMOU//PLyM1QASJ18Ng?=
 =?us-ascii?Q?1s1/Itde42xUS8ldvM8taaejziZJC9T2DmHufqV02s43LJQuzFPULj3yOEL0?=
 =?us-ascii?Q?TQKkRWWU+Le7YVwsNYjGI/7sJW4NsZD2qIqbHCmaQVOZMsSer76t99jJsL8U?=
 =?us-ascii?Q?QKB3G9TNX5ldWGkNgatKsI4i7J5JQnX/At6oBtPV9oYaj2NPMLtkFRLRH5Cn?=
 =?us-ascii?Q?KYQd+UTtrh0QO+D7BYkLgSBpX4+B1sb3y2SH7/NTdZyspkmvJAOcyWJPg2mk?=
 =?us-ascii?Q?Qff6bsEclqZ4cQUlqLbNYhXnAkhmT4Ffphha9NFuC7YaVFd2/MTc70rsB4Hl?=
 =?us-ascii?Q?0Fu3CJCrm+9HJBSr53kVCv4SCa8Lwh2wKeZcaLjryb7KJaXO0HcdrYTnGD6N?=
 =?us-ascii?Q?tBqc8CgDtLFIPbVg7o0Ag3rRCZGMgPxFcDSNB1/VVvZ0WvQN7K8qdmpd9zD5?=
 =?us-ascii?Q?3EGemSYAMQAIM100mAE4XQjNFblFiNSO4Uf4HiURTr7rLWLVbouc743+nw07?=
 =?us-ascii?Q?NwsfLAKb1QSjkJGJ9xFLlFXZhFvbUmpzKl4WLYwialNIuZ+G/6Ti8dA1ThQr?=
 =?us-ascii?Q?LXlIVbLQuNszIGiSeW3X/OReKrw4cyFcbzp933/oJ5NNGqaHqO9ZivuYa9Fg?=
 =?us-ascii?Q?vHiMbsbE63mGlQZCrtyj4sSMPaveHzZYOhqPQknrHAq6Vr1ZjKSMo08kCdGy?=
 =?us-ascii?Q?T3FFBG6HcVYRQBQRJtTf9mrRghsCN7R6J2Kpwbd0MyIo0M7ToQp2gwMUhg9J?=
 =?us-ascii?Q?d28MwX9w00iNM52fsMKZNMgGhmMuSH83sfrI/2KnT02LAACjgFYyGfIGAFnw?=
 =?us-ascii?Q?1LyrENaQVKrxPBYLzhwLlL1KaJttdLcW+zPra0zuPMq0yGTlW2rcb/ntYVZ/?=
 =?us-ascii?Q?WbXL+F+KkzClS/1eokTyJtdlduQTahrMCG2O/ZQKdsTu4G1BFasgYHuQtKlF?=
 =?us-ascii?Q?vzsm47KKzKcB+Ly6z/L4DIu6ASB3s86MvJQ251tZoZtQ2J8s60QY1D4kRd6M?=
 =?us-ascii?Q?B2k0ZQ9f70/rlXtWOS9lbPN2K953rCWIeQnOsn3prax/9Vnw/KAm10PdvZqw?=
 =?us-ascii?Q?DbKH4RXmEfaEMrKZDfGBo2Ty?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-ID: <191EF1BC60CEDA4BA54D63A66D8A2290@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: 7288cdcd-b889-4044-a992-08d95be21e3b
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Aug 2021 09:34:57.5929
 (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: jot+X/jf4GTXyalIHpWZ7ajCVmN4bZQBGRT6bNIfdd1nD7N5agkvggXKd3UTcy6Mq8NJYvGeO51u+ndF38+ORg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4813
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, Aug 10, 2021 at 10:12:38AM +0100, Russell King - ARM Linux admin wr=
ote:
> There's a difference between:
>=20
> 	struct foo bar =3D { 0 };
>=20
> and
>=20
> 	struct foo bar =3D { };
>=20
> The former tells the compiler that you wish to set the first member of
> struct foo, which will be an integer type, to zero. The latter is an
> empty initialiser where all members and sub-members of the structure
> default to a zero value.
>=20
> You should have no problem with the latter. You will encounter problems
> with the former if the first member of struct foo is not an integer
> type.

Ok, that's good to know. Seeing that this patch has not been applied yet
I'll go for a v2.=