Message ID | 20221103114036.1581854-1-steven.price@arm.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp477629wru; Thu, 3 Nov 2022 04:51:00 -0700 (PDT) X-Google-Smtp-Source: AMsMyM6mGWwIDQPgVeEJi4C8LOFZ0U/lsMEpd5ZZkN4DKD/0vPsSIhgcxKOdd+2rjCu9cX/V9k0r X-Received: by 2002:a17:907:7245:b0:78d:ec9c:e743 with SMTP id ds5-20020a170907724500b0078dec9ce743mr28938279ejc.466.1667476259981; Thu, 03 Nov 2022 04:50:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1667476259; cv=none; d=google.com; s=arc-20160816; b=LSuUTrqP8Cj2RUsdjeJokHoFLCVJKSJzdOVeZUym1OCR6SmsLxSPfFnyi4tBse7OXr b6CZ6xZjNmRXxw9Ec9T6luHgxuJWzuIfyV+07dligSUUEVaezwcisR5YDR70MC2jRkZA EhQ9O/e0YKtaAEqfpDZ6tY9y8Bx80o/HHQvsZA9XnllHsMS0SG84yWPClqsGwHv1IXtJ wYN8wexe4eADSYcKqyZrfgMiedNKGEZ/JJe2saZxN8UWEB/1YSCFJhtcpzlb0AVgcyaD CzY3OQMXWwbuhrDwFJtOcdn8Bl8a5Sdn2PO5bbv/uTYtyFXPJDjkSJlpREfoMe0qCSDw esKg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=6cyZKqCmQdu5Q+ba1fp8QZh+aVFtkLhQy+qyEDsmqOg=; b=qVQgdo2f3/rPHapwbSfBIxFo0nSFxKzhOQIEsItkaBtcG35bvFsvZFU0JD4eCV9Y/W 0EBn3ijimg6SaG2R96jitRTP2wKrw7jl0eHjzZ8SNCdp0ik5IZzR28RRq2PMSHk7ET3R mCxLhgRB48hRA0U5muVga4/+kH5/VmkN4+bn6KWJPCBZLRG/vmUbX+eSQrMqg+44GQ1A UZmBBH70wjDjJhrN0FrhWCe+7nGhNzbiaQGmO36j6Vf5nB0pJ2dVCh1HccH+W3xWs7QW Ck8/TIrfdOXT4fyxhnzcQna2IS84Fvjclq8TiaP4+WDhyods+7/mFT58bj2MZM3PAmQH JkNA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id ga23-20020a1709070c1700b00773b8e3b6a1si1159957ejc.805.2022.11.03.04.50.33; Thu, 03 Nov 2022 04:50:59 -0700 (PDT) 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; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229598AbiKCLlH (ORCPT <rfc822;yves.mi.zy@gmail.com> + 99 others); Thu, 3 Nov 2022 07:41:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53202 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229461AbiKCLlF (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Thu, 3 Nov 2022 07:41:05 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 1D4B4B14 for <linux-kernel@vger.kernel.org>; Thu, 3 Nov 2022 04:41:04 -0700 (PDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2F5A01FB; Thu, 3 Nov 2022 04:41:10 -0700 (PDT) Received: from e112269-lin.cambridge.arm.com (e112269-lin.cambridge.arm.com [10.1.194.34]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DBF233F703; Thu, 3 Nov 2022 04:41:02 -0700 (PDT) From: Steven Price <steven.price@arm.com> To: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Cc: Steven Price <steven.price@arm.com>, Rob Herring <robh@kernel.org>, Tomeu Vizoso <tomeu.vizoso@collabora.com>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Alyssa Rosenzweig <alyssa@collabora.com> Subject: [PATCH] drm/panfrost: Remove type name from internal struct again Date: Thu, 3 Nov 2022 11:40:36 +0000 Message-Id: <20221103114036.1581854-1-steven.price@arm.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_NONE 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 X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748475587038235030?= X-GMAIL-MSGID: =?utf-8?q?1748475587038235030?= |
Series |
drm/panfrost: Remove type name from internal struct again
|
|
Commit Message
Steven Price
Nov. 3, 2022, 11:40 a.m. UTC
Commit 72655fb942c1 ("drm/panfrost: replace endian-specific types with
native ones") accidentally reverted part of the parent commit
7228d9d79248 ("drm/panfrost: Remove type name from internal structs")
leading to the situation that the Panfrost UAPI header still doesn't
compile correctly in C++.
Revert the accidental revert and pass me a brown paper bag.
Reported-by: Alyssa Rosenzweig <alyssa@collabora.com>
Fixes: 72655fb942c1 ("drm/panfrost: replace endian-specific types with native ones")
Signed-off-by: Steven Price <steven.price@arm.com>
---
include/uapi/drm/panfrost_drm.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
Reviewed-by: Alyssa Rosenzweig <alyssa@collabora.com> On Thu, Nov 03, 2022 at 11:40:36AM +0000, Steven Price wrote: > Commit 72655fb942c1 ("drm/panfrost: replace endian-specific types with > native ones") accidentally reverted part of the parent commit > 7228d9d79248 ("drm/panfrost: Remove type name from internal structs") > leading to the situation that the Panfrost UAPI header still doesn't > compile correctly in C++. > > Revert the accidental revert and pass me a brown paper bag. > > Reported-by: Alyssa Rosenzweig <alyssa@collabora.com> > Fixes: 72655fb942c1 ("drm/panfrost: replace endian-specific types with native ones") > Signed-off-by: Steven Price <steven.price@arm.com> > --- > include/uapi/drm/panfrost_drm.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/uapi/drm/panfrost_drm.h b/include/uapi/drm/panfrost_drm.h > index 6f93c915cc88..9f231d40a146 100644 > --- a/include/uapi/drm/panfrost_drm.h > +++ b/include/uapi/drm/panfrost_drm.h > @@ -254,7 +254,7 @@ struct panfrost_dump_object_header { > __u64 nbos; > } reghdr; > > - struct pan_bomap_hdr { > + struct { > __u32 valid; > __u64 iova; > __u32 data[2]; > -- > 2.34.1 >
On 07/11/2022 13:22, Alyssa Rosenzweig wrote: > Reviewed-by: Alyssa Rosenzweig <alyssa@collabora.com> Thanks, pushed to drm-misc-fixes: c4299907c09a drm/panfrost: Remove type name from internal struct again Steve > On Thu, Nov 03, 2022 at 11:40:36AM +0000, Steven Price wrote: >> Commit 72655fb942c1 ("drm/panfrost: replace endian-specific types with >> native ones") accidentally reverted part of the parent commit >> 7228d9d79248 ("drm/panfrost: Remove type name from internal structs") >> leading to the situation that the Panfrost UAPI header still doesn't >> compile correctly in C++. >> >> Revert the accidental revert and pass me a brown paper bag. >> >> Reported-by: Alyssa Rosenzweig <alyssa@collabora.com> >> Fixes: 72655fb942c1 ("drm/panfrost: replace endian-specific types with native ones") >> Signed-off-by: Steven Price <steven.price@arm.com> >> --- >> include/uapi/drm/panfrost_drm.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/include/uapi/drm/panfrost_drm.h b/include/uapi/drm/panfrost_drm.h >> index 6f93c915cc88..9f231d40a146 100644 >> --- a/include/uapi/drm/panfrost_drm.h >> +++ b/include/uapi/drm/panfrost_drm.h >> @@ -254,7 +254,7 @@ struct panfrost_dump_object_header { >> __u64 nbos; >> } reghdr; >> >> - struct pan_bomap_hdr { >> + struct { >> __u32 valid; >> __u64 iova; >> __u32 data[2]; >> -- >> 2.34.1 >>
diff --git a/include/uapi/drm/panfrost_drm.h b/include/uapi/drm/panfrost_drm.h index 6f93c915cc88..9f231d40a146 100644 --- a/include/uapi/drm/panfrost_drm.h +++ b/include/uapi/drm/panfrost_drm.h @@ -254,7 +254,7 @@ struct panfrost_dump_object_header { __u64 nbos; } reghdr; - struct pan_bomap_hdr { + struct { __u32 valid; __u64 iova; __u32 data[2];