From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) (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 2F0C63FC8 for ; Tue, 21 Sep 2021 17:08:03 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 583B460E73; Tue, 21 Sep 2021 17:08:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1632244082; bh=euUUx+t+JzzcyJOuKhnF/K2dJ6grlDis26/k37xKIlo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GiWBiAGtqxKww2qoNr7Jt38z+wrN+FQhicejm2iFkmCx1fi9bbLTqC/B1/7rXVRrM tbLFJidwZaj3pUU904Ad6vV2ZTo5lti/gsp2qEupS+zcmjq64nD4TYMjU8Sts/bHmA gM36slXtsui5rfydghw7Q+Gn2yyRzWOzo8Ag61Bc= Date: Tue, 21 Sep 2021 19:08:00 +0200 From: Greg KH To: Sergio Paracuellos Cc: Thomas Bogendoerfer , Bjorn Helgaas , Matthias Brugger , "open list:MIPS" , linux-pci , linux-staging@lists.linux.dev, NeilBrown , linux-kernel Subject: Re: [PATCH v2 3/3] staging: mt7621-pci: set end limit for 'ioport_resource' Message-ID: References: <20210822161005.22467-1-sergio.paracuellos@gmail.com> <20210822161005.22467-4-sergio.paracuellos@gmail.com> <20210902110835.GB7614@alpha.franken.de> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Tue, Sep 21, 2021 at 07:02:55PM +0200, Sergio Paracuellos wrote: > On Tue, Sep 21, 2021 at 5:28 PM Greg KH wrote: > > > > On Thu, Sep 02, 2021 at 01:19:53PM +0200, Sergio Paracuellos wrote: > > > On Thu, Sep 2, 2021 at 1:08 PM Thomas Bogendoerfer > > > wrote: > > > > > > > > On Thu, Sep 02, 2021 at 12:15:12PM +0200, Sergio Paracuellos wrote: > > > > > On Thu, Sep 2, 2021 at 11:15 AM Greg KH wrote: > > > > > > > > > > > > On Sun, Aug 29, 2021 at 05:14:27PM +0200, Sergio Paracuellos wrote: > > > > > > > On Fri, Aug 27, 2021 at 11:01 AM Greg KH wrote: > > > > > > > > > > > > > > > > On Sun, Aug 22, 2021 at 06:10:05PM +0200, Sergio Paracuellos wrote: > > > > > > > > > We have increase IO_SPACE_LIMIT for ralink platform to get PCI IO resources > > > > > > > > > properly handled using PCI core APIs. To align those changes with driver > > > > > > > > > code we have to set 'ioport_resource' end limit to IO_SPACE_LIMIT to avoid > > > > > > > > > errors. > > > > > > > > > > > > > > > > > > Signed-off-by: Sergio Paracuellos > > > > > > > > > > > > > > > > Acked-by: Greg Kroah-Hartman > > > > > > > > > > > > > > Thanks. Since I am planning to move 'mt7621-pci' from staging to > > > > > > > 'drivers/pci/controller' and send v3 after the next merge window, I > > > > > > > prefer this patch to go through the staging tree. For the other two I > > > > > > > don't have any preference and it is ok for me to go through mips or > > > > > > > pci trees. So, Bjorn and Thomas is up to you if you are ok with the > > > > > > > changes. > > > > > > > > > > > > Yes, I would need acks for the other patches in the series if this is to > > > > > > come through the staging tree. > > > > > > > > > > Yes, I know it. Let's wait for Thomas and Bjorn preference for those > > > > > remaining two. > > > > > > > > I've sent my acked-by for the MIPS patch. > > > > > > Thanks! > > > > Ok, I took patches 1 and 3 in my tree now. Please submit patch 2 to the > > PCI developers and maintainer, as that is up to them to take, not me. > > Ok, thanks. I will resend the remaining patch if that is needed. Only > one concern here, only with those two patches applied the driver is > totally broken since it needs the remaining PATCH to make all the pci > subsystem work. Is this ok? Get the change acked by the PCI developers and I will be glad to also take it through my tree :)