On 10:08-20230907, Nishanth Menon wrote:
> 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 <n-fran...@ti.com>
> > ---
> > 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/52a7c0b2-7a87-c74e-e63f-d07821ffc...@ti.com/
>
> 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?
>
I submitted one here if this works:
https://lore.kernel.org/u-boot/20230907180635.89011-1-re...@ti.com/T/#u
-Reid
> --
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
> 849D 1736 249D
