Hi Svyatoslav,
On Thu, 20 Apr 2023 at 17:35, Svyatoslav Ryhel <clamo...@gmail.com> wrote:
>
> чт, 20 квіт. 2023 р. о 01:41 Simon Glass <s...@chromium.org> пише:
> >
> > Hi Svyatoslav,
> >
> > On Wed, 19 Apr 2023 at 13:17, Svyatoslav Ryhel <clamo...@gmail.com> wrote:
> > >
> > > SSD2825 is an innovative and cost-effective MIPI Bridge Chip solution
> > > targeting high resolution smartphones. It can convert 24bit RGB
> > > interface into 4-lane MIPI-DSI interface to drive extremely high
> > > resolution display modules of up to 800 x 1366, while supporting AMOLED,
> > > a-si LCD or LTPS advanced panel technologies for smartphone applications.
> > >
> > > Bridge is wrapped in panel uClass model for wider compatibility.
> > >
> > > Tested-by: Andreas Westman Dorcsak <hed...@yahoo.com> # LG P880 T30
> > > Tested-by: Svyatoslav Ryhel <clamo...@gmail.com> # LG P895 T30
> > > Signed-off-by: Svyatoslav Ryhel <clamo...@gmail.com>
> > > ---
> > > drivers/video/bridge/Kconfig | 7 +
> > > drivers/video/bridge/Makefile | 1 +
> > > drivers/video/bridge/ssd2825.c | 523 +++++++++++++++++++++++++++++++++
> > > 3 files changed, 531 insertions(+)
> > > create mode 100644 drivers/video/bridge/ssd2825.c
> > >
> >
> > [..]
> >
> > > +static int ssd2825_spi_write(struct udevice *dev, int reg,
> > > + const void *buf, int flags)
> > > +{
> > > + struct spi_slave *slave = dev_get_parent_priv(dev);
> > > + u8 command[2];
> > > +
> > > + if (flags & SSD2825_CMD_SEND) {
> > > + command[0] = SSD2825_CMD_MASK;
> > > + command[1] = reg;
> > > + spi_xfer(slave, 9, &command,
> > > + NULL, SPI_XFER_ONCE);
> >
> > Please can you use dm_spi_xfer() instead, thoughout?
> >
>
> I will replace it, but isn't spi_xfer preferable over dm_spi_xfer?
> I remember that I had doubts about which to use.
No, we should use the dm variants where available, since then we can
drop the old ones.
It might be time for the SPI maintainer to take a look.
+Jagan Teki +Vignesh R for that
[..]
Regards,
Simon
