From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 92816C433EF for ; Thu, 18 Nov 2021 04:15:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7440261B42 for ; Thu, 18 Nov 2021 04:15:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242870AbhKRESM (ORCPT ); Wed, 17 Nov 2021 23:18:12 -0500 Received: from mail.kernel.org ([198.145.29.99]:36120 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242046AbhKRESF (ORCPT ); Wed, 17 Nov 2021 23:18:05 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id EB68C61B54; Thu, 18 Nov 2021 04:15:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1637208906; bh=Z6XrZ6YI1n+U0f0O0SgskN+T9tiaEHj7dLG+s1QIXko=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=NSOH1vpNFWSOBaCI9QFszpofxY4DnXsR3rLePkEtbev6ChvWn2ZJ/1DDWMq+J2Dot fA8h7Wt0gXXKZYqpHtZMiy20qwmVpRxFcwrW589jmt3XqP7Kp3NCLN6PLi2IAC69Uq xEsdprMgwgZoeqR5vlBBdpFIrQ9/1gyPx7z8kBx5MU2BddPjATAZQusu7HWZ2lZcGk oXir/2rpAaChKeNH/IjI/YMhIA2Bt8gaMznNKuYIaIq9/+iBQaiCWe7+2+4XYRPQRm L6BTSZX6HUuQjReEndyWQQa4+a3Rmv/IrtHIYEQVz79RBLcqrrw/LAFrkp7X8tSDg7 5vxiIuJN0j6UA== From: Jakub Kicinski To: davem@davemloft.net Cc: netdev@vger.kernel.org, Jakub Kicinski , l.stelmach@samsung.com Subject: [PATCH net-next 1/9] net: ax88796c: don't write to netdev->dev_addr directly Date: Wed, 17 Nov 2021 20:14:53 -0800 Message-Id: <20211118041501.3102861-2-kuba@kernel.org> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20211118041501.3102861-1-kuba@kernel.org> References: <20211118041501.3102861-1-kuba@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The future is here, convert the new driver as we are about to make netdev->dev_addr const. Signed-off-by: Jakub Kicinski --- CC: l.stelmach@samsung.com --- drivers/net/ethernet/asix/ax88796c_main.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/net/ethernet/asix/ax88796c_main.c b/drivers/net/ethernet/asix/ax88796c_main.c index e230d8d0ff73..e7a9f9863258 100644 --- a/drivers/net/ethernet/asix/ax88796c_main.c +++ b/drivers/net/ethernet/asix/ax88796c_main.c @@ -144,12 +144,13 @@ static void ax88796c_set_mac_addr(struct net_device *ndev) static void ax88796c_load_mac_addr(struct net_device *ndev) { struct ax88796c_device *ax_local = to_ax88796c_device(ndev); + u8 addr[ETH_ALEN]; u16 temp; lockdep_assert_held(&ax_local->spi_lock); /* Try the device tree first */ - if (!eth_platform_get_mac_address(&ax_local->spi->dev, ndev->dev_addr) && + if (!platform_get_ethdev_address(&ax_local->spi->dev, ndev) && is_valid_ether_addr(ndev->dev_addr)) { if (netif_msg_probe(ax_local)) dev_info(&ax_local->spi->dev, @@ -159,18 +160,19 @@ static void ax88796c_load_mac_addr(struct net_device *ndev) /* Read the MAC address from AX88796C */ temp = AX_READ(&ax_local->ax_spi, P3_MACASR0); - ndev->dev_addr[5] = (u8)temp; - ndev->dev_addr[4] = (u8)(temp >> 8); + addr[5] = (u8)temp; + addr[4] = (u8)(temp >> 8); temp = AX_READ(&ax_local->ax_spi, P3_MACASR1); - ndev->dev_addr[3] = (u8)temp; - ndev->dev_addr[2] = (u8)(temp >> 8); + addr[3] = (u8)temp; + addr[2] = (u8)(temp >> 8); temp = AX_READ(&ax_local->ax_spi, P3_MACASR2); - ndev->dev_addr[1] = (u8)temp; - ndev->dev_addr[0] = (u8)(temp >> 8); + addr[1] = (u8)temp; + addr[0] = (u8)(temp >> 8); - if (is_valid_ether_addr(ndev->dev_addr)) { + if (is_valid_ether_addr(addr)) { + eth_hw_addr_set(ndev, addr); if (netif_msg_probe(ax_local)) dev_info(&ax_local->spi->dev, "MAC address read from ASIX chip\n"); -- 2.31.1