pull request for atomic

Gedare Bloom gedare at rtems.org
Mon Jul 23 15:32:06 UTC 2012


On Mon, Jul 23, 2012 at 4:45 AM, yangwei weiyang <wei.a.yang at gmail.com> wrote:
> 2012/7/23 Sebastian Huber <sebastian.huber at embedded-brains.de>:
>> Hi,
>>
>> is it possible to squash the commits together?  This makes it easier to
>> review.
> Hi Sebastian, firstly thank you for your comments. The commits are a
> little trivial because all commits are submitted step by step. And i
> do not know how to make a pull request on github with a single patch
> from first commit to last commit. Or i can generate a patch manually
> and send it to mail list.

git rebase -i

should let you specify revision range and squash commits together.
then you can push a new branch and make new request.

>>
>> Since you wrote the files probably on your own they should have your
>> copyright.
>>
> OK, i will add the copyright to the appropriate files.
>> The file "cpukit/score/cpu/i386/rtems/score/atomic_cpu.h" should include a
>> generic file e.g. "cpukit/score/include/rtems/score/genericcpuatomic.h" with
>> the type definitions.  I think we can use this generic file on every CPU
>> port.
>>
> Yeah, agree. And RTEMS support so many architectures so i will think
> about what types should be generic to all cpu architecture.
>> I suggest to use the following type names:
>>
>> Atomic_Int
>> Atomic_Long
>> Atomic_Int32
>> Atomic_Int64
>> Atomic_Pointer
>>
> OK. and which file follow this rule that i can study the name rule on RTEMS?
>> I suggest to rename "atomic_cpu.h" into "cpuatomic.h".  Currently there is
>> no header file in the score with a "_".
>>
> OK.
>>
>> On 07/22/2012 06:08 PM, yangwei weiyang wrote:
>>>
>>> Hi all,
>>>
>>> I have made a pull request for review the source code of atomic, and
>>> could you give me any comments? thank you!
>>>
>>> pull request link is blow:
>>> https://github.com/RTEMS/rtems/pull/1
>>>
>>
>>
>> --
>> Sebastian Huber, embedded brains GmbH
>>
>> Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
>> Phone   : +49 89 18 90 80 79-6
>> Fax     : +49 89 18 90 80 79-9
>> E-Mail  : sebastian.huber at embedded-brains.de
>> PGP     : Public key available on request.
>>
>> Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
>>
>>
>> _______________________________________________
>> rtems-devel mailing list
>> rtems-devel at rtems.org
>> http://www.rtems.org/mailman/listinfo/rtems-devel
>
>
>
> --
> Wei Yang
> Best Regards
>
> wei.a.yang at gmail.com
>
> _______________________________________________
> rtems-devel mailing list
> rtems-devel at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-devel




More information about the devel mailing list