Re: [PATCH][resend] of: Add vendor 2nd prefix for Asahi Kasei Corp

From: Kuninori Morimoto
Date: Wed Nov 12 2014 - 21:19:07 EST



Hi Mark, Geert

Sorry for my super late response about this

> > > Given all that, I think we should get rid of the "ak" prefix entirely,
> > > fix up the seaboard dts, and leave the legacy matching for "ak,ak8975"
> > > up to the i2c framework. However, that would imply deleting from
> > > vendor-prefixes.txt (if we want people to be warned on use of "ak"), and
> > > that's not a good precedent to set.
> >
> > Instead of deleting, we can deprecate, like was done for powervr:
> >
> > powervr PowerVR (deprecated, use img)
>
> Ah. I tried to see if we had any deprecated bindings, but I somehow
> failed to search correctly and missed powervr.
>
> > Still missing is an update for checkpatch to warn when a deprecated
> > vendor prefix is used.
>
> Marking it deprecated and warning with checkpatch is certainly
> preferable.

I guess removing "ak" line from vendor-prefixes.txt is very enough
for warn for a deprecated vendor prefix on checkpatch ?

...
- ak Asahi Kasei Corp.
...
+ asahi-kasei Asahi Kasei Corp.

Best regards
---
Kuninori Morimoto
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/