On 12/1/21 3:08 PM, Tom Rini wrote:
On Wed, Dec 01, 2021 at 02:44:02PM -0500, Sean Anderson wrote:
This xlate function just performs some checking. We can do this in request() instead and use the default xlate.
Signed-off-by: Sean Anderson seanga2@gmail.com
drivers/clk/clk-cdce9xx.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/clk/clk-cdce9xx.c b/drivers/clk/clk-cdce9xx.c index 6634b7b799..c15e9f210e 100644 --- a/drivers/clk/clk-cdce9xx.c +++ b/drivers/clk/clk-cdce9xx.c @@ -86,19 +86,13 @@ static int cdce9xx_reg_write(struct udevice *dev, u8 addr, u8 val) return ret; }
-static int cdce9xx_clk_of_xlate(struct clk *clk,
struct ofnode_phandle_args *args)+static int cdce9xx_clk_of_request(struct clk *clk) { struct cdce9xx_clk_data *data = dev_get_priv(clk->dev);
- if (args->args_count != 1)
- if (clk->id > data->chip->num_outputs) return -EINVAL;
- if (args->args[0] > data->chip->num_outputs)
return -EINVAL;- clk->id = args->args[0];
- return 0; }
@@ -241,7 +235,7 @@ static const struct udevice_id cdce9xx_clk_of_match[] = { };
static const struct clk_ops cdce9xx_clk_ops = {
- .of_xlate = cdce9xx_clk_of_xlate,
- .request = cdce9xx_clk_request, .get_rate = cdce9xx_clk_get_rate, .set_rate = cdce9xx_clk_set_rate, };
Adding Tero...
Thanks. Perhaps .mailmap should be updated?
--Sean