Joel, I have added all 3 id in the documentation patch and I am not sure if that patch has been applied or not. Regards -Vijay On 8/1/19, 11:31 PM, "Joel Stanley" <joel@xxxxxxxxx> wrote: Add pxe1610 as a trivial device On Tue, 23 Jul 2019 at 17:14, Vijay Khemka <vijaykhemka@xxxxxx> wrote: > > On 7/23/19, 7:53 AM, "Rob Herring" <robh+dt@xxxxxxxxxx> wrote: > > On Tue, Jul 23, 2019 at 8:50 AM Rob Herring <robh+dt@xxxxxxxxxx> wrote: > > > > On Mon, Jul 22, 2019 at 6:46 PM Vijay Khemka <vijaykhemka@xxxxxx> wrote: > > > > > > The pxe1610 is a voltage regulator from Infineon. It also supports > > > other VRs pxe1110 and pxm1310 from Infineon. > > What happened to the other compatibles? S/w doesn't need to know the > differences? > As far as driver is concerned, it doesn't need to know differences. You have these three IDs in the driver: pxm1310 pxm1310 pxe1610 So all three could be listed in the documentation? Rob, is this what you wanted Vijay to do?