From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2F7BA5D482 for ; Wed, 22 Nov 2023 18:23:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="cfx7RL1y" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 41197C433C7; Wed, 22 Nov 2023 18:23:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700677428; bh=yZFggWdceu7rn7hUogOUO9F9Pf31Wzul4TPKXAC/3U0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=cfx7RL1y4ISdbaje+SGuou7CtcmZrlVSo+mA0tr2l4eyuKze0qLS9FLGsH/Uvv546 s7lFQoD9HyEbwfoxy4Cvr6kIYavw/xkIhD5Bn5iNAQKfzXemwOgQb3bplHnFm5BIg7 w5kgoNaYBleytc03xf9iMLLT8dpedTeV5o0w4OEXK1C+iS0AJ4kNeMn1VzMRH5PHgQ U4MvOpuzoVT+CffnRb3OADnazbboACsZBf0jhbD2rj8OQrhsqR4YB24oNzNd2p9IGm gmTmjf9ABiSuJpyWpqzQ0LvRp116ghdK8gRzUr9Sq4s4P6kzIC1GKYmZwcBH5PdgmM +k9NrEe+MlV+g== Date: Wed, 22 Nov 2023 10:23:47 -0800 From: Jakub Kicinski To: Christian Marangi Cc: "Russell King (Oracle)" , Andrew Lunn , Heiner Kallweit , "David S. Miller" , Eric Dumazet , Paolo Abeni , Robert Marko , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel test robot Subject: Re: [net-next PATCH] net: phy: aquantia: drop wrong endianness conversion for addr and CRC Message-ID: <20231122102347.0bde86bb@kernel.org> In-Reply-To: <655e4025.df0a0220.50550.3d70@mx.google.com> References: <20231122170813.1222-1-ansuelsmth@gmail.com> <655e4025.df0a0220.50550.3d70@mx.google.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 22 Nov 2023 18:53:39 +0100 Christian Marangi wrote: > So they DO get converted to the HOST endian on reading the firmware from > an nvmem cell or a filesystem? They don't get converted when "reading from nvmem / fs". They get converted when you do: word = get_unaligned((const u32 *)(data + pos)); get_unaligned() is basically: #if BIGENDIAN #define get_unaligned get_unaligned_be32 #else #define get_unaligned get_unaligned_le32 #endif so you'll get different behavior here depending on the CPU.