Re: [PATCH 1/2] ASoC: add es8316 codec driver

From: Mark Brown <broonie@kernel.org>
To: Daniel Drake <drake@endlessm.com>
Cc: alsa-devel@alsa-project.org,
	Linux Upstreaming Team <linux@endlessm.com>,
	Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
	yangxiaohua <yangxiaohua@everest-semi.com>,
	Liam Girdwood <lgirdwood@gmail.com>
Subject: Re: [PATCH 1/2] ASoC: add es8316 codec driver
Date: Sun, 14 May 2017 18:36:02 +0900	[thread overview]
Message-ID: <20170514093602.fzvy44vdej3ukyym@sirena.org.uk> (raw)
In-Reply-To: <CAD8Lp47DsghwLrdn3izM63BCqn56=esdNeKV6L4NU9tdRhNPQg@mail.gmail.com>

[-- Attachment #1.1: Type: text/plain, Size: 898 bytes --]

On Wed, May 10, 2017 at 01:18:20PM -0600, Daniel Drake wrote:
> On Wed, May 10, 2017 at 12:49 PM, Pierre-Louis Bossart
> <pierre-louis.bossart@linux.intel.com> wrote:

> > If the copyright is held by Everest Semiconductor and David Yang is also
> > from Everest Semiconductor, then a signoff from Everest looks very much
> > required?

> The original patches from David do now have his sign-off:
> https://bugzilla.kernel.org/show_bug.cgi?id=189261#c57

> But I have now rearranged things quite a bit. What is the norm here,
> can I just copy over the sign off from before, or do we need to
> request explicit permission again from David?

I'm not going to look at patches that don't have signoffs, sorry.
Please see SubmittingPatches for details of what a signoff means and how
to handle them.  Without knowing how closely what you've done resembles
what you're basing it on it's hard to be sure.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]