[PATCH v5 4/6] fwu: gpt: implement read_mdata and write_mdata callbacks

[Etienne Carriere]

On Tue, 28 Feb 2023 at 01:52, <jassisinghbrar at gmail.com> wrote:
>
> From: Jassi Brar <jaswinder.singh at linaro.org>
>
> Moving towards using common code for meta-data management,
> implement the read/write mdata hooks.
>
> Signed-off-by: Jassi Brar <jaswinder.singh at linaro.org>
> Reviewed-by: Etienne Carriere <etienne.carriere at linaro.org>
> Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> ---
>  drivers/fwu-mdata/gpt_blk.c | 36 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)
>
> diff --git a/drivers/fwu-mdata/gpt_blk.c b/drivers/fwu-mdata/gpt_blk.c
> index 28f5d23e1e..bdaa10cd1d 100644
> --- a/drivers/fwu-mdata/gpt_blk.c
> +++ b/drivers/fwu-mdata/gpt_blk.c
> @@ -272,7 +272,43 @@ static int fwu_mdata_gpt_blk_probe(struct udevice *dev)
>         return 0;
>  }
>
> +static int fwu_gpt_read_mdata(struct udevice *dev, struct fwu_mdata *mdata,
> +                                                bool primary)
> +{
> +       struct fwu_mdata_gpt_blk_priv *priv = dev_get_priv(dev);
> +       struct blk_desc *desc = dev_get_uclass_plat(priv->blk_dev);
> +       int ret;
> +
> +       ret = gpt_get_mdata_partitions(desc);
> +       if (ret < 0) {
> +               log_debug("Error getting the FWU metadata partitions\n");
> +               return -ENOENT;
> +       }
> +
> +       return gpt_read_write_mdata(desc, mdata, MDATA_READ,
> +                                primary ? g_mdata_part[0] : g_mdata_part[1]);

Indentation to fix.


> +}
> +
> +static int fwu_gpt_write_mdata(struct udevice *dev, struct fwu_mdata *mdata,
> +                                                bool primary)
> +{
> +       struct fwu_mdata_gpt_blk_priv *priv = dev_get_priv(dev);
> +       struct blk_desc *desc = dev_get_uclass_plat(priv->blk_dev);
> +       int ret;
> +
> +       ret = gpt_get_mdata_partitions(desc);
> +       if (ret < 0) {
> +               log_debug("Error getting the FWU metadata partitions\n");
> +               return -ENOENT;
> +       }
> +
> +       return gpt_read_write_mdata(desc, mdata, MDATA_WRITE,
> +                                primary ? g_mdata_part[0] : g_mdata_part[1]);

Indentation to fix.
With these 2 comments addressed, my R-B tag applies.

> +}
> +
>  static const struct fwu_mdata_ops fwu_gpt_blk_ops = {
> +       .read_mdata = fwu_gpt_read_mdata,
> +       .write_mdata = fwu_gpt_write_mdata,
>         .get_mdata = fwu_gpt_get_mdata,
>         .update_mdata = fwu_gpt_update_mdata,
>         .get_mdata_part_num = fwu_gpt_get_mdata_partitions,
> --
> 2.34.1
>