[PATCH 0/5] Remaining COMPILE_TEST and smatch cleanups
There were several interactions at the COMPILE_TEST and smatch patch series. While I applied most of them, there are 5 patches that I kept out of it. The omap3 patch that were in my tree was the old one. So, I'm re-posting it. The ioctl32 patches are the latest version. Let's repost it to get some acks, as this patch touches at V4L2 core, so a careful review is always a good idea. Arnd Bergmann (1): media: omap3isp: allow it to build with COMPILE_TEST Laurent Pinchart (1): media: omap3isp: Enable driver compilation on ARM with COMPILE_TEST Mauro Carvalho Chehab (3): omap: omap-iommu.h: allow building drivers with COMPILE_TEST media: v4l2-compat-ioctl32: fix several __user annotations media: v4l2-compat-ioctl32: better name userspace pointers drivers/media/platform/Kconfig| 7 +- drivers/media/platform/omap3isp/isp.c | 8 + drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 623 +- include/linux/omap-iommu.h| 5 + 4 files changed, 338 insertions(+), 305 deletions(-) -- 2.14.3
Re: [PATCH 0/5] Remaining COMPILE_TEST and smatch cleanups
Em Tue, 17 Apr 2018 06:20:10 -0400 Mauro Carvalho Chehab escreveu: > There were several interactions at the COMPILE_TEST and smatch > patch series. While I applied most of them, there are 5 patches that > I kept out of it. The omap3 patch that were in my tree was the old > one. So, I'm re-posting it. > > The ioctl32 patches are the latest version. Let's repost it to get some > acks, as this patch touches at V4L2 core, so a careful review is > always a good idea. Forgot to mention. If anyone wants to test, the patches are at: https://git.linuxtv.org/mchehab/experimental.git/log/?h=compile_test_v6b > > Arnd Bergmann (1): > media: omap3isp: allow it to build with COMPILE_TEST > > Laurent Pinchart (1): > media: omap3isp: Enable driver compilation on ARM with COMPILE_TEST > > Mauro Carvalho Chehab (3): > omap: omap-iommu.h: allow building drivers with COMPILE_TEST > media: v4l2-compat-ioctl32: fix several __user annotations > media: v4l2-compat-ioctl32: better name userspace pointers > > drivers/media/platform/Kconfig| 7 +- > drivers/media/platform/omap3isp/isp.c | 8 + > drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 623 > +- > include/linux/omap-iommu.h| 5 + > 4 files changed, 338 insertions(+), 305 deletions(-) > Thanks, Mauro
Re: [PATCH 0/5] Remaining COMPILE_TEST and smatch cleanups
Em Wed, 18 Apr 2018 12:04:14 +0300 Sakari Ailus escreveu: > On Tue, Apr 17, 2018 at 06:20:10AM -0400, Mauro Carvalho Chehab wrote: > > There were several interactions at the COMPILE_TEST and smatch > > patch series. While I applied most of them, there are 5 patches that > > I kept out of it. The omap3 patch that were in my tree was the old > > one. So, I'm re-posting it. > > > > The ioctl32 patches are the latest version. Let's repost it to get some > > acks, as this patch touches at V4L2 core, so a careful review is > > always a good idea. > > > > Arnd Bergmann (1): > > media: omap3isp: allow it to build with COMPILE_TEST > > > > Laurent Pinchart (1): > > media: omap3isp: Enable driver compilation on ARM with COMPILE_TEST > > > > Mauro Carvalho Chehab (3): > > omap: omap-iommu.h: allow building drivers with COMPILE_TEST > > media: v4l2-compat-ioctl32: fix several __user annotations > > media: v4l2-compat-ioctl32: better name userspace pointers > > > > drivers/media/platform/Kconfig| 7 +- > > drivers/media/platform/omap3isp/isp.c | 8 + > > drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 623 > > +- > > include/linux/omap-iommu.h| 5 + > > 4 files changed, 338 insertions(+), 305 deletions(-) > > For patches 1 and 2: > > Acked-by: Sakari Ailus What about patch 3? > > I'd like to see a new versions of patches 4 and 5; I agree on the naming > change. With what changes? > Could you set the To: header to a valid value going forward? It's not a > valid e-mail address but still contains "<" character which causes trouble > when replying to the patches. I've no idea how to fix it. When I submit patches, I don't add any To: header (as the "to" is meant to be the one that will apply the patches... sending an e-mail to myself seems too mad for my taste). Somewhere between git-send-email, my SMTP local host or the SMTP smart gateway, or something afterwards, a "fake" To: gets introduced. Thanks, Mauro
Re: [PATCH 0/5] Remaining COMPILE_TEST and smatch cleanups
Hi Mauro, On Thu, Apr 19, 2018 at 07:42:28AM -0300, Mauro Carvalho Chehab wrote: > Em Wed, 18 Apr 2018 12:04:14 +0300 > Sakari Ailus escreveu: > > > On Tue, Apr 17, 2018 at 06:20:10AM -0400, Mauro Carvalho Chehab wrote: > > > There were several interactions at the COMPILE_TEST and smatch > > > patch series. While I applied most of them, there are 5 patches that > > > I kept out of it. The omap3 patch that were in my tree was the old > > > one. So, I'm re-posting it. > > > > > > The ioctl32 patches are the latest version. Let's repost it to get some > > > acks, as this patch touches at V4L2 core, so a careful review is > > > always a good idea. > > > > > > Arnd Bergmann (1): > > > media: omap3isp: allow it to build with COMPILE_TEST > > > > > > Laurent Pinchart (1): > > > media: omap3isp: Enable driver compilation on ARM with COMPILE_TEST > > > > > > Mauro Carvalho Chehab (3): > > > omap: omap-iommu.h: allow building drivers with COMPILE_TEST > > > media: v4l2-compat-ioctl32: fix several __user annotations > > > media: v4l2-compat-ioctl32: better name userspace pointers > > > > > > drivers/media/platform/Kconfig| 7 +- > > > drivers/media/platform/omap3isp/isp.c | 8 + > > > drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 623 > > > +- > > > include/linux/omap-iommu.h| 5 + > > > 4 files changed, 338 insertions(+), 305 deletions(-) > > > > For patches 1 and 2: > > > > Acked-by: Sakari Ailus > > What about patch 3? > > > > > I'd like to see a new versions of patches 4 and 5; I agree on the naming > > change. > > With what changes? Hans had comments on patch 5 at least (moving changes to 3rd patch), that may affect 4th patch as well. > > > Could you set the To: header to a valid value going forward? It's not a > > valid e-mail address but still contains "<" character which causes trouble > > when replying to the patches. > > I've no idea how to fix it. When I submit patches, I don't add any To: > header (as the "to" is meant to be the one that will apply the patches... > sending an e-mail to myself seems too mad for my taste). Somewhere > between git-send-email, my SMTP local host or the SMTP smart gateway, > or something afterwards, a "fake" To: gets introduced. To header isn't about who is going to apply the patches but who they concern the most. How about addressing the mail to the linux-media list? That's what everyone else does, unless they are sending the patches to certain recipients, which they could do for a number of reasons. The invalid To: header effectively leads to unintentional off-list discussion. -- Kind regards, Sakari Ailus e-mail: sakari.ai...@iki.fi
Re: [PATCH 0/5] Remaining COMPILE_TEST and smatch cleanups
Em Mon, 23 Apr 2018 17:09:19 +0300 Sakari Ailus escreveu: > Hi Mauro, > > On Thu, Apr 19, 2018 at 07:42:28AM -0300, Mauro Carvalho Chehab wrote: > > Em Wed, 18 Apr 2018 12:04:14 +0300 > > Sakari Ailus escreveu: > > > > > On Tue, Apr 17, 2018 at 06:20:10AM -0400, Mauro Carvalho Chehab wrote: > > > > There were several interactions at the COMPILE_TEST and smatch > > > > patch series. While I applied most of them, there are 5 patches that > > > > I kept out of it. The omap3 patch that were in my tree was the old > > > > one. So, I'm re-posting it. > > > > > > > > The ioctl32 patches are the latest version. Let's repost it to get some > > > > acks, as this patch touches at V4L2 core, so a careful review is > > > > always a good idea. > > > > > > > > Arnd Bergmann (1): > > > > media: omap3isp: allow it to build with COMPILE_TEST > > > > > > > > Laurent Pinchart (1): > > > > media: omap3isp: Enable driver compilation on ARM with COMPILE_TEST > > > > > > > > Mauro Carvalho Chehab (3): > > > > omap: omap-iommu.h: allow building drivers with COMPILE_TEST > > > > media: v4l2-compat-ioctl32: fix several __user annotations > > > > media: v4l2-compat-ioctl32: better name userspace pointers > > > > > > > > drivers/media/platform/Kconfig| 7 +- > > > > drivers/media/platform/omap3isp/isp.c | 8 + > > > > drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 623 > > > > +- > > > > include/linux/omap-iommu.h| 5 + > > > > 4 files changed, 338 insertions(+), 305 deletions(-) > > > > > > For patches 1 and 2: > > > > > > Acked-by: Sakari Ailus > > > > What about patch 3? > > > > > > > > I'd like to see a new versions of patches 4 and 5; I agree on the naming > > > change. > > > > With what changes? > > Hans had comments on patch 5 at least (moving changes to 3rd patch), that > may affect 4th patch as well. All Hans comments were addressed at version 2 sent on Thru, and he already gave his reviewed-by on Friday. > > > Could you set the To: header to a valid value going forward? It's not a > > > valid e-mail address but still contains "<" character which causes trouble > > > when replying to the patches. > > > > I've no idea how to fix it. When I submit patches, I don't add any To: > > header (as the "to" is meant to be the one that will apply the patches... > > sending an e-mail to myself seems too mad for my taste). Somewhere > > between git-send-email, my SMTP local host or the SMTP smart gateway, > > or something afterwards, a "fake" To: gets introduced. > > To header isn't about who is going to apply the patches but who they > concern the most. How about addressing the mail to the linux-media list? > That's what everyone else does, unless they are sending the patches to > certain recipients, which they could do for a number of reasons. > If you look at Documentation/process/submitting-patches.rst, you'll see: ... 5) Select the recipients for your patch --- ... You should also normally choose at least one mailing list to receive a copy of your patch set. The thing is that a "To:" assumes that the one receiving it will be applying the patch. This is sometimes not true for patches I write lately, as they may be meant to be applied at linux-doc instead, when they touches something related to doc-building system. > The invalid To: header effectively leads to unintentional off-list > discussion. Yeah, I know this is bad. I'll seek for some time to look into it more deeply. Thanks, Mauro