[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y8ar1+QfRR++YfKJ@kroah.com>
Date: Tue, 17 Jan 2023 15:08:23 +0100
From: Greg KH <gregkh@...uxfoundation.org>
To: Nipun Gupta <nipun.gupta@....com>
Cc: robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
rafael@...nel.org, eric.auger@...hat.com,
alex.williamson@...hat.com, cohuck@...hat.com,
song.bao.hua@...ilicon.com, mchehab+huawei@...nel.org,
maz@...nel.org, f.fainelli@...il.com, jeffrey.l.hugo@...il.com,
saravanak@...gle.com, Michael.Srba@...nam.cz, mani@...nel.org,
yishaih@...dia.com, jgg@...pe.ca, jgg@...dia.com,
robin.murphy@....com, will@...nel.org, joro@...tes.org,
masahiroy@...nel.org, ndesaulniers@...gle.com,
linux-arm-kernel@...ts.infradead.org, linux-kbuild@...r.kernel.org,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
okaya@...nel.org, harpreet.anand@....com, nikhil.agarwal@....com,
michal.simek@....com, git@....com
Subject: Re: [PATCH 04/19] bus/cdx: add MCDI protocol interface for firmware
interaction
On Tue, Jan 17, 2023 at 07:11:36PM +0530, Nipun Gupta wrote:
> +/** Request/Response structure */
> +#define MCDI_HEADER_OFST 0
> +#define MCDI_HEADER_CODE_LBN 0
> +#define MCDI_HEADER_CODE_WIDTH 7
> +#define MCDI_HEADER_RESYNC_LBN 7
> +#define MCDI_HEADER_RESYNC_WIDTH 1
> +#define MCDI_HEADER_DATALEN_LBN 8
> +#define MCDI_HEADER_DATALEN_WIDTH 8
> +#define MCDI_HEADER_SEQ_LBN 16
> +#define MCDI_HEADER_SEQ_WIDTH 4
> +#define MCDI_HEADER_RSVD_LBN 20
> +#define MCDI_HEADER_RSVD_WIDTH 1
> +#define MCDI_HEADER_NOT_EPOCH_LBN 21
> +#define MCDI_HEADER_NOT_EPOCH_WIDTH 1
> +#define MCDI_HEADER_ERROR_LBN 22
> +#define MCDI_HEADER_ERROR_WIDTH 1
> +#define MCDI_HEADER_RESPONSE_LBN 23
> +#define MCDI_HEADER_RESPONSE_WIDTH 1
> +#define MCDI_HEADER_XFLAGS_LBN 24
> +#define MCDI_HEADER_XFLAGS_WIDTH 8
<snip>
This whole file could use some tabs to align all of the values for the
defines to make it readable. Any chance of doing that?
thanks,
greg k-h
Powered by blists - more mailing lists