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 EA880C433F5 for ; Fri, 15 Oct 2021 08:26:13 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id AE29661181 for ; Fri, 15 Oct 2021 08:26:13 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org AE29661181 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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=/9ul9NASA8FJxPH6tJakqknWHibPKmpprZVkkm+66bI=; b=EIix4WWS/bOBrM 8rLX2oXdq0mwoP2A9BIYqOvnQQeUU6fWMVnM44edA23aPsc0HicUNj2ZEsxOW6TI5t94DcXwKahMy O0wmR6MY1Iklin/QOtHkVQn6GYWQVrd/6tnnAFF0E2pwO3leACRL3cmNZHih5dCd+m6oqDmKHj5hg V0zapfrYt+z5LHgT+J/k309pDFqm3Ig/Ov0PBEUEfZ+c0hxrEVP7J7x9N8f4M6drN/+BwV/yq53bZ RFjigj1rtxv6CxR8cP7mYj3fID4PgZg3hr3vRljsWde5Fc9pMFXAtW2F9HhI2yQ5GOdw2z7Ju4AJn YGMbpaWftpXL9G+H0buw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mbIVm-005y2f-QU; Fri, 15 Oct 2021 08:24:20 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mbIQy-005wB2-Hz; Fri, 15 Oct 2021 08:19:22 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 6028760249; Fri, 15 Oct 2021 08:19:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1634285959; bh=ZtB79vn0N+nDpFFIfpwm8g6mWwuaDQm7mVlJLNDK7nY=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=l6xJIVZQLzDcy+1R0WUSiieiyp674YTinwCtDARIUvlSRKlLeFA68sOGYLhHIUPFw W/9jgsQTsh1xELW5qOnSpfBDer61gUVrp3Mzazvypls8lIfab1INUyODnezZfshRwZ RkxucAwgtUbUjEEzJekRkQzB+Q8ek2+bOyjt/fmwXkkDugGOFAzpJH1xoVCq8ydund xb2rQD/gpPUl8DQDB+U2v5Y3+ndi3FoE7m8FwTpN7gIAWEdjYmUEucGYR3klRhKXti gw5KYdg3+Icx3SAbJdskGoILaYcCIB2o2CdGXwNv2VX/VMQxPLuNt39FdVcwu4GJ4r 728cxCySUaypA== Message-ID: <747ce6bf4b3511fd13408a52a301f63e37210883.camel@kernel.org> Subject: Re: [PATCH] staging: vchiq_arm: Add 36-bit address support From: nicolas saenz julienne To: Mwesigwa Guma , linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , Stefan Wahren , Ojaswin Mujoo , Dan Carpenter , Amarjargal Gundjalam , Phil Elwell , bcm-kernel-feedback-list@broadcom.com, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev, fedora-rpi@googlegroups.com, Joel Savitz , Chukpozohn Toe , Clark Williams Date: Fri, 15 Oct 2021 10:19:13 +0200 In-Reply-To: <20211014223230.451659-1-mguma@redhat.com> References: <20211014223230.451659-1-mguma@redhat.com> User-Agent: Evolution 3.40.4 (3.40.4-2.fc34) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211015_011920_700193_78516BF7 X-CRM114-Status: GOOD ( 17.23 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Mwesigwa, On Thu, 2021-10-14 at 18:32 -0400, Mwesigwa Guma wrote: > Cc: Nicolas Saenz Julienne > Cc: Greg Kroah-Hartman > Cc: Stefan Wahren > Cc: Ojaswin Mujoo > Cc: Dan Carpenter > Cc: Amarjargal Gundjalam > Cc: Phil Elwell > Cc: bcm-kernel-feedback-list@broadcom.com > Cc: linux-rpi-kernel@lists.infradead.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-staging@lists.linux.dev > Cc: fedora-rpi@googlegroups.com > Cc: Joel Savitz > Cc: Chukpozohn Toe > Cc: Clark Williams > > This is a forward port of Phil Elwell's commit from the Raspberry Pi > Linux fork described as follows [1]: > > Conditional on a new compatible string, change the pagelist encoding > such that the top 24 bits are the pfn, leaving 8 bits for run length > (-1), giving a 36-bit address range. > > Manage the split between addresses for the VPU and addresses for the > 40-bit DMA controller with a dedicated DMA device pointer that on non- > BCM2711 platforms is the same as the main VCHIQ device. This allows > the VCHIQ node to stay in the usual place in the DT. > > This commit enables VCHIQ device access on a Raspberry Pi 4B running the > mainline Linux kernel. > > Tested on Fedora Linux running on a Raspberry Pi 4B. > > [1]: https://github.com/raspberrypi/linux/commit/97268fd23eb8d08dc74eac5e3fd697303de26610 > > Signed-off-by: Mwesigwa Guma I see a lot happening on this patch. You're: - Registering a number of child devices that don't seem to exist upstream ('vcsm-cma', 'bcm2835-codec', and 'bcm2825-isp'). - Updating vchiq_register_child(). - Adding brcm,bcm2711-vchiq's compatible string (no dt bindings? see Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt). - Looking for 'brcm,bcm2711-dma' in the devicetree. - Using 'brcm,bcm2711-dma's' device node to re-generate the page lists. Each one of these should at least be a separate patch, which proper justification[1]. You can't just take downstream fixes, rebase them and send them upstream. You really have to own them, undestand what's happening and repurpose everything so it's up to standard even if it means diverging from what downstream is doing. Regards, Nicolas [1] Have a look at Documentation/process/submitting-patches.rst. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel