Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Mon, 27 Nov 2017 20:59:19 +0300
From:   Yury Norov <ynorov@caviumnetworks.com>
To:     Nick Desaulniers <ndesaulniers@google.com>
Cc:     linux-arm-kernel@lists.infradead.org,
        LKML <linux-kernel@vger.kernel.org>,
        Alex Matveev <alxmtvv@gmail.com>,
        Alexander Potapenko <glider@google.com>,
        Andi Kleen <ak@linux.intel.com>,
        Ard Biesheuvel <ard.biesheuvel@linaro.org>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Kees Cook <keescook@chromium.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Sami Tolvanen <samitolvanen@google.com>,
        Stephen Boyd <sboyd@codeaurora.org>,
        Will Deacon <will.deacon@arm.com>,
        Joe Perches <joe@perches.com>,
        Andy Whitcroft <apw@canonical.com>
Subject: Re: [PATCH] arm64: fix missing 'const' qualifiers
Message-ID: <20171127175919.y5hr7qgvhkdwanbm@yury-thinkpad>
References: <20171125094127.7536-1-ynorov@caviumnetworks.com>
 <CAKwvOdm+65Fx1gLZR3h7Vz-Ma4DNWR4W=cOL8KpKZXMgN-nCLg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAKwvOdm+65Fx1gLZR3h7Vz-Ma4DNWR4W=cOL8KpKZXMgN-nCLg@mail.gmail.com>
User-Agent: NeoMutt/20170113 (1.7.2)
Received-SPF: None (protection.outlook.com: cavium.com does not designate
 permitted sender hosts)
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Nov 2017 17:59:38.8222 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 6d6d4619-8e54-4040-a9fd-08d535c0a1bd
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR0701MB3827
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Nov 27, 2017 at 09:15:50AM -0800, Nick Desaulniers wrote:
> Checked call sites, looks like no one tries to modify these pointers.
> I too prefer to make things as const as possible, though I recently
> had a coworker refer to the propagation of const in a code base as a
> "Gordian knot," which I thought was eloquent.
>
> checkpatch mentions that there's one line that's over 80 chars, and
> that there's M$ DOS style line endings (but that may just be my client
> gmail, or yours (exchange?)).  If you fix up that long line, then:

I happily use mutt, but my company's mail server is MS Outlook, so I 
have to send everything with msmtp. It usually works. In fact, this
is the first complain for 2 years. 

My checkpatch:
yury:linux$ scripts/checkpatch.pl const.patch 
total: 0 errors, 0 warnings, 32 lines checked

const.patch has no obvious style problems and is ready for submission.

I guess, because 1 line is exactly 80 chars length, and because MS format
adds another symbol as line delimiter, we end up with 81 chars in line,
which makes checkpatch worrying.

CC Joe Perches and Andy Whitcroft for it.

> Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>

Thanks

From 1585240410857030262@xxx Mon Nov 27 17:17:45 +0000 2017
X-GM-THRID: 1585030578918772458
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread