Re: drm/tegra: sor: Implement sor1_brick clock

[Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 919 bytes --]

On Fri, Jul 01, 2016 at 11:58:13PM +0300, Dan Carpenter wrote:
> Hello Thierry Reding,
> 
> The patch a7ba8310c7f5: "drm/tegra: sor: Implement sor1_brick clock"
> from Oct 1, 2015, leads to the following static checker warning:
> 
> 	drivers/gpu/drm/tegra/sor.c:318 tegra_clk_sor_brick_get_parent()
> 	warn: assigning (-22) to unsigned variable 'parent'
> 
> drivers/gpu/drm/tegra/sor.c
>    314  static u8 tegra_clk_sor_brick_get_parent(struct clk_hw *hw)
>                ^^
> returns u8.
> 
>    315  {
>    316          struct tegra_clk_sor_brick *brick = to_brick(hw);
>    317          struct tegra_sor *sor = brick->sor;
>    318          u8 parent = -EINVAL;

This ultimately has the effect of setting u8 to 0xff, which is the most
natural value to signify that the parent is invalid. But yeah, it's not
nice to rely on this behaviour, so I've used U8_MAX instead now.

Thanks,
Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]