чт, 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.
> Also note that when you add a printf() for errors, the string is
> stored and it increases code size. It might be better to use
> log_debug(), unless the errors are likely.
>
> [..]
>
> Regards,
> Simon
