merged.

Bruce

In message: [linux-yocto][v6.1/standard/preempt-rt/nxp-sdk-6.1/nxp-soc & 
v6.1/standard/nxp-sdk-6.1/nxp-soc][PATCH 1/2] LF-8851: dmaengine: imx-sdma: 
sdma driver code optimization
on 17/07/2023 Xiaolei Wang wrote:

> From: Joy Zou <joy....@nxp.com>
> 
> commit e708cbc250efab668507628d49bfec578b94f3ed from
> https://github.com/nxp-imx/linux-imx.git lf-6.1.y
> 
> The bluetooth starts to use sdma before sdma driver initialization done.
> It will cause NULL pointer access.
> 
> This patch adds sdma is_on check in order to avoid accessing NULL pointer.
> 
> Reviewed-by: Shengjiu Wang <shengjiu.w...@nxp.com>
> Signed-off-by: Joy Zou <joy....@nxp.com>
> Signed-off-by: Xiaolei Wang <xiaolei.w...@windriver.com>
> ---
>  drivers/dma/imx-sdma.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/dma/imx-sdma.c b/drivers/dma/imx-sdma.c
> index a5d5aa7b70ce..97269f6d5eec 100644
> --- a/drivers/dma/imx-sdma.c
> +++ b/drivers/dma/imx-sdma.c
> @@ -2125,8 +2125,8 @@ static int sdma_config_write(struct dma_chan *chan,
>       sdmac->watermark_level = 0;
>       sdma_get_pc(sdmac, sdmac->peripheral_type);
>  
> -     if (!sdmac->sdma->fw_loaded && sdmac->is_ram_script) {
> -             dev_warn_once(sdmac->sdma->dev, "sdma firmware not ready!\n");
> +     if (!sdmac->sdma->is_on || (!sdmac->sdma->fw_loaded && 
> sdmac->is_ram_script)) {
> +             dev_warn_once(sdmac->sdma->dev, "sdma or sdma firmware not 
> ready!\n");
>               return -EPERM;
>       }
>  
> -- 
> 2.25.1
> 

In message: [linux-yocto][v6.1/standard/preempt-rt/nxp-sdk-6.1/nxp-soc & 
v6.1/standard/nxp-sdk-6.1/nxp-soc][PATCH 2/2] drm/bridge: adv7533: Limit 
supported clocks
on 17/07/2023 Xiaolei Wang wrote:

> Refer to commit 323751dadfbe("MLK-21958-13: drm/bridge: adv7511: Limit 
> supported clocks")
> Add Limit supported clocks for adv7533, Some modes are not working with this 
> converter.
> This will cause hdmi link down, so add this workround patch to solve this 
> problem.
> 
> Signed-off-by: Xiaolei Wang <xiaolei.w...@windriver.com>
> ---
>  drivers/gpu/drm/bridge/adv7511/adv7533.c | 31 ++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
> 
> diff --git a/drivers/gpu/drm/bridge/adv7511/adv7533.c 
> b/drivers/gpu/drm/bridge/adv7511/adv7533.c
> index 7316aa092f9b..b09ceb994123 100644
> --- a/drivers/gpu/drm/bridge/adv7511/adv7533.c
> +++ b/drivers/gpu/drm/bridge/adv7511/adv7533.c
> @@ -24,6 +24,26 @@ static const struct reg_sequence 
> adv7533_cec_fixed_registers[] = {
>       { 0x05, 0xc8 },
>  };
>  
> +/*
> + * TODO: Currently, filter-out unsupported modes by their clocks.
> + * Need to find a better way to do this.
> + * These are the pixel clocks that the converter can handle successfully.
> + */
> +
> +static const int valid_clocks[] = {
> +     148500,
> +     135000,
> +     132000,
> +     119000,
> +     108000,
> +     78750,
> +     74250,
> +     65000,
> +     49500,
> +     40000,
> +     31500,
> +};
> +
>  static void adv7511_dsi_config_timing_gen(struct adv7511 *adv)
>  {
>       struct drm_display_mode *mode = &adv->curr_mode;
> @@ -136,6 +156,8 @@ enum drm_mode_status adv7533_mode_valid(struct adv7511 
> *adv,
>       unsigned long max_lane_freq;
>       struct mipi_dsi_device *dsi = adv->dsi;
>       u8 bpp = mipi_dsi_pixel_format_to_bpp(dsi->format);
> +     size_t i, num_modes = ARRAY_SIZE(valid_clocks);
> +     bool clock_ok = false;
>  
>       /* Check max clock for either 7533 or 7535 */
>       if (mode->clock > (adv->type == ADV7533 ? 80000 : 148500))
> @@ -147,6 +169,15 @@ enum drm_mode_status adv7533_mode_valid(struct adv7511 
> *adv,
>       if (mode->clock * bpp > max_lane_freq * adv->num_dsi_lanes)
>               return MODE_CLOCK_HIGH;
>  
> +     for (i = 0; i < num_modes; i++)
> +             if (mode->clock == valid_clocks[i]) {
> +                     clock_ok = true;
> +                     break;
> +     }
> +
> +     if (!clock_ok)
> +             return MODE_NOCLOCK;
> +
>       return MODE_OK;
>  }
>  
> -- 
> 2.25.1
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#12867): 
https://lists.yoctoproject.org/g/linux-yocto/message/12867
Mute This Topic: https://lists.yoctoproject.org/mt/100205922/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: 
https://lists.yoctoproject.org/g/linux-yocto/leave/6687884/21656/624485779/xyzzy
 [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to