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 26BC4CD3448 for ; Tue, 3 Sep 2024 16:00:24 +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=XdgZeiDfpysWm9lWTgXqvjatGdoyBFeI3d1tG17BlSs=; b=aDIQUClDIBhWLQj9fsEaikUpSx MZFrP8pt9mAsbxRL0SUiOgAtF3WAmxMo8FT74ZNJ72OqUxjjBTDcAyc4KVSKopabsHZri7hv762+L slqWdR9sN5hgFBIvUmMDc6SHq6+ubKfOBYCw97PJPjT9KsFo90aJ1231TCweAbuf92NozQKJe+xuj C4N84f3+lS5q4jT25r4mj0dEEDPl7YK6tmSAIu2csOmc0rQPT7P5FKoWTBR4Di+V48D8na9TLVwqS USp4SvnYeDv/e89uURKwYF9Cx6zBlv9aC+UPOT5HpWwqZwcnOEOnyhmaOzDZB/ICtaIjOxC+2cGYS Sm+98vDg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1slVwy-00000000zTF-00zf; Tue, 03 Sep 2024 16:00:12 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1slVdj-00000000uzj-0QZf for linux-arm-kernel@lists.infradead.org; Tue, 03 Sep 2024 15:40: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 3A13FFEC; Tue, 3 Sep 2024 08:40:43 -0700 (PDT) Received: from bogus (unknown [10.57.81.106]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B479C3F66E; Tue, 3 Sep 2024 08:40:14 -0700 (PDT) Date: Tue, 3 Sep 2024 16:40:00 +0100 From: Sudeep Holla To: Florian Fainelli Cc: linux-arm-kernel@lists.infradead.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sudeep Holla , Cristian Marussi , "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: <20240903154000.GA2080277@bogus> References: <20240827182450.3608307-1-florian.fainelli@broadcom.com> <20240827182450.3608307-3-florian.fainelli@broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240827182450.3608307-3-florian.fainelli@broadcom.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240903_084019_234174_6F47224C X-CRM114-Status: GOOD ( 14.16 ) 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, 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. -- Regards, Sudeep