[EXT] Re: [PATCH v4 01/10] drm: bridge: cadence: convert mailbox functions to macro functions

[Sandor Yu]

> -----Original Message-----
> From: Fabio Estevam <festevam at gmail.com>
> Sent: 2022年11月23日 2:09
> To: Sandor Yu <sandor.yu at nxp.com>
> Cc: andrzej.hajda at intel.com; neil.armstrong at linaro.org;
> robert.foss at linaro.org; Laurent.pinchart at ideasonboard.com;
> jonas at kwiboo.se; jernej.skrabec at gmail.com; airlied at gmail.com;
> daniel at ffwll.ch; robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org;
> shawnguo at kernel.org; s.hauer at pengutronix.de; kishon at ti.com;
> vkoul at kernel.org; dri-devel at lists.freedesktop.org;
> devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> linux-kernel at vger.kernel.org; linux-phy at lists.infradead.org;
> alexander.stein at ew.tq-group.com; kernel at pengutronix.de; dl-linux-imx
> <linux-imx at nxp.com>; Oliver Brown <oliver.brown at nxp.com>
> Subject: [EXT] Re: [PATCH v4 01/10] drm: bridge: cadence: convert mailbox
> functions to macro functions
> 
> Caution: EXT Email
> 
> Hi Sandor,
> 
> On Mon, Nov 21, 2022 at 4:27 AM Sandor Yu <Sandor.yu at nxp.com> wrote:
> >
> > Mailbox access functions could be share to other mhdp driver and
> > HDP-TX HDMI/DP PHY drivers, move those functions to head file
> > include/drm/bridge/cdns-mhdp-mailbox.h and convert them to macro
> > functions.
> 
> What is the reason for converting the functions to macro?
Both HDMI PHY driver and HDMI bridge driver need mailbox API function to access register and get status from firmware.
Covert those functions to macro could be easy reused by both PHY and bridge driver.

B.R
Sandor