From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Subject: Re: [PATCH] wcn36xx: Remove Unicode Byte Order Mark from testcode Date: Mon, 11 Jun 2018 17:13:09 +0300 Message-ID: <877en5v2bu.fsf@kamboji.qca.qualcomm.com> References: <1528375527-22761-1-git-send-email-geert@linux-m68k.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: Eyal Ilsar , "David S . Miller" , Arnd Bergmann , netdev@vger.kernel.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, wcn36xx@lists.infradead.org To: Geert Uytterhoeven Return-path: In-Reply-To: <1528375527-22761-1-git-send-email-geert@linux-m68k.org> (Geert Uytterhoeven's message of "Thu, 7 Jun 2018 14:45:27 +0200") Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Geert Uytterhoeven writes: > Older gcc (< 4.4) doesn't like files starting with a Unicode BOM: > > drivers/net/wireless/ath/wcn36xx/testmode.c:1: error: stray =E2=80=98= \357=E2=80=99 in program > drivers/net/wireless/ath/wcn36xx/testmode.c:1: error: stray =E2=80=98= \273=E2=80=99 in program > drivers/net/wireless/ath/wcn36xx/testmode.c:1: error: stray =E2=80=98= \277=E2=80=99 in program > > Remove the BOM, the rest of the file is plain ASCII anyway. > > Output of "file drivers/net/wireless/ath/wcn36xx/testmode.c" before: > > drivers/net/wireless/ath/wcn36xx/testmode.c: C source, UTF-8 Unicode = (with BOM) text > > and after: > > drivers/net/wireless/ath/wcn36xx/testmode.c: C source, ASCII text > > Fixes: 87f825e6e246cee0 ("wcn36xx: Add support for Factory Test Mode (FTM= )") > Signed-off-by: Geert Uytterhoeven I'll queue this for 4.18. --=20 Kalle Valo