Message ID | 20240812230545.GA2358@pendragon.ideasonboard.com |
---|---|
State | New |
Headers | show |
Series | [GIT,PULL,FOR,v6.12] rkisp1 extensible parameters format | expand |
Hi Laurent, On 13/08/2024 01:05, Laurent Pinchart wrote: > Hi Hans, Mauro, commit 5ef3fcc8e42b43523498c7b0ede19a47fbdf8353 (media: rkisp1: Implement extensible params support) failed to push because the commit log contains a local changelog: ... Implement parameter buffer validation for the extensible format at .buf_prepare() time. Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> Reviewed-by: Paul Elder <paul.elder@ideasonboard.com> Tested-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- Changes since v10: - Use %zu to print size_t arguments I can either just take these patches (i.e. I don't merge them), and manually remove this before committing, or you need to update your tree yourself. If you decide to do the latter, then please add my: Reviewed-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> to patches 2 and 3 (uAPI changes). Regards, Hans > > The following changes since commit c80bfa4f9e0ebfce6ac909488d412347acbcb4f9: > > media: ti: cal: use 'time_left' variable with wait_event_timeout() (2024-08-09 07:56:39 +0200) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/next-media-rkisp1-20240813 > > for you to fetch changes up to 32be6029c1c1e8c628077403b2096e9d9085f6c8: > > media: rkisp1: Add support for the companding block (2024-08-12 13:36:50 +0300) > > ---------------------------------------------------------------- > Extensible parameters support for the rkisp1 driver > > ---------------------------------------------------------------- > Jacopo Mondi (7): > media: uapi: rkisp1-config: Add extensible params format > media: uapi: videodev2: Add V4L2_META_FMT_RK_ISP1_EXT_PARAMS > media: rkisp1: Add struct rkisp1_params_buffer > media: rkisp1: Copy the parameters buffer > media: rkisp1: Cache the currently active format > media: rkisp1: Implement extensible params support > media: rkisp1: Implement s_fmt/try_fmt > > Laurent Pinchart (2): > media: rkisp1: Add helper function to swap colour channels > media: rkisp1: Add features mask to extensible block handlers > > Ondrej Jirman (1): > media: rkisp1: Adapt to different SoCs having different size limits > > Paul Elder (3): > media: rkisp1: Add register definitions for the companding block > media: rkisp1: Add feature flags for BLS and compand > media: rkisp1: Add support for the companding block > > Documentation/admin-guide/media/rkisp1.rst | 11 +- > .../userspace-api/media/v4l/metafmt-rkisp1.rst | 57 +- > .../media/platform/rockchip/rkisp1/rkisp1-common.c | 14 + > .../media/platform/rockchip/rkisp1/rkisp1-common.h | 49 +- > .../media/platform/rockchip/rkisp1/rkisp1-csi.c | 5 +- > .../media/platform/rockchip/rkisp1/rkisp1-dev.c | 15 +- > .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 9 +- > .../media/platform/rockchip/rkisp1/rkisp1-params.c | 1037 ++++++++++++++++++-- > .../media/platform/rockchip/rkisp1/rkisp1-regs.h | 23 + > .../platform/rockchip/rkisp1/rkisp1-resizer.c | 4 +- > .../media/platform/rockchip/rkisp1/rkisp1-stats.c | 51 +- > drivers/media/v4l2-core/v4l2-ioctl.c | 1 + > include/uapi/linux/rkisp1-config.h | 578 +++++++++++ > include/uapi/linux/videodev2.h | 1 + > 14 files changed, 1689 insertions(+), 166 deletions(-) >
On Wed, Aug 14, 2024 at 12:02:39PM +0200, Hans Verkuil wrote: > Hi Laurent, > > On 13/08/2024 01:05, Laurent Pinchart wrote: > > Hi Hans, Mauro, > > commit 5ef3fcc8e42b43523498c7b0ede19a47fbdf8353 (media: rkisp1: Implement extensible params support) > failed to push because the commit log contains a local changelog: > > ... > Implement parameter buffer validation for the extensible format at > .buf_prepare() time. > > Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> > Reviewed-by: Paul Elder <paul.elder@ideasonboard.com> > Tested-by: Kieran Bingham <kieran.bingham@ideasonboard.com> > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > Changes since v10: > > - Use %zu to print size_t arguments > > I can either just take these patches (i.e. I don't merge them), and manually remove > this before committing, or you need to update your tree yourself. > > If you decide to do the latter, then please add my: > > Reviewed-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> > > to patches 2 and 3 (uAPI changes). Oooops. I'll resend now. I wonder how I didn't notice that. > > > > The following changes since commit c80bfa4f9e0ebfce6ac909488d412347acbcb4f9: > > > > media: ti: cal: use 'time_left' variable with wait_event_timeout() (2024-08-09 07:56:39 +0200) > > > > are available in the Git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/next-media-rkisp1-20240813 > > > > for you to fetch changes up to 32be6029c1c1e8c628077403b2096e9d9085f6c8: > > > > media: rkisp1: Add support for the companding block (2024-08-12 13:36:50 +0300) > > > > ---------------------------------------------------------------- > > Extensible parameters support for the rkisp1 driver > > > > ---------------------------------------------------------------- > > Jacopo Mondi (7): > > media: uapi: rkisp1-config: Add extensible params format > > media: uapi: videodev2: Add V4L2_META_FMT_RK_ISP1_EXT_PARAMS > > media: rkisp1: Add struct rkisp1_params_buffer > > media: rkisp1: Copy the parameters buffer > > media: rkisp1: Cache the currently active format > > media: rkisp1: Implement extensible params support > > media: rkisp1: Implement s_fmt/try_fmt > > > > Laurent Pinchart (2): > > media: rkisp1: Add helper function to swap colour channels > > media: rkisp1: Add features mask to extensible block handlers > > > > Ondrej Jirman (1): > > media: rkisp1: Adapt to different SoCs having different size limits > > > > Paul Elder (3): > > media: rkisp1: Add register definitions for the companding block > > media: rkisp1: Add feature flags for BLS and compand > > media: rkisp1: Add support for the companding block > > > > Documentation/admin-guide/media/rkisp1.rst | 11 +- > > .../userspace-api/media/v4l/metafmt-rkisp1.rst | 57 +- > > .../media/platform/rockchip/rkisp1/rkisp1-common.c | 14 + > > .../media/platform/rockchip/rkisp1/rkisp1-common.h | 49 +- > > .../media/platform/rockchip/rkisp1/rkisp1-csi.c | 5 +- > > .../media/platform/rockchip/rkisp1/rkisp1-dev.c | 15 +- > > .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 9 +- > > .../media/platform/rockchip/rkisp1/rkisp1-params.c | 1037 ++++++++++++++++++-- > > .../media/platform/rockchip/rkisp1/rkisp1-regs.h | 23 + > > .../platform/rockchip/rkisp1/rkisp1-resizer.c | 4 +- > > .../media/platform/rockchip/rkisp1/rkisp1-stats.c | 51 +- > > drivers/media/v4l2-core/v4l2-ioctl.c | 1 + > > include/uapi/linux/rkisp1-config.h | 578 +++++++++++ > > include/uapi/linux/videodev2.h | 1 + > > 14 files changed, 1689 insertions(+), 166 deletions(-)
On Wed, Aug 14, 2024 at 04:19:15PM +0300, Laurent Pinchart wrote: > On Wed, Aug 14, 2024 at 12:02:39PM +0200, Hans Verkuil wrote: > > On 13/08/2024 01:05, Laurent Pinchart wrote: > > > Hi Hans, Mauro, > > > > commit 5ef3fcc8e42b43523498c7b0ede19a47fbdf8353 (media: rkisp1: Implement extensible params support) > > failed to push because the commit log contains a local changelog: > > > > ... > > Implement parameter buffer validation for the extensible format at > > .buf_prepare() time. > > > > Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> > > Reviewed-by: Paul Elder <paul.elder@ideasonboard.com> > > Tested-by: Kieran Bingham <kieran.bingham@ideasonboard.com> > > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > --- > > Changes since v10: > > > > - Use %zu to print size_t arguments > > > > I can either just take these patches (i.e. I don't merge them), and manually remove > > this before committing, or you need to update your tree yourself. > > > > If you decide to do the latter, then please add my: > > > > Reviewed-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> > > > > to patches 2 and 3 (uAPI changes). > > Oooops. I'll resend now. I wonder how I didn't notice that. Pull request resent, and local scripts updated to make sure this won't happen again. > > > > > > The following changes since commit c80bfa4f9e0ebfce6ac909488d412347acbcb4f9: > > > > > > media: ti: cal: use 'time_left' variable with wait_event_timeout() (2024-08-09 07:56:39 +0200) > > > > > > are available in the Git repository at: > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/next-media-rkisp1-20240813 > > > > > > for you to fetch changes up to 32be6029c1c1e8c628077403b2096e9d9085f6c8: > > > > > > media: rkisp1: Add support for the companding block (2024-08-12 13:36:50 +0300) > > > > > > ---------------------------------------------------------------- > > > Extensible parameters support for the rkisp1 driver > > > > > > ---------------------------------------------------------------- > > > Jacopo Mondi (7): > > > media: uapi: rkisp1-config: Add extensible params format > > > media: uapi: videodev2: Add V4L2_META_FMT_RK_ISP1_EXT_PARAMS > > > media: rkisp1: Add struct rkisp1_params_buffer > > > media: rkisp1: Copy the parameters buffer > > > media: rkisp1: Cache the currently active format > > > media: rkisp1: Implement extensible params support > > > media: rkisp1: Implement s_fmt/try_fmt > > > > > > Laurent Pinchart (2): > > > media: rkisp1: Add helper function to swap colour channels > > > media: rkisp1: Add features mask to extensible block handlers > > > > > > Ondrej Jirman (1): > > > media: rkisp1: Adapt to different SoCs having different size limits > > > > > > Paul Elder (3): > > > media: rkisp1: Add register definitions for the companding block > > > media: rkisp1: Add feature flags for BLS and compand > > > media: rkisp1: Add support for the companding block > > > > > > Documentation/admin-guide/media/rkisp1.rst | 11 +- > > > .../userspace-api/media/v4l/metafmt-rkisp1.rst | 57 +- > > > .../media/platform/rockchip/rkisp1/rkisp1-common.c | 14 + > > > .../media/platform/rockchip/rkisp1/rkisp1-common.h | 49 +- > > > .../media/platform/rockchip/rkisp1/rkisp1-csi.c | 5 +- > > > .../media/platform/rockchip/rkisp1/rkisp1-dev.c | 15 +- > > > .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 9 +- > > > .../media/platform/rockchip/rkisp1/rkisp1-params.c | 1037 ++++++++++++++++++-- > > > .../media/platform/rockchip/rkisp1/rkisp1-regs.h | 23 + > > > .../platform/rockchip/rkisp1/rkisp1-resizer.c | 4 +- > > > .../media/platform/rockchip/rkisp1/rkisp1-stats.c | 51 +- > > > drivers/media/v4l2-core/v4l2-ioctl.c | 1 + > > > include/uapi/linux/rkisp1-config.h | 578 +++++++++++ > > > include/uapi/linux/videodev2.h | 1 + > > > 14 files changed, 1689 insertions(+), 166 deletions(-)