From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-17.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4B196C2B9F4 for ; Tue, 22 Jun 2021 12:32:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 31CCA6128E for ; Tue, 22 Jun 2021 12:32:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231750AbhFVMeY (ORCPT ); Tue, 22 Jun 2021 08:34:24 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:51056 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231959AbhFVMeT (ORCPT ); Tue, 22 Jun 2021 08:34:19 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1624365122; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=vP7JF6KY1/ahu18Tz6/GObDLrGSrV/SofAI1KrpMPF4=; b=AA+F5ILI95fQR5WCMROFXT3L+UISvkKHtqoShKsHXj89miH6K6NJLwtyu8ZGhbA3YfOBI8 nAOqoiVMMUtZv8GJMa7ohZn8EhVaFtqPfKVW7ep2Am0kwaC7L5+hk+X8WPwnn0GAabqO6W G8GYT/D8rv1X2A8Liev11ZnJHSGAz0o= Received: from mail-ot1-f72.google.com (mail-ot1-f72.google.com [209.85.210.72]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-154-3FstR5i0O2S89RvWkpM4CQ-1; Tue, 22 Jun 2021 08:32:01 -0400 X-MC-Unique: 3FstR5i0O2S89RvWkpM4CQ-1 Received: by mail-ot1-f72.google.com with SMTP id e16-20020a0568302010b02903feaaa5cf10so11040420otp.8 for ; Tue, 22 Jun 2021 05:32:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=vP7JF6KY1/ahu18Tz6/GObDLrGSrV/SofAI1KrpMPF4=; b=M/+1sFMutv2yjp3SNbU1+amEDibsWY1J0Ek61Gjpz6CIRYZHoU2WpgXx4vjxvMhblq ibEhutVyswQx2n5hRO8YIaKCrnkEZ6otz8X9+/WxD4aqckZoAdVVw3zqRSY3YaKmoAhm Jef8zmziDH7HrflvbuQ9Pp31k4rwe7lx8utpPSkMXz9hM7y4tFEb2SaXuo2t8UoNKD6p 2pyKY1EhrtanSUlL/z0+qQoyNw6kT4jgD6uciYjG2YmDtsH7csw1+/kECvtw2eESl/9v eYlMRIejmIAVaT2LNG/TuaiyIQpfF4zHZEwKAIoL00kehqCjY1L02kD/bgh8+0/HjVJ0 epJw== X-Gm-Message-State: AOAM530jsnrHX1p9ZRpbbdoDmmcrH2yVDkaDUcZuu9KgQ0i9JsiWmzF6 rn6oUIX7R0zcTRjlCEMwsPjzTlsw9KUhBnELdtrjcHvpHh/NncAK+t7xmvhFX80iKWGS2o5EFhi aJD9F9eU7PxoiNY0JgJri X-Received: by 2002:a54:4694:: with SMTP id k20mr3030296oic.134.1624365120365; Tue, 22 Jun 2021 05:32:00 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx8IUtPHHs01259DCSqd8l9v2KUFLEpzLtTrLK2iyrtXZYfJpqcLl9xh2coGZ1Y7YXxQNxOVQ== X-Received: by 2002:a54:4694:: with SMTP id k20mr3030276oic.134.1624365120167; Tue, 22 Jun 2021 05:32:00 -0700 (PDT) Received: from localhost.localdomain (075-142-250-213.res.spectrum.com. [75.142.250.213]) by smtp.gmail.com with ESMTPSA id b2sm2613572oic.56.2021.06.22.05.31.58 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 22 Jun 2021 05:31:59 -0700 (PDT) Subject: Re: [PATCH 2/4] fpga: dfl: Move DFH header register macros to linux/dfl.h To: Xu Yilun Cc: =?UTF-8?Q?Martin_Hundeb=c3=b8ll?= , Wu Hao , Moritz Fischer , Jean Delvare , Guenter Roeck , Lee Jones , Mark Brown , =?UTF-8?Q?Martin_Hundeb=c3=b8ll?= , linux-fpga@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-spi@vger.kernel.org, Debarati Biswas , Russ Weight References: <20210621070621.431482-1-mhu@silicom.dk> <20210621070621.431482-3-mhu@silicom.dk> <81975a85-e9d6-bd4b-7666-56d1d1d581bc@redhat.com> <20210622045613.GA27046@yilunxu-OptiPlex-7050> From: Tom Rix Message-ID: Date: Tue, 22 Jun 2021 05:31:57 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 MIME-Version: 1.0 In-Reply-To: <20210622045613.GA27046@yilunxu-OptiPlex-7050> Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=trix@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Precedence: bulk List-ID: X-Mailing-List: linux-spi@vger.kernel.org On 6/21/21 9:56 PM, Xu Yilun wrote: > On Mon, Jun 21, 2021 at 06:56:28AM -0700, Tom Rix wrote: >> On 6/21/21 12:06 AM, Martin Hundebøll wrote: >>> From: Debarati Biswas >>> >>> Device Feature List (DFL) drivers may be defined in subdirectories other >>> than drivers/fpga, and each DFL driver should have access to the Device >>> Feature Header (DFH) register, which contains revision and type >>> information. This change moves the macros specific to the DFH register >>> from drivers/fpga/dfl.h to include/linux/dfl.h. >>> >>> Signed-off-by: Debarati Biswas >>> Signed-off-by: Russ Weight >>> Signed-off-by: Martin Hundebøll >>> --- >>> drivers/fpga/dfl.h | 48 +---------------------------------------- >>> include/linux/dfl.h | 52 +++++++++++++++++++++++++++++++++++++++++++++ >>> 2 files changed, 53 insertions(+), 47 deletions(-) >>> >>> diff --git a/drivers/fpga/dfl.h b/drivers/fpga/dfl.h >>> index 2b82c96ba56c..6ed0353e9a99 100644 >>> --- a/drivers/fpga/dfl.h >>> +++ b/drivers/fpga/dfl.h >>> @@ -17,6 +17,7 @@ >>> #include >> bitfield.h was added to linux/dfl.h >> >> Likely both aren't needed, try removing this one. > The DFL register definitions are in dfl.h, and Source files which include > dfl.h are likely to use bitfield ops for DFL register access, so could we > keep it here? Not sure which dfl.h you mean. It looks like you are copying header includes in both files and including the one file in the other. So it is likely you are including headers you don't need. Tom > > Thanks, > Yilun > >> Tom >> >>> #include >>> #include >>> +#include >>> #include >>> #include >>> #include >>> @@ -53,32 +54,6 @@ >>> #define PORT_FEATURE_ID_UINT 0x12 >>> #define PORT_FEATURE_ID_STP 0x13 >>> -/* >>> - * Device Feature Header Register Set >>> - * >>> - * For FIUs, they all have DFH + GUID + NEXT_AFU as common header registers. >>> - * For AFUs, they have DFH + GUID as common header registers. >>> - * For private features, they only have DFH register as common header. >>> - */ >>> -#define DFH 0x0 >>> -#define GUID_L 0x8 >>> -#define GUID_H 0x10 >>> -#define NEXT_AFU 0x18 >>> - >>> -#define DFH_SIZE 0x8 >>> - >>> -/* Device Feature Header Register Bitfield */ >>> -#define DFH_ID GENMASK_ULL(11, 0) /* Feature ID */ >>> -#define DFH_ID_FIU_FME 0 >>> -#define DFH_ID_FIU_PORT 1 >>> -#define DFH_REVISION GENMASK_ULL(15, 12) /* Feature revision */ >>> -#define DFH_NEXT_HDR_OFST GENMASK_ULL(39, 16) /* Offset to next DFH */ >>> -#define DFH_EOL BIT_ULL(40) /* End of list */ >>> -#define DFH_TYPE GENMASK_ULL(63, 60) /* Feature type */ >>> -#define DFH_TYPE_AFU 1 >>> -#define DFH_TYPE_PRIVATE 3 >>> -#define DFH_TYPE_FIU 4 >>> - >>> /* Next AFU Register Bitfield */ >>> #define NEXT_AFU_NEXT_DFH_OFST GENMASK_ULL(23, 0) /* Offset to next AFU */ >>> @@ -403,27 +378,6 @@ struct device *dfl_fpga_pdata_to_parent(struct dfl_feature_platform_data *pdata) >>> return pdata->dev->dev.parent->parent; >>> } >>> -static inline bool dfl_feature_is_fme(void __iomem *base) >>> -{ >>> - u64 v = readq(base + DFH); >>> - >>> - return (FIELD_GET(DFH_TYPE, v) == DFH_TYPE_FIU) && >>> - (FIELD_GET(DFH_ID, v) == DFH_ID_FIU_FME); >>> -} >>> - >>> -static inline bool dfl_feature_is_port(void __iomem *base) >>> -{ >>> - u64 v = readq(base + DFH); >>> - >>> - return (FIELD_GET(DFH_TYPE, v) == DFH_TYPE_FIU) && >>> - (FIELD_GET(DFH_ID, v) == DFH_ID_FIU_PORT); >>> -} >>> - >>> -static inline u8 dfl_feature_revision(void __iomem *base) >>> -{ >>> - return (u8)FIELD_GET(DFH_REVISION, readq(base + DFH)); >>> -} >>> - >>> /** >>> * struct dfl_fpga_enum_info - DFL FPGA enumeration information >>> * >>> diff --git a/include/linux/dfl.h b/include/linux/dfl.h >>> index 6cc10982351a..1cd86b2e7cb1 100644 >>> --- a/include/linux/dfl.h >>> +++ b/include/linux/dfl.h >>> @@ -8,7 +8,9 @@ >>> #ifndef __LINUX_DFL_H >>> #define __LINUX_DFL_H >>> +#include >>> #include >>> +#include >>> #include >>> /** >>> @@ -83,4 +85,54 @@ void dfl_driver_unregister(struct dfl_driver *dfl_drv); >>> module_driver(__dfl_driver, dfl_driver_register, \ >>> dfl_driver_unregister) >>> +/* >>> + * Device Feature Header Register Set >>> + * >>> + * For FIUs, they all have DFH + GUID + NEXT_AFU as common header registers. >>> + * For AFUs, they have DFH + GUID as common header registers. >>> + * For private features, they only have DFH register as common header. >>> + */ >>> +#define DFH 0x0 >>> +#define GUID_L 0x8 >>> +#define GUID_H 0x10 >>> +#define NEXT_AFU 0x18 >>> + >>> +#define DFH_SIZE 0x8 >>> + >>> +/* Device Feature Header Register Bitfield */ >>> +#define DFH_ID GENMASK_ULL(11, 0) /* Feature ID */ >>> +#define DFH_ID_FIU_FME 0 >>> +#define DFH_ID_FIU_PORT 1 >>> +#define DFH_REVISION GENMASK_ULL(15, 12) >>> +#define DFH_NEXT_HDR_OFST GENMASK_ULL(39, 16) /* Offset to next DFH */ >>> +#define DFH_EOL BIT_ULL(40) /* End of list */ >>> +#define DFH_TYPE GENMASK_ULL(63, 60) /* Feature type */ >>> +#define DFH_TYPE_AFU 1 >>> +#define DFH_TYPE_PRIVATE 3 >>> +#define DFH_TYPE_FIU 4 >>> + >>> +/* Function to read from DFH and check if the Feature type is FME */ >>> +static inline bool dfl_feature_is_fme(void __iomem *base) >>> +{ >>> + u64 v = readq(base + DFH); >>> + >>> + return (FIELD_GET(DFH_TYPE, v) == DFH_TYPE_FIU) && >>> + (FIELD_GET(DFH_ID, v) == DFH_ID_FIU_FME); >>> +} >>> + >>> +/* Function to read from DFH and check if the Feature type is port*/ >>> +static inline bool dfl_feature_is_port(void __iomem *base) >>> +{ >>> + u64 v = readq(base + DFH); >>> + >>> + return (FIELD_GET(DFH_TYPE, v) == DFH_TYPE_FIU) && >>> + (FIELD_GET(DFH_ID, v) == DFH_ID_FIU_PORT); >>> +} >>> + >>> +/* Function to read feature revision from DFH */ >>> +static inline u8 dfl_feature_revision(void __iomem *base) >>> +{ >>> + return (u8)FIELD_GET(DFH_REVISION, readq(base + DFH)); >>> +} >>> + >>> #endif /* __LINUX_DFL_H */