All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Mark Rutland <mark.rutland@arm.com>, Andrew Lunn <andrew@lunn.ch>,
	Marcin Wojtas <mw@semihalf.com>,
	Jason Cooper <jason@lakedaemon.net>,
	devicetree@vger.kernel.org,
	Antoine Tenart <antoine.tenart@bootlin.com>,
	Grzegorz Jaszczyk <jaz@semihalf.com>,
	Gregory Clement <gregory.clement@bootlin.com>,
	linux-kernel@vger.kernel.org, Evan Wang <xswang@marvell.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Nadav Haklai <nadavh@marvell.com>,
	Rob Herring <robh+dt@kernel.org>,
	Maxime Chevallier <maxime.chevallier@bootlin.com>,
	linux-arm-kernel@lists.infradead.org,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [PATCH v4 3/7] phy: add A3700 COMPHY support
Date: Mon, 17 Dec 2018 16:56:03 +0100	[thread overview]
Message-ID: <20181217165603.602a7a42@xps13> (raw)
In-Reply-To: <20181217165223.6f98b8a4@windsurf>

Hi Thomas,

Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote on Mon, 17 Dec
2018 16:52:23 +0100:

> Hello,
> 
> On Mon, 17 Dec 2018 16:48:57 +0100, Miquel Raynal wrote:
> 
> > I received the following bug report because I added a dependency on
> > COMPILE_TEST in Kconfig:
> > 
> >         config: i386-allmodconfig (attached as .config) compiler:
> >         gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce:
> >                 # save the attached .config to linux build tree
> >                 make ARCH=i386 
> > 
> >         All errors (new ones prefixed by >>):
> >   
> >         >> ERROR: "__arm_smccc_smc" [drivers/phy/marvell/phy-mvebu-a3700-comphy.ko] undefined!      
> > 
> > Unless someone complains, I will drop the dependency on COMPILE_TEST in
> > Kconfig.  
> 
> Keep COMPILE_TEST, and add:
> 
> 	depends on HAVE_ARM_SMCCC
> 
> perhaps ?

Oh that's a cool Kconfig entry. Forgot about it, thanks for the
pointer!


Thanks,
Miquèl

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Gregory Clement <gregory.clement@bootlin.com>,
	Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Antoine Tenart <antoine.tenart@bootlin.com>,
	Maxime Chevallier <maxime.chevallier@bootlin.com>,
	Nadav Haklai <nadavh@marvell.com>,
	Marcin Wojtas <mw@semihalf.com>,
	Grzegorz Jaszczyk <jaz@semihalf.com>,
	Evan Wang <xswang@marvell.com>
Subject: Re: [PATCH v4 3/7] phy: add A3700 COMPHY support
Date: Mon, 17 Dec 2018 16:56:03 +0100	[thread overview]
Message-ID: <20181217165603.602a7a42@xps13> (raw)
In-Reply-To: <20181217165223.6f98b8a4@windsurf>

Hi Thomas,

Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote on Mon, 17 Dec
2018 16:52:23 +0100:

> Hello,
> 
> On Mon, 17 Dec 2018 16:48:57 +0100, Miquel Raynal wrote:
> 
> > I received the following bug report because I added a dependency on
> > COMPILE_TEST in Kconfig:
> > 
> >         config: i386-allmodconfig (attached as .config) compiler:
> >         gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce:
> >                 # save the attached .config to linux build tree
> >                 make ARCH=i386 
> > 
> >         All errors (new ones prefixed by >>):
> >   
> >         >> ERROR: "__arm_smccc_smc" [drivers/phy/marvell/phy-mvebu-a3700-comphy.ko] undefined!      
> > 
> > Unless someone complains, I will drop the dependency on COMPILE_TEST in
> > Kconfig.  
> 
> Keep COMPILE_TEST, and add:
> 
> 	depends on HAVE_ARM_SMCCC
> 
> perhaps ?

Oh that's a cool Kconfig entry. Forgot about it, thanks for the
pointer!


Thanks,
Miquèl

WARNING: multiple messages have this Message-ID (diff)
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Gregory Clement <gregory.clement@bootlin.com>,
	Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	<devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Antoine Tenart <antoine.tenart@bootlin.com>,
	Maxime Chevallier <maxime.chevallier@bootlin.com>,
	Nadav Haklai <nadavh@marvell.com>,
	Marcin Wojtas <mw@semihalf.com>,
	Grzegorz Jaszczyk <jaz@semihalf.com>,
	Evan Wang <xswang@marvell.com>
Subject: Re: [PATCH v4 3/7] phy: add A3700 COMPHY support
Date: Mon, 17 Dec 2018 16:56:03 +0100	[thread overview]
Message-ID: <20181217165603.602a7a42@xps13> (raw)
In-Reply-To: <20181217165223.6f98b8a4@windsurf>

Hi Thomas,

Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote on Mon, 17 Dec
2018 16:52:23 +0100:

> Hello,
> 
> On Mon, 17 Dec 2018 16:48:57 +0100, Miquel Raynal wrote:
> 
> > I received the following bug report because I added a dependency on
> > COMPILE_TEST in Kconfig:
> > 
> >         config: i386-allmodconfig (attached as .config) compiler:
> >         gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce:
> >                 # save the attached .config to linux build tree
> >                 make ARCH=i386 
> > 
> >         All errors (new ones prefixed by >>):
> >   
> >         >> ERROR: "__arm_smccc_smc" [drivers/phy/marvell/phy-mvebu-a3700-comphy.ko] undefined!      
> > 
> > Unless someone complains, I will drop the dependency on COMPILE_TEST in
> > Kconfig.  
> 
> Keep COMPILE_TEST, and add:
> 
> 	depends on HAVE_ARM_SMCCC
> 
> perhaps ?

Oh that's a cool Kconfig entry. Forgot about it, thanks for the
pointer!


Thanks,
Miquèl

  reply	other threads:[~2018-12-17 15:56 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-13  8:39 [PATCH v4 0/7] Add Armada 3700 COMPHY support Miquel Raynal
2018-12-13  8:39 ` Miquel Raynal
2018-12-13  8:39 ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 1/7] phy: mvebu-cp110-comphy: fix port check in ->xlate() Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 2/7] phy: enumerate SATA PHY mode Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 3/7] phy: add A3700 COMPHY support Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-17 15:48   ` Miquel Raynal
2018-12-17 15:48     ` Miquel Raynal
2018-12-17 15:48     ` Miquel Raynal
2018-12-17 15:52     ` Thomas Petazzoni
2018-12-17 15:52       ` Thomas Petazzoni
2018-12-17 15:52       ` Thomas Petazzoni
2018-12-17 15:56       ` Miquel Raynal [this message]
2018-12-17 15:56         ` Miquel Raynal
2018-12-17 15:56         ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 4/7] dt-bindings: phy: mvebu-comphy: extend the file to describe a3700 bindings Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 5/7] MAINTAINERS: phy: add entry for Armada 3700 COMPHY driver Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 6/7] ARM64: dts: marvell: armada-37xx: fix SATA node scope Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39 ` [PATCH v4 7/7] ARM64: dts: marvell: armada-37xx: declare the COMPHY node Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal
2018-12-13  8:39   ` Miquel Raynal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181217165603.602a7a42@xps13 \
    --to=miquel.raynal@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=antoine.tenart@bootlin.com \
    --cc=devicetree@vger.kernel.org \
    --cc=gregory.clement@bootlin.com \
    --cc=jason@lakedaemon.net \
    --cc=jaz@semihalf.com \
    --cc=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=maxime.chevallier@bootlin.com \
    --cc=mw@semihalf.com \
    --cc=nadavh@marvell.com \
    --cc=robh+dt@kernel.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=xswang@marvell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.