Re: [PATCHv3 5/6] dt-bindings: fpga: add authenticate-fpga-config property

[Richard Gong]

On 1/25/21 11:05 PM, Moritz Fischer wrote:
> On Mon, Jan 25, 2021 at 02:56:27PM -0600, richard.gong@xxxxxxxxxxxxxxx wrote:
> > From: Richard Gong <richard.gong@xxxxxxxxx>
> >
> > Add authenticate-fpga-config property for FPGA bitstream authentication,
> > which makes sure a signed bitstream has valid signatures.
> >
> > Signed-off-by: Richard Gong <richard.gong@xxxxxxxxx>
> > ---
> > v3: no change
> > v2: put authenticate-fpga-config above partial-fpga-config
> >      update commit messages
> > ---
> >   Documentation/devicetree/bindings/fpga/fpga-region.txt | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/fpga/fpga-region.txt b/Documentation/devicetree/bindings/fpga/fpga-region.txt
> > index e811cf8..d0d3234 100644
> > --- a/Documentation/devicetree/bindings/fpga/fpga-region.txt
> > +++ b/Documentation/devicetree/bindings/fpga/fpga-region.txt
> > @@ -182,6 +182,7 @@ Optional properties:
> >   	This property is optional if the FPGA Manager handles the bridges.
> >           If the fpga-region is  the child of a fpga-bridge, the list should not
> >           contain the parent bridge.
> > +- authenticate-fpga-config : boolean, set if do bitstream authentication only.
> I don't understand. Can I do authenticate-fpga-config AND
> partial-fpga-config?

Yes, but not simultaneously.

Flag authenticate-fpga-config is used to first check the integrity of 
the bitstream. If the authentication passes, the user can perform a full 
or partial configuration to actually configure the bistream to device.

> >   - partial-fpga-config : boolean, set if partial reconfiguration is to be done,
> >   	otherwise full reconfiguration is done.
> >   - external-fpga-config : boolean, set if the FPGA has already been configured
> > --
> > 2.7.4
> Please clarify,
>
> Moritz
Regards,
Richard