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 AC88ED30004 for ; Fri, 18 Oct 2024 13:03:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=VSQO7r0VffbAVkvnKCwDGUDsOojO1/89Cgk1LLaHVF0=; b=JWlZIHmjUmyrWKebO4JY+QsHwd UeT+D2jPjPrfKT26Dhs13oRuiv+im4hxvrAKycK1yUheBbA5D37qE2PXK/xMeR2AgXmfWM4j37pnO fOLdWTZyUn6DMRZd8RSy6SZquoZzPOfhTZfNMF2EnPGr0467U6PstKjc5TQNaHNMt60CDFwTf1udv d1nlS50E7lk8nLhmKBPi+yN7keA5RBlB91iRYqBFpfdykzS8IGhg6qndgHf6V8hW8sSgn6luT0U/3 rDmUNFFoNk92yKgPYkMp/NLGMbistnz1unco5y6nW45rsTIhfY1XoTm4QMSRRTioNsnDQnsNxnGfH TwZmMRwA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t1mdL-00000000pAn-0fUR; Fri, 18 Oct 2024 13:03:11 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t1mXe-00000000oWQ-10cD for linux-arm-kernel@lists.infradead.org; Fri, 18 Oct 2024 12:57:20 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5E4A5106F; Fri, 18 Oct 2024 05:57:44 -0700 (PDT) Received: from bogus (e133711.arm.com [10.1.196.55]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8EBC63F7F5; Fri, 18 Oct 2024 05:57:12 -0700 (PDT) Date: Fri, 18 Oct 2024 13:57:09 +0100 From: Sudeep Holla To: Florian Fainelli Cc: linux-arm-kernel@lists.infradead.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Cristian Marussi , Sudeep Holla , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , "open list:SYSTEM CONTROL & POWER/MANAGEMENT INTERFACE" , justin.chen@broadcom.com, opendmb@gmail.com, kapil.hali@broadcom.com, bcm-kernel-feedback-list@broadcom.com, Arnd Bergmann Subject: Re: [PATCH v4 2/2] firmware: arm_scmi: Support 'reg-io-width' property for shared memory Message-ID: References: <20240827182450.3608307-1-florian.fainelli@broadcom.com> <20240827182450.3608307-3-florian.fainelli@broadcom.com> <20240903154000.GA2080277@bogus> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240903154000.GA2080277@bogus> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241018_055718_396091_59C07F8A X-CRM114-Status: GOOD ( 23.05 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Sep 03, 2024 at 04:40:00PM +0100, Sudeep Holla wrote: > On Tue, Aug 27, 2024 at 11:24:50AM -0700, Florian Fainelli wrote: > > Some shared memory areas might only support a certain access width, > > such as 32-bit, which memcpy_{from,to}_io() does not adhere to at least > > on ARM64 by making both 8-bit and 64-bit accesses to such memory. > > > > Update the shmem layer to support reading from and writing to such > > shared memory area using the specified I/O width in the Device Tree. The > > various transport layers making use of the shmem.c code are updated > > accordingly to pass the I/O accessors that they store. > > > > This looks good to me now, much simpler. I will push this to -next soon, > but it won't be for v6.12. I have already sent PR for that. I want this > to be in -next for longer just to see if anyone has any comments and > doesn't break any platform(which it shouldn't anyways). > > Just hoping if anyone looks at it and have feedback once it is in -next. > I will apply formally at v6.12-rc1 and report back if no one complains > until then. > Hi Florian, Just thought I will check with you if the content is -next are fine as I now recall I did the rebase as this patch was original posted before the rework of transport as modules were merged. Please confirm if you are happy with the rebase as you see in -next. I also had to rebase it on recent fixes that Justin added as there were trivial conflicts. Another thing I wanted to check is if [1] series has any impact on this. IIUC no, but it would be good to give a go in terms of testing just in case that as well lands in -next. -- Regards, Sudeep [1] https://lore.kernel.org/all/20241010123627.695191-1-jvetter@kalrayinc.com