[Skiboot] [PATCH 1/2] flash: Add documentation for blocklevel_smart_write()

Cyril Bur cyrilbur at gmail.com
Wed Jul 27 18:25:59 AEST 2016


On Wed, 27 Jul 2016 12:19:13 +1000
Michael Neuling <mikey at neuling.org> wrote:

> A function called "smart" doesn't give the user any clues as to why
> they would want to use it over a another calls.
> 
> This adds some documentation so users can determine when to best use
> this call.
> 

Thanks Mikey

Reviewed-by: Cyril Bur <cyrilbur at gmail.com>

> Signed-off-by: Michael Neuling <mikey at neuling.org>
> ---
>  libflash/blocklevel.h | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/libflash/blocklevel.h b/libflash/blocklevel.h
> index 9f4285e..9770e11 100644
> --- a/libflash/blocklevel.h
> +++ b/libflash/blocklevel.h
> @@ -64,7 +64,14 @@ int blocklevel_erase(struct blocklevel_device *bl, uint32_t pos, uint32_t len);
>  int blocklevel_get_info(struct blocklevel_device *bl, const char **name, uint32_t *total_size,
>  		uint32_t *erase_granule);
>  
> -/* Convienience functions */
> +/*
> + * blocklevel_smart_write() performs reads on the data to see if it
> + * can skip erase or write calls. This is likely more conveient for
> + * the caller since they don't need to perform these checks
> + * themselves. Depending on the new and old data, this may be faster
> + * or slower than the just using blocklevel_erase/write calls.
> + * directly.
> + */
>  int blocklevel_smart_write(struct blocklevel_device *bl, uint32_t pos, const void *buf, uint32_t len);
>  
>  /* Implemented in software at this level */



More information about the Skiboot mailing list