On Tue, Jun 02, 2015 at 07:00:39PM +0300, Andrew Andrianov wrote: > From: Andrew 'Necromant' Andrianov <and...@ncrmnt.org> > > This patch adds a generic ion driver that allows > ion heaps to be added via devicetree. It provides > a simple and generic way to feed physical memory regions > to ion without writing a custom driver, e.g. > > ion_sram: ion@0x00100000 { > compatible = "ion,physmem"; > reg = <0x00100000 0x40000>; > reg-names = "memory"; > ion-heap-id = <1>; > ion-heap-type = <ION_HEAP_TYPE_DMA>; > ion-heap-align = <0x10>; > ion-heap-name = "SRAM"; > }; > > Signed-off-by: Andrew Andrianov <and...@ncrmnt.org>
Your From: name and Signed-off-by: name is not matching. But why you are using this extra From: line? you email header From: is same as your Signed-off-by. And since you are adding new files it would be better if you can fix few checkpatch warnings it has. like: CHECK: Prefer kzalloc(sizeof(*ipdev)...) over kzalloc(sizeof(struct physmem_ion_dev)...) CHECK: Alignment should match open parenthesis CHECK: No space is necessary after a cast CHECK: Please don't use multiple blank lines regards sudip -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/