From: Moritz Fischer <mdf@kernel.org>
To: Tom Rix <trix@redhat.com>
Cc: Richard Gong <richard.gong@linux.intel.com>,
mdf@kernel.org, gregkh@linuxfoundation.org,
linux-fpga@vger.kernel.org, linux-kernel@vger.kernel.org,
Richard Gong <richard.gong@intel.com>
Subject: Re: [PATCHv5 1/7] firmware: stratix10-svc: reset COMMAND_RECONFIG_FLAG_PARTIAL to 0
Date: Thu, 18 Mar 2021 09:57:43 -0700 [thread overview]
Message-ID: <YFOGh+KKYcp17aeB@epycbox.lan> (raw)
In-Reply-To: <12eb2031-db0a-4c00-117e-f3ee8dd15978@redhat.com>
On Mon, Feb 15, 2021 at 06:32:16AM -0800, Tom Rix wrote:
>
> On 2/15/21 6:41 AM, Richard Gong wrote:
> > Hi Tom,
> >
> > On 2/13/21 9:44 AM, Tom Rix wrote:
> >>
> >> On 2/9/21 2:20 PM, richard.gong@linux.intel.com wrote:
> >>> From: Richard Gong <richard.gong@intel.com>
> >>>
> >>> Clean up COMMAND_RECONFIG_FLAG_PARTIAL flag by resetting it to 0, which
> >>> aligns with the firmware settings.
> >>>
> >>> Fixes: 36847f9e3e56 ("firmware: stratix10-svc: correct reconfig flag and timeout values")
> >>> Signed-off-by: Richard Gong <richard.gong@intel.com>
> >>> ---
> >>> v5: new add, add the missing standalone patch
> >>> ---
> >>> include/linux/firmware/intel/stratix10-svc-client.h | 2 +-
> >>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/include/linux/firmware/intel/stratix10-svc-client.h b/include/linux/firmware/intel/stratix10-svc-client.h
> >>> index a93d859..f843c6a 100644
> >>> --- a/include/linux/firmware/intel/stratix10-svc-client.h
> >>> +++ b/include/linux/firmware/intel/stratix10-svc-client.h
> >>> @@ -56,7 +56,7 @@
> >>> * COMMAND_RECONFIG_FLAG_PARTIAL:
> >>> * Set to FPGA configuration type (full or partial).
> >>> */
> >>> -#define COMMAND_RECONFIG_FLAG_PARTIAL 1
> >>> +#define COMMAND_RECONFIG_FLAG_PARTIAL 0
> >>
> >> Is this the stand alone fix split from v3's patch 1 ?
> >>
> >> https://lore.kernel.org/linux-fpga/YBFW50LPP%2FyEbvEW@kroah.com/
> >> Yes, it is a stand-alone patch.
>
> Thanks.
>
> Reviewed-by: Tom Rix <trix@redhat.com>
>
> >
> >> Tom
> >>
> >>> /**
> >>> * Timeout settings for service clients:
> >>
> > Regards,
> > Richard
> >
>
Applied to fixes.
Thanks,
Moritz
next prev parent reply other threads:[~2021-03-18 16:58 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-09 22:20 [PATCHv5 0/7] Extend Intel service layer, FPGA manager and region richard.gong
2021-02-09 22:20 ` [PATCHv5 1/7] firmware: stratix10-svc: reset COMMAND_RECONFIG_FLAG_PARTIAL to 0 richard.gong
2021-02-13 15:44 ` Tom Rix
2021-02-15 14:41 ` Richard Gong
2021-02-15 14:32 ` Tom Rix
2021-03-18 16:57 ` Moritz Fischer [this message]
2021-02-09 22:20 ` [PATCHv5 2/7] firmware: stratix10-svc: add COMMAND_AUTHENTICATE_BITSTREAM flag richard.gong
2021-02-09 22:20 ` [PATCHv5 3/7] firmware: stratix10-svc: extend SVC driver to get the firmware version richard.gong
2021-03-28 20:52 ` Moritz Fischer
2021-02-09 22:20 ` [PATCHv5 4/7] fpga: fpga-mgr: add FPGA_MGR_BITSTREAM_AUTHENTICATE flag richard.gong
2021-02-09 22:20 ` [PATCHv5 5/7] fpga: of-fpga-region: add authenticate-fpga-config property richard.gong
2021-02-09 22:20 ` [PATCHv5 6/7] dt-bindings: fpga: " richard.gong
2021-02-09 22:20 ` [PATCHv5 7/7] fpga: stratix10-soc: extend driver for bitstream authentication richard.gong
2021-02-25 13:07 ` [PATCHv5 0/7] Extend Intel service layer, FPGA manager and region Gong, Richard
2021-02-25 13:28 ` Tom Rix
2021-02-25 16:58 ` Richard Gong
2021-03-19 23:22 ` Richard Gong
2021-03-20 14:35 ` Tom Rix
[not found] ` <MWHPR11MB0015516D86D02A0FE5423D6387669@MWHPR11MB0015.namprd11.prod.outlook.com>
2021-03-21 21:05 ` FW: " Richard Gong
2021-03-22 13:53 ` Tom Rix
2021-03-22 15:53 ` Richard Gong
2021-03-27 18:09 ` Moritz Fischer
2021-03-28 15:40 ` Tom Rix
2021-03-28 17:20 ` Moritz Fischer
2021-03-31 18:47 ` Russ Weight
2021-03-31 22:16 ` Moritz Fischer
2021-04-07 23:27 ` Russ Weight
2021-05-02 18:43 ` Moritz Fischer
2021-04-13 1:41 ` Richard Gong
2021-04-29 14:56 ` Richard Gong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YFOGh+KKYcp17aeB@epycbox.lan \
--to=mdf@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-fpga@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=richard.gong@intel.com \
--cc=richard.gong@linux.intel.com \
--cc=trix@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox