[PATCH v3 1/2] drm/bridge: Add Cadence DSI driver

Archit Taneja architt at codeaurora.org
Wed Oct 11 10:13:15 UTC 2017



On 10/11/2017 02:56 PM, Boris Brezillon wrote:
> Hi Archit,
> 
> On Thu, 7 Sep 2017 15:06:13 +0530
> Archit Taneja <architt at codeaurora.org> wrote:
> 
>>
>>> +
>>> +static void cdns_dsi_bridge_disable(struct drm_bridge *bridge)
>>> +{
>>> +	struct cdns_dsi_input *input = bridge_to_cdns_dsi_input(bridge);
>>> +	struct cdns_dsi *dsi = input_to_dsi(input);
>>> +	u32 val;
>>> +
>>> +	val = readl(dsi->regs + MCTL_MAIN_DATA_CTL);
>>> +	val &= ~(IF_VID_SELECT_MASK | IF_VID_MODE | VID_EN | HOST_EOT_GEN |
>>> +		 DISP_EOT_GEN);
>>
>> I see some truncation related sparse warnings here and a couple of other places
>> when building against arm32. Those would be nice to fix.
> 
> I had a look and it seems to happen everytime I use GENMASK() (the
> truncation is harmless, but sparse complains).
> If you don't mind, I'd prefer to keep GENMASK() rather than manually
> defining masks, but maybe you have an idea how to fix these warnings
> without getting rid of GENMASK().

I don't know either. Let's live with the warnings for now.

Thanks,
Archit

> 
>>
>>> +	writel(val, dsi->regs + MCTL_MAIN_DATA_CTL);
>>> +
>>> +	val = readl(dsi->regs + MCTL_MAIN_EN) & ~IF_EN(input->id);
>>> +	writel(val, dsi->regs + MCTL_MAIN_EN);
>>> +}
> 

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project


More information about the dri-devel mailing list