From: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> To: Sakari Ailus <sakari.ailus@linux.intel.com> Cc: linux-media@vger.kernel.org, heiko.stuebner@theobroma-systems.com, heiko@sntech.de, laurent.pinchart@ideasonboard.com, helen.koike@collabora.com, ezequiel@collabora.com, hverkuil@xs4all.nl, kernel@collabora.com, dafna3@gmail.com, linux-rockchip@lists.infradead.org, mchehab@kernel.org, tfiga@chromium.org Subject: Re: [PATCH v7 1/5] media: rkisp1: uapi: change hist_bins array type from __u16 to __u32 Date: Thu, 21 Jan 2021 13:48:58 +0100 [thread overview] Message-ID: <12823703-5a32-f08a-5b4d-3a5ead231b33@collabora.com> (raw) In-Reply-To: <20210120213207.GN11878@paasikivi.fi.intel.com> Am 20.01.21 um 22:32 schrieb Sakari Ailus: > Hi Dafna, > > Thanks for the update. > > On Wed, Jan 20, 2021 at 05:44:42PM +0100, Dafna Hirschfeld wrote: >> Each entry in the array is a 20 bits value composed of 16 >> bits unsigned integer and 4 bits fractional part. So the >> type should change to __u32. >> In addition add a documentation of how the measurements >> are done. > > The commit message lines wrap at 74, under 60 is not much. > >> >> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> > > Should there be a Fixes: line? The patch is changing UAPI... > > The file has been recently introduced. Should it go to fixes or to a stable > kernel, too? The file was in the staging directory and will be first destaged in the coming 5.11, so I don't think this is necessary. > >> --- >> include/uapi/linux/rkisp1-config.h | 12 ++++++++---- >> 1 file changed, 8 insertions(+), 4 deletions(-) >> >> diff --git a/include/uapi/linux/rkisp1-config.h b/include/uapi/linux/rkisp1-config.h >> index 6e449e784260..f75f8d698fb4 100644 >> --- a/include/uapi/linux/rkisp1-config.h >> +++ b/include/uapi/linux/rkisp1-config.h >> @@ -844,13 +844,17 @@ struct rkisp1_cif_isp_af_stat { >> /** >> * struct rkisp1_cif_isp_hist_stat - statistics histogram data >> * >> - * @hist_bins: measured bin counters >> + * @hist_bins: measured bin counters. Each bin is a 20 bits unsigned fixed point type. >> + * Bits 0-4 are the fractional part and bits 5-19 are the integer part. >> * >> - * Measurement window divided into 25 sub-windows, set >> - * with ISP_HIST_XXX >> + * The window of the measurements area is divided to 5x5 sub-windows. The histogram >> + * is then computed for each sub-window independently and the final result is a weighted >> + * average of the histogram measurements on all sub-windows. >> + * The window of the measurements area and the weight of each sub-window are configurable >> + * using struct @rkisp1_cif_isp_hst_config. > > Please wrap the lines before 80. ok, Thanks, Dafna > >> */ >> struct rkisp1_cif_isp_hist_stat { >> - __u16 hist_bins[RKISP1_CIF_ISP_HIST_BIN_N_MAX]; >> + __u32 hist_bins[RKISP1_CIF_ISP_HIST_BIN_N_MAX]; >> }; >> >> /** >> -- >> 2.17.1 >> >
WARNING: multiple messages have this Message-ID (diff)
From: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> To: Sakari Ailus <sakari.ailus@linux.intel.com> Cc: mchehab@kernel.org, heiko@sntech.de, heiko.stuebner@theobroma-systems.com, dafna3@gmail.com, tfiga@chromium.org, hverkuil@xs4all.nl, linux-rockchip@lists.infradead.org, helen.koike@collabora.com, laurent.pinchart@ideasonboard.com, kernel@collabora.com, ezequiel@collabora.com, linux-media@vger.kernel.org Subject: Re: [PATCH v7 1/5] media: rkisp1: uapi: change hist_bins array type from __u16 to __u32 Date: Thu, 21 Jan 2021 13:48:58 +0100 [thread overview] Message-ID: <12823703-5a32-f08a-5b4d-3a5ead231b33@collabora.com> (raw) In-Reply-To: <20210120213207.GN11878@paasikivi.fi.intel.com> Am 20.01.21 um 22:32 schrieb Sakari Ailus: > Hi Dafna, > > Thanks for the update. > > On Wed, Jan 20, 2021 at 05:44:42PM +0100, Dafna Hirschfeld wrote: >> Each entry in the array is a 20 bits value composed of 16 >> bits unsigned integer and 4 bits fractional part. So the >> type should change to __u32. >> In addition add a documentation of how the measurements >> are done. > > The commit message lines wrap at 74, under 60 is not much. > >> >> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> > > Should there be a Fixes: line? The patch is changing UAPI... > > The file has been recently introduced. Should it go to fixes or to a stable > kernel, too? The file was in the staging directory and will be first destaged in the coming 5.11, so I don't think this is necessary. > >> --- >> include/uapi/linux/rkisp1-config.h | 12 ++++++++---- >> 1 file changed, 8 insertions(+), 4 deletions(-) >> >> diff --git a/include/uapi/linux/rkisp1-config.h b/include/uapi/linux/rkisp1-config.h >> index 6e449e784260..f75f8d698fb4 100644 >> --- a/include/uapi/linux/rkisp1-config.h >> +++ b/include/uapi/linux/rkisp1-config.h >> @@ -844,13 +844,17 @@ struct rkisp1_cif_isp_af_stat { >> /** >> * struct rkisp1_cif_isp_hist_stat - statistics histogram data >> * >> - * @hist_bins: measured bin counters >> + * @hist_bins: measured bin counters. Each bin is a 20 bits unsigned fixed point type. >> + * Bits 0-4 are the fractional part and bits 5-19 are the integer part. >> * >> - * Measurement window divided into 25 sub-windows, set >> - * with ISP_HIST_XXX >> + * The window of the measurements area is divided to 5x5 sub-windows. The histogram >> + * is then computed for each sub-window independently and the final result is a weighted >> + * average of the histogram measurements on all sub-windows. >> + * The window of the measurements area and the weight of each sub-window are configurable >> + * using struct @rkisp1_cif_isp_hst_config. > > Please wrap the lines before 80. ok, Thanks, Dafna > >> */ >> struct rkisp1_cif_isp_hist_stat { >> - __u16 hist_bins[RKISP1_CIF_ISP_HIST_BIN_N_MAX]; >> + __u32 hist_bins[RKISP1_CIF_ISP_HIST_BIN_N_MAX]; >> }; >> >> /** >> -- >> 2.17.1 >> > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip
next prev parent reply other threads:[~2021-01-21 12:50 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-01-20 16:44 [PATCH v7 0/5] Fix the rkisp1 userspace API for later IP versions Dafna Hirschfeld 2021-01-20 16:44 ` Dafna Hirschfeld 2021-01-20 16:44 ` [PATCH v7 1/5] media: rkisp1: uapi: change hist_bins array type from __u16 to __u32 Dafna Hirschfeld 2021-01-20 16:44 ` Dafna Hirschfeld 2021-01-20 21:32 ` Sakari Ailus 2021-01-20 21:32 ` Sakari Ailus 2021-01-21 12:48 ` Dafna Hirschfeld [this message] 2021-01-21 12:48 ` Dafna Hirschfeld 2021-01-21 13:01 ` Sakari Ailus 2021-01-21 13:01 ` Sakari Ailus 2021-01-20 16:44 ` [PATCH v7 2/5] media: rkisp1: stats: remove a wrong cast to u8 Dafna Hirschfeld 2021-01-20 16:44 ` Dafna Hirschfeld 2021-01-21 9:31 ` Heiko Stuebner 2021-01-21 9:31 ` Heiko Stuebner 2021-01-21 10:07 ` Tomasz Figa 2021-01-21 10:07 ` Tomasz Figa 2021-01-21 11:47 ` Dafna Hirschfeld 2021-01-21 11:47 ` Dafna Hirschfeld 2021-01-20 16:44 ` [PATCH v7 3/5] media: rockchip: rkisp1: reduce number of histogram grid elements in uapi Dafna Hirschfeld 2021-01-20 16:44 ` Dafna Hirschfeld 2021-01-20 16:44 ` [PATCH v7 4/5] media: rockchip: rkisp1: carry ip version information Dafna Hirschfeld 2021-01-20 16:44 ` Dafna Hirschfeld 2021-01-20 16:44 ` [PATCH v7 5/5] media: rockchip: rkisp1: extend uapi array sizes Dafna Hirschfeld 2021-01-20 16:44 ` Dafna Hirschfeld 2021-01-21 10:16 ` Tomasz Figa 2021-01-21 10:16 ` Tomasz Figa
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=12823703-5a32-f08a-5b4d-3a5ead231b33@collabora.com \ --to=dafna.hirschfeld@collabora.com \ --cc=dafna3@gmail.com \ --cc=ezequiel@collabora.com \ --cc=heiko.stuebner@theobroma-systems.com \ --cc=heiko@sntech.de \ --cc=helen.koike@collabora.com \ --cc=hverkuil@xs4all.nl \ --cc=kernel@collabora.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-media@vger.kernel.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=mchehab@kernel.org \ --cc=sakari.ailus@linux.intel.com \ --cc=tfiga@chromium.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.