From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org ([140.211.169.12]:46778 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751059AbdJCI6K (ORCPT ); Tue, 3 Oct 2017 04:58:10 -0400 Date: Tue, 3 Oct 2017 10:58:18 +0200 From: Greg KH To: Robert Jarzmik Cc: stable@vger.kernel.org, Andrea Adami Subject: Re: Issue with pxa_dma driver on linux 4.4.y Message-ID: <20171003085818.GG15659@kroah.com> References: <87ing6bgzt.fsf@belgarion.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87ing6bgzt.fsf@belgarion.home> Sender: stable-owner@vger.kernel.org List-ID: On Mon, Sep 25, 2017 at 11:33:42PM +0200, Robert Jarzmik wrote: > Dear maintainers, > > In the tagging of fixing commits, I have submitted this patch which was merged > into the stable tree : > - 6bab1c6afdca ("dmaengine: pxa_dma: fix the maximum requestor line") > which translated into this commit in the stable tree : > - 94d75e190f19 ("dmaengine: pxa_dma: fix the maximum requestor line") > > Unfortunately I have not tagged the dependant patches this one relies on, and > more specifically : > - build breakage fix : > - c283e41ef324 ("dmaengine: mmp-pdma: add number of requestors") > - functionality breakage fix : > - 72b195cb7162 ("ARM: pxa: add the number of DMA requestor lines") > - 4c35430ad18f ("ARM: pxa: fix the number of DMA requestor lines") > > I see 2 ways out of this situation : > - either we revert stable commit 94d75e190f19 > => functionnality will remain broken, but only for a subset of DMA lines > - or apply the dependant patches, ie. c283e41ef324, 72b195cb7162, 4c35430ad18f > > It's up to you, but if you could choose one and apply it, the build breakage > will be gone. I've applied the extra patches, thanks for letting me know. > I must mention that there is no defconfig file so far with covers this specific > build, and this was seen by Andrea who reported the issue. Ah, that's why none of the build systems caught this. Perhaps we should add a defconfig for this? Is there one in Linus's tree yet? thanks, greg k-h