[nsp] Route-Reflector question

[TMS]

Hello
I have question regarding BGP Route-Reflector. I have BGP network
which looks like that:

[router1] <---> [router2] <---> [router3]
 1.1.1.1         1.1.1.2         1.1.1.3

router1 is r1
router2 is r2
router3 is r3

r1 and r2 are in "IBGP mesh"
r3 is RR-Client for r2
r2 is RR-Client for r3

So, i configured this that:

r1:

router bgp 100
  neighbor 1.1.1.2 remote-as 100
  neighbor 1.1.1.2 next-hop-self
!

r2:

router bgp 100
  neighbor 1.1.1.1 remote-as 100
  neighbor 1.1.1.1 next-hop-self
  neighbor 1.1.1.3 remote-as 100
  neighbor 1.1.1.3 next-hop-self
  neighbor 1.1.1.3 route-reflector-client
!

r3:

router bgp 100
  neighbor 1.1.1.2 remote-as 100
  neighbor 1.1.1.2 next-hop-self
  neighbor 1.1.1.2 route-reflector-client
!

Is above BGP Configurations are correct ? Maybe I should delete line:
"neighbor 1.1.1.2 route-reflector-client" from r3 configuration ?

best regards,
Tommy