| Message ID | 1408524184-31928-5-git-send-email-p.zabel@pengutronix.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Aug 20, 2014 at 3:43 AM, Philipp Zabel <p.zabel@pengutronix.de> wrote: > Currently there is a wild mixture of isl, isil, and intersil > compatibles in the kernel. At this point, changing the vendor > symbol to the most often used variant, which is equal to the > NASDAQ symbol, isil, should not hurt, since the isl1208 driver > doesn't care either way. > > Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de> > --- > arch/powerpc/boot/dts/ppa8548.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts > index 27b0699..000262b 100644 > --- a/arch/powerpc/boot/dts/ppa8548.dts > +++ b/arch/powerpc/boot/dts/ppa8548.dts > @@ -89,7 +89,7 @@ > &soc { > i2c@3000 { > rtc@6f { > - compatible = "intersil,isl1208"; > + compatible = "isil,isl1208"; This is a mature platform. You cannot change it. You can add the new string, but the dts and kernel need both strings. I would just leave it all alone and add the new prefix and mark the others deprecated. Rob > reg = <0x6f>; > }; > }; > -- > 2.1.0.rc1 >
diff --git a/arch/powerpc/boot/dts/ppa8548.dts b/arch/powerpc/boot/dts/ppa8548.dts index 27b0699..000262b 100644 --- a/arch/powerpc/boot/dts/ppa8548.dts +++ b/arch/powerpc/boot/dts/ppa8548.dts @@ -89,7 +89,7 @@ &soc { i2c@3000 { rtc@6f { - compatible = "intersil,isl1208"; + compatible = "isil,isl1208"; reg = <0x6f>; }; };
Currently there is a wild mixture of isl, isil, and intersil compatibles in the kernel. At this point, changing the vendor symbol to the most often used variant, which is equal to the NASDAQ symbol, isil, should not hurt, since the isl1208 driver doesn't care either way. Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de> --- arch/powerpc/boot/dts/ppa8548.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)