Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

treewide: use uniform vendor for british telecom #4842

Closed
wants to merge 1 commit into from
Closed

treewide: use uniform vendor for british telecom #4842

wants to merge 1 commit into from

Conversation

mwarning
Copy link
Contributor

@mwarning mwarning commented Dec 11, 2021 •
edited

British Telecom is mostly known as BT.

Signed-off-by: Moritz Warning moritzwarning@web.de

@mwarning mwarning changed the title vendor: set uniform vendor for british telecom [WIP] vendor: set uniform vendor for british telecom Dec 11, 2021
@mwarning mwarning changed the title [WIP] vendor: set uniform vendor for british telecom vendor: set uniform vendor for british telecom Dec 11, 2021
@mans0n
Copy link
Member

mans0n commented Dec 11, 2021

I would use "treewide: ..." prefix.
And I see two more BT devices in bcm63xx, are they also from British Telecom?

@mans0n mans0n added the treewide pull request/issue with change across more than single place label Dec 11, 2021
@mwarning
Copy link
Contributor Author

mwarning commented Dec 11, 2021 •
edited

@mans0n good point, thank you.

@mwarning mwarning changed the title vendor: set uniform vendor for british telecom treewide: set uniform vendor for british telecom Dec 11, 2021
@mwarning
treewide: use uniform vendor value for british telecom
4989a4d 
British Telecom is mostly known as BT.

Signed-off-by: Moritz Warning <moritzwarning@web.de>
@mwarning mwarning changed the title treewide: set uniform vendor for british telecom treewide: use uniform vendor for british telecom Dec 11, 2021
@aparcar
Copy link
Member

aparcar commented Dec 12, 2021

Looks good

@hauke
Copy link
Member

hauke commented Dec 12, 2021

Thank your for the patch, I applied it to master.

@hauke hauke closed this Dec 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
treewide pull request/issue with change across more than single place
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@mwarning @mans0n @aparcar @hauke