[2/5] utils: tuning: rkisp1: Add skeletal AGC to the rkisp1 tuning script
diff mbox series

Message ID 20240405145335.3010479-3-paul.elder@ideasonboard.com
State Superseded
Headers show
Series
  • utils: tuning: Add AGC and CCM
Related show

Commit Message

Paul Elder April 5, 2024, 2:53 p.m. UTC
Add the skeletal AGC module to the rkisp1 tuning script. For now it just
spits out hardcoded values.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
---
 utils/tuning/rkisp1.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Stefan Klug April 12, 2024, 7:47 a.m. UTC | #1
Hi Paul,

thanks for the patch.
Looks good to me.

Reviewed-by: Stefan Klug <stefan.klug@ideasonboard.com> 

Cheers,
Stefan

On Fri, Apr 05, 2024 at 11:53:32PM +0900, Paul Elder wrote:
> Add the skeletal AGC module to the rkisp1 tuning script. For now it just
> spits out hardcoded values.
> 
> Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> ---
>  utils/tuning/rkisp1.py | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/utils/tuning/rkisp1.py b/utils/tuning/rkisp1.py
> index 1cea6ddb..d42646de 100755
> --- a/utils/tuning/rkisp1.py
> +++ b/utils/tuning/rkisp1.py
> @@ -11,6 +11,7 @@ import libtuning as lt
>  from libtuning.parsers import YamlParser
>  from libtuning.generators import YamlOutput
>  from libtuning.modules.lsc import LSCRkISP1
> +from libtuning.modules.agc import AGCRkISP1
>  
>  tuner = lt.Tuner('RkISP1')
>  tuner.add(LSCRkISP1(
> @@ -32,9 +33,10 @@ tuner.add(LSCRkISP1(
>            # values.  This can also be a custom function.
>            smoothing_function=lt.smoothing.MedianBlur(3),
>            ))
> +tuner.add(AGCRkISP1(debug=[lt.Debug.Plot], hw_rev='V10'))
>  tuner.set_input_parser(YamlParser())
>  tuner.set_output_formatter(YamlOutput())
> -tuner.set_output_order([LSCRkISP1])
> +tuner.set_output_order([AGCRkISP1, LSCRkISP1])
>  
>  if __name__ == '__main__':
>      sys.exit(tuner.run(sys.argv))
> -- 
> 2.39.2
>
Kieran Bingham April 12, 2024, 9:04 a.m. UTC | #2
Quoting Stefan Klug (2024-04-12 08:47:53)
> Hi Paul,
> 
> thanks for the patch.
> Looks good to me.
> 
> Reviewed-by: Stefan Klug <stefan.klug@ideasonboard.com> 
> 
> Cheers,
> Stefan
> 
> On Fri, Apr 05, 2024 at 11:53:32PM +0900, Paul Elder wrote:
> > Add the skeletal AGC module to the rkisp1 tuning script. For now it just
> > spits out hardcoded values.
> > 
> > Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> > ---
> >  utils/tuning/rkisp1.py | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> > diff --git a/utils/tuning/rkisp1.py b/utils/tuning/rkisp1.py
> > index 1cea6ddb..d42646de 100755
> > --- a/utils/tuning/rkisp1.py
> > +++ b/utils/tuning/rkisp1.py
> > @@ -11,6 +11,7 @@ import libtuning as lt
> >  from libtuning.parsers import YamlParser
> >  from libtuning.generators import YamlOutput
> >  from libtuning.modules.lsc import LSCRkISP1
> > +from libtuning.modules.agc import AGCRkISP1
> >  
> >  tuner = lt.Tuner('RkISP1')
> >  tuner.add(LSCRkISP1(
> > @@ -32,9 +33,10 @@ tuner.add(LSCRkISP1(
> >            # values.  This can also be a custom function.
> >            smoothing_function=lt.smoothing.MedianBlur(3),
> >            ))
> > +tuner.add(AGCRkISP1(debug=[lt.Debug.Plot], hw_rev='V10'))

Are the AGC tuning parameters expected to differ between hw_revs ?

> >  tuner.set_input_parser(YamlParser())
> >  tuner.set_output_formatter(YamlOutput())
> > -tuner.set_output_order([LSCRkISP1])
> > +tuner.set_output_order([AGCRkISP1, LSCRkISP1])
> >  
> >  if __name__ == '__main__':
> >      sys.exit(tuner.run(sys.argv))
> > -- 
> > 2.39.2
> >
Kieran Bingham April 12, 2024, 9:08 a.m. UTC | #3
Quoting Kieran Bingham (2024-04-12 10:04:36)
> Quoting Stefan Klug (2024-04-12 08:47:53)
> > Hi Paul,
> > 
> > thanks for the patch.
> > Looks good to me.
> > 
> > Reviewed-by: Stefan Klug <stefan.klug@ideasonboard.com> 
> > 
> > Cheers,
> > Stefan
> > 
> > On Fri, Apr 05, 2024 at 11:53:32PM +0900, Paul Elder wrote:
> > > Add the skeletal AGC module to the rkisp1 tuning script. For now it just
> > > spits out hardcoded values.
> > > 
> > > Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> > > ---
> > >  utils/tuning/rkisp1.py | 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/utils/tuning/rkisp1.py b/utils/tuning/rkisp1.py
> > > index 1cea6ddb..d42646de 100755
> > > --- a/utils/tuning/rkisp1.py
> > > +++ b/utils/tuning/rkisp1.py
> > > @@ -11,6 +11,7 @@ import libtuning as lt
> > >  from libtuning.parsers import YamlParser
> > >  from libtuning.generators import YamlOutput
> > >  from libtuning.modules.lsc import LSCRkISP1
> > > +from libtuning.modules.agc import AGCRkISP1
> > >  
> > >  tuner = lt.Tuner('RkISP1')
> > >  tuner.add(LSCRkISP1(
> > > @@ -32,9 +33,10 @@ tuner.add(LSCRkISP1(
> > >            # values.  This can also be a custom function.
> > >            smoothing_function=lt.smoothing.MedianBlur(3),
> > >            ))
> > > +tuner.add(AGCRkISP1(debug=[lt.Debug.Plot], hw_rev='V10'))
> 
> Are the AGC tuning parameters expected to differ between hw_revs ?

Aha, yes I see it probably is. So doesn't that mean we'll need to
distinguish between these when loading the files?

I.e. - the tuning files will have to be rkisp1 version specfiic in the
file name that gets loaded. That could lead to quite a bit of tuning
file duplication ;-(

Or otherwise, the tuning process should write both AGC configurations to
the same file, and then the IPA would load and only read the one it is
interested in, without having to duplicate all of the LSC etc.

--
Kieran


> 
> > >  tuner.set_input_parser(YamlParser())
> > >  tuner.set_output_formatter(YamlOutput())
> > > -tuner.set_output_order([LSCRkISP1])
> > > +tuner.set_output_order([AGCRkISP1, LSCRkISP1])
> > >  
> > >  if __name__ == '__main__':
> > >      sys.exit(tuner.run(sys.argv))
> > > -- 
> > > 2.39.2
> > >

Patch
diff mbox series

diff --git a/utils/tuning/rkisp1.py b/utils/tuning/rkisp1.py
index 1cea6ddb..d42646de 100755
--- a/utils/tuning/rkisp1.py
+++ b/utils/tuning/rkisp1.py
@@ -11,6 +11,7 @@  import libtuning as lt
 from libtuning.parsers import YamlParser
 from libtuning.generators import YamlOutput
 from libtuning.modules.lsc import LSCRkISP1
+from libtuning.modules.agc import AGCRkISP1
 
 tuner = lt.Tuner('RkISP1')
 tuner.add(LSCRkISP1(
@@ -32,9 +33,10 @@  tuner.add(LSCRkISP1(
           # values.  This can also be a custom function.
           smoothing_function=lt.smoothing.MedianBlur(3),
           ))
+tuner.add(AGCRkISP1(debug=[lt.Debug.Plot], hw_rev='V10'))
 tuner.set_input_parser(YamlParser())
 tuner.set_output_formatter(YamlOutput())
-tuner.set_output_order([LSCRkISP1])
+tuner.set_output_order([AGCRkISP1, LSCRkISP1])
 
 if __name__ == '__main__':
     sys.exit(tuner.run(sys.argv))