On Mon, Jan 25, 2021 at 02:56:26PM -0600, richard.gong@xxxxxxxxxxxxxxx wrote: > From: Richard Gong <richard.gong@xxxxxxxxx> > > Add authenticate-fpga-config property to support FPGA bitstream > authentication, which makes sure a signed bitstream has valid signatures. > > Signed-off-by: Richard Gong <richard.gong@xxxxxxxxx> > --- > v3: no change > v2: changed in alphabetical order > --- > drivers/fpga/of-fpga-region.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/fpga/of-fpga-region.c b/drivers/fpga/of-fpga-region.c > index e405309..3840883 100644 > --- a/drivers/fpga/of-fpga-region.c > +++ b/drivers/fpga/of-fpga-region.c > @@ -219,6 +219,9 @@ static struct fpga_image_info *of_fpga_region_parse_ov( > info->overlay = overlay; > > /* Read FPGA region properties from the overlay */ > + if (of_property_read_bool(overlay, "authenticate-fpga-config")) > + info->flags |= FPGA_MGR_BITSTREM_AUTHENTICATION; > + Should you check here that no new nodes are being added as you *only* authenticate? > if (of_property_read_bool(overlay, "partial-fpga-config")) > info->flags |= FPGA_MGR_PARTIAL_RECONFIG; > > -- > 2.7.4 > Thanks, Moritz
