[v2,0/4] drivers: clk: zynqmp: minor bux fixes for zynqmp clock driver

Message

Jolly Shah March 2, 2020, 9:50 p.m. UTC

This patchset includes below fixes for clock driver
1> Fix Divider2 calculation
2> Memory leak in clock registration
3> Fix invalid name queries
4> Limit bestdiv with maxdiv

v2:
 - Updated subject for cover letter and patches 
   to add prefix

Quanyang Wang (1):
  drivers: clk: zynqmp: fix memory leak in zynqmp_register_clocks

Rajan Vaja (2):
  drivers: clk: zynqmp: Limit bestdiv with maxdiv
  drivers: clk: zynqmp: Fix invalid clock name queries

Tejas Patel (1):
  drivers: clk: zynqmp: Fix divider2 calculation

 drivers/clk/zynqmp/clkc.c    | 20 ++++++++++++++------
 drivers/clk/zynqmp/divider.c | 19 ++++++++++++++-----
 2 files changed, 28 insertions(+), 11 deletions(-)

Comments

Jolly Shah March 23, 2020, 9:15 p.m. UTC | #1

A gentle reminder for review.

Thanks,
Jolly Shah

On 3/2/20, 1:51 PM, "Jolly Shah" <[e-mail]> wrote:

    This patchset includes below fixes for clock driver
    1> Fix Divider2 calculation
    2> Memory leak in clock registration
    3> Fix invalid name queries
    4> Limit bestdiv with maxdiv
    
    v2:
     - Updated subject for cover letter and patches 
       to add prefix
    
    Quanyang Wang (1):
      drivers: clk: zynqmp: fix memory leak in zynqmp_register_clocks
    
    Rajan Vaja (2):
      drivers: clk: zynqmp: Limit bestdiv with maxdiv
      drivers: clk: zynqmp: Fix invalid clock name queries
    
    Tejas Patel (1):
      drivers: clk: zynqmp: Fix divider2 calculation
    
     drivers/clk/zynqmp/clkc.c    | 20 ++++++++++++++------
     drivers/clk/zynqmp/divider.c | 19 ++++++++++++++-----
     2 files changed, 28 insertions(+), 11 deletions(-)
    
    -- 
    2.7.4

Jolly Shah April 9, 2020, 8:44 p.m. UTC | #2

Hi Stephan,

Ping. Please review.

Thanks,
Jolly Shah

 > ------Original Message------
 > From: Jolly Shah <[e-mail]>
 > Sent:  Monday, March 02, 2020 1:50PM
 > To: Olof <[e-mail]>, Mturquette <[e-mail]>, 
Sboyd <[e-mail]>, Michal Simek <[e-mail]>, Arm 
<[e-mail]>, Linux-clk <[e-mail]>
 > Cc: Rajan Vaja <[e-mail]>, 
[e-mail] 
<[e-mail]>, [e-mail] 
<[e-mail]>, Jolly Shah <[e-mail]>
 > Subject: [PATCH v2 0/4] drivers: clk: zynqmp: minor bux fixes for 
zynqmp clock driver
 >
> This patchset includes below fixes for clock driver
> 1> Fix Divider2 calculation
> 2> Memory leak in clock registration
> 3> Fix invalid name queries
> 4> Limit bestdiv with maxdiv
> 
> v2:
>   - Updated subject for cover letter and patches
>     to add prefix
> 
> Quanyang Wang (1):
>    drivers: clk: zynqmp: fix memory leak in zynqmp_register_clocks
> 
> Rajan Vaja (2):
>    drivers: clk: zynqmp: Limit bestdiv with maxdiv
>    drivers: clk: zynqmp: Fix invalid clock name queries
> 
> Tejas Patel (1):
>    drivers: clk: zynqmp: Fix divider2 calculation
> 
>   drivers/clk/zynqmp/clkc.c    | 20 ++++++++++++++------
>   drivers/clk/zynqmp/divider.c | 19 ++++++++++++++-----
>   2 files changed, 28 insertions(+), 11 deletions(-)
>

Michal Simek April 21, 2020, 7:50 a.m. UTC | #3

Hi Stephen,

On 02. 03. 20 22:50, Jolly Shah wrote:
> This patchset includes below fixes for clock driver
> 1> Fix Divider2 calculation
> 2> Memory leak in clock registration
> 3> Fix invalid name queries
> 4> Limit bestdiv with maxdiv
> 
> v2:
>  - Updated subject for cover letter and patches 
>    to add prefix
> 
> Quanyang Wang (1):
>   drivers: clk: zynqmp: fix memory leak in zynqmp_register_clocks
> 
> Rajan Vaja (2):
>   drivers: clk: zynqmp: Limit bestdiv with maxdiv
>   drivers: clk: zynqmp: Fix invalid clock name queries
> 
> Tejas Patel (1):
>   drivers: clk: zynqmp: Fix divider2 calculation
> 
>  drivers/clk/zynqmp/clkc.c    | 20 ++++++++++++++------
>  drivers/clk/zynqmp/divider.c | 19 ++++++++++++++-----
>  2 files changed, 28 insertions(+), 11 deletions(-)
> 

Do you see any issue with this patchset? Or do you want me to take it
via my tree? I can't see any dependency that's why I think it shouldn't
be a problem to take it via your tree.

Thanks,
Michal