Re: [PATCH] ARM: dts: Add trackpad to exynos5250-snow-rev5

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 20, 2017 at 08:13:21PM +0100, Leandro Santiago wrote:
> I am sending the v2 of the patch, with the following issues addressed:
> - properties ordering by importance
> - nodes ordering by address
> - fixed commit message
> - fixed interrupts property with edge failing interruption.

You make reading the patches especially difficult. Let me make it clear:
1. The previous one was v2 so this is the same?
2. Changelog goes after --- separator.
3. git format-patch -v2 (or whatever number)
4. scripts/get_maintainer.pl v2-....
5. git send-email --to ..... v2-...

Instead you attached v2 to some old conversation. Also your patch seems
to miss the mailing lists even though you CC-ed one of them here.
Probably you need to subscribe it first.

Missing the mailing list means:
1. It is not public.
2. It is not present on Patchwork which I use to apply the patches:
https://patchwork.kernel.org/project/linux-samsung-soc/list/

The patch's subject is still wrong - no changes here.

Beside the formal stuff, Javier's points remain valid. This should be
added to proper revision.

Best regards,
Krzysztof

> 
> Javier, I really I don't know exactly about the revision of my chromebook,
> but with the chromeos kernel it boots using the revision 5 dtb.
> 
> From a59ffe148a03e9b66918c30a553d564db32b885f Mon Sep 17 00:00:00 2001
> From: Leandro Santiago <leandrosansilva@xxxxxxxxx>
> Date: Sun, 19 Feb 2017 16:50:18 +0100
> Subject: [PATCH v2] ARM: dts: Add atmel maxtouch trackpad to
>  exynos5250-snow-rev5
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux