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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2DAC7CAC5A7 for ; Thu, 25 Sep 2025 10:13:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:Cc:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Ypn4YSrLQpxAQToXhjHon9t+Gs4Iwq0igqbrwfc7+V8=; b=CaHkRdFU70xA0v Q0l03DFSanLhqFbUvJyMehHls9Go6PM0PJmeE9BzS0e7sV+4tUcc/bhTuEY5kGKMu+a2X2zeexmGT AgakEZmjVYvqzv3gxqCo9fljDefmWpFpG+h41XCSOal+itihHU191B4TrOM3Fbvy4oyo6AgdoJd95 ku2lXK5VA33UMcOvwD/lXOSaxDylMdy+XcxNf8xXMygK0Ss8jZ+71V881zGxgrnSpvF9mIDyYTx0h /kXuf3LwRDftu7CpyAEEw/7SKShj06mqLTsRu/kkybPb6dpgkQO0soB1J/jWXzD6S1JC4boAzY84z 1//hEitEaR5y+IVaImYA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1v1iyR-00000007yJg-3xIe; Thu, 25 Sep 2025 10:13:15 +0000 Received: from mail-wm1-x32b.google.com ([2a00:1450:4864:20::32b]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1v1iyQ-00000007yJE-0vbL for linux-i3c@lists.infradead.org; Thu, 25 Sep 2025 10:13:15 +0000 Received: by mail-wm1-x32b.google.com with SMTP id 5b1f17b1804b1-45f2acb5f42so3767525e9.1 for ; Thu, 25 Sep 2025 03:13:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1758795192; x=1759399992; darn=lists.infradead.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=HmljixeQc8xgr6LVqYOS1SBQvDhtnhXHQXUSYqpft9s=; b=WmiJb39ZWU0smQK9hwrssGOxY05GWAB45E9v1in5gIlrCZVpz6DUf5YtoFH78z5aCR NAuj03pQCaRUFMhP44pLSXsqdOeltj08okYvwQEEL8JQ+yIMbBqFc38exqXY0Bo0KPKV Tkuev9yIpv88YjKysqH17cUn+SOpdFg18tTasyZIKiDeZ+PcrL0chklPjNW4cUwOUdf1 M+4sqHikgkD+l12J08upcjX/ylgM9qJsplaoMjI0r1NtpvpyE8RI6wIKr56cUAHTRCN6 f5/BjDSGtm28FA7Hg4GxBelDViCOe+tTKtURGTlwhfl6fEZGnI887Bcqt2fMMujbaYyy 1qfg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758795192; x=1759399992; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=HmljixeQc8xgr6LVqYOS1SBQvDhtnhXHQXUSYqpft9s=; b=iMXEybhQgvKZbbyFDGHq6DKBWpWDPaTMx7+4+kZpmw/kAREFqfEiF4YwVKG07AMtQf MtZsHHbsu+2oPEPCwvjbt6wd31Lyl2Z4BiXMKq5nQ5Pnif6ySyk7eEl31x4MLse79tYu +g/7luELAtGklvDS22pCeC2bcENgQRfgQ7GsrKfvv5jSMO/vNPSN5oe9DhFfVewbnDFM KGfLpVsyzDjkxCeXuFS7fxBuibEHWi2WP6e3i7j/iOzWu08RQuAPVy+ZM9Zd43cdEa31 e1ZxpdjNSBJFDWRTWF0iLhH/fTDVHWAgsN58JZcp7/+jWiyWr/0x6S+UzWzIjK1GQUc2 lDcA== X-Forwarded-Encrypted: i=1; AJvYcCWNEPsFWG1Uu+bGfVxV9EyoIWrS0Nn60qZ3E7bBXyTJpQDuSU1dNG96PBFJnAfe6Y+1X3nSgvvO25k=@lists.infradead.org X-Gm-Message-State: AOJu0YwMl9/j9Cc7Zg7KkstXf2ywZnH/ONZtqE/nnUBiylagoZNfFtK4 TpW620CqQWIrAXwGUBikwI6ayn4nBvld3zAlqyGdBwkIfgjNhpFmakLz X-Gm-Gg: ASbGncvN3jbuai2XtX3983XRfLTF0tscV4WnSa2ig8Y3vxdQUzROsdpKRA6OfnZBXbk o75WUsW7ysxhr2+DaurAywtExDJRau7hOzcHrAZnt43a4qUWXx0FA+f75jzkkYZJrPphTAycoq1 hsBoQK7sOV7PH7yChXOfHrmAr7GHdqh1H8V9rGtK8YLP0LvqsFgobp6Zdc5n35Sk0iVTQ46XIJ0 kQDGwKONcStbCdblpQjjwl6aRvn/QC0Lnqd4uUbiXbOd8fZLVHnNJBhfDD6cQ9RuL3SHJu0FTfm fTdUL8Lqm9ViyvD4r9CNSHd/+k/PxQ3elQ8abiiMTpl41L4Gh9NC9YanD8P6BHDD1AKnkl0XMsW iYvGt2vtSav7o/8+SZRuiJ+EnGA1Ut+k= X-Google-Smtp-Source: AGHT+IEC57Rdx2+h+FGeZVFTjH1ezVMChuJ93kmZqX1ZaXIDR/t+6gNu7QdgglKgxzNMt7eo3aLEiA== X-Received: by 2002:a05:600c:5d1:b0:46d:83e7:45ec with SMTP id 5b1f17b1804b1-46e33ca94eamr14208235e9.11.1758795191990; Thu, 25 Sep 2025 03:13:11 -0700 (PDT) Received: from [192.168.1.187] ([161.230.67.253]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-40fb9768bdesm2438865f8f.23.2025.09.25.03.13.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 25 Sep 2025 03:13:11 -0700 (PDT) Message-ID: Subject: Re: [PATCH] [v2] i3c: fix big-endian FIFO transfers From: Nuno =?ISO-8859-1?Q?S=E1?= To: Arnd Bergmann , Manikanta Guntupalli , Jorge Marques , Arnd Bergmann Cc: Alexandre Belloni , Jorge Marques , Wolfram Sang , Frank Li , "linux-i3c@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "git (AMD-Xilinx)" , Michal Simek Date: Thu, 25 Sep 2025 11:13:39 +0100 In-Reply-To: <91e73a29-96e3-4a52-addb-0cb954f46c04@app.fastmail.com> References: <20250924201837.3691486-1-arnd@kernel.org> <2wtpklapw5ogsevuvk2l4ngvw7hymer2y4cc454h47u2d7tq44@4mknmpk5yzil> <37d47af4f4d5220764efc5870630fdfc1e9be2c9.camel@gmail.com> <91e73a29-96e3-4a52-addb-0cb954f46c04@app.fastmail.com> User-Agent: Evolution 3.58.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250925_031314_279691_1D73B398 X-CRM114-Status: GOOD ( 27.91 ) X-BeenThere: linux-i3c@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-i3c" Errors-To: linux-i3c-bounces+linux-i3c=archiver.kernel.org@lists.infradead.org T24gVGh1LCAyMDI1LTA5LTI1IGF0IDExOjM1ICswMjAwLCBBcm5kIEJlcmdtYW5uIHdyb3RlOgo+ IE9uIFRodSwgU2VwIDI1LCAyMDI1LCBhdCAxMDo1OCwgTnVubyBTw6Egd3JvdGU6Cj4gPiBPbiBU aHUsIDIwMjUtMDktMjUgYXQgMDg6NDcgKzAwMDAsIEd1bnR1cGFsbGksIE1hbmlrYW50YSB3cm90 ZToKPiA+ID4gPiAoQU1ELVhpbGlueCkgPGdpdEBhbWQuY29tPjsgU2ltZWssIE1pY2hhbCA8bWlj aGFsLnNpbWVrQGFtZC5jb20+Cj4gPiA+ID4gU3ViamVjdDogUmU6IFtQQVRDSF0gW3YyXSBpM2M6 IGZpeCBiaWctZW5kaWFuIEZJRk8gdHJhbnNmZXJzCj4gPiA+ID4gT24gVGh1LCAyMDI1LTA5LTI1 IGF0IDA3OjM3ICswMDAwLCBHdW50dXBhbGxpLCBNYW5pa2FudGEgd3JvdGU6Cj4gPiA+ID4gPiA+ IGkzY0BsaXN0cy5pbmZyYWRlYWQub3JnOyBsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3JnCj4g PiA+ID4gPiA+IFN1YmplY3Q6IFJlOiBbUEFUQ0hdIFt2Ml0gaTNjOiBmaXggYmlnLWVuZGlhbiBG SUZPIHRyYW5zZmVycwo+ID4gPiA+ID4gPiBPbiBXZWQsIFNlcCAyNCwgMjAyNSBhdCAxMDoxODoz M1BNICswMjAwLCBBcm5kIEJlcmdtYW5uIHdyb3RlOgo+ID4gPiA+IAo+ID4gPiA+IEkgd291bGQg YXJndWUgdGhhdCdzIHNvbWV0aGluZyBmb3IgY2FsbGVycyBvZiB0aGVzZSBmdW5jdGlvbnMgdG8g Y2FyZQo+ID4gPiA+IGFib3V0Lgo+ID4gPiBJZiBlYWNoIEkzQyBkcml2ZXIgaGFzIHRvIGhhbmRs ZSBGSUZPIGVuZGlhbm5lc3MgaW5kaXZpZHVhbGx5LCBpdAo+ID4gPiBpbnRyb2R1Y2VzCj4gPiA+ IHVubmVjZXNzYXJ5IGR1cGxpY2F0aW9uIGFuZCBvdmVyaGVhZCBhY3Jvc3MgZHJpdmVycy4gQ2Vu dHJhbGl6aW5nIHRoaXMgaW4KPiA+ID4gdGhlCj4gPiA+IEZJRk8gYWNjZXNzIGhlbHBlcnMga2Vl cHMgdGhlIGxvZ2ljIGNvbnNpc3RlbnQsIGF2b2lkcyByZXBlYXRlZAo+ID4gPiBib2lsZXJwbGF0 ZSwKPiA+ID4gYW5kIHJlZHVjZXMgdGhlIGNoYW5jZSBvZiBzdWJ0bGUgYnVncy4KPiA+IAo+ID4g SSBtZWFuLCB0aGF0J3Mgd2hhdCBzcGkgYW5kIGkyYyBkcml2ZXJzIGRvIGFscmVhZHkuwqBXaXRo IGVudW0KPiA+IGkzY19maWZvX2VuZGlhbgo+ID4geW91J3JlIGFscmVhZHkgZm9yY2luZyB1c2Vy cyB0byBjYXJlIChvciBrbm93KSBhYm91dCBlbmRpYW5pc20gc28gdGhleSBtaWdodAo+ID4gYXMK PiA+IHdlbGwganVzdCBwYXNzIHRoZSBkYXRhIGluIHRoZSBwcm9wZXIgb3JkZXIgYWxyZWFkeSAo bm90IHN1cmUgaWYgaXQncyBzdWNoIGEKPiA+IGJpZwo+ID4gJ2J1cmRlbicpLgo+IAo+IENhbiB5 b3UgZ2l2ZSBhbiBleGFtcGxlIG9mIGFuIHNwaSBvciBpMmMgZHJpdmVyIGhhbmRsZXMgYSBzaW1p bGFyCj4gc2l0dWF0aW9uIHRvIHRoZSBuZXcgaTNjIGRyaXZlcj8gQXMgZmFyIGFzIEkgY2FuIHRl bGwsIHN3YXBwaW5nCj4gdGhlIGJ5dGVzIGluIGEgRklGTyByZWdpc3RlciBpcyB2ZXJ5IHVudXN1 YWwgZm9yIGEgaGFyZHdhcmUgZGVzaWduCj4gYW5kIHByb2JhYmx5IGEgbWlzdGFrZSByYXRoZXIg dGhhbiBhbiBpbnRlbnRpb25hbCBkZWNpc2lvbi4KPiAKCkkgbWVhbnQgdGhhdCBpMmMgYW5kIHNw aSBkcml2ZXJzIChhbmQgSSBtZWFudCBvbiB0aGUgZGV2aWNlIHNpZGUpIGFscmVhZHkgYXJlCnRo ZSBvbmVzIGhhdmluZyB0byBjYXJlIGFib3V0IHB1dHRpbmcgdGhlIGRhdGEgaW4gdGhlIHByb3Bl ciBlbmRpYW5pc20gc28gdGhhdApjb250cm9sbGVycyBkb24ndCBoYXZlIHRvIGNhcmUgKEFGQUlL KS4KCkJ1dCBJIHNvIHNlZSBub3cgdGhhdCB0aGUgYWJvdmUgaXMga2luZCBvZiB1bnJlbGF0ZWQu Cgo+IE9uIHRoZSBvdGhlciBoYW5kLCBJIGNhbiBmaW5kIGRyaXZlcnMgdGhhdCBhcmUgb2J2aW91 c2x5IHdyb25nCj4gb24gYmlnLWVuZGlhbiBrZXJuZWxzLCBzdWNoIGFzIFRlZ3JhJ3MgaTJjX3dy aXRlc2xfdmkoKSBmdW5jdGlvbgo+IGJlaW5nIHVuaW50ZW50aW9uYWxseSBzd2FwcGVkIGZyb20g aTJjX3dyaXRlc2woKSBvbiBiaWctZW5kaWFuLgo+IAo+IEZvciB0aGUgaTNjIGhlbHBlciwgSSB0 aGluayBKb3JnZSdzIGN1cnJlbnQgdmVyc2lvbiB3aXRoIG15Cj4gZml4IHNob3VsZCB3b3JrIGZv ciBldmVyeSBub3JtYWwgZHJpdmVyLCBhbmQgSSB3b3VsZCBub3QKPiB3YW50IHRvIG1ha2UgaXQg bW9yZSBjb21wbGljYXRlZCBmb3IgYW4gb2JzY3VyZSBjYXNlLiBUaGUKPiB2ZXJzaW9uIGZvciB0 aGUgQU1EIGRyaXZlciBjYW4ganVzdCBiZSBpbiB0aGF0IGRyaXZlciwgb3IKPiBpdCBjb3VsZCBi ZSBhIHNlcGFyYXRlIGZ1bmN0aW9uIG5hbWUgaW4gdGhlIGNvbW1vbiBoZWFkZXIKPiBpZiB0aGVy ZSBpcyBhIGNoYW5jZSB3ZSdsbCBuZWVkIGl0IGFnYWluLgo+IAoKSSBkbyBhZ3JlZSB3aXRoIHRo ZSBhYm92ZS4KCi0gTnVubyBTw6EKCj4gwqDCoMKgwqDCoCBBcm5kCgotLSAKbGludXgtaTNjIG1h aWxpbmcgbGlzdApsaW51eC1pM2NAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5m cmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWkzYwo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 02011502BE for ; Thu, 25 Sep 2025 10:13:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758795195; cv=none; b=Ksjn7OMdsE9vYUISz/R/B6DNpE3PAP676AB/Oj9I/ZPs20jQ4Bkc9/inqJ5Y7DkSGXz548lJxmk9Y9afkvJkbG+1kEOWMlxN/WTC9OO0qcmKyWdla22XDhWvuymZUMCyLuxyAsDn6Ls1iUdsHuBz/CtcgsHtpItt6CJZDB5hQvk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758795195; c=relaxed/simple; bh=HmljixeQc8xgr6LVqYOS1SBQvDhtnhXHQXUSYqpft9s=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=DfubHLYt7MbFCv54crk8sh8KZjfHZpzy3Kd23G8OkZCiK2TyaKF+JYxGtuMr7XgWyFOCP1CmwZKBi4tgjdJMxzoQzp3b2/opleHRsnbx1AymWNh+HeyquT8ZrmKpE/0MjCBlX8MJ8rLRDgIRIzbejQE54df3wX5bFenzecvlymM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=NFfQXCjf; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="NFfQXCjf" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-45f2acb5f42so3767555e9.1 for ; Thu, 25 Sep 2025 03:13:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1758795192; x=1759399992; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=HmljixeQc8xgr6LVqYOS1SBQvDhtnhXHQXUSYqpft9s=; b=NFfQXCjf6etA9/+U2b26pkIqP2Av9aJ+9uNI4guStBKsK6XO8y6mgBEd+YCNokqtj9 nhy4klZb8xQswIqn5O7hwGbR7bUCerniOBI1FspJ1dOhXEDeJ6GB7M8lc1AbSGHojr10 9o5sS0kUoErTWsnv4gLh/g9MvtVRsmYuRs4umwPhXYSxmhMhUzytR3Q3KNlztiBEwOpZ p6PqA6h1WTipyig/I6ixlTcJmZg08eQS1yPw963ZspnHLuIUFHK1+p0aSUsaSJ6kp1As 0WHvaPyx2q8so6NhzUX56mLXYettjds50Lgj0QXjep3pvnBuzFKykxAH3TnvfWFbymYM 3njA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758795192; x=1759399992; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=HmljixeQc8xgr6LVqYOS1SBQvDhtnhXHQXUSYqpft9s=; b=XcCVD1WrxzaGdy/Jjo3N1s0V5wJRS+cbqiFDfgFiFE2oOzpSKLdA/JlWF2nv58IlD+ YifxSi5BapQQ7swcye5JMDVKO5OSbJgbxxilG54gNVtOzj8Z7nxaLhnPVjJjog3iRVi7 AjbgxNcTZEOFTl9Y4ro/Njihes5EHul/lI+W7JfzA39Hh1ItY3OkS/b14DgtWsWftx5+ AT5Kew+OinjMuWSt2QB29anZlL22+CIOsp1mZbvCqw3PzUvOqSeRS8WH+BMA3AYcjUQC gw9Qg/VpPawunj1PZKSKAaZM4Bc0Pi1bHqv6Ozxg2z4SXNgsbi+SyeAsZ58wLl553pSb MTZQ== X-Forwarded-Encrypted: i=1; AJvYcCWMAgVeQL2Up13Gfu9EPpIIubLI8xuKNDNSNdp++fkD/K0nBj4vXJ6wwcJnfOtJ4GPxBG2JIIUP7oHty40=@vger.kernel.org X-Gm-Message-State: AOJu0YwnnpCeMkwwMbISVRNsrQunTfE6Vu5BKX7mjtZjYjvQQC/xJZfU ubNp7z5REoX/q1ExogauY+fOcrjxSQJDTHTAHS/Jn9c0wGJc8kSPLfSQ X-Gm-Gg: ASbGncuFqYfwoSgL/uukqNaRuPEdQTsTBsz3CUEVPAyL+3//tu7kacQHLi3hMKvC7n4 9Li5+4KLpIOhC32MsyJmU5WcaAD3JRkVdavNztZ8B/vKBIjPyG9VYPT8u+Axc/3X2osaV3Q4UHi bT8KkNht01y7JQjf4dzKBC8r4GyZQjUMyhow5L3YdMSS2O5FnhZ+17pcNMQyg70OaEAwoYCgRUF uGwRBMAuKckzoldq3W7eevunY59KTdxpWbpIjyrjunYVqGNg0y4X/HLfC4jYtF/P3wR4iODa9fa 4uJ6tJ89Z3fMGx9OYaEiuRSpD9yamr0QfNatXXxZQkIVBk8Pi+4iI7RkHEVimDRVcLTZTga7bac P86O9lynUaY9TG9U7q6RPROQN97xqM5M= X-Google-Smtp-Source: AGHT+IEC57Rdx2+h+FGeZVFTjH1ezVMChuJ93kmZqX1ZaXIDR/t+6gNu7QdgglKgxzNMt7eo3aLEiA== X-Received: by 2002:a05:600c:5d1:b0:46d:83e7:45ec with SMTP id 5b1f17b1804b1-46e33ca94eamr14208235e9.11.1758795191990; Thu, 25 Sep 2025 03:13:11 -0700 (PDT) Received: from [192.168.1.187] ([161.230.67.253]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-40fb9768bdesm2438865f8f.23.2025.09.25.03.13.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 25 Sep 2025 03:13:11 -0700 (PDT) Message-ID: Subject: Re: [PATCH] [v2] i3c: fix big-endian FIFO transfers From: Nuno =?ISO-8859-1?Q?S=E1?= To: Arnd Bergmann , Manikanta Guntupalli , Jorge Marques , Arnd Bergmann Cc: Alexandre Belloni , Jorge Marques , Wolfram Sang , Frank Li , "linux-i3c@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "git (AMD-Xilinx)" , Michal Simek Date: Thu, 25 Sep 2025 11:13:39 +0100 In-Reply-To: <91e73a29-96e3-4a52-addb-0cb954f46c04@app.fastmail.com> References: <20250924201837.3691486-1-arnd@kernel.org> <2wtpklapw5ogsevuvk2l4ngvw7hymer2y4cc454h47u2d7tq44@4mknmpk5yzil> <37d47af4f4d5220764efc5870630fdfc1e9be2c9.camel@gmail.com> <91e73a29-96e3-4a52-addb-0cb954f46c04@app.fastmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Thu, 2025-09-25 at 11:35 +0200, Arnd Bergmann wrote: > On Thu, Sep 25, 2025, at 10:58, Nuno S=C3=A1 wrote: > > On Thu, 2025-09-25 at 08:47 +0000, Guntupalli, Manikanta wrote: > > > > (AMD-Xilinx) ; Simek, Michal > > > > Subject: Re: [PATCH] [v2] i3c: fix big-endian FIFO transfers > > > > On Thu, 2025-09-25 at 07:37 +0000, Guntupalli, Manikanta wrote: > > > > > > i3c@lists.infradead.org; linux-kernel@vger.kernel.org > > > > > > Subject: Re: [PATCH] [v2] i3c: fix big-endian FIFO transfers > > > > > > On Wed, Sep 24, 2025 at 10:18:33PM +0200, Arnd Bergmann wrote: > > > >=20 > > > > I would argue that's something for callers of these functions to ca= re > > > > about. > > > If each I3C driver has to handle FIFO endianness individually, it > > > introduces > > > unnecessary duplication and overhead across drivers. Centralizing thi= s in > > > the > > > FIFO access helpers keeps the logic consistent, avoids repeated > > > boilerplate, > > > and reduces the chance of subtle bugs. > >=20 > > I mean, that's what spi and i2c drivers do already.=C2=A0With enum > > i3c_fifo_endian > > you're already forcing users to care (or know) about endianism so they = might > > as > > well just pass the data in the proper order already (not sure if it's s= uch a > > big > > 'burden'). >=20 > Can you give an example of an spi or i2c driver handles a similar > situation to the new i3c driver? As far as I can tell, swapping > the bytes in a FIFO register is very unusual for a hardware design > and probably a mistake rather than an intentional decision. >=20 I meant that i2c and spi drivers (and I meant on the device side) already a= re the ones having to care about putting the data in the proper endianism so t= hat controllers don't have to care (AFAIK). But I so see now that the above is kind of unrelated. > On the other hand, I can find drivers that are obviously wrong > on big-endian kernels, such as Tegra's i2c_writesl_vi() function > being unintentionally swapped from i2c_writesl() on big-endian. >=20 > For the i3c helper, I think Jorge's current version with my > fix should work for every normal driver, and I would not > want to make it more complicated for an obscure case. The > version for the AMD driver can just be in that driver, or > it could be a separate function name in the common header > if there is a chance we'll need it again. >=20 I do agree with the above. - Nuno S=C3=A1 > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Arnd