From: Nandhini Srikandan <[email protected]>
Fixes: 2cc8d9227bbb ("spi: dw: Introduce Synopsys IP-core versions interface")
Signed-off-by: Nandhini Srikandan <[email protected]>
---
drivers/spi/spi-dw.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/spi/spi-dw.h b/drivers/spi/spi-dw.h
index d5ee5130601e..79d853f6d192 100644
--- a/drivers/spi/spi-dw.h
+++ b/drivers/spi/spi-dw.h
@@ -23,7 +23,7 @@
((_dws)->ip == DW_ ## _ip ## _ID)
#define __dw_spi_ver_cmp(_dws, _ip, _ver, _op) \
- (dw_spi_ip_is(_dws, _ip) && (_dws)->ver _op DW_ ## _ip ## _ver)
+ (dw_spi_ip_is(_dws, _ip) && (_dws)->ver _op DW_ ## _ip ## _ ## _ver)
#define dw_spi_ver_is(_dws, _ip, _ver) __dw_spi_ver_cmp(_dws, _ip, _ver, ==)
--
2.17.1
On Wed, Jul 06, 2022 at 12:20:36PM +0800, [email protected] wrote:
> From: Nandhini Srikandan <[email protected]>
>
> Fixes: 2cc8d9227bbb ("spi: dw: Introduce Synopsys IP-core versions interface")
> Signed-off-by: Nandhini Srikandan <[email protected]>
What is the problem and how does this patch fix it?
On Wed, Jul 06, 2022 at 02:56:30PM +0000, Srikandan, Nandhini wrote:
> > wrote:
> > > From: Nandhini Srikandan <[email protected]>
> > >
> > > Fixes: 2cc8d9227bbb ("spi: dw: Introduce Synopsys IP-core versions
> > interface")
> > > Signed-off-by: Nandhini Srikandan <[email protected]>
> > What is the problem and how does this patch fix it?
> The macro was introduced by Serge Semin in the patch set (2cc8d9227bbb) but it was unused.
> It is used by one of the patches in the current patchset and without the fix it leads to compilation error. I had included this in v4 and Serge Semin had asked me to add the "Fixes: 2cc8d9227bbb" tag and keep it at the top of the series. So, I have kept it as first patch in this patchset series.
This is information that ought to be in the changelog.
Please fix your mail client to word wrap within paragraphs at something
substantially less than 80 columns. Doing this makes your messages much
easier to read and reply to.
> -----Original Message-----
> From: Mark Brown <[email protected]>
> Sent: Wednesday, July 6, 2022 5:28 PM
> To: Srikandan, Nandhini <[email protected]>
> Cc: [email protected]; [email protected]; linux-
> [email protected]; [email protected];
> [email protected]; [email protected]; Pan, Kris
> <[email protected]>; Demakkanavar, Kenchappa
> <[email protected]>; Zhou, Furong
> <[email protected]>; Sangannavar, Mallikarjunappa
> <[email protected]>; Vaidya, Mahesh R
> <[email protected]>; A, Rashmi <[email protected]>
> Subject: Re: [PATCH v5 1/4] spi: dw: Fix IP-core versions macro
>
> On Wed, Jul 06, 2022 at 12:20:36PM +0800, [email protected]
> wrote:
> > From: Nandhini Srikandan <[email protected]>
> >
> > Fixes: 2cc8d9227bbb ("spi: dw: Introduce Synopsys IP-core versions
> interface")
> > Signed-off-by: Nandhini Srikandan <[email protected]>
>
> What is the problem and how does this patch fix it?
The macro was introduced by Serge Semin in the patch set (2cc8d9227bbb) but it was unused.
It is used by one of the patches in the current patchset and without the fix it leads to compilation error. I had included this in v4 and Serge Semin had asked me to add the "Fixes: 2cc8d9227bbb" tag and keep it at the top of the series. So, I have kept it as first patch in this patchset series.
- Nandhini
Hi Nandhini
On Wed, Jul 06, 2022 at 12:20:36PM +0800, [email protected] wrote:
> From: Nandhini Srikandan <[email protected]>
>
> Fixes: 2cc8d9227bbb ("spi: dw: Introduce Synopsys IP-core versions interface")
> Signed-off-by: Nandhini Srikandan <[email protected]>
As Mark already noted each patch is supposed to have a changelog [1].
Please add a detailed explanation of the problem above the tags
block [1].
[1] "Patch formatting and changelogs" Documentation/process/5.Posting.rst
-Sergey
> ---
> drivers/spi/spi-dw.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/spi/spi-dw.h b/drivers/spi/spi-dw.h
> index d5ee5130601e..79d853f6d192 100644
> --- a/drivers/spi/spi-dw.h
> +++ b/drivers/spi/spi-dw.h
> @@ -23,7 +23,7 @@
> ((_dws)->ip == DW_ ## _ip ## _ID)
>
> #define __dw_spi_ver_cmp(_dws, _ip, _ver, _op) \
> - (dw_spi_ip_is(_dws, _ip) && (_dws)->ver _op DW_ ## _ip ## _ver)
> + (dw_spi_ip_is(_dws, _ip) && (_dws)->ver _op DW_ ## _ip ## _ ## _ver)
>
> #define dw_spi_ver_is(_dws, _ip, _ver) __dw_spi_ver_cmp(_dws, _ip, _ver, ==)
>
> --
> 2.17.1
>