On 19:44-20230907, Neha Malcom Francis wrote: > The U-Boot AVS driver works on the VTM (Voltage and Thermal Management) > module, also used by the Linux TI Bandgap temperature sensor driver > (drivers/thermal/k3_j72xx_bandgap.c). Although the purpose and > functionalities that these two implement are different, the hardware is > the same, so ensure that their compatibles are in sync. Thus, add > ti,j721e-vtm compatible to the AVS driver. > > Signed-off-by: Neha Malcom Francis <[email protected]> > --- > drivers/misc/k3_avs.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/misc/k3_avs.c b/drivers/misc/k3_avs.c > index 840148d090..3008cf9810 100644 > --- a/drivers/misc/k3_avs.c > +++ b/drivers/misc/k3_avs.c > @@ -382,6 +382,7 @@ static struct vd_config am654_vd_config = { > static const struct udevice_id k3_avs_ids[] = { > { .compatible = "ti,am654-avs", .data = (ulong)&am654_vd_config }, > { .compatible = "ti,j721e-avs", .data = (ulong)&j721e_vd_config }, > + { .compatible = "ti,j721e-vtm", .data = (ulong)&j721e_vd_config }, > {} > }; > > -- > 2.34.1 > https://lore.kernel.org/all/[email protected]/
we are going to end up creating a dependency nightmare. Can we just do a single series(might be just a single patch) for compatible additions for k3_avs.c and make the dts syncs dependent on this? -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
