From: mchehab@osg•samsung.com (Mauro Carvalho Chehab)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 13/18] [media] media-entity.h: rename entity.type to entity.function
Date: Tue, 24 Nov 2015 08:32:08 -0200 [thread overview]
Message-ID: <20151124083208.590bebbb@recife.lan> (raw)
In-Reply-To: <7197567.LfPs5Bvqpi@avalon>
Em Mon, 23 Nov 2015 19:51:41 +0200
Laurent Pinchart <laurent.pinchart@ideasonboard•com> escreveu:
> Hi Mauro,
>
> Thank you for the patch.
>
> On Sunday 06 September 2015 14:30:56 Mauro Carvalho Chehab wrote:
> > Entities should have one or more functions. Calling it as a
> > type proofed to not be correct, as an entity could eventually
>
> s/proofed/proved/
>
> > have more than one type.
> >
> > So, rename the field as function.
>
> s/as/to/
>
> > Please notice that this patch doesn't extend support for
> > multiple function entities. Such change will happen when
> > we have real case drivers using it.
> >
> > No functional changes.
> >
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@osg•samsung.com>
>
> [snip]
>
> > diff --git a/include/media/media-entity.h b/include/media/media-entity.h
> > index 8bdc10dcc5e7..10f7d5f0eb66 100644
> > --- a/include/media/media-entity.h
> > +++ b/include/media/media-entity.h
> > @@ -152,7 +152,8 @@ struct media_entity_operations {
> > *
> > * @graph_obj: Embedded structure containing the media object common data.
> > * @name: Entity name.
> > - * @type: Entity type, as defined at uapi/media.h (MEDIA_ENT_T_*)
> > + * @function: Entity main function, as defined at uapi/media.h
> > + * (MEDIA_ENT_F_*)
>
> I would squash this patch with "uapi/media.h: Rename entities types to
> functions" as they essentially touch the same lines.
They touch at the same lines, but the rationale is different. Keeping
it separate is, IMHO, better, as one logical change per patch is preferred.
> If you want to keep them
> separate I would use MEDIA_ENT_T_* here and rename it to MEDIA_ENT_F_* in
> "uapi/media.h: Rename entities types to functions".
IMHO, that would be confusing, as this patch would be incomplete, and
the other one would be renaming from MEDIA_ENT_T_* to MEDIA_ENT_F_*
due to a reason that it is on a different patch.
>
> > * @revision: Entity revision - OBSOLETE - should be removed soon.
> > * @flags: Entity flags, as defined at uapi/media.h (MEDIA_ENT_FL_*)
> > * @group_id: Entity group ID - OBSOLETE - should be removed soon.
>
> [snip]
>
next prev parent reply other threads:[~2015-11-24 10:32 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1441559233.git.mchehab@osg.samsung.com>
2015-09-06 17:30 ` [PATCH 13/18] [media] media-entity.h: rename entity.type to entity.function Mauro Carvalho Chehab
2015-09-11 15:25 ` Hans Verkuil
2015-11-23 17:51 ` Laurent Pinchart
2015-11-24 10:32 ` Mauro Carvalho Chehab [this message]
2015-09-06 17:30 ` [PATCH 15/18] [media] uapi/media.h: Rename entities types to functions Mauro Carvalho Chehab
2015-09-11 15:36 ` Hans Verkuil
2015-12-10 19:54 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151124083208.590bebbb@recife.lan \
--to=mchehab@osg$(echo .)samsung.com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox