Submitting patches question

Dean Michael Ancajas dbancajas at gmail.com
Mon Dec 8 16:05:45 EST 2014


Hi,
  I have submitted single line patches. I was wondering what is the
policy on the # of changes per patch? For instance the code below:

        if (Index) {
                data = ft1000_read_reg(dev, FT1000_REG_MAG_DPDATAL);
        } else {
                data = ft1000_read_reg(dev, FT1000_REG_MAG_DPDATAH);
        }

the braces are not necessary according to the rules. Should I

A. submit 1 patch for the "if" portion and another for the "else" on
separate emails (i.e. two patches on separate emails)?

B. 1 patch for the whole thing (considering this is trivial change) ?

C.  2 separate patches but in 1-email only?

Thanks



More information about the Kernelnewbies mailing list