Yes, but it still doesn't answer my question: do you expect me to resend the patch with a different subject line?

On 03/06/15 16:22, Bill Fischofer wrote:
The agreed to format for git send-email is --subject-prefix="API-NEXT
PATCH"  (or PATCHv2, etc.)

On Wed, Jun 3, 2015 at 10:00 AM, Zoltan Kiss <zoltan.k...@linaro.org
<mailto:zoltan.k...@linaro.org>> wrote:

    Hi,

    I think there might be some ambiguity here: I understand that the
    correct format of subject line is [PATCH API-NEXT], rather than
    lower case, I'll do my best not to forget that next time. But do you
    expect me to resend the patch with a different subject line? I
    assume however picks this up (Maxim?) is already aware of this minor
    mistake and picks it up anyway, but I might be wrong.
    Or, do you wait for me to send it again with an another patch
    implementing this behaviour across our codebase? I thought I should
    send it once it gets commited to api-next.
    And finally, should the subject line of this next patch start with
    [PATCH API-NEXT] as well?

    Regards,

    Zoli

    On 28/05/15 15:28, Bill Fischofer wrote:

        We agreed that *any* change to the include/odp directory would be
        flagged API-NEXT (avoids having to make decisions).  We can then
        decide
        which to cherry-pick into the mainline on an expedited basis for
        simple
        things like documentation only changes, but these would be
        reviewed on a
        case-by-case basis.

        On Thu, May 28, 2015 at 9:25 AM, Zoltan Kiss
        <zoltan.k...@linaro.org <mailto:zoltan.k...@linaro.org>
        <mailto:zoltan.k...@linaro.org <mailto:zoltan.k...@linaro.org>>>
        wrote:

             Yes, I missed that prefix. Do you want me to resend the patch?
             I'll send an another patch to implement this behaviour in
        the repo
             wherever odp_pktio_send is called. But I think we can't
        create a
             unit test for it, can we?

             On 28/05/15 14:56, Bill Fischofer wrote:

                 This needs to be API-NEXT as it is a change to an API file.

                 On Thu, May 28, 2015 at 7:53 AM, Savolainen, Petri (Nokia -
                 FI/Espoo)
                 <petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>
        <mailto:petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>>
                 <mailto:petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>
                 <mailto:petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>>>> wrote:

                      Reviewed-by: Petri Savolainen
        <petri.savolai...@nokia.com <mailto:petri.savolai...@nokia.com>
                 <mailto:petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>>
                      <mailto:petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>
                 <mailto:petri.savolai...@nokia.com
        <mailto:petri.savolai...@nokia.com>>>>


                       > -----Original Message-----
                       > From: lng-odp
        [mailto:lng-odp-boun...@lists.linaro.org
        <mailto:lng-odp-boun...@lists.linaro.org>
                 <mailto:lng-odp-boun...@lists.linaro.org
        <mailto:lng-odp-boun...@lists.linaro.org>>
                      <mailto:lng-odp-boun...@lists.linaro.org
        <mailto:lng-odp-boun...@lists.linaro.org>
                 <mailto:lng-odp-boun...@lists.linaro.org
        <mailto:lng-odp-boun...@lists.linaro.org>>>] On Behalf Of ext
                       > Zoltan Kiss
                       > Sent: Thursday, May 28, 2015 3:19 PM
                       > To: lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>
                 <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>>
                 <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>>>
                       > Subject: [lng-odp] [PATCH] api-next: packet_io:
        clarify what
                      happens when
                       > not all packets are sent
                       >
                       > Currently our examples are not handling this
        situation
                 as well.
                       >
                       > Signed-off-by: Zoltan Kiss
        <zoltan.k...@linaro.org <mailto:zoltan.k...@linaro.org>
                 <mailto:zoltan.k...@linaro.org
        <mailto:zoltan.k...@linaro.org>>
                      <mailto:zoltan.k...@linaro.org
        <mailto:zoltan.k...@linaro.org>

                 <mailto:zoltan.k...@linaro.org
        <mailto:zoltan.k...@linaro.org>>>>

                       > ---
                       >  include/odp/api/packet_io.h | 4 +++-
                       >  1 file changed, 3 insertions(+), 1 deletion(-)
                       >
                       > diff --git a/include/odp/api/packet_io.h
                      b/include/odp/api/packet_io.h
                       > index 89356a6..b97b2b8 100644
                       > --- a/include/odp/api/packet_io.h
                       > +++ b/include/odp/api/packet_io.h
                       > @@ -111,7 +111,9 @@ int
        odp_pktio_recv(odp_pktio_t pktio,
                      odp_packet_t
                       > pkt_table[], int len);
                       >   * @param pkt_table[]  Array of packets to send
                       >   * @param len          length of pkt_table[]
                       >   *
                       > - * @return Number of packets sent
                       > + * @return Number of packets sent. If it is
        less than
                 'len', the
                       > remaining
                       > + * packets at the end of pkt_table[] are left
        intact,
                 and caller
                      has to
                       > take
                       > + * care of them.
                       >   * @retval <0 on failure
                       >   */
                       >  int odp_pktio_send(odp_pktio_t pktio, odp_packet_t
                 pkt_table[],
                      int len);
                       > --
                       > 1.9.1
                       >
                       > _______________________________________________
                       > lng-odp mailing list
                       > lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>
                 <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>>
                 <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>>>
                       > https://lists.linaro.org/mailman/listinfo/lng-odp
                      _______________________________________________
                      lng-odp mailing list
        lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>>
                 <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>>>
        https://lists.linaro.org/mailman/listinfo/lng-odp




_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to