Re: [PATCH 0/6] [v2] omap:mailbox-enhancements and fixes

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

 



Hi Hari,

On 11/10/2010 1:45 PM, Hari Kanigeri wrote:
Thanks to Rene Sapiens and Omar Ramirez for their inputs on initial patch
set.
http://www.mail-archive.com/linux-omap@xxxxxxxxxxxxxxx/msg37278.html

The patch set addresses the following review comments from Rene and Omar.
http://www.mail-archive.com/linux-omap@xxxxxxxxxxxxxxx/msg37626.html
https://patchwork.kernel.org/patch/255091/
https://patchwork.kernel.org/patch/255081/

Following patches are changed because of above review comments:
omap:mailbox-send message in process context
omap:mailbox-add notification support for multiple readers

Following patch is dropped from initial patch set
omap:mailbox-resolve multiple receiver problem

The patch set is tested on omap4 SDP board.

Fernando Guzman Lugo (1):
   mailbox: change full flag per mailbox queue instead of global

Hari Kanigeri (5):
   omap:mailbox: fix rx interrupt disable in omap4
   omap:mailbox-fix checkpatch warnings

That one is weird? How can you submit a patch that fix checkpatch?
In theory you should not send any patch that generate checkpatch error or warning.

   omap:mailbox-send message in process context
   omap:mailbox-add notification support for multiple readers
   omap:clocks44x-add dummy clock for mailbox

We are trying to enforce some consistency in the subjects name so you should name your patches like that:

    OMAP: mailbox: fix rx interrupt disable in omap4
    OMAP: mailbox: fix checkpatch warnings
    OMAP: mailbox: send message in process context
    OMAP: mailbox: add notification support for multiple readers
    OMAP4: clocks: add dummy clock for mailbox


Regards,
Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux