as the title says i need help with some wiring for taillights.

i currently have the skyline taillight mod that is hooked up only as running lights, and my stock bulbs are turn signal/brake lights

Now, Im going to buy a new set of lights and redo it so that the led lights are the only lights in the housing, so they will function as brake lights, tail lights, turn signal etc etc...



now i know this has been done before and Im not doing it to be original or anything just doing it because its what i want. i have searched the threads already and just want to make sure i have this wiring diagram correct before i mess something up. lol so to all the wiring guru's out there is this correct?




for the relay i have :

30 running to the black ground wire from the car
85 running to the + input on the pwm (pulse width modulator to dim running lights)
87 running to the negative on the led rings
87a running to the - input on the pwm (pulse width modulator to dim running lights)
86 running to the brown running lights from the car

the positive on the led rings run to the + output on the pwm
negative on the led rings run both to the 87 post on the relay and the - output on the pwm
and a resistor wired to the black ground from the car to the + output to the led rings (so they don't hyper flash when used as turn signals)


do i have it all set up correctly from what you can see or do i have to make any adjustments to make this work correctly

thanks in advance for your help!