чт, 20 квіт. 2023 р. о 19:30 Simon Glass <s...@chromium.org> пише: > > 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. >
I have switched to the dm version and it seems to fit even better, everything works perfectly fine. > It might be time for the SPI maintainer to take a look. > > +Jagan Teki +Vignesh R for that > > [..] > > Regards, > Simon