DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EBF1B20862
Subject: Re: [PATCH v2 1/3] lib: cordic: Move cordic macros and defines to
 header file
To:     Priit Laes <plaes@plaes.org>
References: <cover.9ec98292c595a111ae78f31383ff8618c9cb04da.1541446422.git-series.plaes@plaes.org>
 <3be396e01b33905e9ac5c80e7038b7aacf72eaf8.1541446422.git-series.plaes@plaes.org>
 <8eb0e232-d14b-60d9-e3c8-008c46659a30@broadcom.com>
 <20181106071455.ch6wooqibm76tyjl@plaes.org>
Cc:     linux-kernel@vger.kernel.org,
        linux-wireless <linux-wireless@vger.kernel.org>
From:   Arend van Spriel <arend.vanspriel@broadcom.com>
Message-ID: <2dbd0169-6a07-2ff0-6ea7-99acd624db9f@broadcom.com>
Date:   Tue, 6 Nov 2018 13:49:56 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <20181106071455.ch6wooqibm76tyjl@plaes.org>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On 11/6/2018 8:14 AM, Priit Laes wrote:
> On Mon, Nov 05, 2018 at 11:02:35PM +0100, Arend van Spriel wrote:
>> On 11/5/2018 8:37 PM, Priit Laes wrote:
>>> Also append CORDIC_ prefix to nonprefixed macros.
>>
>> Bit annoying that you made me look at LKML for this patch. Can you just post
>> the entire series to linux-wireless so Kalle can take it all through his
>> tree. Also the commit message is a bit thin. Please explain why these are
>> moved to the header file.
>
> Sorry, I messed up again while trying to automate the send-email with
> get_maintainers script (dry-run argument did not work .gitconfig :S)

No worries. It was only a bit annoying ;-)

Regards,
Arend