From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: MIME-Version: 1.0 References: <1524869998-2805-1-git-send-email-wesley@sifive.com> <1524869998-2805-3-git-send-email-wesley@sifive.com> <72c263ef-09c3-a05f-3c16-1bc8df667d38@suse.de> In-Reply-To: <72c263ef-09c3-a05f-3c16-1bc8df667d38@suse.de> From: Wesley Terpstra Date: Sat, 28 Apr 2018 22:37:56 +0000 Message-ID: Subject: Re: [PATCH 2/3] dt-bindings: Add "sifive" vendor prefix Content-Type: multipart/alternative; boundary="00000000000040277c056af046b0" To: =?UTF-8?Q?Andreas_F=C3=A4rber?= Cc: Thierry Reding , Rob Herring , Mark Rutland , =?UTF-8?Q?Noralf_Tr=C3=B8nnes?= , David Lechner , Alexandre Belloni , SZ Lin , linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org List-ID: --00000000000040277c056af046b0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Will do, thanks! On Sat, Apr 28, 2018, 4:21 AM Andreas F=C3=A4rber wrote: > Am 28.04.2018 um 00:59 schrieb Wesley W. Terpstra: > > This adds a vendor prefix "sifive" for SiFive, Inc. > > We make chips. > > > > Signed-off-by: Wesley W. Terpstra > > --- > > Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > > 1 file changed, 1 insertion(+) > > Reviewed-by: Andreas F=C3=A4rber > > but please reorder the patches, so that the prefix definition comes > before its first use in the pwm binding. > > Regards, > Andreas > > -- > SUSE Linux GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany > GF: Felix Imend=C3=B6rffer, Jane Smithard, Graham Norton > HRB 21284 (AG N=C3=BCrnberg) > --00000000000040277c056af046b0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Will do, thanks!=C2=A0

On Sat, Apr 28, 2018, 4:21 AM Andreas F=C3=A4rber <= ;afaerber@suse.de> wrote:
Am 28.04.2018 um 00:59 schrieb Wesley W. = Terpstra:
> This adds a vendor prefix "sifive" for SiFive, Inc.
> We make chips.
>
> Signed-off-by: Wesley W. Terpstra <wesley@sifive.com>
> ---
>=C2=A0 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>=C2=A0 1 file changed, 1 insertion(+)

Reviewed-by: Andreas F=C3=A4rber <afaerber@suse.de>

but please reorder the patches, so that the prefix definition comes
before its first use in the pwm binding.

Regards,
Andreas

--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany
GF: Felix Imend=C3=B6rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N=C3=BCrnberg)
--00000000000040277c056af046b0--