11 Feb
2019
11 Feb
'19
3:50 p.m.
On 2/11/19 14:41, Krzysztof Kozlowski wrote:
@@ -1469,12 +1506,16 @@ static int samsung_i2s_remove(struct platform_device *pdev) struct samsung_i2s_priv *priv = dev_get_drvdata(&pdev->dev); struct i2s_dai *pri_dai = samsung_i2s_get_pri_dai(&pdev->dev);
if (!priv)
This is not obvious. Maybe comment that it is secondary device remove()?
Sure, I will a comment here, I was considering it earlier but somehow it got missed.
@@ -1573,6 +1614,9 @@ static const struct platform_device_id samsung_i2s_driver_ids[] = { .name = "samsung-i2s", .driver_data = (kernel_ulong_t)&i2sv3_dai_type, },
{
Nit - squash with previous line.
Will change it.
--
Thanks,
Sylwester