Make applying patches easier and better Readme

[joe2k01]

No description provided.

[gw0]

Can you update the Readme to be cleaner and use features of Markdown? Such as:

• usually there is a new line after each title
• single-line commands should be written between ` and `, they render like command
• multi-line commands should be written between ``` and ```
• add command to git clone this repo
• my preference would also be to remove the numbered list and just use paragraphs or just a multi-line block of commands

[ikalkov]

I agree with @gw0.
Additionally, I have to admit I don't like the idea of fixed patches being stored in the device tree. How about creating a dedicated repository and adding it to the manifest?

[joe2k01]

I stored patches in the device tree because the patches in the wiki are outdated and because this is the way I manage patches in other android device trees, surely a dedicated repo could be a better option.
BTW, @gw0 what do you mean by 'add command to git clone this repo', just the git clone and then device repo link?

[ikalkov]

True, the patching tutorial in the wiki may be outdated, but I don't see yet how keeping required changes as files ist going to improve the situation.

[joe2k01]

I think this is a better way to manage patches because of the using of an automated script for applying them, but as you said a dedicated repo it's better than keeping patches in the device tree

[gw0]

@joe2k01 I am building Android for the first time. With "add command to git clone this repo" I mean there is no instruction into where you should clone this repository, something like (unfortunately this fails):

$ git clone https://github.com/android-rpi/device_brcm_rpi3 ./device/brcm/rpi3
fatal: destination path './device/brcm/rpi3' already exists and is not an empty directory.

[joe2k01]

Actually, there is the instruction. It's described in the first link

[gw0]

Oh, you mean this in local_manifests in default.xml:

<project path="device/brcm/rpi3" name="device_brcm_rpi3" revision="nougat" remote="arpi"/>

[joe2k01]

@ikalkov are you considering the merge?
@gw0 yes

[ikalkov]

@joe2k01, yes, if you remove those patch files. I'm out of office now, but I'll take a closer look at this merge next week.

[joe2k01]

Removed patches, should I pull request them in the hardware repo?