From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from galahad.ideasonboard.com ([185.26.127.97]:38925 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751455AbeCUJ5v (ORCPT ); Wed, 21 Mar 2018 05:57:51 -0400 From: Laurent Pinchart To: Arnd Bergmann Cc: Frank.Rowand@sony.com, David Airlie , niklas.soderlund+renesas@ragnatech.se, dri-devel , Linux-Renesas , Linux Kernel Mailing List , Frank Rowand Subject: Re: [PATCH] [RFC] drm: rcar-du: keep temporary dtb files around during build Date: Wed, 21 Mar 2018 11:58:57 +0200 Message-ID: <2654454.izifTNoijI@avalon> In-Reply-To: References: <20180315153730.3798004-1-arnd@arndb.de> <8748007.hiNNEuVPPY@avalon> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Arnd, On Wednesday, 21 March 2018 09:38:35 EET Arnd Bergmann wrote: > On Tue, Mar 20, 2018 at 9:15 PM, Laurent Pinchart wrote: > > On Friday, 16 March 2018 10:25:25 EET Arnd Bergmann wrote: > >> On Fri, Mar 16, 2018 at 2:39 AM, wrote: > >>> On Thursday, March 15, 2018 8:37 AM, Arnd Bergmann wrote: > >>>> The *.dtb and *.dtb.S files get removed by 'make' during the build > >>>> process, and later seem to be missed during the 'modpost' stage: > >>>> > >>>> rm drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7795.dtb > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7791.dtb > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7791.dtb.S > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7795.dtb.S > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7790.dtb.S > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7793.dtb > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7796.dtb > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7790.dtb > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7796.dtb.S > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7793.dtb.S > >>>> WARNING: could not open > >>>> drivers/gpu/drm/rcar-du/rcar_du_of_lvds_r8a7790.dtb.S: No such file or > >>>> directory > >>>> > >>>> As a workaround, this adds all those files to the 'extra-y' target > >>>> list, but that's really ugly. Any ideas for a better fix? > >>> > >>> Does this work for you (untested, but the way it is done in > >>> drivers/of/unittest-data/Makefile): > >>> > >>> .PRECIOUS: \ > >>> $(obj)/%.dtb.S \ > >>> $(obj)/%.dtb > >> > >> Yes, that works and looks much better than my version. > > > > Thank you for your patch, and sorry for breaking the build. Do you plan to > > submit a new version based on Frank's approach ? > > I'm currently at Linaro Connect and won't be able to send a tested patch > before mid next week. If you want it earlier, feel free to apply that patch > with my original description and 'Reported-by: Arnd Bergmann > '. I was about to do so, but realized I can't reproduce the problem here. If you don't mind I'll let you submit a v2. Alternatively I can work on it if you give me instruction to reproduce the issue. -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH] [RFC] drm: rcar-du: keep temporary dtb files around during build Date: Wed, 21 Mar 2018 11:58:57 +0200 Message-ID: <2654454.izifTNoijI@avalon> References: <20180315153730.3798004-1-arnd@arndb.de> <8748007.hiNNEuVPPY@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from galahad.ideasonboard.com (galahad.ideasonboard.com [IPv6:2001:4b98:dc2:45:216:3eff:febb:480d]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2492B88007 for ; Wed, 21 Mar 2018 09:57:51 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Arnd Bergmann Cc: niklas.soderlund+renesas@ragnatech.se, David Airlie , Linux Kernel Mailing List , dri-devel , Linux-Renesas , Frank Rowand , Frank.Rowand@sony.com List-Id: dri-devel@lists.freedesktop.org SGkgQXJuZCwKCk9uIFdlZG5lc2RheSwgMjEgTWFyY2ggMjAxOCAwOTozODozNSBFRVQgQXJuZCBC ZXJnbWFubiB3cm90ZToKPiBPbiBUdWUsIE1hciAyMCwgMjAxOCBhdCA5OjE1IFBNLCBMYXVyZW50 IFBpbmNoYXJ0IHdyb3RlOgo+ID4gT24gRnJpZGF5LCAxNiBNYXJjaCAyMDE4IDEwOjI1OjI1IEVF VCBBcm5kIEJlcmdtYW5uIHdyb3RlOgo+ID4+IE9uIEZyaSwgTWFyIDE2LCAyMDE4IGF0IDI6Mzkg QU0sICA8RnJhbmsuUm93YW5kQHNvbnkuY29tPiB3cm90ZToKPiA+Pj4gT24gVGh1cnNkYXksIE1h cmNoIDE1LCAyMDE4IDg6MzcgQU0sIEFybmQgQmVyZ21hbm4gd3JvdGU6Cj4gPj4+PiBUaGUgKi5k dGIgYW5kICouZHRiLlMgZmlsZXMgZ2V0IHJlbW92ZWQgYnkgJ21ha2UnIGR1cmluZyB0aGUgYnVp bGQKPiA+Pj4+IHByb2Nlc3MsIGFuZCBsYXRlciBzZWVtIHRvIGJlIG1pc3NlZCBkdXJpbmcgdGhl ICdtb2Rwb3N0JyBzdGFnZToKPiA+Pj4+IAo+ID4+Pj4gcm0gZHJpdmVycy9ncHUvZHJtL3JjYXIt ZHUvcmNhcl9kdV9vZl9sdmRzX3I4YTc3OTUuZHRiCj4gPj4+PiBkcml2ZXJzL2dwdS9kcm0vcmNh ci1kdS9yY2FyX2R1X29mX2x2ZHNfcjhhNzc5MS5kdGIKPiA+Pj4+IGRyaXZlcnMvZ3B1L2RybS9y Y2FyLWR1L3JjYXJfZHVfb2ZfbHZkc19yOGE3NzkxLmR0Yi5TCj4gPj4+PiBkcml2ZXJzL2dwdS9k cm0vcmNhci1kdS9yY2FyX2R1X29mX2x2ZHNfcjhhNzc5NS5kdGIuUwo+ID4+Pj4gZHJpdmVycy9n cHUvZHJtL3JjYXItZHUvcmNhcl9kdV9vZl9sdmRzX3I4YTc3OTAuZHRiLlMKPiA+Pj4+IGRyaXZl cnMvZ3B1L2RybS9yY2FyLWR1L3JjYXJfZHVfb2ZfbHZkc19yOGE3NzkzLmR0Ygo+ID4+Pj4gZHJp dmVycy9ncHUvZHJtL3JjYXItZHUvcmNhcl9kdV9vZl9sdmRzX3I4YTc3OTYuZHRiCj4gPj4+PiBk cml2ZXJzL2dwdS9kcm0vcmNhci1kdS9yY2FyX2R1X29mX2x2ZHNfcjhhNzc5MC5kdGIKPiA+Pj4+ IGRyaXZlcnMvZ3B1L2RybS9yY2FyLWR1L3JjYXJfZHVfb2ZfbHZkc19yOGE3Nzk2LmR0Yi5TCj4g Pj4+PiBkcml2ZXJzL2dwdS9kcm0vcmNhci1kdS9yY2FyX2R1X29mX2x2ZHNfcjhhNzc5My5kdGIu Uwo+ID4+Pj4gV0FSTklORzogY291bGQgbm90IG9wZW4KPiA+Pj4+IGRyaXZlcnMvZ3B1L2RybS9y Y2FyLWR1L3JjYXJfZHVfb2ZfbHZkc19yOGE3NzkwLmR0Yi5TOiBObyBzdWNoIGZpbGUgb3IKPiA+ Pj4+IGRpcmVjdG9yeQo+ID4+Pj4gCj4gPj4+PiBBcyBhIHdvcmthcm91bmQsIHRoaXMgYWRkcyBh bGwgdGhvc2UgZmlsZXMgdG8gdGhlICdleHRyYS15JyB0YXJnZXQKPiA+Pj4+IGxpc3QsIGJ1dCB0 aGF0J3MgcmVhbGx5IHVnbHkuIEFueSBpZGVhcyBmb3IgYSBiZXR0ZXIgZml4Pwo+ID4+PiAKPiA+ Pj4gRG9lcyB0aGlzIHdvcmsgZm9yIHlvdSAodW50ZXN0ZWQsIGJ1dCB0aGUgd2F5IGl0IGlzIGRv bmUgaW4KPiA+Pj4gZHJpdmVycy9vZi91bml0dGVzdC1kYXRhL01ha2VmaWxlKToKPiA+Pj4gCj4g Pj4+IC5QUkVDSU9VUzogXAo+ID4+PiAgICAgICAgICQob2JqKS8lLmR0Yi5TIFwKPiA+Pj4gICAg ICAgICAkKG9iaikvJS5kdGIKPiA+PiAKPiA+PiBZZXMsIHRoYXQgd29ya3MgYW5kIGxvb2tzIG11 Y2ggYmV0dGVyIHRoYW4gbXkgdmVyc2lvbi4KPiA+IAo+ID4gVGhhbmsgeW91IGZvciB5b3VyIHBh dGNoLCBhbmQgc29ycnkgZm9yIGJyZWFraW5nIHRoZSBidWlsZC4gRG8geW91IHBsYW4gdG8KPiA+ IHN1Ym1pdCBhIG5ldyB2ZXJzaW9uIGJhc2VkIG9uIEZyYW5rJ3MgYXBwcm9hY2ggPwo+IAo+IEkn bSBjdXJyZW50bHkgYXQgTGluYXJvIENvbm5lY3QgYW5kIHdvbid0IGJlIGFibGUgdG8gc2VuZCBh IHRlc3RlZCBwYXRjaAo+IGJlZm9yZSBtaWQgbmV4dCB3ZWVrLiBJZiB5b3Ugd2FudCBpdCBlYXJs aWVyLCBmZWVsIGZyZWUgdG8gYXBwbHkgdGhhdCBwYXRjaAo+IHdpdGggbXkgb3JpZ2luYWwgZGVz Y3JpcHRpb24gYW5kICdSZXBvcnRlZC1ieTogQXJuZCBCZXJnbWFubgo+IDxhcm5kQGFybmRiLmRl PicuCgpJIHdhcyBhYm91dCB0byBkbyBzbywgYnV0IHJlYWxpemVkIEkgY2FuJ3QgcmVwcm9kdWNl IHRoZSBwcm9ibGVtIGhlcmUuIElmIHlvdSAKZG9uJ3QgbWluZCBJJ2xsIGxldCB5b3Ugc3VibWl0 IGEgdjIuIEFsdGVybmF0aXZlbHkgSSBjYW4gd29yayBvbiBpdCBpZiB5b3UgCmdpdmUgbWUgaW5z dHJ1Y3Rpb24gdG8gcmVwcm9kdWNlIHRoZSBpc3N1ZS4KCi0tIApSZWdhcmRzLAoKTGF1cmVudCBQ aW5jaGFydAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K ZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0 dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==