Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1484907 > unrolled thread

Re: [PATCH v2 17/17] UBI: introduce the VID buffer concept

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2 17/17] UBI: introduce the VID buffer concept Richard Weinberger <richard@nod.at> - 2016-09-16 13:40 +0200
    Re: [PATCH v2 17/17] UBI: introduce the VID buffer concept Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-16 13:50 +0200

#1484907 — Re: [PATCH v2 17/17] UBI: introduce the VID buffer concept

Boris,

On 05.09.2016 17:05, Boris Brezillon wrote:
> +/**
> + * ubi_get_vid_hdr - Get the VID header attached to a VID buffer
> + * @vidb: the VID buffer to free

Copy paste error?

> + */
> +static inline struct ubi_vid_hdr *ubi_get_vid_hdr(struct ubi_vid_io_buf *vidb)
> +{
> +	return vidb->hdr;

I guess in future there will be a second parameter to return the nth header located
on the buffer?

Thanks,
//richard

[toc] | [next] | [standalone]

#1484913

On Fri, 16 Sep 2016 13:38:27 +0200
Richard Weinberger <richard@nod.at> wrote:

> Boris,
> 
> On 05.09.2016 17:05, Boris Brezillon wrote:
> > +/**
> > + * ubi_get_vid_hdr - Get the VID header attached to a VID buffer
> > + * @vidb: the VID buffer to free  
> 
> Copy paste error?

Yep. Will fix that.

> 
> > + */
> > +static inline struct ubi_vid_hdr *ubi_get_vid_hdr(struct ubi_vid_io_buf *vidb)
> > +{
> > +	return vidb->hdr;  
> 
> I guess in future there will be a second parameter to return the nth header located
> on the buffer?

Exactly.

> 
> Thanks,
> //richard

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web