clk: si570: Simplify probe
authorBiju Das <biju.das.jz@bp.renesas.com>
Sat, 9 Sep 2023 16:47:38 +0000 (17:47 +0100)
committerStephen Boyd <sboyd@kernel.org>
Tue, 24 Oct 2023 02:36:56 +0000 (19:36 -0700)
commit15f5e2e4225a94fe76302e87ecdf95e56e571382
tree96e90d27a6af3622a6cf8e3e6b153e701ee490e4
parentf234f028727f8634973f8a9ef2ec933eb0eade5b
clk: si570: Simplify probe

The driver has an OF match table, still, it uses an ID lookup table for
retrieving match data. Currently, the driver is working on the
assumption that an I2C device registered via OF will always match a
legacy I2C device ID. The correct approach is to have an OF device ID
table using i2c_get_match_data() if the devices are registered via OF/ID.

Unify the OF/ID table by adding struct clk_si570_info as match data
instead of clk_si570_variant and replace the ID lookup table for
the match data by i2c_get_match_data(). This allows to simplify
probe().

Drop enum clk_si570_variant as there is no user.

While at it, remove the trailing comma in the terminator entry for the OF
table making code robust against (theoretical) misrebases or other similar
things where the new entry goes _after_ the termination without the
compiler noticing.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Link: https://lore.kernel.org/r/20230909164738.47708-1-biju.das.jz@bp.renesas.com
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk-si570.c