Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1670888

Re: [PATCH v4 6/6] arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K

Show all headers | View raw

Hi Thomas,
 
 On mar., juin 20 2017, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Hello,
>
> On Tue, 20 Jun 2017 16:56:35 +0200, Gregory CLEMENT wrote:
>
>> > +#include <dt-bindings/interrupt-controller/mvebu-icu.h>  
>> 
>> With this line you created a dependency with the patch "irqchip:
>> irq-mvebu-icu: new driver for Marvell ICU". And without it the dtb is
>> not buidable.
>> 
>> So either I wait for the next kernel release to apply it or I will need
>> a stable branch with the commit adding the mvebu-icu.h file.
>
> Alternatively, I could change the patch to use hardcoded values for now
> (i.e replace ICU_GRP_NSR everywhere by 0x0), and then change it during
> the next kernel cycle to use the #define value. I would really prefer
> such a solution to a solution where merging the entire ICU DT change
> would be delayed by one cycle.

As we start to be short in time then not having an external dependency
is easier so use the hardcoded value in your v5 series, and we will
switch to #define value in the next cycle.

Thanks,

Gregory

>
> Let me know what you prefer. Maybe Marc can say if the irqchip
> maintainers can provide a stable branch containing the mvebu-icu.h file.
>
> Best regards,
>
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v4 0/6] Add support for the ICU unit in Marvell Armada 7K/8K Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-06-20 16:10 +0200
  [PATCH v4 6/6] arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-06-20 16:10 +0200
    Re: [PATCH v4 6/6] arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K Gregory CLEMENT <gregory.clement@free-electrons.com> - 2017-06-20 17:00 +0200
      Re: [PATCH v4 6/6] arm64: dts: marvell: enable GICP and ICU on  Armada 7K/8K Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-06-20 17:40 +0200
        Re: [PATCH v4 6/6] arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K Gregory CLEMENT <gregory.clement@free-electrons.com> - 2017-06-20 17:50 +0200
  [PATCH v4 5/6] arm64: marvell: enable ICU and GICP drivers Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-06-20 16:10 +0200
    Re: [PATCH v4 5/6] arm64: marvell: enable ICU and GICP drivers Marc Zyngier <marc.zyngier@arm.com> - 2017-06-21 10:00 +0200
  [PATCH v4 4/6] irqchip: irq-mvebu-icu: new driver for Marvell ICU Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-06-20 16:10 +0200
    Re: [PATCH v4 4/6] irqchip: irq-mvebu-icu: new driver for Marvell ICU Marc Zyngier <marc.zyngier@arm.com> - 2017-06-21 10:00 +0200
      Re: [PATCH v4 4/6] irqchip: irq-mvebu-icu: new driver for Marvell  ICU Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-06-21 15:40 +0200

csiph-web