linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus
@ 2015-09-04 12:49 Josh Boyer
  2015-09-06 11:20 ` Maxime Ripard
  0 siblings, 1 reply; 6+ messages in thread
From: Josh Boyer @ 2015-09-04 12:49 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Olof Johansson
  Cc: Josef Gajdusek, Maxime Ripard, devicetree, linux-arm-kernel,
	linux-kernel

Commit 79ae3e66f8d (ARM: dts: sun4i: Add Iteaduino Plus A10) added a new
make target for the sun4i-a10-itead-iteaduino-plus dts file, but mistakenly
used .dts instead of the correct .dtb suffix.  This resulted in a build error
like:

        scripts/Makefile.dtbinst:42: target 'sun4i-a10-itead-iteaduino-plus.dts' doesn't match the target pattern

when doing a make dtbs_install.

Fix it to use the proper file name.

Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
---
 arch/arm/boot/dts/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 233159d2eaab..bb8fa023d574 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -578,7 +578,7 @@ dtb-$(CONFIG_MACH_SUN4I) += \
 	sun4i-a10-hackberry.dtb \
 	sun4i-a10-hyundai-a7hd.dtb \
 	sun4i-a10-inet97fv2.dtb \
-	sun4i-a10-itead-iteaduino-plus.dts \
+	sun4i-a10-itead-iteaduino-plus.dtb \
 	sun4i-a10-jesurun-q5.dtb \
 	sun4i-a10-marsboard.dtb \
 	sun4i-a10-mini-xplus.dtb \
-- 
2.4.3


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus
  2015-09-04 12:49 [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus Josh Boyer
@ 2015-09-06 11:20 ` Maxime Ripard
  2015-09-29 14:32   ` Josh Boyer
  2015-10-05 22:55   ` Ben Hutchings
  0 siblings, 2 replies; 6+ messages in thread
From: Maxime Ripard @ 2015-09-06 11:20 UTC (permalink / raw)
  To: Josh Boyer
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Olof Johansson,
	Josef Gajdusek, devicetree, linux-arm-kernel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 712 bytes --]

On Fri, Sep 04, 2015 at 08:49:34AM -0400, Josh Boyer wrote:
> Commit 79ae3e66f8d (ARM: dts: sun4i: Add Iteaduino Plus A10) added a new
> make target for the sun4i-a10-itead-iteaduino-plus dts file, but mistakenly
> used .dts instead of the correct .dtb suffix.  This resulted in a build error
> like:
> 
>         scripts/Makefile.dtbinst:42: target 'sun4i-a10-itead-iteaduino-plus.dts' doesn't match the target pattern
> 
> when doing a make dtbs_install.
> 
> Fix it to use the proper file name.
> 
> Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>

Queued, thanks!

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus
  2015-09-06 11:20 ` Maxime Ripard
@ 2015-09-29 14:32   ` Josh Boyer
  2015-10-01 10:11     ` Maxime Ripard
  2015-10-05 22:55   ` Ben Hutchings
  1 sibling, 1 reply; 6+ messages in thread
From: Josh Boyer @ 2015-09-29 14:32 UTC (permalink / raw)
  To: Maxime Ripard
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Olof Johansson,
	Josef Gajdusek, open list:OPEN FIRMWARE AND...,
	linux-arm-kernel, Linux-Kernel@Vger. Kernel. Org

On Sun, Sep 6, 2015 at 7:20 AM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Fri, Sep 04, 2015 at 08:49:34AM -0400, Josh Boyer wrote:
>> Commit 79ae3e66f8d (ARM: dts: sun4i: Add Iteaduino Plus A10) added a new
>> make target for the sun4i-a10-itead-iteaduino-plus dts file, but mistakenly
>> used .dts instead of the correct .dtb suffix.  This resulted in a build error
>> like:
>>
>>         scripts/Makefile.dtbinst:42: target 'sun4i-a10-itead-iteaduino-plus.dts' doesn't match the target pattern
>>
>> when doing a make dtbs_install.
>>
>> Fix it to use the proper file name.
>>
>> Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
>
> Queued, thanks!

Are you going to push this to Olof or Linus for the 4.3 release?  I
don't see it in Linus' tree and this is still broken there.

josh

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus
  2015-09-29 14:32   ` Josh Boyer
@ 2015-10-01 10:11     ` Maxime Ripard
  0 siblings, 0 replies; 6+ messages in thread
From: Maxime Ripard @ 2015-10-01 10:11 UTC (permalink / raw)
  To: Josh Boyer
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Olof Johansson,
	Josef Gajdusek, open list:OPEN FIRMWARE AND...,
	linux-arm-kernel, Linux-Kernel@Vger. Kernel. Org

[-- Attachment #1: Type: text/plain, Size: 1129 bytes --]

Hi,

On Tue, Sep 29, 2015 at 10:32:59AM -0400, Josh Boyer wrote:
> On Sun, Sep 6, 2015 at 7:20 AM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> > On Fri, Sep 04, 2015 at 08:49:34AM -0400, Josh Boyer wrote:
> >> Commit 79ae3e66f8d (ARM: dts: sun4i: Add Iteaduino Plus A10) added a new
> >> make target for the sun4i-a10-itead-iteaduino-plus dts file, but mistakenly
> >> used .dts instead of the correct .dtb suffix.  This resulted in a build error
> >> like:
> >>
> >>         scripts/Makefile.dtbinst:42: target 'sun4i-a10-itead-iteaduino-plus.dts' doesn't match the target pattern
> >>
> >> when doing a make dtbs_install.
> >>
> >> Fix it to use the proper file name.
> >>
> >> Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
> >
> > Queued, thanks!
> 
> Are you going to push this to Olof or Linus for the 4.3 release?  I
> don't see it in Linus' tree and this is still broken there.

I was planning on sending the pull request for 4.3 sometime this week.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus
  2015-09-06 11:20 ` Maxime Ripard
  2015-09-29 14:32   ` Josh Boyer
@ 2015-10-05 22:55   ` Ben Hutchings
  2015-10-06  8:37     ` Maxime Ripard
  1 sibling, 1 reply; 6+ messages in thread
From: Ben Hutchings @ 2015-10-05 22:55 UTC (permalink / raw)
  To: Maxime Ripard, Josh Boyer
  Cc: Rob Herring, Pawel Moll, Mark Rutland, Olof Johansson,
	Josef Gajdusek, devicetree, linux-arm-kernel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 849 bytes --]

On Sun, 2015-09-06 at 13:20 +0200, Maxime Ripard wrote:
> On Fri, Sep 04, 2015 at 08:49:34AM -0400, Josh Boyer wrote:
> > Commit 79ae3e66f8d (ARM: dts: sun4i: Add Iteaduino Plus A10) added a new
> > make target for the sun4i-a10-itead-iteaduino-plus dts file, but mistakenly
> > used .dts instead of the correct .dtb suffix.  This resulted in a build error
> > like:
> > 
> >         scripts/Makefile.dtbinst:42: target 'sun4i-a10-itead-iteaduino-plus.dts' doesn't match the target pattern
> > 
> > when doing a make dtbs_install.
> > 
> > Fix it to use the proper file name.
> > 
> > Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
> 
> Queued, thanks!

This still isn't in Linus's tree; please make sure it gets to him
before 4.3.

Ben.

-- 
Ben Hutchings
compatible: Gracefully accepts erroneous data from any source

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 811 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus
  2015-10-05 22:55   ` Ben Hutchings
@ 2015-10-06  8:37     ` Maxime Ripard
  0 siblings, 0 replies; 6+ messages in thread
From: Maxime Ripard @ 2015-10-06  8:37 UTC (permalink / raw)
  To: Ben Hutchings
  Cc: Josh Boyer, Rob Herring, Pawel Moll, Mark Rutland,
	Olof Johansson, Josef Gajdusek, devicetree, linux-arm-kernel,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1039 bytes --]

On Mon, Oct 05, 2015 at 11:55:17PM +0100, Ben Hutchings wrote:
> On Sun, 2015-09-06 at 13:20 +0200, Maxime Ripard wrote:
> > On Fri, Sep 04, 2015 at 08:49:34AM -0400, Josh Boyer wrote:
> > > Commit 79ae3e66f8d (ARM: dts: sun4i: Add Iteaduino Plus A10) added a new
> > > make target for the sun4i-a10-itead-iteaduino-plus dts file, but mistakenly
> > > used .dts instead of the correct .dtb suffix.  This resulted in a build error
> > > like:
> > > 
> > >         scripts/Makefile.dtbinst:42: target 'sun4i-a10-itead-iteaduino-plus.dts' doesn't match the target pattern
> > > 
> > > when doing a make dtbs_install.
> > > 
> > > Fix it to use the proper file name.
> > > 
> > > Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
> > 
> > Queued, thanks!
> 
> This still isn't in Linus's tree; please make sure it gets to him
> before 4.3.

The pull request has been sent on sunday to arm-soc.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-10-06  8:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-04 12:49 [PATCH RESEND] ARM: dts: Fix Makefile target for sun4i-a10-itead-iteaduino-plus Josh Boyer
2015-09-06 11:20 ` Maxime Ripard
2015-09-29 14:32   ` Josh Boyer
2015-10-01 10:11     ` Maxime Ripard
2015-10-05 22:55   ` Ben Hutchings
2015-10-06  8:37     ` Maxime Ripard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).