Received: by 2002:a05:6a10:1d13:0:0:0:0 with SMTP id pp19csp2044473pxb; Sat, 28 Aug 2021 02:48:39 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwysqjxEQrxeOLDui5vgvD+7/jzViE3LoQiE36dyKn4OU3y9EiTLipdfwuWLczp/pnZPSYt X-Received: by 2002:a02:6a55:: with SMTP id m21mr12149164jaf.74.1630144119477; Sat, 28 Aug 2021 02:48:39 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1630144119; cv=none; d=google.com; s=arc-20160816; b=WpC+rLAn6z/csDJlagkxY2RgFP4qZQeTCHaCzmSSgZ1j846gvFh4I2N76mxsEWVER6 fh0F2gKImVZVFOd0//v63b8ymxana3xdB/ezBFaeqbTsUOLTeONNjGy1QurAwJVdJsEn cFaqWEn2mkcYxAMG1uuw5Ors34e2k+Ms4Q748Ux25tZyndDUPUEZVzXAfk36OxPT+d1g y3hglBYUbSdOI2blkPkRtcUX0S4YzorFj22soQN5kEB0MkzA03ygu47pwE99iaPK7tVl k+Xlq15Td39Vz/yoSyI40MomWm74c7MUVv0R5zz3IoMvI9Vk63qvBC3w0xQYNOZJspEl G1AQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=3lqXp+QI0u/rwbZ+Pn8R6fFCKXyodwZVuK3fVSXp69c=; b=YVKjyvjC08Jpin6Zysr7bs+smmfxovMMEL1LAWbCcCipRJJFhLqdTGzV8yfHUR9zsL WbOgyl7vFYlnhHM5c1nj6XqDDPnz7ekL4LvjUAWYJ+5ZmfKEStjszj8+TWmr52kCDRJW 0rgeqcTwOs6xoVKR+ELKOKG7EpP7XbFyO67LZ5pd7XpOpUmYbb7CGi+0YMu11yGyRjWU 6eggCL0KvcRxht7kl3q82+fC3S0zq4e+qadnzRXxnt/lI4OdRuMe69rqF+kmloGk97UR hmRFW6LOIcWf/WMs2py32QW3I+Tgj/0COXsrFTKY/c7MEXoaxPOad+SX7ZRJ9Dw0P6Bm O4fw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=nRLnpRQm; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id f14si8085627jaj.115.2021.08.28.02.48.28; Sat, 28 Aug 2021 02:48:39 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=nRLnpRQm; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233688AbhH1Jr1 (ORCPT + 99 others); Sat, 28 Aug 2021 05:47:27 -0400 Received: from mail.kernel.org ([198.145.29.99]:33518 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230444AbhH1Jr0 (ORCPT ); Sat, 28 Aug 2021 05:47:26 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 9C60E60ED3; Sat, 28 Aug 2021 09:46:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1630143996; bh=SiGJxzk16eUk9XGSCPM4wzNg/e5oL3uafMhtUaj0g+E=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nRLnpRQmpIhLLvrnzK8oTpqSxuKwXAyg6z2i3RQI8WrSVMKyOCUoPlfxipAj/AkJr JVHH1bg9v042ynF0TL5l85Yf5eUzUlbcWLa+MTNhQtX7u2y5VVR3GlIIYchj2gX6Oh 056W8pFQNRi6EBbDOi6mDmeOaXp78mH9HoeoB0R8= Date: Sat, 28 Aug 2021 11:46:33 +0200 From: Greg KH To: Krish Jain Cc: Bryan Brattlof , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Declare the file_operations struct as const Message-ID: References: <3634721.RBzQ2xsved@localhost.localdomain> <1742120.GgyQvu0Ciu@localhost.localdomain> <20210827233835.px4az5hyqks2n4o5@h510> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A: http://en.wikipedia.org/wiki/Top_post Q: Were do I find info about this thing called top-posting? A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Sat, Aug 28, 2021 at 11:37:50AM +0200, Krish Jain wrote: > Hi. Thanks for your response. Changing to "const static" would fix > the first error but looking at the second error indicates that it > can't be a const, right? So checkpatch.pl was wrong? checkpatch.pl is a perl script that does its best here. You always have to then look at the code itself to see if what it is asking you to do is correct. And you always have to at the very least, test build your changes to verify that they do not break anything. thanks, greg k-h