Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Jonathan McDowell <noodles@fb.com>
To:     sunliming <sunliming@kylinos.cn>
CC:     "dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
        "bhe@redhat.com" <bhe@redhat.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "frowand.list@gmail.com" <frowand.list@gmail.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kelulanainsley@gmail.com" <kelulanainsley@gmail.com>,
        kernel test robot <lkp@intel.com>
Subject: Re: [PATCH RESEND] of: Add missing function declarations
Thread-Topic: [PATCH RESEND] of: Add missing function declarations
Thread-Index: AQHYfiw95YJNL9wORUehks5+IVdrAa1NDIAA
Date:   Mon, 13 Jun 2022 09:03:23 +0000
Message-ID: <Yqb9VoYe+D69KYTY@noodles-fedora.dhcp.thefacebook.com>
References: <20220612071535.1179039-1-sunliming@kylinos.cn>
In-Reply-To: <20220612071535.1179039-1-sunliming@kylinos.cn>
Accept-Language: en-GB, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?D5PS99Ae7wwbd2Np4G4Xhk4Vc2g0qkajHCnj70KLVwxnmMImn9qFsUQrhuUP?=
 =?us-ascii?Q?yYnmPkKQSVT21syFIxTInO4VmTnX2/KGgHnYdlrA9AkrAG9j/lobsWcT+O5/?=
 =?us-ascii?Q?Qzybe/k7LQL+qTpOWjftOUYXixDY3TBOeQ6leYZRG9SO6J2GNhTu9tJGRGFj?=
 =?us-ascii?Q?vcs4g/tCuLN33LRF0Joh4nzuWJn44pla/qh2EuZXlVL/h1/nhBWCS4HZIxsK?=
 =?us-ascii?Q?xmp+ZysiZef08iKEy3C2mlDv5y+zZRu6MrcwT9xudulQdZyGa1JKlt5VNQBe?=
 =?us-ascii?Q?f9r3fLuxblRKExNa5ZEe+re3xMUd/cXFrzIAetvZ9bE9ui1AhBvGiSEWza2u?=
 =?us-ascii?Q?RlV9qMvC6Dey8eg66krit7SEMfAVjn9nF1+gZYIjMdke9scFmbpMGt0iAdtM?=
 =?us-ascii?Q?hLx7xfQ1x2fhAA8wtmhBt3cVKtswIDtQBUs+5TjwP3k6zTWSPVdFEoqzu4mS?=
 =?us-ascii?Q?4UNi0U8y8xpaNa16ci7dMzTSDejx4oWUn2IziCykT0hTUVDaFkx1GnIfhyT+?=
 =?us-ascii?Q?xZQpCDtHMsz2QSAiK7ausT/+UAiMK7cqQx4Fhjlrm3Qia4vIEN1Mwmbya7nl?=
 =?us-ascii?Q?4bZzKtPppJ4p/QpdbH7nH1r9i2qBvGHQNYu6+kozOsVfiYnqmRVyYrl1on/N?=
 =?us-ascii?Q?60WAIg0Cb0un6+NoEqBYddkUHGxWHdhD1/IaUaVjvJvdr3h7+tCiB3iaig7S?=
 =?us-ascii?Q?thjhgLxUauT52esR/nXnjdCnSUDu5zSyUYJhu8WTFG8xxF8svueVSwoIsYzz?=
 =?us-ascii?Q?d3fBaYxIloGYEUFrpn25JLh4vC5tDPlni0YG14npswz/vXEg+TFtbSey+bxH?=
 =?us-ascii?Q?tdbmKWaDelQ8T+o85/oTG5Zey6UgyfpXAMN5ZyF2PELaVaaaSUlXMiRU+gce?=
 =?us-ascii?Q?NkPFCMqTCWWCwVM7Vm9rwdr4QwUUnS8k31MtAYMs+RLHWP0AGsz5mTu1KghT?=
 =?us-ascii?Q?R6M5ywNtT3nLsOeUbWynPudeiUqZ7DFeMeKTj6Zx3ev4WqdSYfTQuf3qynPn?=
 =?us-ascii?Q?EVUbIUIE3hi6lAw2rXjap++X5ibXreyvXyLkEL8XuUSOJ/GCJUYbAsRTJRoq?=
 =?us-ascii?Q?U3SlVf1qMcie051WHYysDLpOJr/JB7L50tTSPksmXRgGUPy3DOJ/oRnYmPgF?=
 =?us-ascii?Q?PNeCAeA7VB1LwfKbHfyj1fRn9zcvGzWd20FXNGprtN756CcxsQNNNFfsrmet?=
 =?us-ascii?Q?SmpXjYusnKfdaLImlbbG2Jm7JIrcsVVKEAvTWCQWxMgzTlAJo//cXN0z/XIG?=
 =?us-ascii?Q?mdgMlPKWRwt2wAohRV7OJ7KeMksj1eAZAulpluxKZ5H91aohv62RlVgCkKQw?=
 =?us-ascii?Q?/gd82RhkG2PQ7BHD4cOs1NzgxBAOtak0We6+lAeOJgx6c+MMphWDlQE/PeV0?=
 =?us-ascii?Q?3gRR1lnjtUNlu6b5oku+P7Vdy9Fr15Pc2GZH8zttu+KacoDV945uuwENzd30?=
 =?us-ascii?Q?gzQOr3Ew+fHRyK3FQrXEQlhGpV3Bc2ic76OfVNf2rG8JTwTikxqKyC8fY8Tu?=
 =?us-ascii?Q?xjo1DeCyJa57wHchMn2y3BF0+h9PvQsszYenZ3tk7vS5frVTLAkQl6aAwMR/?=
 =?us-ascii?Q?t4gvtFx84iCM1Sor+TdN4q0zaThR5j/zL39obva9ob5xMi6kuq7mEYGhTof8?=
 =?us-ascii?Q?s4w9WZbIy9sbcMpA7XaUKB1MBKx2IrRLzfcqDsfNo90YIkVDC83QrRgKoj60?=
 =?us-ascii?Q?kWn2ZwzQorHUl7Bwb0ST1u9Wpg6Nz8FoeoSZgsuaReEtn6uCnZzJqYjMenMd?=
 =?us-ascii?Q?Aqiow0Qusg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <D205169C2118E74D97D5C5A7D9990AD1@namprd15.prod.outlook.com>
MIME-Version: 1.0
X-OriginatorOrg: fb.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR15MB4552.namprd15.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 422b1e07-6d2d-4b4f-f8a7-08da4d1b91f6
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Jun 2022 09:03:23.4215
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 8ae927fe-1255-47a7-a2af-5f3a069daaa2
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: MhutSHhy7oecXF47A8OFnm4xUBImwkakL+3TwrnBvEGYiMiyukrUD89Gw4WU7vov
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLAPR15MB3841
X-Proofpoint-ORIG-GUID: Q7ag7divlyoYuZlBjjGJ5Aea81WiSBLA
X-Proofpoint-GUID: Q7ag7divlyoYuZlBjjGJ5Aea81WiSBLA
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.874,Hydra:6.0.517,FMLib:17.11.64.514
 definitions=2022-06-13_03,2022-06-09_02,2022-02-23_01
X-Spam-Status: No, score=-4.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

NAK. This is for a patch of mine that I don't believe has hit mainline
yet - I've only seen it in daveh's test tree. The prototypes for these
functions are now guarded by CONFIG_HAVE_IMA_KEXEC but all OF platforms
build them even when the platform doesn't actually support IMA over
kexec. I'll update my patch to avoid that.

On Sun, Jun 12, 2022 at 03:15:35PM +0800, sunliming wrote:
> Fixes the following w1 warning:
> 
> drivers/of/kexec.c:126:5: warning: no previous prototype for 'ima_get_kexec_buffer' [-Wmissing-prototypes]
> drivers/of/kexec.c:153:5: warning: no previous prototype for 'ima_free_kexec_buffer' [-Wmissing-prototypes
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: sunliming <sunliming@kylinos.cn>
> ---
>  include/linux/of.h | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/include/linux/of.h b/include/linux/of.h
> index f0a5d6b10c5a..3a166a1c4ef6 100644
> --- a/include/linux/of.h
> +++ b/include/linux/of.h
> @@ -844,6 +844,16 @@ static inline phys_addr_t of_dma_get_max_cpu_address(struct device_node *np)
>  	return PHYS_ADDR_MAX;
>  }
>  
> +static inline int ima_get_kexec_buffer(void **addr, size_t *size)
> +{
> +        return -ENOSYS;
> +}
> +
> +static inline int ima_free_kexec_buffer(void **addr, size_t *size)
> +{
> +        return -ENOSYS;
> +}
> +
>  #define of_match_ptr(_ptr)	NULL
>  #define of_match_node(_matches, _node)	NULL
>  #endif /* CONFIG_OF */
> -- 
> 2.25.1
>